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

Refactored SQS test - native image execution #375

Merged
merged 8 commits into from
Jun 5, 2023
Merged

Conversation

msupic
Copy link
Contributor

@msupic msupic commented Jun 2, 2023

Refactored SQS test so it can be executed in a native image

  • replaced groovy with java test
  • SQS container managed by test resources service

@graemerocher
Copy link
Contributor

need to remove the snapshot repo before merging

@msupic
Copy link
Contributor Author

msupic commented Jun 2, 2023

The snapshot repo can be removed after a new release of micronaut-test-resources is created since code from this PR is needed: micronaut-projects/micronaut-test-resources#299
If it is not an issue, I can merge this to master with the snapshot repo and later remove the snapshot repo and version when we get new release of micronaut-test-resources

msupic added 3 commits June 5, 2023 10:27
…into sqs-native-test

# Conflicts:
#	buildSrc/src/main/groovy/io.micronaut.build.internal.jms-tests.gradle
#	gradle/libs.versions.toml
#	tests/tasks-activemq-artemis/build.gradle
#	tests/tasks-activemq-classic/build.gradle
#	tests/tasks-sqs/build.gradle
#	tests/tasks-sqs/src/main/resources/logback.xml
#	tests/tasks-sqs/src/test/groovy/example/TasksSpec.groovy
@sdelamo
Copy link
Contributor

sdelamo commented Jun 5, 2023

@msupic please merge master into this branch.

msupic added 2 commits June 5, 2023 12:42
…into sqs-native-test

# Conflicts:
#	tests/tasks-sqs/src/test/groovy/example/TasksSpec.groovy
…tory which uses config values from sqs container created by micronaut-test-resources
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 5, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@msupic
Copy link
Contributor Author

msupic commented Jun 5, 2023

@graemerocher - the snapshot repo has been removed
@sdelamo - the master has been merged into this branch

@graemerocher graemerocher merged commit e30742b into master Jun 5, 2023
@graemerocher graemerocher deleted the sqs-native-test branch June 5, 2023 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants