No need to set repoAnalysisStarted again

DaveCTurner · DaveCTurner · commit 0e58906bf915 · 2025-05-20T06:05:19.000+01:00
diff --git a/x-pack/plugin/snapshot-repo-test-kit/qa/s3/src/javaRestTest/java/org/elasticsearch/repositories/blobstore/testkit/analyze/S3RepositoryAnalysisRestIT.java b/x-pack/plugin/snapshot-repo-test-kit/qa/s3/src/javaRestTest/java/org/elasticsearch/repositories/blobstore/testkit/analyze/S3RepositoryAnalysisRestIT.java
@@ -66,8 +66,8 @@ private static void ensurePurposeParameterPresent(S3HttpHandler.S3Request reques
                 // getRepositoryData looking for root index-N blob will have different/missing x-purpose parameter
                 return;
             }
+            repoAnalysisStarted.set(true);
         }
-        repoAnalysisStarted.set(true);
         assertTrue(request.toString(), request.hasQueryParamOnce("x-purpose"));
         assertEquals(request.toString(), "RepositoryAnalysis", request.getQueryParamOnce("x-purpose"));
     }

Original file line number	Diff line number	Diff line change
`@@ -66,8 +66,8 @@ private static void ensurePurposeParameterPresent(S3HttpHandler.S3Request reques`
`66`	`66`	`// getRepositoryData looking for root index-N blob will have different/missing x-purpose parameter`
`67`	`67`	`return;`
`68`	`68`	`}`
	`69`	`+ repoAnalysisStarted.set(true);`
`69`	`70`	`}`
`70`		`- repoAnalysisStarted.set(true);`
`71`	`71`	`assertTrue(request.toString(), request.hasQueryParamOnce("x-purpose"));`
`72`	`72`	`assertEquals(request.toString(), "RepositoryAnalysis", request.getQueryParamOnce("x-purpose"));`
`73`	`73`	`}`