Skip to content

Commit

Permalink
test(policychecks): remove flaky executeCommand check #6009
Browse files Browse the repository at this point in the history
## Problem
Flaky tests: #5996

## Solution
The check for executeCommand is not important to these tests, and can be
removed.

fix #5996
  • Loading branch information
kevluu-aws authored Nov 13, 2024
1 parent 226636f commit 09a0db7
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -369,7 +369,6 @@ describe('customChecks', function () {
assert.deepStrictEqual(actualCommand.referencePolicyType, policyType)

assert(onCustomPolicyCheckResponseFireSpy.notCalled)
assert(executeCommandStub.notCalled)
})

it('checkNoNewAccess should handle CloudFormation document type correctly', async function () {
Expand Down Expand Up @@ -400,7 +399,6 @@ describe('customChecks', function () {
assert.deepStrictEqual(actualCommand.referencePolicyType, policyType)

assert(onCustomPolicyCheckResponseFireSpy.notCalled)
assert(executeCommandStub.notCalled)
})

it('checkNoNewAccess should handle missing reference document', async function () {
Expand Down

0 comments on commit 09a0db7

Please sign in to comment.