Restrict the type of exception that can be thrown during target pattern parsing (and general target-oriented graph visitation) to QueryException. In practice, this is the only exception that can be thrown, but that fact is extremely non-obvious. We make it obvious by restricting the type of Exception that can be thrown to QueryExceptionMarkerInterface, an interface that is only non-trivially implemented by QueryException. PiperOrigin-RevId: 410355114
diff --git a/src/test/java/com/google/devtools/build/lib/cmdline/BUILD b/src/test/java/com/google/devtools/build/lib/cmdline/BUILD index cfca030..ef53ba5 100644 --- a/src/test/java/com/google/devtools/build/lib/cmdline/BUILD +++ b/src/test/java/com/google/devtools/build/lib/cmdline/BUILD
@@ -18,9 +18,13 @@ deps = [ "//src/main/java/com/google/devtools/build/lib/cmdline", "//src/main/java/com/google/devtools/build/lib/cmdline:LabelValidator", + "//src/main/java/com/google/devtools/build/lib/cmdline:batch_callback", + "//src/main/java/com/google/devtools/build/lib/cmdline:parallel_visitor", + "//src/main/java/com/google/devtools/build/lib/cmdline:query_exception_marker_interface", + "//src/main/java/com/google/devtools/build/lib/concurrent", "//src/main/java/com/google/devtools/build/lib/vfs:pathfragment", "//src/main/java/net/starlark/java/eval", - "//src/test/java/com/google/devtools/build/lib/testutil:TestUtils", + "//src/test/java/com/google/devtools/build/lib/testutil:TestThread", "//third_party:guava", "//third_party:guava-testlib", "//third_party:junit4",