Delete defunct test suite infrastructure
Everything but supportedOs is never read in practice. They were probably used
at some point, but we've migrated much of the test suite infrastructure over to
regular old BUILD-defined tests. Deleted the attributes that were easy, but left
size since it'll be a larger effort to clean up all of its vestigial uses.
`tolerateEmptyTestSuites` is no longer used. In fact it was actually harmful
since it gave the impression that it actually worked - `TautologyTest` would get
picked up by the classpath-scanning test search of c.g.devtools.build.lib, which
every use of the test suite builder would use, so test suites were never really
empty in practice and would blindly succeed due to `TautologyTest` blindly
succeeding.
Alternatively we _could_ go the other direction and leave things as they are
with the eventual goal of improving the test-suite infra. However, we've already
moved quite a bit away from the point where it'd make sense to do that, which is
why it was so easy do delete all of this functionality. Also, as noted in the
previous paragraph, the framework actually hides bugs, so removing it reduces
room for error.
PiperOrigin-RevId: 377972815
diff --git a/src/test/java/com/google/devtools/build/lib/windows/WindowsSubprocessTest.java b/src/test/java/com/google/devtools/build/lib/windows/WindowsSubprocessTest.java
index b737521..1ac68f6 100644
--- a/src/test/java/com/google/devtools/build/lib/windows/WindowsSubprocessTest.java
+++ b/src/test/java/com/google/devtools/build/lib/windows/WindowsSubprocessTest.java
@@ -34,11 +34,9 @@
import org.junit.runner.RunWith;
import org.junit.runners.JUnit4;
-/**
- * Unit tests for {@link WindowsSubprocess}.
- */
+/** Unit tests for {@link WindowsSubprocess}. */
@RunWith(JUnit4.class)
-@TestSpec(localOnly = true, supportedOs = OS.WINDOWS)
+@TestSpec(supportedOs = OS.WINDOWS)
public class WindowsSubprocessTest {
private String mockSubprocess;
private String mockBinary;