Fix test in org.eclipse.compare.tests eclipse-platform#525

This commit reactivates and fixes the test testVerify() from FilterTest.java in org.eclipse.compare.tests. The test was outcommented in the past probably because Filter.validateResourceFilters(String) could not be performed anymore. Now validateResourceFilters(String) is performed on CompareResourceFilter. Contributes to eclipse-platform#525.
CodeLtDave · Sep 13, 2023 · f41dfc3 · f41dfc3
1 parent 7ed493c
commit f41dfc3
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/team/tests/org.eclipse.compare.tests/src/org/eclipse/compare/tests/FilterTest.java b/team/tests/org.eclipse.compare.tests/src/org/eclipse/compare/tests/FilterTest.java
@@ -61,9 +61,8 @@ public void testMultiFilter() {
 
 	@Test
 	public void testVerify() {
-		// Assert.assertNull("filters don't verify",
-		// Filter.validateResourceFilters("*.class, .cvsignore, bin/"));
-		// Assert.assertNotNull("filters shouldn't verify",
-		// Filter.validateResourceFilters("bin//"));
+		Assert.assertNull("filters don't verify",
+				CompareResourceFilter.validateResourceFilters("*.class, .cvsignore, bin/"));
+		Assert.assertNotNull("filters shouldn't verify", CompareResourceFilter.validateResourceFilters("bin//"));
 	}
 }