test: tweak espresso (#278)

* tweak test_permissions

* fix more

* fix rubocop

Author: KazuCocoa

test/functional/android/android/device_test.rb

@@ -85,12 +85,20 @@ def test_background_reset
 
         @driver.background_app(-1)
         sleep 1 # to wait the app has gone
-        error = assert_raises ::Selenium::WebDriver::Error::WebDriverError do
-          @driver.find_element :accessibility_id, 'App'
-        end
-        assert 'An element could not be located on the page using the given search parameters.', error.message
+        assert @driver.app_state('io.appium.android.apis') != :running_in_foreground
+
+        # Instrumentation process will crash in Espresso
+        if @@core.automation_name == :espresso
+          @driver.activate_app('io.appium.android.apis')
+          @@core.wait { assert @driver.app_state('io.appium.android.apis') == :running_in_foreground }
+        else
+          error = assert_raises ::Selenium::WebDriver::Error::WebDriverError do
+            @driver.find_element :accessibility_id, 'App'
+          end
+          assert 'An element could not be located on the page using the given search parameters.', error.message
 
-        @driver.reset
+          @driver.reset
+        end
 
         e = @@core.wait(timeout: 60) { @driver.find_element :accessibility_id, 'App' }
         assert_equal 'App', e.text
@@ -132,6 +140,8 @@ def test_app_string
       end
 
       def test_re_install
+        skip 'Instrumentation process will stop by remove_app in Espresso' if @@core.automation_name == :espresso
+
         assert @driver.app_installed?('io.appium.android.apis')
 
         @driver.remove_app 'io.appium.android.apis'
@@ -146,8 +156,11 @@ def test_re_install
       def test_app_management
         assert @driver.app_state('io.appium.android.apis') == :running_in_foreground
 
-        assert @driver.terminate_app('io.appium.android.apis')
-        @@core.wait { assert @driver.app_state('io.appium.android.apis') == :not_running }
+        # Instrumentation process will crash in Espresso
+        unless @@core.automation_name == :espresso
+          assert @driver.terminate_app('io.appium.android.apis')
+          @@core.wait { assert @driver.app_state('io.appium.android.apis') == :not_running }
+        end
 
         assert @driver.activate_app('io.appium.android.apis').nil?
         @@core.wait { assert @driver.app_state('io.appium.android.apis') == :running_in_foreground }

test/functional/android/android/mobile_commands_test.rb

@@ -54,6 +54,9 @@ def test_permissions
         assert @driver.execute_script('mobile: getPermissions',
                                       { type: type[:requested], appPackage: package }).member?(permissions.first)
 
+        # Espresso driver works only getPermissions, so finish
+        return if @core.automation_name == :espresso
+
         @driver.execute_script 'mobile: changePermissions',
                                { action: action[:revoke], appPackage: package, permissions: permissions }