fix: improve ac tests

thousandeyes · May 10, 2024 · cf60211 · cf60211
1 parent 03f213c
commit cf60211
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/thousandeyes/resource_alert_rule_test.go b/thousandeyes/resource_alert_rule_test.go
@@ -40,10 +40,17 @@ func TestAccThousandEyesAlertRule(t *testing.T) {
 				return testAccCheckResourceDestroy(resourceList, state)
 			},
 			checkCreateFunc: []resource.TestCheckFunc{
+				//alert rule is created
+				//with 3 required violating rounds
 				resource.TestCheckResourceAttr(alertRuleResourceName, "rounds_violating_required", "3"),
+				//and with one tests associated
+				resource.TestCheckResourceAttr(alertRuleResourceName, "test_ids.#", "1"),
 			},
 			checkUpdateFunc: []resource.TestCheckFunc{
+				//alert rule is updated
+				//to 4 required violating rounds
 				resource.TestCheckResourceAttr(alertRuleResourceName, "rounds_violating_required", "4"),
+				//and the test association is maintained
 				resource.TestCheckResourceAttr(alertRuleResourceName, "test_ids.#", "1"),
 			},
 		},