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

issue 607: Rework unit-test for VersionSuite #726

Merged
merged 4 commits into from
Oct 31, 2024
Merged

issue 607: Rework unit-test for VersionSuite #726

merged 4 commits into from
Oct 31, 2024

Conversation

omordyk
Copy link
Contributor

@omordyk omordyk commented Oct 31, 2024

issue 607: Rework unit-test for version
Description: #607
Changes:

  • Increased the number of test cases to cover corner cases.
  • Fixed issues below that were found during refactoring:
    assert(!VersionRange("").isValid)         // Empty string
    assert(!VersionRange(",2").isValid)       // Invalid start

- Increased the number of test cases to cover corner cases.
- Fixed minor issues found during refactoring.

Signed-off-by: Oleksandr Mordyk <[email protected]>
@omordyk omordyk marked this pull request as ready for review October 31, 2024 10:38
@omordyk omordyk self-assigned this Oct 31, 2024
- Fix checking for empty version and invalid start.

Signed-off-by: Oleksandr Mordyk <[email protected]>
@omordyk omordyk changed the title issue 607: Rework unit-test for version issue 607: Rework unit-test for VersionSuite Oct 31, 2024
Copy link
Member

@naphelps naphelps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@omordyk LGTM. The only change I would make is moving the test suite package from src/test/scala/org/openhorizon/exchangeapi/route/version/TestVersion.scala to src/test/scala/org/openhorizon/exchangeapi/utility/version/TestVersion.scala. The suite will have the equivalent package as the source.

@omordyk
Copy link
Contributor Author

omordyk commented Oct 31, 2024

@naphelps moved.

@omordyk omordyk requested a review from naphelps October 31, 2024 12:41
@naphelps naphelps merged commit 02f87d8 into master Oct 31, 2024
2 checks passed
@naphelps naphelps deleted the issue_607 branch October 31, 2024 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants