improve PhoneNumberAnalyzerTests#testTelPrefixSearch (opensearch-project#17016)

rursprung · meetvm · commit 7b371c1f5b32 · 2025-01-16T18:23:07.000+05:30
this way we ensure that it doesn't include any additional tokens which we don't want. this is a follow-up to commit 4d94399 / opensearch-project#16993. Signed-off-by: Ralph Ursprung <Ralph.Ursprung@avaloq.com>
diff --git a/plugins/analysis-phonenumber/src/test/java/org/opensearch/analysis/phone/PhoneNumberAnalyzerTests.java b/plugins/analysis-phonenumber/src/test/java/org/opensearch/analysis/phone/PhoneNumberAnalyzerTests.java
@@ -159,11 +159,11 @@ public void testSipWithoutDomainPart() throws IOException {
     }
 
     public void testTelPrefix() throws IOException {
-        assertTokensInclude("tel:+1228", Arrays.asList("1228", "122", "228"));
+        assertTokensInclude(phoneAnalyzer, "tel:+1228", Arrays.asList("tel:+1228", "tel:", "1228", "122", "228"));
     }
 
     public void testTelPrefixSearch() throws IOException {
-        assertTokensInclude("tel:+1228", Arrays.asList("1228"));
+        assertTokensAreInAnyOrder(phoneSearchAnalyzer, "tel:+1228", Arrays.asList("tel:+1228", "1228"));
     }
 
     public void testNumberPrefix() throws IOException {

Original file line number	Diff line number	Diff line change
`@@ -159,11 +159,11 @@ public void testSipWithoutDomainPart() throws IOException {`
`159`	`159`	`}`
`160`	`160`
`161`	`161`	`public void testTelPrefix() throws IOException {`
`162`		`- assertTokensInclude("tel:+1228", Arrays.asList("1228", "122", "228"));`
	`162`	`+ assertTokensInclude(phoneAnalyzer, "tel:+1228", Arrays.asList("tel:+1228", "tel:", "1228", "122", "228"));`
`163`	`163`	`}`
`164`	`164`
`165`	`165`	`public void testTelPrefixSearch() throws IOException {`
`166`		`- assertTokensInclude("tel:+1228", Arrays.asList("1228"));`
	`166`	`+ assertTokensAreInAnyOrder(phoneSearchAnalyzer, "tel:+1228", Arrays.asList("tel:+1228", "1228"));`
`167`	`167`	`}`
`168`	`168`
`169`	`169`	`public void testNumberPrefix() throws IOException {`