commit 7561e441c6a1484d475f761299b27e469bb3884f
author cushon <cushon@google.com> Tue Jun 04 15:51:07 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Tue Jun 04 15:52:28 2019 -0700
tree 3e90b18cada7ce5b97440facf72aaa4dfd155625
parent f718a4249739f63e3a0c6e008faa3c7d47fa30f9

Use 'cast' diagnostics instead of 'unchecked' in output filter tests

PiperOrigin-RevId: 251533233

src/test/java/com/google/devtools/build/lib/outputfilter/OutputFilterTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/outputfilter/OutputFilterTest.java b/src/test/java/com/google/devtools/build/lib/outputfilter/OutputFilterTest.java
index f52c8f5..a7b4acd 100644
--- a/src/test/java/com/google/devtools/build/lib/outputfilter/OutputFilterTest.java
+++ b/src/test/java/com/google/devtools/build/lib/outputfilter/OutputFilterTest.java
@@ -40,9 +40,9 @@
   private EventCollector stderr = new EventCollector(EventKind.STDERR);
   private Path workspaceScript;
 
-  // Unchecked warnings are silenced by default.
-  private void enableUncheckedWarnings() throws Exception {
-    addOptions("--javacopt=\"-Xlint:unchecked\"");
+  // Cast warnings are silenced by default.
+  private void enableCastWarnings() throws Exception {
+    addOptions("--javacopt=\"-Xlint:cast\"");
   }
 
   // Deprecation warnings are silenced by default.
@@ -93,26 +93,28 @@
         "java_library(name = 'd',",
         "             srcs = ['D.java'],",
         "             deps = ['//java/e'])");
-    write("java/d/D.java",
+    write(
+        "java/d/D.java",
         "package d;",
         "import java.lang.Integer;",
         "import java.util.ArrayList;",
         "public class D {",
         "  public static void d() {",
-        "    ArrayList<Integer> l = new ArrayList();",
+        "    int i = (int) 0;",
         "    e.E.e();",
         "  }",
         "}");
     write("java/e/BUILD",
         "java_library(name = 'e',",
         "             srcs = ['E.java'])");
-    write("java/e/E.java",
+    write(
+        "java/e/E.java",
         "package e;",
         "import java.lang.Integer;",
         "import java.util.LinkedList;",
         "public class E {",
         "  public static void e() {",
-        "    LinkedList<Integer> l = new LinkedList();",
+        "    int i = (int) 0;",
         "  }",
         "}");
     write("javatests/a/BUILD",
@@ -153,20 +155,21 @@
     write("javatests/e/BUILD",
         "java_library(name = 'e',",
         "             srcs = ['ETest.java'])");
-    write("javatests/e/ETest.java",
+    write(
+        "javatests/e/ETest.java",
         "package e;",
         "import java.lang.Integer;",
         "import java.util.LinkedList;",
         "public class ETest {",
         "  public static void eTest() {",
-        "    LinkedList<Integer> l = new LinkedList();",
+        "    int i = (int) 0;",
         "  }",
         "}");
     workspaceScript = write("wrk", "echo STATUS_CMD_HAS_RUN >&2");
     workspaceScript.setExecutable(true);
 
-    // Always enable unchecked warnings.
-    enableUncheckedWarnings();
+    // Always enable cast warnings.
+    enableCastWarnings();
   }
 
   @Test
@@ -188,8 +191,8 @@
     env.getReporter().addHandler(stderr);
     buildTarget("//java/d");
 
-    assertEvent("D.java:6: warning: [unchecked] unchecked conversion");
-    assertNoEvent("E.java:6: warning: [unchecked] unchecked conversion");
+    assertEvent("D.java:6: warning: [cast] redundant cast to int");
+    assertNoEvent("E.java:6: warning: [cast] redundant cast to int");
   }
 
   @Test
@@ -235,8 +238,8 @@
     env.getReporter().addHandler(stderr);
     buildTarget("//java/d");
 
-    assertEvent("D.java:6: warning: [unchecked] unchecked conversion");
-    assertNoEvent("E.java:6: warning: [unchecked] unchecked conversion");
+    assertEvent("D.java:6: warning: [cast] redundant cast to int");
+    assertNoEvent("E.java:6: warning: [cast] redundant cast to int");
   }
 
   @Test
@@ -320,9 +323,9 @@
     env.getReporter().addHandler(stderr);
     buildTarget("//javatests/d");
 
-    assertEvent("D.java:6: warning: [unchecked] unchecked conversion");
-    assertNoEvent("E.java:6: warning: [unchecked] unchecked conversion");
-    assertNoEvent("ETest.java:6: warning: [unchecked] unchecked conversion");
+    assertEvent("D.java:6: warning: [cast] redundant cast to int");
+    assertNoEvent("E.java:6: warning: [cast] redundant cast to int");
+    assertNoEvent("ETest.java:6: warning: [cast] redundant cast to int");
   }
 
   @Test