Update fragment issue from warning to error

Also update constants to stay in sync with sdk/ version.

Change-Id: I8e454a03d83c81036bf8cc73609babc96a9c7739

2 files changed, 24 insertions, 8 deletions

diff --git a/lint/cli/src/test/java/com/android/tools/lint/checks/FragmentDetectorTest.java b/lint/cli/src/test/java/com/android/tools/lint/checks/FragmentDetectorTest.java
index f732f71..0654ddc 100644
--- a/lint/cli/src/test/java/com/android/tools/lint/checks/FragmentDetectorTest.java
+++ b/lint/cli/src/test/java/com/android/tools/lint/checks/FragmentDetectorTest.java
@@ -27,26 +27,25 @@ public class FragmentDetectorTest extends AbstractCheckTest {
 
     public void test() throws Exception {
         assertEquals(
-            "src/test/pkg/FragmentTest.java:10: Warning: This fragment class should be public (test.pkg.FragmentTest.Fragment1) [ValidFragment]\n" +
+            "src/test/pkg/FragmentTest.java:10: Error: This fragment class should be public (test.pkg.FragmentTest.Fragment1) [ValidFragment]\n" +
             " private static class Fragment1 extends Fragment {\n" +
             " ^\n" +
-            "src/test/pkg/FragmentTest.java:15: Warning: This fragment inner class should be static (test.pkg.FragmentTest.Fragment2) [ValidFragment]\n" +
+            "src/test/pkg/FragmentTest.java:15: Error: This fragment inner class should be static (test.pkg.FragmentTest.Fragment2) [ValidFragment]\n" +
             " public class Fragment2 extends Fragment {\n" +
             " ^\n" +
-            "src/test/pkg/FragmentTest.java:21: Warning: The default constructor must be public [ValidFragment]\n" +
+            "src/test/pkg/FragmentTest.java:21: Error: The default constructor must be public [ValidFragment]\n" +
             "  private Fragment3() {\n" +
             "          ~~~~~~~~~\n" +
-            "src/test/pkg/FragmentTest.java:26: Warning: This fragment should provide a default constructor (a public constructor with no arguments) (test.pkg.FragmentTest.Fragment4) [ValidFragment]\n" +
+            "src/test/pkg/FragmentTest.java:26: Error: This fragment should provide a default constructor (a public constructor with no arguments) (test.pkg.FragmentTest.Fragment4) [ValidFragment]\n" +
             " public static class Fragment4 extends Fragment {\n" +
             "                     ~~~~~~~~~\n" +
-            "src/test/pkg/FragmentTest.java:27: Warning: Avoid non-default constructors in fragments: use a default constructor plus Fragment#setArguments(Bundle) instead [ValidFragment]\n" +
+            "src/test/pkg/FragmentTest.java:27: Error: Avoid non-default constructors in fragments: use a default constructor plus Fragment#setArguments(Bundle) instead [ValidFragment]\n" +
             "  private Fragment4(int dummy) {\n" +
             "          ~~~~~~~~~\n" +
-            "src/test/pkg/FragmentTest.java:36: Warning: Avoid non-default constructors in fragments: use a default constructor plus Fragment#setArguments(Bundle) instead [ValidFragment]\n" +
+            "src/test/pkg/FragmentTest.java:36: Error: Avoid non-default constructors in fragments: use a default constructor plus Fragment#setArguments(Bundle) instead [ValidFragment]\n" +
             "  public Fragment5(int dummy) {\n" +
             "         ~~~~~~~~~\n" +
-            "0 errors, 6 warnings\n" +
-            "",
+            "6 errors, 0 warnings\n",
 
             lintProject(
                 "bytecode/FragmentTest$Fragment1.class.data=>bin/classes/test/pkg/FragmentTest$Fragment1.class",
diff --git a/lint/cli/src/test/java/com/android/tools/lint/checks/MissingClassDetectorTest.java b/lint/cli/src/test/java/com/android/tools/lint/checks/MissingClassDetectorTest.java
index 27875e8..b3ff056 100644
--- a/lint/cli/src/test/java/com/android/tools/lint/checks/MissingClassDetectorTest.java
+++ b/lint/cli/src/test/java/com/android/tools/lint/checks/MissingClassDetectorTest.java
@@ -373,4 +373,21 @@ public class MissingClassDetectorTest extends AbstractCheckTest {
                         "bytecode/OnClickActivity.class.data=>bin/classes/test/pkg/OnClickActivity.class"
                 ));
     }
+
+    public void testFragments() throws Exception {
+        // Ensure that we don't do instantiation checks here since they are handled by
+        // the FragmentDetector
+        assertEquals(
+                "No warnings.",
+
+                lintProject(
+                        "bytecode/FragmentTest$Fragment1.class.data=>bin/classes/test/pkg/FragmentTest$Fragment1.class",
+                        "bytecode/FragmentTest$Fragment2.class.data=>bin/classes/test/pkg/FragmentTest$Fragment2.class",
+                        "bytecode/FragmentTest$Fragment3.class.data=>bin/classes/test/pkg/FragmentTest$Fragment3.class",
+                        "bytecode/FragmentTest$Fragment4.class.data=>bin/classes/test/pkg/FragmentTest$Fragment4.class",
+                        "bytecode/FragmentTest$Fragment5.class.data=>bin/classes/test/pkg/FragmentTest$Fragment5.class",
+                        "bytecode/FragmentTest$Fragment6.class.data=>bin/classes/test/pkg/FragmentTest$Fragment6.class",
+                        "bytecode/FragmentTest$NotAFragment.class.data=>bin/classes/test/pkg/FragmentTest$NotAFragment.class",
+                        "bytecode/FragmentTest.java.txt=>src/test/pkg/FragmentTest.java"));
+    }
 }