-
Notifications
You must be signed in to change notification settings - Fork 1
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
Implement physical attributes NOMIS to DPS synchronisation #666
Implement physical attributes NOMIS to DPS synchronisation #666
Conversation
ea2de39
to
4787f7d
Compare
77afd31
to
424107c
Compare
|
||
sendPhysicalAttributesChangedEvent() | ||
|
||
verifyResults(type = "ignored", reason = "New physical attributes are empty") |
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.
Question; are physical attributes not copied by NOMIS for new bookings (like it does for Alerts)?
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.
They are yes. This test is for a brand new booking - if you go into the physical info NOMIS screen it creates this "empty" record which makes no sense to synchronise.
} | ||
|
||
@Test | ||
fun `should put message on DLQ if booking has no physical attributes`() = runTest { |
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.
why is this an error?
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.
We shouldn't be receiving events for bookings without any physical attributes - but I chose not to assume that in the code, so had to handle the error scenario. Should never happen.
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.
lgtm
No description provided.