Refactor unit test profiles, try to speed things up #1244
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Yuqi-Du
approved these changes
Jul 9, 2024
|
Unfortunately it looks like I cannot get rid of global test resources, after all. So will just inline profile classes to keep things bit tidier. |
amorton
added a commit
that referenced
this pull request
Jul 16, 2024
This commit fixes failing unit and integration tests. It cherry picks multiple commits from the ajm/tables branch below. -> SKIPPED - did not cherry pick the original see below <- commit 0b53fba Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 17:34:14 2024 -0700 Revert UT->IT cherry-picked merge commit 951a092 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 17:32:20 2024 -0700 Minor clean up of test contexts, javadocs commit 28c9ebb Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 17:21:20 2024 -0700 Minor clean up commit 9801566 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 17:09:18 2024 -0700 Fix import stmt -> merge commit - not cherry picked <- commit b14dfa2 Merge: 13678a5 a9cb436 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 16:49:10 2024 -0700 Merge branch 'main' into ajm/tables -> came from merge, not cherry picked <- commit a9cb436 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 16:47:00 2024 -0700 Update Quarkus Data API uses to 3.9.5 (from parent 3.6.x) (#1246) -> merge commit - not cherry picked <- commit 13678a5 Merge: c7eccc0 dd42485 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 12:29:49 2024 -0700 Merge branch 'main' into ajm/tables -> came from merge, not cherry picked <- commit dd42485 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 11:40:21 2024 -0700 Fix a problem that would prevent Quarkus 3.12 upgrade (due to change in Smallrye lib) (#1248) -> came from merge, not cherry picked <- commit 15cd3f6 Author: YuqiDu <istimdu@gmail.com> Date: Tue Jul 9 10:31:03 2024 -0700 fix http port to 8181, refactor application.yaml as alphabetic order (#1247) -> came from merge, not cherry picked <- commit ce2b337 Author: YuqiDu <istimdu@gmail.com> Date: Tue Jul 9 09:19:11 2024 -0700 Bridge-Removal - Detach the quarkus-common-module dependency (OFF WE GO!) (#1191) Co-authored-by: Tatu Saloranta <tatu.saloranta@datastax.com> commit c7eccc0 Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 19:31:31 2024 +1200 fmt fix commit 64930f5 Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 19:28:42 2024 +1200 Unit test fixes for Namespace commands -> merge commit - not cherry picked <- commit e7a6d15 Merge: baa6ea5 5796eca Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 19:18:55 2024 +1200 Merge branch 'main' into ajm/tables commit baa6ea5 Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 19:14:03 2024 +1200 Fixes for Namespace Commands so they work on DatabaseSchemaObject commit cfbbc8c Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 14:57:20 2024 +1200 fix CreateCollectionResolverVectorizeDisabledTest -> SKIPPED - later reverted , broke IT's <- commit c50d7fd Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Mon Jul 8 19:30:57 2024 -0700 Fixes #1245: change 3 unit tests to integration tests to speed up unit test runtime git cherry-pick 1996697 -> came from merge, not cherry picked <- commit 5796eca Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Mon Jul 8 19:08:23 2024 -0700 Refactor unit test profiles, try to speed things up (#1244) -> came from merge, not cherry picked <- commit 30c7ae0 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Mon Jul 8 18:36:41 2024 -0700 Fix accidental dependencies to shaded packages (#1243) -> merge commit, not cherry picked <- commit 18a804b Merge: 7ee94cd 35ecb00 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Mon Jul 8 18:33:51 2024 -0700 Merge branch 'main' into ajm/tables commit 7ee94cd Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 13:13:47 2024 +1200 fix CreateCollectionResolverVectorDisabledTest commit b12c484 Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 12:45:23 2024 +1200 Fix for failing Collection based tests -> came from merge, not cherry picked <- commit 35ecb00 Author: YuqiDu <istimdu@gmail.com> Date: Mon Jul 8 17:32:05 2024 -0700 Bridge-Removal - miscellaneous cleanups (#1149) Co-authored-by: Tatu Saloranta <tatu.saloranta@datastax.com> commit 4681ccd Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 11:52:31 2024 +1200 fix for createCollection and other commands that run against the namespace FindIntegrationTest works, waiting to find out about others
Yuqi-Du
pushed a commit
that referenced
this pull request
Jul 19, 2024
This commit fixes failing unit and integration tests. It cherry picks multiple commits from the ajm/tables branch below. -> SKIPPED - did not cherry pick the original see below <- commit 0b53fba Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 17:34:14 2024 -0700 Revert UT->IT cherry-picked merge commit 951a092 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 17:32:20 2024 -0700 Minor clean up of test contexts, javadocs commit 28c9ebb Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 17:21:20 2024 -0700 Minor clean up commit 9801566 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 17:09:18 2024 -0700 Fix import stmt -> merge commit - not cherry picked <- commit b14dfa2 Merge: 13678a5 a9cb436 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 16:49:10 2024 -0700 Merge branch 'main' into ajm/tables -> came from merge, not cherry picked <- commit a9cb436 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 16:47:00 2024 -0700 Update Quarkus Data API uses to 3.9.5 (from parent 3.6.x) (#1246) -> merge commit - not cherry picked <- commit 13678a5 Merge: c7eccc0 dd42485 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 12:29:49 2024 -0700 Merge branch 'main' into ajm/tables -> came from merge, not cherry picked <- commit dd42485 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Tue Jul 9 11:40:21 2024 -0700 Fix a problem that would prevent Quarkus 3.12 upgrade (due to change in Smallrye lib) (#1248) -> came from merge, not cherry picked <- commit 15cd3f6 Author: YuqiDu <istimdu@gmail.com> Date: Tue Jul 9 10:31:03 2024 -0700 fix http port to 8181, refactor application.yaml as alphabetic order (#1247) -> came from merge, not cherry picked <- commit ce2b337 Author: YuqiDu <istimdu@gmail.com> Date: Tue Jul 9 09:19:11 2024 -0700 Bridge-Removal - Detach the quarkus-common-module dependency (OFF WE GO!) (#1191) Co-authored-by: Tatu Saloranta <tatu.saloranta@datastax.com> commit c7eccc0 Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 19:31:31 2024 +1200 fmt fix commit 64930f5 Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 19:28:42 2024 +1200 Unit test fixes for Namespace commands -> merge commit - not cherry picked <- commit e7a6d15 Merge: baa6ea5 5796eca Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 19:18:55 2024 +1200 Merge branch 'main' into ajm/tables commit baa6ea5 Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 19:14:03 2024 +1200 Fixes for Namespace Commands so they work on DatabaseSchemaObject commit cfbbc8c Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 14:57:20 2024 +1200 fix CreateCollectionResolverVectorizeDisabledTest -> SKIPPED - later reverted , broke IT's <- commit c50d7fd Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Mon Jul 8 19:30:57 2024 -0700 Fixes #1245: change 3 unit tests to integration tests to speed up unit test runtime git cherry-pick 1996697 -> came from merge, not cherry picked <- commit 5796eca Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Mon Jul 8 19:08:23 2024 -0700 Refactor unit test profiles, try to speed things up (#1244) -> came from merge, not cherry picked <- commit 30c7ae0 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Mon Jul 8 18:36:41 2024 -0700 Fix accidental dependencies to shaded packages (#1243) -> merge commit, not cherry picked <- commit 18a804b Merge: 7ee94cd 35ecb00 Author: Tatu Saloranta <tatu.saloranta@datastax.com> Date: Mon Jul 8 18:33:51 2024 -0700 Merge branch 'main' into ajm/tables commit 7ee94cd Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 13:13:47 2024 +1200 fix CreateCollectionResolverVectorDisabledTest commit b12c484 Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 12:45:23 2024 +1200 Fix for failing Collection based tests -> came from merge, not cherry picked <- commit 35ecb00 Author: YuqiDu <istimdu@gmail.com> Date: Mon Jul 8 17:32:05 2024 -0700 Bridge-Removal - miscellaneous cleanups (#1149) Co-authored-by: Tatu Saloranta <tatu.saloranta@datastax.com> commit 4681ccd Author: Aaron Morton <aaron.morton@datastax.com> Date: Tue Jul 9 11:52:31 2024 +1200 fix for createCollection and other commands that run against the namespace FindIntegrationTest works, waiting to find out about others
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does:
Changes test profiles used by unit tests to avoid global resource construction; inlines ones used by a single test class.
Which issue(s) this PR fixes:
N/A
Checklist