Add a comment in tests
The purpose of the first test of enable_logging wasn't quite clear. It's future-proofing against the assumptions made about AWS behavior.
This commit is contained in:
parent
f98dbbb580
commit
c9eeb161e0
|
@ -51,6 +51,8 @@ func TestAccAWSCloudTrail_enable_logging(t *testing.T) {
|
||||||
Config: testAccAWSCloudTrailConfig,
|
Config: testAccAWSCloudTrailConfig,
|
||||||
Check: resource.ComposeTestCheckFunc(
|
Check: resource.ComposeTestCheckFunc(
|
||||||
testAccCheckCloudTrailExists("aws_cloudtrail.foobar", &trail),
|
testAccCheckCloudTrailExists("aws_cloudtrail.foobar", &trail),
|
||||||
|
// This is a warning test. AWS sets up new trails with logging disabled
|
||||||
|
// Should that change in the future, this test should fail.
|
||||||
testAccCheckCloudTrailLoggingEnabled("aws_cloudtrail.foobar", false, &trail),
|
testAccCheckCloudTrailLoggingEnabled("aws_cloudtrail.foobar", false, &trail),
|
||||||
),
|
),
|
||||||
},
|
},
|
||||||
|
|
Loading…
Reference in New Issue