Skip to content
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

Change aware tests #1727

Open
bharathkkb opened this issue Jul 25, 2023 · 3 comments
Open

Change aware tests #1727

bharathkkb opened this issue Jul 25, 2023 · 3 comments
Assignees
Labels

Comments

@bharathkkb
Copy link
Member

Currently we run all tests for each PR irrespective of which files have changed. It would be nice if we can test just what was modified based on changed files.

Each blueprint test knows which root module it actuates so an initial version of this could do a git diff and understand if changed files affect current module and t.Skip if is not changed. However we likely need some rules like any changes to a module/submodule should override this heuristic and run all tests regardless.

Further if we can become config aware, i.e example foo which is the test root module uses module bar, we could restrict the tests that way too.

@bharathkkb
Copy link
Member Author

@rogerthatdev also suggested doing this for lint tests

@msampathkumar
Copy link
Member

+1, this will reduce our (terraform-docs-samples GH) CICD from hrs to mins.

@bharathkkb
Copy link
Member Author

Awais is exploring this area

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants