-
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
[AB2D-6147] increase ab2d-events-client
coverage
#399
Conversation
@@ -121,7 +121,7 @@ public boolean equals(final Object o) { | |||
if (otherTimeOfEvent != null) { | |||
return false; | |||
} | |||
} else if (thisTimeOfEvent.toEpochSecond() != otherTimeOfEvent.toEpochSecond()) { | |||
} else if (otherTimeOfEvent == null || thisTimeOfEvent.toEpochSecond() != otherTimeOfEvent.toEpochSecond()) { |
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.
I found this missing null check when writing the tests
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.
Good catch
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.
Do you plan to increase library version? may be later?
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.
Oh, thanks for catching that @smirnovaae! I forgot that part. I'll do it in another PR.
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.
Looks good to me!
🎫 Ticket
https://jira.cms.gov/browse/AB2D-6147
🛠 Changes
A very long and repetitious PR for testing
LoggableEvent
. 80% of the diff is testing the extremely longequals
function, in such a way that hits every single possible case. While doing this I hit a null pointer exception (on the date handling), so I added a line of code to fix that.ℹ️ Context
This is a part of my journey towards 90% code coverage