Skip to content

feat(interu): Add --check-test-definitions flag#30

Merged
Techassi merged 4 commits into
mainfrom
feat/run-integration-test-check-test-options
Jan 14, 2025
Merged

feat(interu): Add --check-test-definitions flag#30
Techassi merged 4 commits into
mainfrom
feat/run-integration-test-check-test-options

Conversation

@Techassi
Copy link
Copy Markdown
Member

@Techassi Techassi commented Jan 13, 2025

Part of stackabletech/issues#659

This adds the --check-test-definitions flag which validates that the test or test-suite is defined in the tests/test-definition.yaml file. The default file path can be overridden by using --test-definitions.

Comment thread tools/interu/src/config/tests.rs
Comment thread tools/interu/fixtures/interu.yaml
NickLarsenNZ
NickLarsenNZ previously approved these changes Jan 13, 2025
Copy link
Copy Markdown
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

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

LGTM, left a suggestion

Comment thread tools/interu/src/cli.rs Outdated
Comment thread tools/interu/src/config/profile.rs Outdated
NickLarsenNZ
NickLarsenNZ previously approved these changes Jan 13, 2025
Copy link
Copy Markdown
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

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

LGTM

@Techassi Techassi changed the title feat(interu): Add --check-test-options flag feat(interu): Add --check-test-definitions flag Jan 14, 2025
Copy link
Copy Markdown
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

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

LGTM

@Techassi Techassi merged commit 0dce603 into main Jan 14, 2025
@Techassi Techassi deleted the feat/run-integration-test-check-test-options branch January 14, 2025 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants