Flesh out the test a bit more

Author: AngledLuffa

test/src/edu/stanford/nlp/semgraph/semgrex/SemgrexTest.java

@@ -1575,7 +1575,7 @@ public void testBatchUniq() {
   public void testVariableGroups() {
     // first, a basic test that it is capturing the variable groups correctly
     SemgrexPattern pattern = SemgrexPattern.compile("{word:/(.*ill.*)/#1%name}");
-    SemanticGraph graph = SemanticGraph.valueOf("[ate-2 subj> Bill-1 obj>[muffins-5 compound> Blueberry-3 compound> filled-4]]");
+    SemanticGraph graph = SemanticGraph.valueOf("[ate-2 subj> Bill-1 obj>[muffins-6 compound> Blueberry-3 compound> Flueberry-4 compound> filled-5]]");
     Set<String> matches = new HashSet<>();
     SemgrexMatcher matcher = pattern.matcher(graph);
     while (matcher.find()) {
@@ -1586,11 +1586,30 @@ public void testVariableGroups() {
     Set<String> expectedMatches = Stream.of("Bill", "filled").collect(Collectors.toCollection(HashSet::new));
     assertEquals(expectedMatches, matches);
 
+    // test a basic use case of a single variable string matching
     pattern = SemgrexPattern.compile("{word:/(.*)ill/#1%name} .. {word:/(.*)lueberry/#1%name}");
     matcher = pattern.matcher(graph);
     assertTrue(matcher.find());
     assertEquals("B", matcher.variableStrings.getString("name"));
+    // this should not match Flueberry
     assertFalse(matcher.find());
+
+    // this time, because the variable names are different,
+    // both Blueberry and Flueberry should match
+    pattern = SemgrexPattern.compile("{word:/(.*)ill/#1%name} .. {word:/(.*)lueberry/#1%letter}");
+    matcher = pattern.matcher(graph);
+    matches.clear();
+    assertTrue(matcher.find());
+    assertEquals("B", matcher.variableStrings.getString("name"));
+    assertNotNull(matcher.variableStrings.getString("letter"));
+    matches.add(matcher.variableStrings.getString("letter"));
+    assertTrue(matcher.find());
+    assertEquals("B", matcher.variableStrings.getString("name"));
+    assertNotNull(matcher.variableStrings.getString("letter"));
+    matches.add(matcher.variableStrings.getString("letter"));
+    assertFalse(matcher.find());
+    expectedMatches = Stream.of("B", "F").collect(Collectors.toCollection(HashSet::new));
+    assertEquals(expectedMatches, matches);
   }
 
   public static void outputBatchResults(SemgrexPattern pattern, List<CoreMap> sentences) {