Add test cases for the change.

permissions-dispatcher · Dec 2, 2018 · 8a0fa29 · 8a0fa29
1 parent 2c011d8
commit 8a0fa29
Show file tree

Hide file tree

Showing 3 changed files with 95 additions and 1 deletion.
diff --git a/...ermissions/dispatcher/processor/exception/SpecialPermissionsWithNeverAskAgainException.kt b/...ermissions/dispatcher/processor/exception/SpecialPermissionsWithNeverAskAgainException.kt
@@ -1,3 +1,3 @@
 package permissions.dispatcher.processor.exception
 
-class SpecialPermissionsWithNeverAskAgainException : RuntimeException("@NeverAskAgain annotated method never being called with WRITE_SETTINGS or SYSTEM_ALERT_WINDOW permission.")
+class SpecialPermissionsWithNeverAskAgainException : RuntimeException("'@NeverAskAgain' annotated method never being called with 'WRITE_SETTINGS' or 'SYSTEM_ALERT_WINDOW' permission.")
diff --git a/processor/src/test/java/permissions/dispatcher/processor/ProcessorTestSuite.java b/processor/src/test/java/permissions/dispatcher/processor/ProcessorTestSuite.java
@@ -131,6 +131,16 @@ public class ProcessorTestSuite extends TestSuite {
         assertJavaSource(Source.needsPermissionMethodOverloadFragment);
     }
 
+    @Test public void systemAlertWindowWithOnNeverAskAgain() {
+        expectRuntimeException("'@NeverAskAgain' annotated method never being called with 'WRITE_SETTINGS' or 'SYSTEM_ALERT_WINDOW' permission.");
+        assertJavaSource(Source.systemAlertWindowWithOnNeverAskAgain);
+    }
+
+    @Test public void writeSettingsWithOnNeverAskAgain() {
+        expectRuntimeException("'@NeverAskAgain' annotated method never being called with 'WRITE_SETTINGS' or 'SYSTEM_ALERT_WINDOW' permission.");
+        assertJavaSource(Source.writeSettingsWithOnNeverAskAgain);
+    }
+
     @Test public void methodOverloadWithoutNeedsPermission() {
         assertJavaSource(Source.methodOverloadWithoutNeedsPermission);
     }

diff --git a/processor/src/test/java/permissions/dispatcher/processor/data/Source.java b/processor/src/test/java/permissions/dispatcher/processor/data/Source.java
@@ -6001,6 +6001,90 @@ protected String[] getExpectSource() {
         }
     };
 
+    public static final BaseTest systemAlertWindowWithOnNeverAskAgain = new BaseTest() {
+        @Override
+        protected String getName() {
+            return "MyFragment";
+        }
+
+        @Override
+        protected String[] getActualSource() {
+            return new String[]{
+                    "package test;",
+                    "import android.Manifest;",
+                    "import androidx.fragment.app.Fragment;",
+                    "import permissions.dispatcher.NeedsPermission;",
+                    "import permissions.dispatcher.OnNeverAskAgain;",
+                    "import permissions.dispatcher.OnPermissionDenied;",
+                    "import permissions.dispatcher.OnShowRationale;",
+                    "import permissions.dispatcher.PermissionRequest;",
+                    "import permissions.dispatcher.RuntimePermissions;",
+                    "@RuntimePermissions",
+                    "public class MyFragment extends Fragment {",
+                    "    @NeedsPermission(Manifest.permission.SYSTEM_ALERT_WINDOW)",
+                    "    void systemAlertWindow() {",
+                    "    }",
+                    "    @OnShowRationale(Manifest.permission.SYSTEM_ALERT_WINDOW)",
+                    "    void systemAlertWindowOnShowRationale(PermissionRequest request) {",
+                    "    }",
+                    "    @OnNeverAskAgain(Manifest.permission.SYSTEM_ALERT_WINDOW)",
+                    "    void systemAlertWindowOnNeverAskAgain() {",
+                    "    }",
+                    "    @OnPermissionDenied(Manifest.permission.SYSTEM_ALERT_WINDOW)",
+                    "    void systemAlertWindowOnPermissionDenied() {",
+                    "    }",
+                    "}"
+            };
+        }
+
+        @Override
+        protected String[] getExpectSource() {
+            return EMPTY_SOURCE;
+        }
+    };
+
+    public static final BaseTest writeSettingsWithOnNeverAskAgain = new BaseTest() {
+        @Override
+        protected String getName() {
+            return "MyFragment";
+        }
+
+        @Override
+        protected String[] getActualSource() {
+            return new String[]{
+                    "package test;",
+                    "import android.Manifest;",
+                    "import androidx.fragment.app.Fragment;",
+                    "import permissions.dispatcher.NeedsPermission;",
+                    "import permissions.dispatcher.OnNeverAskAgain;",
+                    "import permissions.dispatcher.OnPermissionDenied;",
+                    "import permissions.dispatcher.OnShowRationale;",
+                    "import permissions.dispatcher.PermissionRequest;",
+                    "import permissions.dispatcher.RuntimePermissions;",
+                    "@RuntimePermissions",
+                    "public class MyFragment extends Fragment {",
+                    "    @NeedsPermission(Manifest.permission.WRITE_SETTINGS)",
+                    "    void writeSettings() {",
+                    "    }",
+                    "    @OnShowRationale(Manifest.permission.WRITE_SETTINGS)",
+                    "    void writeSettingOnShowRationale(PermissionRequest request) {",
+                    "    }",
+                    "    @OnNeverAskAgain(Manifest.permission.WRITE_SETTINGS)",
+                    "    void writeSettingOnNeverAskAgain() {",
+                    "    }",
+                    "    @OnPermissionDenied(Manifest.permission.WRITE_SETTINGS)",
+                    "    void writeSettingOnPermissionDenied() {",
+                    "    }",
+                    "}"
+            };
+        }
+
+        @Override
+        protected String[] getExpectSource() {
+            return EMPTY_SOURCE;
+        }
+    };
+
     public static final BaseTest methodOverloadWithoutNeedsPermission = new BaseTest() {
         @Override
         protected String getName() {