Avoid REPOSITORY_ANALYSIS purpose

DaveCTurner · DaveCTurner · commit bf2563075105 · 2025-07-02T12:54:41.000+01:00
diff --git a/test/framework/src/main/java/org/elasticsearch/repositories/blobstore/ESBlobStoreRepositoryIntegTestCase.java b/test/framework/src/main/java/org/elasticsearch/repositories/blobstore/ESBlobStoreRepositoryIntegTestCase.java
@@ -230,7 +230,11 @@ public void testDeleteBlobs() throws IOException {
             assertEquals(container.listBlobs(randomPurpose()).size(), 2);
             container.deleteBlobsIgnoringIfNotExists(randomPurpose(), blobNames.iterator());
             assertTrue(container.listBlobs(randomPurpose()).isEmpty());
-            container.deleteBlobsIgnoringIfNotExists(randomPurpose(), blobNames.iterator()); // does not raise when blobs don't exist
+            container.deleteBlobsIgnoringIfNotExists(
+                // does not raise when blobs don't exist, except for REPOSITORY_ANALYSIS which is strict
+                randomValueOtherThan(OperationPurpose.REPOSITORY_ANALYSIS, BlobStoreTestUtil::randomPurpose),
+                blobNames.iterator()
+            );
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -230,7 +230,11 @@ public void testDeleteBlobs() throws IOException {`
`230`	`230`	`assertEquals(container.listBlobs(randomPurpose()).size(), 2);`
`231`	`231`	`container.deleteBlobsIgnoringIfNotExists(randomPurpose(), blobNames.iterator());`
`232`	`232`	`assertTrue(container.listBlobs(randomPurpose()).isEmpty());`
`233`		`- container.deleteBlobsIgnoringIfNotExists(randomPurpose(), blobNames.iterator()); // does not raise when blobs don't exist`
	`233`	`+ container.deleteBlobsIgnoringIfNotExists(`
	`234`	`+ // does not raise when blobs don't exist, except for REPOSITORY_ANALYSIS which is strict`
	`235`	`+ randomValueOtherThan(OperationPurpose.REPOSITORY_ANALYSIS, BlobStoreTestUtil::randomPurpose),`
	`236`	`+ blobNames.iterator()`
	`237`	`+ );`
`234`	`238`	`}`
`235`	`239`	`}`
`236`	`240`