conformance/discovery: delete all manifests before attempting to dele…

…te blobs, disregard 404 responses Signed-off-by: wayne warren <wayne.warren.s@gmail.com>
opencontainers · Oct 12, 2023 · 5afc3ee · 5afc3ee
1 parent d164db1
commit 5afc3ee
Showing 1 changed file with 47 additions and 20 deletions.
diff --git a/conformance/03_discovery_test.go b/conformance/03_discovery_test.go
@@ -352,16 +352,29 @@ var test03ContentDiscovery = func() {
 				g.Specify("Delete created manifest & associated tags", func() {
 					SkipIfDisabled(contentDiscovery)
 					RunOnlyIf(runContentDiscoverySetup)
-					req := client.NewRequest(reggie.DELETE, "/v2/<name>/manifests/<digest>", reggie.WithDigest(manifests[2].Digest))
-					resp, err := client.Do(req)
-					Expect(err).To(BeNil())
-					Expect(resp.StatusCode()).To(SatisfyAny(
-						SatisfyAll(
-							BeNumerically(">=", 200),
-							BeNumerically("<", 300),
-						),
-						Equal(http.StatusMethodNotAllowed),
-					))
+					references := []string{
+						refsIndexArtifactDigest,
+						manifests[2].Digest,
+						refsManifestAConfigArtifactDigest,
+						refsManifestAConfigArtifactDigest,
+						refsManifestALayerArtifactDigest,
+						testTagName,
+						refsManifestBConfigArtifactDigest,
+						refsManifestBLayerArtifactDigest,
+					}
+					for _, ref := range references {
+						req := client.NewRequest(reggie.DELETE, "/v2/<name>/manifests/<digest>", reggie.WithDigest(ref))
+						resp, err := client.Do(req)
+						Expect(err).To(BeNil())
+						Expect(resp.StatusCode()).To(SatisfyAny(
+							SatisfyAll(
+								BeNumerically(">=", 200),
+								BeNumerically("<", 300),
+							),
+							Equal(http.StatusMethodNotAllowed),
+							Equal(http.StatusNotFound),
+						))
+					}
 				})
 			}
 
@@ -399,16 +412,29 @@ var test03ContentDiscovery = func() {
 				g.Specify("Delete created manifest & associated tags", func() {
 					SkipIfDisabled(contentDiscovery)
 					RunOnlyIf(runContentDiscoverySetup)
-					req := client.NewRequest(reggie.DELETE, "/v2/<name>/manifests/<digest>", reggie.WithDigest(manifests[2].Digest))
-					resp, err := client.Do(req)
-					Expect(err).To(BeNil())
-					Expect(resp.StatusCode()).To(SatisfyAny(
-						SatisfyAll(
-							BeNumerically(">=", 200),
-							BeNumerically("<", 300),
-						),
-						Equal(http.StatusMethodNotAllowed),
-					))
+					references := []string{
+						refsIndexArtifactDigest,
+						manifests[2].Digest,
+						refsManifestAConfigArtifactDigest,
+						refsManifestAConfigArtifactDigest,
+						refsManifestALayerArtifactDigest,
+						testTagName,
+						refsManifestBConfigArtifactDigest,
+						refsManifestBLayerArtifactDigest,
+					}
+					for _, ref := range references {
+						req := client.NewRequest(reggie.DELETE, "/v2/<name>/manifests/<digest>", reggie.WithDigest(ref))
+						resp, err := client.Do(req)
+						Expect(err).To(BeNil())
+						Expect(resp.StatusCode()).To(SatisfyAny(
+							SatisfyAll(
+								BeNumerically(">=", 200),
+								BeNumerically("<", 300),
+							),
+							Equal(http.StatusMethodNotAllowed),
+							Equal(http.StatusNotFound),
+						))
+					}
 				})
 			}
 
@@ -425,6 +451,7 @@ var test03ContentDiscovery = func() {
 							BeNumerically("<", 300),
 						),
 						Equal(http.StatusMethodNotAllowed),
+						Equal(http.StatusNotFound),
 					))
 				}