android: refactor

android/app/src/main/java/com/reactnativenavigation/controllers/NavigationActivity.java

@@ -47,6 +47,7 @@ public class NavigationActivity extends AppCompatActivity implements DefaultHard
         host.getReactInstanceManager().addReactInstanceEventListener(new ReactInstanceManager.ReactInstanceEventListener() {
             @Override
             public void onReactContextInitialized(ReactContext context) {
+                // TODO should we implement this line also if context already exists onCreate?
                 new NavigationEventEmitter(context).emitAppLaunched();
             }
         });

playground/android/app/src/androidTest/java/com/reactnativenavigation/playground/ApplicationLifecycleTest.java

@@ -78,7 +78,10 @@ public class ApplicationLifecycleTest {
         rule.launchActivity(null);
     }
 
-    private void acceptPermission() throws Exception {
+    private void acceptPermissionIfNeeded() throws Exception {
+        if (Settings.canDrawOverlays(getInstrumentation().getContext())) {
+            return;
+        }
         UiDevice.getInstance(getInstrumentation()).findObject(new UiSelector().text("Playground")).click();
         UiDevice.getInstance(getInstrumentation()).findObject(new UiSelector().text("Permit drawing over other apps")).click();
         UiDevice.getInstance(getInstrumentation()).pressBack();
@@ -90,7 +93,7 @@ public class ApplicationLifecycleTest {
     public void startApp_HandleOverlayPermissions_LoadsBridge_ThenShowsWelcomeScreen() throws Exception {
         assumeFalse(Settings.canDrawOverlays(getInstrumentation().getContext()));
         launchActivity();
-        acceptPermission();
+        acceptPermissionIfNeeded();
         onView(withText("React Native Navigation!")).check(matches(isDisplayed()));
     }
 
@@ -100,4 +103,8 @@ public class ApplicationLifecycleTest {
         launchActivity();
         onView(withText("React Native Navigation!")).check(matches(isDisplayed()));
     }
+
+    @Test
+    public void () {
+    }
 }