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

[4.x] Improve examples test coverage #8338

Merged
merged 2 commits into from Feb 14, 2024

Conversation

tvallin
Copy link
Member

@tvallin tvallin commented Feb 6, 2024

Description

fix #8337

Introduce integration test using test containers :

  • Database example is tested with a database ran in a docker container.
    • H2
    • MySql
    • Oracle-XE
  • Tracing example is tested with a tracing service ran in docker container.

Documentation

no impact

Signed-off-by: tvallin <thibault.vallin@oracle.com>
@tvallin tvallin added enhancement New feature or request testing 4.x Version 4.x labels Feb 6, 2024
@tvallin tvallin self-assigned this Feb 6, 2024
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Feb 6, 2024
Copy link
Contributor

@romain-grecourt romain-grecourt left a comment

Choose a reason for hiding this comment

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

The only description is "Improve examples test coverage" in both the referenced issue and this PR.

Can you please:

  • Update the issue and describe the problem that needs to be solved (E.g. foo and bar examples are broken because the unit tests aren't exercising critical parts ; we need to add integration tests to ensure that the examples are working.)
  • Update this PR description to describe the changes (E.g. add integration test for mysql and oracle db using test containers)

Signed-off-by: tvallin <thibault.vallin@oracle.com>
@romain-grecourt romain-grecourt merged commit a690501 into helidon-io:main Feb 14, 2024
12 checks passed
hrstoyanov pushed a commit to hrstoyanov/helidon that referenced this pull request Feb 23, 2024
* Container test

Signed-off-by: tvallin <thibault.vallin@oracle.com>

* refactor tests

Signed-off-by: tvallin <thibault.vallin@oracle.com>

---------

Signed-off-by: tvallin <thibault.vallin@oracle.com>
tvallin added a commit to tvallin/helidon that referenced this pull request Feb 28, 2024
* Container test

Signed-off-by: tvallin <thibault.vallin@oracle.com>

* refactor tests

Signed-off-by: tvallin <thibault.vallin@oracle.com>

---------

Signed-off-by: tvallin <thibault.vallin@oracle.com>
@tvallin tvallin deleted the container-tests branch April 18, 2024 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x Version 4.x enhancement New feature or request OCA Verified All contributors have signed the Oracle Contributor Agreement. testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[4.x] Improve examples test coverage
2 participants