increase w3c functional tests for android (#157)

* increase w3c functional tests

* make sure the test in w3c

* fix rubocop

* add double tap and many actions

* fix rubocop

* add click_and_hold and tweak test cases

* define espresso name as environment variable

Author: KazuCocoa

README.md

@@ -34,7 +34,8 @@ $ appium --relaxed-security # To run all tests in local
 - Conduct tests
  ```bash
 $ bundle install
-$ rake test:func:android # Andorid 
+$ rake test:func:android # Andorid, uiautomator2
+$ AUTOMATION_NAME=espresso rake test:func:android # Andorid, uiautomator2
 $ rake test:func:ios     # iOS
 ```
 

Rakefile

@@ -57,8 +57,18 @@ namespace :test do
 end
 
 namespace :android do
+  desc('uninstall all of test apks from a test device')
+  task :uninstall_test_apks do |_t, _args|
+    `adb uninstall io.appium.uiautomator2.server.test`
+    `adb uninstall io.appium.uiautomator2.server`
+    `adb uninstall io.appium.settings`
+    `adb uninstall io.appium.android.ime`
+    `adb uninstall io.appium.espressoserver.test`
+  end
+
+
   desc('Generate and launch android emulators')
-  task :gen_device  do |_t, _args|
+  task :gen_device do |_t, _args|
     SWARMER_VERSION = '0.2.4'
     CPU_ARCHITECTURE = 'x86'
     IMAGE = 'google_apis'

test/functional/android/webdriver/device_test.rb

@@ -8,9 +8,6 @@ class DeviceTest < AppiumLibCoreTest::Function::TestCase
       def setup
         @@core ||= ::Appium::Core.for(Caps.android)
         @driver = @@core.start_driver
-
-        require 'pry'
-        binding.pry
       end
 
       def teardown
@@ -71,7 +68,7 @@ def test_page_source
         xml = REXML::Document.new s_source
 
         assert s_source.include?('io.appium.android.apis')
-        assert_equal expected, xml[1].elements.each('//*'){ |v| v }.map(&:name)
+        assert_equal expected, xml[1].elements.each('//*') { |v| v }.map(&:name)
       end
 
       # def test_location

test/functional/android/webdriver/w3c_actions_test.rb

@@ -15,14 +15,13 @@ def teardown
         @@core.quit_driver
       end
 
-      def test_tap
+      def test_tap_scroll
         skip if @driver.dialect == :oss
 
         el = @@core.wait { @driver.find_element(:accessibility_id, 'Views') }
         @driver.action.click(el).perform
 
         el = @@core.wait { @driver.find_element(:accessibility_id, 'Custom') }
-        # same as @driver.action.click_and_hold(el).move_to_location(0, 700).release.perform
 
         rect1 = el.rect.dup
         @driver
@@ -39,15 +38,81 @@ def test_tap
           .action
           .move_to_location(rect2.x, rect2.y)
           .pointer_down(:left)
-          .move_to_location(0, rect2.y - rect2.height * 2)
+          .move_to_location(0, rect2.y - rect2.height * 3) # gone the element
           .release
           .perform
-        assert rect2.y > el.rect.y
+
+        @driver.manage.timeouts.implicit_wait = 3
+        assert_raises ::Selenium::WebDriver::Error::NoSuchElementError do
+          @driver.find_element(:accessibility_id, 'Custom')
+        end
+        @driver.manage.timeouts.implicit_wait = @@core.default_wait
 
         el = @@core.wait { @driver.find_element(:accessibility_id, 'ImageButton') }
         assert_equal 'ImageButton', el.text
       end
 
+      def test_double_tap
+        skip if @driver.dialect == :oss
+
+        el = @@core.wait { @driver.find_element(:accessibility_id, 'Views') }
+        @driver.action.click(el).perform
+
+        el = @@core.wait { @driver.find_element(:accessibility_id, 'Buttons') }
+        @driver.action.click(el).perform
+
+        el = @driver.find_element(:id, 'io.appium.android.apis:id/button_toggle')
+        assert_equal 'OFF', el.text
+        @driver.action.click(el).perform
+
+        assert_equal 'ON', el.text
+
+        # double tap action with pause
+        action_builder = @driver.action
+        input = action_builder.pointer_inputs[0]
+        action_builder
+          .move_to(el)
+          .pointer_down(:left)
+          .pause(input, 0.05) # seconds
+          .pointer_up(:left)
+          .pause(input, 0.05) # seconds
+          .pointer_down(:left)
+          .pause(input, 0.05) # seconds
+          .pointer_up(:left)
+          .perform
+        assert_equal 'ON', el.text
+
+        error = assert_raises ::Selenium::WebDriver::Error::UnknownError do
+          @driver.action.double_click(el).perform
+        end
+        assert error.message.include?('You cannot perform')
+      end
+
+      def test_actions_with_many_down_up
+        skip if @driver.dialect == :oss
+
+        el = @@core.wait { @driver.find_element(:accessibility_id, 'Views') }
+        @driver.action.click_and_hold(el).release.perform
+
+        el = @@core.wait { @driver.find_element(:accessibility_id, 'Custom') }
+
+        rect1 = el.rect.dup
+
+        error = assert_raises ::Selenium::WebDriver::Error::UnknownError do
+          @driver
+            .action
+            .move_to(el)
+            .pointer_down(:left) # should insert pause
+            .pointer_down(:left)
+            .pointer_down(:left)
+            .move_to_location(0, rect1.y - rect1.height)
+            .release
+            .release
+            .perform
+        end
+        assert error.message.include?('You cannot perform')
+      end
+
       # Note: Works with Espresso Driver
       def test_multiple_actions
         f1 = @driver.action.add_pointer_input(:touch, 'finger1')

test/functional/ios/webdriver/device_test.rb

@@ -56,14 +56,14 @@ def test_page_source
 
         assert !source.include?('AppiumAUT')
         assert source.include?('XCUIElementTypeApplication type')
-        assert xml[2].elements.each('//*'){ |v| v }.map(&:name) == 77
+        assert xml[2].elements.each('//*') { |v| v }.map(&:name) == 77
 
         s_source = @@driver.page_source
         s_xml = REXML::Document.new s_source
 
         assert s_source.include?('AppiumAUT')
         assert s_source.include?('XCUIElementTypeApplication type')
-        assert s_xml[2].elements.each('//*'){ |v| v }.map(&:name) == 81
+        assert s_xml[2].elements.each('//*') { |v| v }.map(&:name) == 81
       end
 
       def test_location

test/test_helper.rb

@@ -80,7 +80,7 @@ def android
       {
         caps: {
           platformName: :android,
-          automationName: 'uiautomator2',
+          automationName: ENV['AUTOMATION_NAME'] || 'uiautomator2',
           app: 'test/functional/app/api.apk.zip',
           udid: get_udid_name,
           deviceName: 'Android Emulator',
@@ -109,7 +109,7 @@ def android_web
         caps: {
           browserName: :chrome,
           platformName: :android,
-          automationName: 'uiautomator2',
+          automationName: ENV['AUTOMATION_NAME'] || 'uiautomator2',
           chromeOptions: { androidPackage: 'com.android.chrome', args: ['--disable-popup-blocking'] },
           # refer: https://github.com/appium/appium/blob/master/docs/en/writing-running-appium/web/chromedriver.md
           # chromedriverExecutable: "#{Dir.pwd}/test/functional/app/chromedriver",