Skip to content

refactor: move integration tests to feature packages and sub packages based on type #2483

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

Merged
merged 10 commits into from
Aug 8, 2024

Conversation

csviri
Copy link
Collaborator

@csviri csviri commented Aug 8, 2024

No description provided.

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 8, 2024
@csviri
Copy link
Collaborator Author

csviri commented Aug 8, 2024

For now 2 categories:

  • base api
  • dependents
  • workflows

Signed-off-by: Attila Mészáros <[email protected]>
@csviri csviri changed the title refactor: integration tests moved to feature packages and sub packages based on type refactor: move integration tests to feature packages and sub packages based on type Aug 8, 2024
csviri added 6 commits August 8, 2024 13:26
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
@csviri csviri marked this pull request as ready for review August 8, 2024 16:13
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 8, 2024
@csviri csviri requested a review from metacosm August 8, 2024 16:13
@openshift-ci openshift-ci bot requested a review from adam-sandor August 8, 2024 16:13
@@ -42,14 +43,14 @@ void cleanerIsCalledOnBuiltInResource() {
operator.delete(service);

await().untilAsserted(() -> {
assertThat(operator.getReconcilerOfType(ObservedGenerationTestReconciler.class)
assertThat(operator.getReconcilerOfType(BuiltInResourceCleanerReconciler.class)
.getCleanCount()).isPositive();
});
}

Service testService() {
Service service = ReconcilerUtils.loadYaml(Service.class, StandaloneDependentResourceIT.class,
Copy link
Collaborator

Choose a reason for hiding this comment

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

That looks suspicious…

@metacosm metacosm merged commit 02b876b into next Aug 8, 2024
20 checks passed
@metacosm metacosm deleted the it-refactor branch August 8, 2024 19:18
csviri added a commit that referenced this pull request Aug 15, 2024
… based on type (#2483)

* refactor: integration tests moved to feature packages and sub packages pased on type

Signed-off-by: Attila Mészáros <[email protected]>

* fix resource paths

Signed-off-by: Attila Mészáros <[email protected]>

* fixes

Signed-off-by: Attila Mészáros <[email protected]>

* link and test fixes

Signed-off-by: Attila Mészáros <[email protected]>

* format

Signed-off-by: Attila Mészáros <[email protected]>

* wip

Signed-off-by: Attila Mészáros <[email protected]>

* old service

Signed-off-by: Attila Mészáros <[email protected]>

* test fix

Signed-off-by: Attila Mészáros <[email protected]>

* refactor: remove unneeded classes

Signed-off-by: Chris Laprun <[email protected]>

* fix: typo in package name

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
metacosm added a commit that referenced this pull request Aug 16, 2024
… based on type (#2483)

* refactor: integration tests moved to feature packages and sub packages pased on type

Signed-off-by: Attila Mészáros <[email protected]>

* fix resource paths

Signed-off-by: Attila Mészáros <[email protected]>

* fixes

Signed-off-by: Attila Mészáros <[email protected]>

* link and test fixes

Signed-off-by: Attila Mészáros <[email protected]>

* format

Signed-off-by: Attila Mészáros <[email protected]>

* wip

Signed-off-by: Attila Mészáros <[email protected]>

* old service

Signed-off-by: Attila Mészáros <[email protected]>

* test fix

Signed-off-by: Attila Mészáros <[email protected]>

* refactor: remove unneeded classes

Signed-off-by: Chris Laprun <[email protected]>

* fix: typo in package name

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
metacosm added a commit that referenced this pull request Aug 29, 2024
… based on type (#2483)

* refactor: integration tests moved to feature packages and sub packages pased on type

Signed-off-by: Attila Mészáros <[email protected]>

* fix resource paths

Signed-off-by: Attila Mészáros <[email protected]>

* fixes

Signed-off-by: Attila Mészáros <[email protected]>

* link and test fixes

Signed-off-by: Attila Mészáros <[email protected]>

* format

Signed-off-by: Attila Mészáros <[email protected]>

* wip

Signed-off-by: Attila Mészáros <[email protected]>

* old service

Signed-off-by: Attila Mészáros <[email protected]>

* test fix

Signed-off-by: Attila Mészáros <[email protected]>

* refactor: remove unneeded classes

Signed-off-by: Chris Laprun <[email protected]>

* fix: typo in package name

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
csviri added a commit that referenced this pull request Sep 20, 2024
… based on type (#2483)

* refactor: integration tests moved to feature packages and sub packages pased on type

Signed-off-by: Attila Mészáros <[email protected]>

* fix resource paths

Signed-off-by: Attila Mészáros <[email protected]>

* fixes

Signed-off-by: Attila Mészáros <[email protected]>

* link and test fixes

Signed-off-by: Attila Mészáros <[email protected]>

* format

Signed-off-by: Attila Mészáros <[email protected]>

* wip

Signed-off-by: Attila Mészáros <[email protected]>

* old service

Signed-off-by: Attila Mészáros <[email protected]>

* test fix

Signed-off-by: Attila Mészáros <[email protected]>

* refactor: remove unneeded classes

Signed-off-by: Chris Laprun <[email protected]>

* fix: typo in package name

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
metacosm added a commit that referenced this pull request Oct 10, 2024
… based on type (#2483)

* refactor: integration tests moved to feature packages and sub packages pased on type

Signed-off-by: Attila Mészáros <[email protected]>

* fix resource paths

Signed-off-by: Attila Mészáros <[email protected]>

* fixes

Signed-off-by: Attila Mészáros <[email protected]>

* link and test fixes

Signed-off-by: Attila Mészáros <[email protected]>

* format

Signed-off-by: Attila Mészáros <[email protected]>

* wip

Signed-off-by: Attila Mészáros <[email protected]>

* old service

Signed-off-by: Attila Mészáros <[email protected]>

* test fix

Signed-off-by: Attila Mészáros <[email protected]>

* refactor: remove unneeded classes

Signed-off-by: Chris Laprun <[email protected]>

* fix: typo in package name

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
metacosm added a commit that referenced this pull request Nov 5, 2024
… based on type (#2483)

* refactor: integration tests moved to feature packages and sub packages pased on type

Signed-off-by: Attila Mészáros <[email protected]>

* fix resource paths

Signed-off-by: Attila Mészáros <[email protected]>

* fixes

Signed-off-by: Attila Mészáros <[email protected]>

* link and test fixes

Signed-off-by: Attila Mészáros <[email protected]>

* format

Signed-off-by: Attila Mészáros <[email protected]>

* wip

Signed-off-by: Attila Mészáros <[email protected]>

* old service

Signed-off-by: Attila Mészáros <[email protected]>

* test fix

Signed-off-by: Attila Mészáros <[email protected]>

* refactor: remove unneeded classes

Signed-off-by: Chris Laprun <[email protected]>

* fix: typo in package name

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
metacosm added a commit that referenced this pull request Nov 6, 2024
… based on type (#2483)

* refactor: integration tests moved to feature packages and sub packages pased on type

Signed-off-by: Attila Mészáros <[email protected]>

* fix resource paths

Signed-off-by: Attila Mészáros <[email protected]>

* fixes

Signed-off-by: Attila Mészáros <[email protected]>

* link and test fixes

Signed-off-by: Attila Mészáros <[email protected]>

* format

Signed-off-by: Attila Mészáros <[email protected]>

* wip

Signed-off-by: Attila Mészáros <[email protected]>

* old service

Signed-off-by: Attila Mészáros <[email protected]>

* test fix

Signed-off-by: Attila Mészáros <[email protected]>

* refactor: remove unneeded classes

Signed-off-by: Chris Laprun <[email protected]>

* fix: typo in package name

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
csviri added a commit that referenced this pull request Nov 13, 2024
… based on type (#2483)

* refactor: integration tests moved to feature packages and sub packages pased on type

Signed-off-by: Attila Mészáros <[email protected]>

* fix resource paths

Signed-off-by: Attila Mészáros <[email protected]>

* fixes

Signed-off-by: Attila Mészáros <[email protected]>

* link and test fixes

Signed-off-by: Attila Mészáros <[email protected]>

* format

Signed-off-by: Attila Mészáros <[email protected]>

* wip

Signed-off-by: Attila Mészáros <[email protected]>

* old service

Signed-off-by: Attila Mészáros <[email protected]>

* test fix

Signed-off-by: Attila Mészáros <[email protected]>

* refactor: remove unneeded classes

Signed-off-by: Chris Laprun <[email protected]>

* fix: typo in package name

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
metacosm added a commit that referenced this pull request Nov 19, 2024
… based on type (#2483)

* refactor: integration tests moved to feature packages and sub packages pased on type

Signed-off-by: Attila Mészáros <[email protected]>

* fix resource paths

Signed-off-by: Attila Mészáros <[email protected]>

* fixes

Signed-off-by: Attila Mészáros <[email protected]>

* link and test fixes

Signed-off-by: Attila Mészáros <[email protected]>

* format

Signed-off-by: Attila Mészáros <[email protected]>

* wip

Signed-off-by: Attila Mészáros <[email protected]>

* old service

Signed-off-by: Attila Mészáros <[email protected]>

* test fix

Signed-off-by: Attila Mészáros <[email protected]>

* refactor: remove unneeded classes

Signed-off-by: Chris Laprun <[email protected]>

* fix: typo in package name

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
metacosm added a commit that referenced this pull request Nov 20, 2024
… based on type (#2483)

* refactor: integration tests moved to feature packages and sub packages pased on type

Signed-off-by: Attila Mészáros <[email protected]>

* fix resource paths

Signed-off-by: Attila Mészáros <[email protected]>

* fixes

Signed-off-by: Attila Mészáros <[email protected]>

* link and test fixes

Signed-off-by: Attila Mészáros <[email protected]>

* format

Signed-off-by: Attila Mészáros <[email protected]>

* wip

Signed-off-by: Attila Mészáros <[email protected]>

* old service

Signed-off-by: Attila Mészáros <[email protected]>

* test fix

Signed-off-by: Attila Mészáros <[email protected]>

* refactor: remove unneeded classes

Signed-off-by: Chris Laprun <[email protected]>

* fix: typo in package name

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
metacosm added a commit that referenced this pull request Nov 27, 2024
… based on type (#2483)

* refactor: integration tests moved to feature packages and sub packages pased on type

Signed-off-by: Attila Mészáros <[email protected]>

* fix resource paths

Signed-off-by: Attila Mészáros <[email protected]>

* fixes

Signed-off-by: Attila Mészáros <[email protected]>

* link and test fixes

Signed-off-by: Attila Mészáros <[email protected]>

* format

Signed-off-by: Attila Mészáros <[email protected]>

* wip

Signed-off-by: Attila Mészáros <[email protected]>

* old service

Signed-off-by: Attila Mészáros <[email protected]>

* test fix

Signed-off-by: Attila Mészáros <[email protected]>

* refactor: remove unneeded classes

Signed-off-by: Chris Laprun <[email protected]>

* fix: typo in package name

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
csviri added a commit that referenced this pull request Dec 6, 2024
… based on type (#2483)

* refactor: integration tests moved to feature packages and sub packages pased on type

Signed-off-by: Attila Mészáros <[email protected]>

* fix resource paths

Signed-off-by: Attila Mészáros <[email protected]>

* fixes

Signed-off-by: Attila Mészáros <[email protected]>

* link and test fixes

Signed-off-by: Attila Mészáros <[email protected]>

* format

Signed-off-by: Attila Mészáros <[email protected]>

* wip

Signed-off-by: Attila Mészáros <[email protected]>

* old service

Signed-off-by: Attila Mészáros <[email protected]>

* test fix

Signed-off-by: Attila Mészáros <[email protected]>

* refactor: remove unneeded classes

Signed-off-by: Chris Laprun <[email protected]>

* fix: typo in package name

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
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.

Refactor integration tests to use feature packages and topic based tree structure
2 participants