-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Include test cases for other types #29
Comments
I've added the test cases for other and reallyunknowntype in 2152. So can this be closed? |
Yes, there are test cases (https://github.com/JabRef/jabref/pull/2152/files). No, the test case do not read and write these files. With the feature that an entry is only rewritten if it changed, these bib files should not change too. However, I'd like to be 100% sure. The keyword is "roundtrip". The intention of the issue is that there might be impact of using a whole bibdatabase instead of a single bibentry object. |
The tests check that the correct type gets parsed and ensure that the correct string is written. So there is nothing which can go wrong in a roundtrip, except if somebody changes the BibEntry in between. (Actually, this is the nice thing about unit-testing the parser and writer: you can be sure that the roundtrip works too, but if something goes wrong you know exactly where the error is). So I also vote for closing this issue. |
I now understood the comments. The BibEntries are read and then compared. Thus, the discussion on writing misses the point. |
othertype.bib and reallyunknowntype.bib. When reading and writing the file, the output should be equal to the original file. Especially, no type should have been changed.
The text was updated successfully, but these errors were encountered: