CtScannerTest is buggy

src/test/java/spoon/reflect/visitor/CtScannerTest.java

@@ -38,8 +38,11 @@
 
 import java.lang.annotation.Annotation;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Comparator;
+import java.util.HashSet;
 import java.util.List;
+import java.util.Set;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
@@ -108,15 +111,23 @@ public void testScannerCallsAllProperties() throws Exception {
 		CtClass<?> scanner = (CtClass<?>)launcher.getFactory().Type().get(CtScanner.class);
 
 		List<String> problems = new ArrayList<>();
+		Set<String> ignoredInvocations = new HashSet(Arrays.asList("scan", "enter", "exit"));
 
 		for (CtType<?> t : SpoonTestHelpers.getAllInstantiableMetamodelInterfaces()) {
 			CtMethod<?> visitMethod = scanner.getMethodsByName("visit"+t.getSimpleName()).get(0);
+			Set<String> calledMethods = new HashSet<>();
+			Set<String> checkedMethods = new HashSet<>();
 			for (CtMethod<?> m : SpoonTestHelpers.getAllMetamodelMethods(t)) {
 				if (isMetamodelProperty(t, m)) {
+					checkedMethods.add(m.getSignature());
 					//System.out.println("checking "+m.getSignature() +" in "+visitMethod.getSignature());
 					CtInvocation invocation = visitMethod.filterChildren(new TypeFilter<CtInvocation>(CtInvocation.class) {
 						@Override
 						public boolean matches(CtInvocation element) {
+							if(ignoredInvocations.contains(element.getExecutable().getSimpleName())) {
+								return false;
+							}
+							calledMethods.add(element.getExecutable().getSignature());
 							return super.matches(element) && element.getExecutable().getSimpleName().equals(m.getSimpleName());
 						}
 					}).first();
@@ -131,6 +142,8 @@ public boolean matches(CtInvocation element) {
 					}
 				}
 			}
+			calledMethods.removeAll(checkedMethods);
+			assertEquals("CtScanner " + visitMethod.getPosition() + " calls unexpected methods: "+calledMethods, 0, calledMethods.size());
 		}
 		if(problems.size()>0) {
 			fail(problems.toString());