Skip to content

Commit 93dfdde

Browse files
committed
investigate: test passes even without the proper SA permissions
The "reset namespaces" test passes even though the PR which adds the "member" service account the, in theory, required permission to delete namespaces has not been merged. The test does not pass when running the tests locally, and in stage and production the feature does not work without that permission. SANDBOX-1067
1 parent 97cf9ce commit 93dfdde

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

test/e2e/namespaces_reset_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -107,4 +107,6 @@ func (s *namespaceResetFeatureIntegrationSuite) TestResetNamespaces() {
107107

108108
require.NotEqual(s.T(), timestamp, fetchedNamespace.CreationTimestamp.Time, `the namespace "%s" appears to not have been recreated due to having the same creation timestamp as before`, fetchedNamespace.Name)
109109
}
110+
111+
require.FailNow(s.T(), "the test succeeded but we are failing to gather information")
110112
}

0 commit comments

Comments
 (0)