diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/domain/TestFrameworkModule.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/domain/TestFrameworkModule.java index 37b0c649588..b25f7145862 100644 --- a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/domain/TestFrameworkModule.java +++ b/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); diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/domain/buildsystem/ProxyTestModule.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/domain/buildsystem/ProxyTestModule.java index bd50ba88dc3..5603bab9c20 100644 --- a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/domain/buildsystem/ProxyTestModule.java +++ b/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); diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/domain/headless/HeadlessTestModule.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/domain/headless/HeadlessTestModule.java index 2d8a66135e9..145a5a25ff8 100644 --- a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/domain/headless/HeadlessTestModule.java +++ b/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); diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/events/NoOpTestEventsHandler.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/events/NoOpTestEventsHandler.java index 99ac9994829..b6852d4ffba 100644 --- a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/events/NoOpTestEventsHandler.java +++ b/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) { diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/events/TestEventsHandlerImpl.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/events/TestEventsHandlerImpl.java index 4a93eab7600..58487a15459 100644 --- a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/events/TestEventsHandlerImpl.java +++ b/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) { diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/test/ExecutionStrategy.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/test/ExecutionStrategy.java index 03717942cad..7cdacf572fc 100644 --- a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/test/ExecutionStrategy.java +++ b/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()); } diff --git a/internal-api/src/main/java/datadog/trace/api/civisibility/events/TestEventsHandler.java b/internal-api/src/main/java/datadog/trace/api/civisibility/events/TestEventsHandler.java index 65b4dd15abf..4709fa25a05 100644 --- a/internal-api/src/main/java/datadog/trace/api/civisibility/events/TestEventsHandler.java +++ b/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();