Fix failing data-prepper-expression code coverage verfication (#5265)

Signed-off-by: Krishna Kondaka <[email protected]>
opensearch-project · Dec 14, 2024 · 956a89a · 956a89a
1 parent a12471c
commit 956a89a
Show file tree

Hide file tree

Showing 13 changed files with 13 additions and 0 deletions.
diff --git a/...epper-expression/src/test/java/org/opensearch/dataprepper/expression/AndOperatorTest.java b/...epper-expression/src/test/java/org/opensearch/dataprepper/expression/AndOperatorTest.java
@@ -35,6 +35,7 @@ void testShouldEvaluate() {
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(true));
         when(ctx.getRuleIndex()).thenReturn(-1);
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(false));
+        assertThat(objectUnderTest.isBooleanOperator(), is(true));
     }
 
     @Test

diff --git a/...per-expression/src/test/java/org/opensearch/dataprepper/expression/EqualOperatorTest.java b/...per-expression/src/test/java/org/opensearch/dataprepper/expression/EqualOperatorTest.java
@@ -36,6 +36,7 @@ void testShouldEvaluate() {
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(true));
         when(ctx.getRuleIndex()).thenReturn(-1);
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(false));
+        assertThat(objectUnderTest.isBooleanOperator(), is(true));
     }
 
     @Test

diff --git a/...xpression/src/test/java/org/opensearch/dataprepper/expression/GenericNotOperatorTest.java b/...xpression/src/test/java/org/opensearch/dataprepper/expression/GenericNotOperatorTest.java
@@ -68,6 +68,7 @@ void testGivenMatchingRuleThenReturnTrue() {
                 .getRuleIndex();
 
         assertThat(genericNotOperator.shouldEvaluate(context), is(true));
+        assertThat(genericNotOperator.isBooleanOperator(), is(true));
     }
 
     @Test

diff --git a/...pression/src/test/java/org/opensearch/dataprepper/expression/GreaterThanOperatorTest.java b/...pression/src/test/java/org/opensearch/dataprepper/expression/GreaterThanOperatorTest.java
@@ -35,6 +35,7 @@ void testShouldEvaluate() {
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(true));
         when(ctx.getRuleIndex()).thenReturn(-1);
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(false));
+        assertThat(objectUnderTest.isBooleanOperator(), is(true));
     }
 
     @Test

diff --git a/...n/src/test/java/org/opensearch/dataprepper/expression/GreaterThanOrEqualOperatorTest.java b/...n/src/test/java/org/opensearch/dataprepper/expression/GreaterThanOrEqualOperatorTest.java
@@ -35,6 +35,7 @@ void testShouldEvaluate() {
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(true));
         when(ctx.getRuleIndex()).thenReturn(-1);
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(false));
+        assertThat(objectUnderTest.isBooleanOperator(), is(true));
     }
 
     @Test

diff --git a/...-expression/src/test/java/org/opensearch/dataprepper/expression/LessThanOperatorTest.java b/...-expression/src/test/java/org/opensearch/dataprepper/expression/LessThanOperatorTest.java
@@ -35,6 +35,7 @@ void testShouldEvaluate() {
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(true));
         when(ctx.getRuleIndex()).thenReturn(-1);
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(false));
+        assertThat(objectUnderTest.isBooleanOperator(), is(true));
     }
 
     @Test

diff --git a/...sion/src/test/java/org/opensearch/dataprepper/expression/LessThanOrEqualOperatorTest.java b/...sion/src/test/java/org/opensearch/dataprepper/expression/LessThanOrEqualOperatorTest.java
@@ -35,6 +35,7 @@ void testShouldEvaluate() {
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(true));
         when(ctx.getRuleIndex()).thenReturn(-1);
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(false));
+        assertThat(objectUnderTest.isBooleanOperator(), is(true));
     }
 
     @Test

diff --git a/...-expression/src/test/java/org/opensearch/dataprepper/expression/NotInSetOperatorTest.java b/...-expression/src/test/java/org/opensearch/dataprepper/expression/NotInSetOperatorTest.java
@@ -38,6 +38,7 @@ void testShouldEvaluate() {
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(true));
         when(ctx.getRuleIndex()).thenReturn(-1);
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(false));
+        assertThat(objectUnderTest.isBooleanOperator(), is(true));
     }
 
     @Test

diff --git a/...epper-expression/src/test/java/org/opensearch/dataprepper/expression/NotOperatorTest.java b/...epper-expression/src/test/java/org/opensearch/dataprepper/expression/NotOperatorTest.java
@@ -35,6 +35,7 @@ void testShouldEvaluate() {
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(true));
         when(ctx.getRuleIndex()).thenReturn(-1);
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(false));
+        assertThat(objectUnderTest.isBooleanOperator(), is(true));
     }
 
     @Test

diff --git a/...repper-expression/src/test/java/org/opensearch/dataprepper/expression/OrOperatorTest.java b/...repper-expression/src/test/java/org/opensearch/dataprepper/expression/OrOperatorTest.java
@@ -35,6 +35,7 @@ void testShouldEvaluate() {
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(true));
         when(ctx.getRuleIndex()).thenReturn(-1);
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(false));
+        assertThat(objectUnderTest.isBooleanOperator(), is(true));
     }
 
     @Test

diff --git a/...xpression/src/test/java/org/opensearch/dataprepper/expression/RegexEqualOperatorTest.java b/...xpression/src/test/java/org/opensearch/dataprepper/expression/RegexEqualOperatorTest.java
@@ -36,6 +36,7 @@ void testShouldEvaluate() {
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(true));
         when(ctx.getRuleIndex()).thenReturn(-1);
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(false));
+        assertThat(objectUnderTest.isBooleanOperator(), is(true));
     }
 
     @Test

diff --git a/...ession/src/test/java/org/opensearch/dataprepper/expression/RegexNotEqualOperatorTest.java b/...ession/src/test/java/org/opensearch/dataprepper/expression/RegexNotEqualOperatorTest.java
@@ -36,6 +36,7 @@ void testShouldEvaluate() {
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(true));
         when(ctx.getRuleIndex()).thenReturn(-1);
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(false));
+        assertThat(objectUnderTest.isBooleanOperator(), is(true));
     }
 
     @Test

diff --git a/...er-expression/src/test/java/org/opensearch/dataprepper/expression/TypeOfOperatorTest.java b/...er-expression/src/test/java/org/opensearch/dataprepper/expression/TypeOfOperatorTest.java
@@ -44,6 +44,7 @@ void testShouldEvaluate() {
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(true));
         when(ctx.getRuleIndex()).thenReturn(-1);
         assertThat(objectUnderTest.shouldEvaluate(ctx), is(false));
+        assertThat(objectUnderTest.isBooleanOperator(), is(true));
     }
 
     @Test
-Original file line number
+Diff line change
@@ Expand Up / @@ -68,6 +68,7 @@ void testGivenMatchingRuleThenReturnTrue() { @@
                     .getRuleIndex();
             assertThat(genericNotOperator.shouldEvaluate(context), is(true));
+            assertThat(genericNotOperator.isBooleanOperator(), is(true));
         }
         @Test
@@ Expand Down @@