Skip to content

Commit 2f0dcd6

Browse files
daniel-mohedanodevflow.devflow-routing-intake
andauthored
Improve test coverage for CiVis classes (#11768)
fix: ExecutionAggregation coverage fix: TestExecutionPolicy coverage Co-authored-by: devflow.devflow-routing-intake <devflow.devflow-routing-intake@kubernetes.us1.ddbuild.io>
1 parent 673986e commit 2f0dcd6

2 files changed

Lines changed: 55 additions & 0 deletions

File tree

internal-api/build.gradle.kts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -197,6 +197,7 @@ extra["excludedClassesCoverage"] = listOf(
197197
"datadog.trace.api.cache.FixedSizeCache.IdentityHash",
198198
"datadog.trace.api.cache.FixedSizeWeakKeyCache",
199199
// Interface with default method
200+
"datadog.trace.api.civisibility.execution.TestExecutionPolicy",
200201
"datadog.trace.api.iast.Taintable",
201202
"datadog.trace.api.Stateful",
202203
"datadog.trace.api.Stateful.1",
Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,54 @@
1+
package datadog.trace.api.civisibility.execution;
2+
3+
import static datadog.trace.api.civisibility.execution.ExecutionAggregation.MIXED;
4+
import static datadog.trace.api.civisibility.execution.ExecutionAggregation.NONE;
5+
import static datadog.trace.api.civisibility.execution.ExecutionAggregation.ONLY_FAILED;
6+
import static datadog.trace.api.civisibility.execution.ExecutionAggregation.ONLY_PASSED;
7+
import static org.junit.jupiter.api.Assertions.assertEquals;
8+
9+
import org.junit.jupiter.api.Test;
10+
11+
class ExecutionAggregationTest {
12+
13+
@Test
14+
void noneAfterFailIsOnlyFailed() {
15+
assertEquals(ONLY_FAILED, NONE.withExecution(TestStatus.fail));
16+
}
17+
18+
@Test
19+
void noneAfterPassIsOnlyPassed() {
20+
assertEquals(ONLY_PASSED, NONE.withExecution(TestStatus.pass));
21+
}
22+
23+
@Test
24+
void noneAfterSkipIsOnlyPassed() {
25+
assertEquals(ONLY_PASSED, NONE.withExecution(TestStatus.skip));
26+
}
27+
28+
@Test
29+
void onlyFailedAfterFailStaysOnlyFailed() {
30+
assertEquals(ONLY_FAILED, ONLY_FAILED.withExecution(TestStatus.fail));
31+
}
32+
33+
@Test
34+
void onlyFailedAfterPassIsMixed() {
35+
assertEquals(MIXED, ONLY_FAILED.withExecution(TestStatus.pass));
36+
}
37+
38+
@Test
39+
void onlyPassedAfterPassStaysOnlyPassed() {
40+
assertEquals(ONLY_PASSED, ONLY_PASSED.withExecution(TestStatus.pass));
41+
}
42+
43+
@Test
44+
void onlyPassedAfterFailIsMixed() {
45+
assertEquals(MIXED, ONLY_PASSED.withExecution(TestStatus.fail));
46+
}
47+
48+
@Test
49+
void mixedAfterAnyStatusStaysMixed() {
50+
assertEquals(MIXED, MIXED.withExecution(TestStatus.pass));
51+
assertEquals(MIXED, MIXED.withExecution(TestStatus.fail));
52+
assertEquals(MIXED, MIXED.withExecution(TestStatus.skip));
53+
}
54+
}

0 commit comments

Comments
 (0)