test: fail the build when TablesNamesFinder misses a visitor dispatch method

TablesNamesFinder dispatches through 8 visitor interfaces. Abstract
dispatch methods are compiler-enforced, but the interfaces progressively
gain default ones (e.g. SelectVisitor#visit(PivotQuery, context)), and
for a default declared by a single interface a missing implementation
compiles silently and loses the tables of that node type without any
test failing. Pin the complete post-#2479/#2517 state (214 methods).

Signed-off-by: 付典 <fudianchn@gmail.com>

Author: fudianchn

src/test/java/net/sf/jsqlparser/util/TablesNamesFinderCompletenessTest.java

@@ -0,0 +1,64 @@
+package net.sf.jsqlparser.util;
+
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+import java.lang.reflect.Method;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Set;
+
+import org.junit.jupiter.api.Test;
+
+/**
+ * TablesNamesFinder collects tables by dispatching through the visitor interfaces it declares. Most
+ * dispatch methods are abstract, so a missing implementation fails the build already. But the
+ * interfaces progressively gain default methods (e.g. SelectVisitor#visit(PivotQuery, context)
+ * returns null), and for those a missing TablesNamesFinder implementation compiles silently and
+ * loses the tables of that node type without any test failing.
+ *
+ * This test fails the build when a declared visitor interface has a dispatch method visit(Node,
+ * context) for an AST node that TablesNamesFinder does not implement itself. It does not catch
+ * empty-shell implementations (an override that visits nothing) nor visitor families the finder
+ * does not declare at all: both remain manual review duties.
+ */
+public class TablesNamesFinderCompletenessTest {
+
+    // GroupByElement: ExpressionVisitor's default forwards the group-by expressions and the
+    // grouping sets to the visitor, TablesNamesFinder relies on it from visit(PlainSelect).
+    private static final Set<String> INHERITED_DISPATCH_NODES =
+            Set.of("net.sf.jsqlparser.statement.select.GroupByElement");
+
+    @Test
+    void testImplementsEveryDispatchMethodOfEveryDeclaredVisitorInterface()
+            throws NoSuchMethodException {
+        List<String> gaps = new ArrayList<>();
+        for (Class<?> visitorInterface : TablesNamesFinder.class.getInterfaces()) {
+            for (Method dispatch : visitorInterface.getMethods()) {
+                if (!dispatch.getName().equals("visit") || dispatch.getParameterCount() != 2) {
+                    continue;
+                }
+                Class<?> nodeType = dispatch.getParameterTypes()[0];
+                // skips bulk dispatch helpers carrying a java.util collection
+                if (!nodeType.getName().startsWith("net.sf.jsqlparser.")) {
+                    continue;
+                }
+                if (INHERITED_DISPATCH_NODES.contains(nodeType.getName())) {
+                    continue;
+                }
+                Method implementation = TablesNamesFinder.class.getMethod(dispatch.getName(),
+                        dispatch.getParameterTypes());
+                if (implementation.getDeclaringClass() != TablesNamesFinder.class) {
+                    gaps.add(visitorInterface.getSimpleName() + "#" + dispatch.getName() + "("
+                            + nodeType.getSimpleName() + ", context)");
+                }
+            }
+        }
+
+        assertTrue(gaps.isEmpty(), () -> "TablesNamesFinder must implement every dispatch"
+                + " method of the visitor interfaces it declares, otherwise the tables of that"
+                + " node type are silently not collected. Missing:\n" + String.join("\n", gaps)
+                + "\nImplement visit(Node, context) in TablesNamesFinder, or - when the"
+                + " inherited default already forwards every child - document the node in"
+                + " INHERITED_DISPATCH_NODES.");
+    }
+}