Remove unused TestEventsHandler methods

Author: nikita-tkachenko-datadog

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/domain/TestFrameworkModule.java

@@ -28,8 +28,6 @@ TestSuiteImpl testSuiteStart(
    */
   boolean isNew(@Nonnull TestIdentifier test);
 
-  boolean isFlaky(@Nonnull TestIdentifier test);
-
   boolean isModified(@Nonnull TestSourceData testSourceData);
 
   boolean isQuarantined(TestIdentifier test);

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/domain/buildsystem/ProxyTestModule.java

@@ -98,11 +98,6 @@ public boolean isNew(@Nonnull TestIdentifier test) {
     return executionStrategy.isNew(test);
   }
 
-  @Override
-  public boolean isFlaky(@Nonnull TestIdentifier test) {
-    return executionStrategy.isFlaky(test);
-  }
-
   @Override
   public boolean isModified(@Nonnull TestSourceData testSourceData) {
     return executionStrategy.isModified(testSourceData);

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/domain/headless/HeadlessTestModule.java

@@ -84,11 +84,6 @@ public boolean isNew(@Nonnull TestIdentifier test) {
     return executionStrategy.isNew(test);
   }
 
-  @Override
-  public boolean isFlaky(@Nonnull TestIdentifier test) {
-    return executionStrategy.isFlaky(test);
-  }
-
   @Override
   public boolean isModified(@Nonnull TestSourceData testSourceData) {
     return executionStrategy.isModified(testSourceData);

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/events/NoOpTestEventsHandler.java

@@ -105,16 +105,6 @@ public TestExecutionPolicy executionPolicy(
     return Regular.INSTANCE;
   }
 
-  @Override
-  public boolean isNew(@Nonnull TestIdentifier test) {
-    return false;
-  }
-
-  @Override
-  public boolean isFlaky(@Nonnull TestIdentifier test) {
-    return false;
-  }
-
   @Override
   public int executionPriority(
       @Nullable TestIdentifier test, @Nonnull TestSourceData testSourceData) {

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/events/TestEventsHandlerImpl.java

@@ -305,16 +305,6 @@ public int executionPriority(@Nullable TestIdentifier test, @Nonnull TestSourceD
     return testModule.executionPriority(test, testSource);
   }
 
-  @Override
-  public boolean isNew(@Nonnull TestIdentifier test) {
-    return testModule.isNew(test);
-  }
-
-  @Override
-  public boolean isFlaky(@Nonnull TestIdentifier test) {
-    return testModule.isFlaky(test);
-  }
-
   @Nullable
   @Override
   public SkipReason skipReason(TestIdentifier test) {

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

@@ -60,7 +60,7 @@ public boolean isNew(@Nonnull TestIdentifier test) {
         && !executionSettings.isKnown(test.toFQN());
   }
 
-  public boolean isFlaky(@Nonnull TestIdentifier test) {
+  private boolean isFlaky(@Nonnull TestIdentifier test) {
     return executionSettings.isFlaky(test.toFQN());
   }
 

internal-api/src/main/java/datadog/trace/api/civisibility/events/TestEventsHandler.java

@@ -107,10 +107,6 @@ TestExecutionPolicy executionPolicy(
   @Nullable
   SkipReason skipReason(TestIdentifier test);
 
-  boolean isNew(@Nonnull TestIdentifier test);
-
-  boolean isFlaky(@Nonnull TestIdentifier test);
-
   @Override
   void close();