-
Notifications
You must be signed in to change notification settings - Fork 70
Closed
Labels
good first issueDenotes an issue ready for a new contributor, according to the "help wanted" guidelines.Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.
Description
In this test we delete the test catalog:
operator-controller/test/e2e/install_test.go
Lines 145 to 146 in 7f61f6d
| // Delete the catalog first | |
| Expect(c.Delete(ctx, operatorCatalog)).To(Succeed()) |
And later recreate it here:
operator-controller/test/e2e/install_test.go
Lines 163 to 164 in 7f61f6d
| operatorCatalog, err = createTestCatalog(ctx, testCatalogName, os.Getenv(testCatalogRefEnvVar)) | |
| Expect(err).ToNot(HaveOccurred()) |
The issue is that - if any of the assertions before catalog recreation fails all tests which use the test catalog will also fail.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueDenotes an issue ready for a new contributor, according to the "help wanted" guidelines.Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.