Improve PersonMatchesQueryPredicate test cases

AY2324S2-CS2103T-W12-3 · Apr 12, 2024 · 0ed71aa · 0ed71aa
1 parent e55374c
commit 0ed71aa
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/src/test/java/seedu/address/model/person/PersonMatchesQueryPredicateTest.java b/src/test/java/seedu/address/model/person/PersonMatchesQueryPredicateTest.java
@@ -61,6 +61,14 @@ public void test_nameMatchesQuery_returnsTrue() {
         // Query without whitespace
         predicate = new PersonMatchesQueryPredicate("xy");
         assertTrue(predicate.test(new PersonBuilder().withName("Alex Yeoh").build()));
+
+        // Query without tags
+        predicate = new PersonMatchesQueryPredicate("Alice Bob");
+        assertTrue(predicate.test(new PersonBuilder().withName("Alice Bob").withTags(new String[0]).build()));
+
+        // Query without assets
+        predicate = new PersonMatchesQueryPredicate("Alice Bob");
+        assertTrue(predicate.test(new PersonBuilder().withName("Alice Bob").withAssets(new String[0]).build()));
     }
 
     @Test
@@ -120,6 +128,10 @@ public void test_tagMatchesQuery_returnsTrue() {
         // Short query, long tag
         predicate = new PersonMatchesQueryPredicate("a");
         assertTrue(predicate.test(new PersonBuilder().withTags("colleagues").build()));
+
+        // No assets
+        predicate = new PersonMatchesQueryPredicate("friends");
+        assertTrue(predicate.test(new PersonBuilder().withTags("friends").withAssets(new String[0]).build()));
     }
 
     @Test
@@ -150,6 +162,10 @@ public void test_assetMatchesQuery_returnsTrue() {
         // Query without whitespace
         predicate = new PersonMatchesQueryPredicate("rsc");
         assertTrue(predicate.test(new PersonBuilder().withAssets("hammer screw").build()));
+
+        // No tags
+        predicate = new PersonMatchesQueryPredicate("hammer");
+        assertTrue(predicate.test(new PersonBuilder().withAssets("hammer").withTags(new String[0]).build()));
     }
 
     @Test