move test management check to isQuarantined

Author: daniel-mohedano

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/test/ExecutionStrategy.java

@@ -8,6 +8,7 @@
 import datadog.trace.api.civisibility.telemetry.tag.SkipReason;
 import datadog.trace.civisibility.config.EarlyFlakeDetectionSettings;
 import datadog.trace.civisibility.config.ExecutionSettings;
+import datadog.trace.civisibility.config.TestManagementSettings;
 import datadog.trace.civisibility.execution.Regular;
 import datadog.trace.civisibility.execution.RetryUntilSuccessful;
 import datadog.trace.civisibility.execution.RunNTimes;
@@ -62,6 +63,10 @@ public boolean isFlaky(TestIdentifier test) {
   }
 
   public boolean isQuarantined(TestIdentifier test) {
+    TestManagementSettings testManagementSettings = executionSettings.getTestManagementSettings();
+    if (!testManagementSettings.isEnabled()) {
+      return false;
+    }
     Collection<TestIdentifier> quarantinedTests = executionSettings.getQuarantinedTests();
     return quarantinedTests.contains(test.withoutParameters());
   }
@@ -105,7 +110,7 @@ public TestExecutionPolicy executionPolicy(TestIdentifier test, TestSourceData t
           config.getCiVisibilityFlakyRetryCount(), isQuarantined(test), autoRetriesUsed);
     }
 
-    if (executionSettings.getTestManagementSettings().isEnabled() && isQuarantined(test)) {
+    if (isQuarantined(test)) {
       return new RunOnceIgnoreOutcome();
     }