Skip to content

Conversation

@tynorantoni
Copy link
Collaborator

Changes are introduced for removing unnecessary repetitions,
setting up tagged dependency versions and using existing functions
for detecting run architecture.

@tynorantoni tynorantoni requested a review from roxell November 17, 2025 12:36
@tynorantoni tynorantoni force-pushed the integration-test-refactor branch from 647e393 to 4a1240e Compare November 17, 2025 13:34
Introduced code for dynamic finding of lava test directory

Signed-off-by: Pawel Szymaszek <[email protected]>
@tynorantoni tynorantoni force-pushed the integration-test-refactor branch from 4a1240e to 14068a5 Compare November 17, 2025 16:53
…ction

Use detect_api with case syntax for cleaner approach
of finding right architecture for further testing

Signed-off-by: Pawel Szymaszek <[email protected]>
… GECKODRIVER, UV

Refactor code for using tagged version of required packages:
- spire
- uv
- geckodriver

Signed-off-by: Pawel Szymaszek <[email protected]>
Replace path repetitions with UTILS_PATH var for cleaner look

Signed-off-by: Pawel Szymaszek <[email protected]>
variable

Delete redundant git checkout cmd and fetch selected
branch directly with usage of BRANCH_NAME variable

Signed-off-by: Pawel Szymaszek <[email protected]>
@tynorantoni tynorantoni force-pushed the integration-test-refactor branch from 14068a5 to f65dd87 Compare November 18, 2025 06:07
@roxell roxell merged commit e9bffbe into master Nov 18, 2025
6 checks passed
@bhcopeland
Copy link
Member

The only comment I was going to leave was should use "" on the {} vars. Like the mv, and uv commands. Anything with $. Just to be safe from variable expansions, but only minor.

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.

4 participants