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

Patch/72 new test data set for ehrbase/project management#72 #202

Conversation

jakesmolka
Copy link
Contributor

@jakesmolka jakesmolka commented Mar 24, 2020

Changes

  • Adds first contribution-context checks
    • Adds check if a version with change type CREATION also has preceding_version_uid set, or with type MODIFICATION has the version uid not set. Both are invalid.
  • Also fixed integration test for the first case.
  • Small change to existing test data

Related issue

fixes https://github.com/ehrbase/project_management/issues/72

Additional information and checks

  • Pull request linked in changelog

Wlad Wagner and others added 3 commits December 3, 2019 22:06
…w-Test-Data-Set-for-ehrbase/project_management#72
Adds check if a version with change type CREATION also has
preceding_version_uid set, or with type MODIFICATION has the version uid
not set. Both are invalid.
Also fixed intergration test for the first case.

see https://github.com/ehrbase/project_management/issues/72
@jakesmolka jakesmolka self-assigned this Mar 24, 2020
@testautomation
Copy link
Contributor

quick command to run related test (on this branch!)

robot -i 72 -d results -L TRACE robot/CONTRIBUTION_TESTS

@jakesmolka
Copy link
Contributor Author

(added to current sprint - was not transferred from last to current from the beginning)

@jakesmolka jakesmolka changed the title WIP: Patch/72 new test data set for ehrbase/project management#72 Patch/72 new test data set for ehrbase/project management#72 Apr 6, 2020
@jakesmolka jakesmolka merged commit c198348 into develop Apr 7, 2020
@jakesmolka jakesmolka deleted the patch/72_New-Test-Data-Set-for-ehrbase/project_management#72 branch April 7, 2020 13:48
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.

None yet

3 participants