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

use exact regex expressions for test names #230

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

anniehedgpeth
Copy link
Contributor

@anniehedgpeth anniehedgpeth commented Oct 9, 2023

Background

TF-8610
Right now the tests use the contain function for the test names, which causes more tests to run than you desire. This will force the use to use the exact name but only the test they want to run will run. The downside is that you can't put notes in that comment because it will cause it to not match exactly, but the user can simply add a follow up comment instead.

How Has This Been Tested

Will be tested post-merge.

@anniehedgpeth anniehedgpeth requested a review from a team as a code owner October 9, 2023 15:06
@anniehedgpeth anniehedgpeth merged commit 88da84c into main Oct 9, 2023
3 checks passed
@anniehedgpeth anniehedgpeth deleted the ah/TF-8610-testnames branch October 9, 2023 18:08
@anniehedgpeth
Copy link
Contributor Author

anniehedgpeth commented Oct 9, 2023

/help

Command Description
/test <all|test case name...> [destroy=false] Run the Terraform test workflow on the modules in the tests/ directory. Unnamed arguments can be "all" to run all test cases or specific test case names to only run selected cases. The named argument "destroy=false" will disable the destruction of test infrastructure for debugging purposes.
/destroy <all|test case name...> Destroy any resources that may still be in Terraform state from previous tests. Unnamed arguments can be "all" to destroy all resources from all test cases or specific test case names to only destroy selected test case resources.
/help Shows this help message

Test Case Names

FDO:

  • private-active-active
  • private-tcp-active-active
  • public-active-active
  • standalone-external
  • standalone-mounted-disk

Replicated:

  • private-active-active-replicated
  • private-tcp-active-active-replicated
  • public-active-active-replicated
  • standalone-external-replicated
  • standalone-mounted-disk-replicated

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.

2 participants