Skip to content

Commit

Permalink
Prevent CI to be run for whole collection (#728) (#732)
Browse files Browse the repository at this point in the history
[PR #728/6d0e2948 backport][stable-3] Prevent CI to be run for whole collection

This is a backport of PR #728 as merged into main (6d0e294).
SUMMARY

Add a fake integration suite including an aliases file listing every module name with missing integration tests.
This fake suite is necessary for the new CI ansible-test-splitter behaviour. If one of the modules (listed in the aliases file) without a test suite is modified, the CI is run for the entire collection since the ansible-test-splitter won't find any target match. This fake integration suite helps handle this situation by avoiding running the CI for the whole collection. Furthermore, since the modules listed in the aliases file are marked as disabled, tests are automatically skipped.
Track Issue: #729

ISSUE TYPE


Feature Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Jill R <None>
  • Loading branch information
patchback[bot] authored Mar 25, 2022
1 parent 6d87ac7 commit 2e8ab93
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 0 deletions.
5 changes: 5 additions & 0 deletions tests/integration/targets/legacy_missing_tests/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
## Fake integration suite

This is a fake integration suite including an aliases file listing every module name with missing integration tests (some of them are covered by units).

This fake suite is necessary for the new CI ansible-test-splitter behaviour. Namely, if one of the modules (listed in the aliases file) without a test suite is modified, the CI is run for the entire collection since the ansible-test-splitter won't find any target match. This fake integration suite helps handle this situation by avoiding running the CI for the whole collection. Furthermore, since the modules listed in the aliases file are marked as disabled, tests are automatically skipped.
7 changes: 7 additions & 0 deletions tests/integration/targets/legacy_missing_tests/aliases
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
disabled

# Lookup plugins
aws_ssm # covered by unit tests

# Callback plugin
aws_resource_actions

0 comments on commit 2e8ab93

Please sign in to comment.