-
Notifications
You must be signed in to change notification settings - Fork 27
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
Feature/cdr 715 fix partial date time serialization #420
Feature/cdr 715 fix partial date time serialization #420
Conversation
441938c
to
61743d8
Compare
Fixing this bug uncovered another bug, where the parser will accept partial dates with an invalid month like 2017-00 or 2017-13 instead of throwing an error. This is tested in COMPOSITION-tests-1 B.6.n which is the one that makes the pipeline fail |
…ialization # Conflicts: # serialisation/src/main/java/org/ehrbase/serialisation/jsonencoding/DateTimeSerializer.java
…x-partial-date-time-serialization # Conflicts: # CHANGELOG.md
serialisation/src/main/java/org/ehrbase/serialisation/xmlencoding/CanonicalXML.java
Show resolved
Hide resolved
b5216e5
to
a808c92
Compare
…ialization # Conflicts: # CHANGELOG.md
…ialization # Conflicts: # bom/pom.xml
Kudos, SonarCloud Quality Gate passed! |
Note that 8 tests from COMPOSITION-tests-1 fail, because they compare error messages in addition to the status code. With the workaround implementation we now get more meaningful error messages that no longer match the ones expected by the tests. Affected test cases are: B.6.7.a |
No description provided.