-
Notifications
You must be signed in to change notification settings - Fork 0
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
73 generate assertions endpoint #139
Conversation
…-assertions-endpoint
…ate-assertions-endpoint
…rameters, badData and other parent-class utilities.
…o ensure no interference with real database.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two minor comments about log messages
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LOGGER.debug(String.format("%s %s.", prefix, "Received Generate Assertions request."));
ends in two periods.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also LOGGER.debug(String.format("%s %s.", prefix, "Completed assertion generation for all IRV contests."));
same double period.
final String prefix = "[generateAssertions]";
doesn't match the function name.
This does the main work of the generate-assertions endpoint, with tests.
Note that there are not (yet) any API tests - these tests only check the behaviour of the function that does the work of the endpoint.