From 8ae5d2b9046e106d658679c7ceba817dac94212b Mon Sep 17 00:00:00 2001 From: Rian-Ismael <106334437+Rian-Ismael@users.noreply.github.com> Date: Thu, 30 Jan 2025 14:16:17 -0300 Subject: [PATCH] Update OrJanusPredicateTest.java Improves the readability of the test. The assertions are clearer and include descriptive messages, making it easier to identify which assertion fails. This enhances code maintainability, understanding and debugging efficiency. Signed-off-by: Rian-Ismael <106334437+Rian-Ismael@users.noreply.github.com> --- .../graphdb/predicate/OrJanusPredicateTest.java | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/janusgraph-test/src/test/java/org/janusgraph/graphdb/predicate/OrJanusPredicateTest.java b/janusgraph-test/src/test/java/org/janusgraph/graphdb/predicate/OrJanusPredicateTest.java index 2b869d2aea..b3e1f632f8 100644 --- a/janusgraph-test/src/test/java/org/janusgraph/graphdb/predicate/OrJanusPredicateTest.java +++ b/janusgraph-test/src/test/java/org/janusgraph/graphdb/predicate/OrJanusPredicateTest.java @@ -47,8 +47,10 @@ ConnectiveJanusPredicate getNegatePredicate(List childPredi @Test public void testIsQNF() { - assertTrue(getPredicate(Arrays.asList(Text.PREFIX, Cmp.EQUAL)).isQNF()); - assertTrue(getPredicate(Arrays.asList(Text.PREFIX, Cmp.EQUAL, new OrJanusPredicate(Arrays.asList(Text.PREFIX, Cmp.EQUAL)))).isQNF()); - assertFalse(getPredicate(Arrays.asList(Text.PREFIX, Cmp.EQUAL, new AndJanusPredicate(Arrays.asList(Text.PREFIX, Cmp.EQUAL)))).isQNF()); + assertTrue(getPredicate(Arrays.asList(Text.PREFIX, Cmp.EQUAL)).isQNF(), "Single PREFIX EQUAL should be QNF"); + assertTrue(getPredicate(Arrays.asList(Text.PREFIX, Cmp.EQUAL, new OrJanusPredicate(Arrays.asList(Text.PREFIX, Cmp.EQUAL)))).isQNF(), + "PREFIX EQUAL with OR(JANUS(PREFIX EQUAL)) should be QNF"); + assertFalse(getPredicate(Arrays.asList(Text.PREFIX, Cmp.EQUAL, new AndJanusPredicate(Arrays.asList(Text.PREFIX, Cmp.EQUAL)))).isQNF(), + "PREFIX EQUAL with AND(JANUS(PREFIX EQUAL)) should not be QNF"); } }