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 #126

Merged
merged 3 commits into from
Oct 9, 2023
Merged

Conversation

anniehedgpeth
Copy link
Contributor

@anniehedgpeth anniehedgpeth commented Oct 9, 2023

Background

As with this change to the workflow in the Azure TFE module, I am removing the contains function to more strictly enforce the correct regex for the test you are choosing to run. This is due to the doubling up of tests since FDO and that they are similarly named.

How has this been tested?

The functionality worked in this PR, but I will test again post merge.

TFE Modules

Did you add a new setting?

No

@anniehedgpeth anniehedgpeth marked this pull request as draft October 9, 2023 22:43
@anniehedgpeth
Copy link
Contributor Author

anniehedgpeth commented Oct 9, 2023

/help

Command Description
/test <all|full-aws|full-azure|full-google|test case name...> [destroy=false] Run the Terraform test workflow on the modules in the tests/ directory in each of the terraform-aws-terraform-enterprise, terraform-azurerm-terraform-enterprise, and terraform-google-terraform-enterprise module repositories. Unnamed arguments can be "all" to run all test cases, full-[aws|azure|google] to run all cloud specific 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|full-aws|full-azure|full-google|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, full-[aws|azure|google] to destroy all resources from cloud specific cases, or specific test case names to only destroy selected test case resources.
/help Shows this help message

@anniehedgpeth anniehedgpeth changed the title test use exact regex expressions for test names Oct 9, 2023
@anniehedgpeth anniehedgpeth marked this pull request as ready for review October 9, 2023 23:23
@anniehedgpeth anniehedgpeth requested a review from a team as a code owner October 9, 2023 23:23
@anniehedgpeth anniehedgpeth merged commit 58cea4e into main Oct 9, 2023
2 checks passed
@anniehedgpeth anniehedgpeth deleted the ah/cleanup branch October 9, 2023 23:24
arnvid pushed a commit to arnvid/terraform-random-tfe-utility that referenced this pull request Nov 2, 2023
remove contains function from test name commands
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.

3 participants