Skip to content

Conversation

@chudilka1
Copy link
Contributor

@chudilka1 chudilka1 commented Aug 27, 2025

CRE-764: CRE v1/v2 system tests refactoring

  1. Added test suite with tests grouped by CRE version.
  2. Pre-conditions, such as test environment setup, moved to the before_suite.go.
  3. Externalised tests to dedicated test files.
  4. Moved duplications and reused test steps to the helpers_test.go.
  5. Utility methods used by tests moved to the corresponding system-tests/lib to preserve the Single Responsibility Principle.

@chudilka1 chudilka1 force-pushed the CRE-764-system-tests-refactoring branch 10 times, most recently from 88c1487 to 8df931f Compare August 28, 2025 08:40
@chudilka1 chudilka1 changed the title Cre 764 system tests refactoring [CRE-764] System tests refactoring Aug 28, 2025
@chudilka1 chudilka1 force-pushed the CRE-764-system-tests-refactoring branch 2 times, most recently from 42879f9 to 975192a Compare August 28, 2025 09:49
@chudilka1 chudilka1 requested a review from Tofel August 28, 2025 09:50
@chudilka1 chudilka1 force-pushed the CRE-764-system-tests-refactoring branch 7 times, most recently from d111e54 to c14dd06 Compare August 28, 2025 15:45
@chudilka1 chudilka1 marked this pull request as ready for review August 28, 2025 15:46
@chudilka1 chudilka1 requested review from a team as code owners August 28, 2025 15:46
@chudilka1 chudilka1 requested a review from Tofel August 28, 2025 15:46
@chudilka1 chudilka1 requested review from krehermann and removed request for patrickhuie19 August 28, 2025 15:47
@chudilka1 chudilka1 force-pushed the CRE-764-system-tests-refactoring branch 5 times, most recently from e64bc6f to 39434e1 Compare August 29, 2025 11:00
Refactoring: move re-used common utility methods in the tests to smoke-tests/cmd package

Refactoring: move re-used logically grouped steps to helpers_test.go

Refactoring: externalize CREv1 PoR smoke-test to a separate file.

Refactoring: externalize CREv1 PoR smoke-test to a separate file.

Refactoring: externalize CREv2 smoke-tests to separate files.

Refactoring: update system-test files names and README.
@chudilka1 chudilka1 force-pushed the CRE-764-system-tests-refactoring branch 4 times, most recently from e2613e6 to 07b8004 Compare August 29, 2025 13:14
@chudilka1 chudilka1 requested a review from Tofel August 29, 2025 13:17
@chudilka1 chudilka1 force-pushed the CRE-764-system-tests-refactoring branch from 07b8004 to ed3f677 Compare August 29, 2025 13:21
Tofel
Tofel previously approved these changes Aug 29, 2025
Copy link
Contributor

@Tofel Tofel left a comment

Choose a reason for hiding this comment

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

nice job!

@cl-sonarqube-production
Copy link

@Tofel Tofel added this pull request to the merge queue Aug 29, 2025
Merged via the queue into develop with commit b63d9a3 Aug 29, 2025
152 of 154 checks passed
@Tofel Tofel deleted the CRE-764-system-tests-refactoring branch August 29, 2025 14:49
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.

3 participants