Skip to content
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

GOML JB Week5 #3

Conversation

jabesse
Copy link
Collaborator

@jabesse jabesse commented Feb 10, 2019

#4614
Tried to find the class causing this bug. Unit tests showed that this issue could be related to JavaFX in the DuplicateResolverDialog class because all non-JavaFX tests were successful. Testing will continue once my IDE is setup for JavaFX unit testing because most of its functionality is inside a private method only accessible through a constructor that requires JavaFX dependent classes. It could also be in the ImportInspectionDialog class, but it was the same situation where all the methods of interest needed JavaFX tests. Testing coverage: 2 Classes, 4 Methods, and 12 Lines.


  • Change in CHANGELOG.md described
  • Tests created for changes
  • Manually tested changed features in running JabRef
  • Screenshots added in PR description (for bigger UI changes)
  • Ensured that the git commit message is a good one
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

@jabesse jabesse merged commit 2e62b01 into master Feb 10, 2019
benjagooder added a commit that referenced this pull request Apr 22, 2019
benjagooder added a commit that referenced this pull request Apr 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants