percy-appium-app 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 85a82be9fb7970cea7e6bfbf92341377a6cf4647f07ce09b3a3c4bb52ff25c11
-  data.tar.gz: 03be307ae51ff85c7853cf0284ded40713a91f8bbf4ce36f8f6b7ae69165d26e
+  metadata.gz: cb456ecc7624f0fa71f10f0d4697af3d212a1a611a47ba9af19148f6f4e6314b
+  data.tar.gz: e987cb68808d8dd1fd3ea1a021836e5eaef6a38821106d9b347093d2cf07d067
 SHA512:
-  metadata.gz: 986183165da611f44bfd356562c084bdb942d7f87a2e38418746562c1d056894e8a373c4fe3e61a5aa2c3a44a48c9c42d7701e845b30eec7727fdafbe2a78faa
-  data.tar.gz: a21a495cb3cda29740602eb262e23ef67ed7c3fce651d191d1ec91c73bc56b0540f6c8cf55772816da6acdcadf1d07c6040e35a49f4fdfae95e1ca0016ccf2fe
+  metadata.gz: 29d57b272a4bedec154cfde69921a9c468554998892c224bfc2443a250742fdac8b0000af483e449d84f9330e8473d026736691bf7f8e9e58f9d2c1a3c9778e7
+  data.tar.gz: e90a50582005f371cc002e59dc37004bebc234620e6044ae79d65d0646c2c7e608d54910f69137dcf9b0ccec89262c4fd82e89fe7e95a7420da1be68dbb0c904

data/.github/workflows/test.yml

@@ -6,6 +6,7 @@ on:
   workflow_dispatch:
     inputs:
       branch:
+        description: Which branch of @percy/cli to build from source for this run
         required: false
         type: string
         default: master
@@ -20,7 +21,13 @@ jobs:
       matrix:
         os: [ubuntu-latest]
         ruby: ['2.6', '2.7', '3.0']
+        include:
+          - os: ubuntu-latest
+            ruby: '3.3'
+            appium_lib: '~> 16.0'
     runs-on: ${{ matrix.os }}
+    env:
+      APPIUM_LIB_VERSION: ${{ matrix.appium_lib || '~> 12.0' }}
     steps:
       - uses: actions-ecosystem/action-regex-match@v2
         id: regex-match
@@ -42,6 +49,24 @@ jobs:
           key: v1/${{ runner.os }}/ruby-${{ matrix.ruby }}/${{ hashFiles('**/Gemfile.lock') }}
           restore-keys: v1/${{ runner.os }}/ruby-${{ matrix.ruby }}/
       - run: bundle install
+      - name: Set up @percy/cli from git
+        if: ${{ github.event_name == 'workflow_dispatch' }}
+        env:
+          BRANCH: ${{ github.event.inputs.branch }}
+          WORKSPACE: ${{ github.workspace }}
+        run: |
+          cd /tmp
+          git clone --branch "$BRANCH" --depth 1 https://github.com/percy/cli
+          cd cli
+          PERCY_PACKAGES=`find packages -mindepth 1 -maxdepth 1 -type d | sed -e 's/packages/@percy/g' | tr '\n' ' '`
+          yarn
+          yarn build
+          yarn global:link
+          cd "$WORKSPACE"
+          yarn link $PERCY_PACKAGES >/dev/null 2>&1 || true
+          echo "$(yarn global bin)" >> "$GITHUB_PATH"
+          export PATH="$(yarn global bin):$PATH"
+          percy --version
       # Run each spec via the SimpleCov runner so per-process coverage is
       # recorded and merged, then collate the merged result and enforce the gate.
       - run: for file in specs/*.rb; do echo "$file"; bundle exec ruby specs/support/run_spec.rb "$file"; done

data/Gemfile

@@ -2,14 +2,17 @@
 
 source 'https://rubygems.org'
 
-# Pin appium_lib to the 12.x line so all Ruby legs (2.6/2.7/3.0) resolve
-# appium_lib_core ~> 5.0, which still exposes Appium::Core::Base::SearchContext.
-# Unpinned, Ruby 3.0 resolved appium_lib 15 / appium_lib_core 9.x, which removed
-# that constant and broke the specs. appium_console 3.0.0 requires appium_lib = 12.0.0.
-gem 'appium_console', '~> 3.0'
-gem 'appium_lib', '~> 12.0'
+# appium_lib defaults to the 12.x line so the Ruby 2.6/2.7/3.0 legs resolve
+# appium_lib_core ~> 5.0. CI also runs a leg on the current major via
+# APPIUM_LIB_VERSION, since appium_lib_core 11+ changed where the driver keeps its
+# hub URL. appium_console 3.0.0 requires appium_lib = 12.0.0, so it only joins the
+# 12.x resolution.
+appium_lib_version = ENV.fetch('APPIUM_LIB_VERSION', '~> 12.0')
+gem 'appium_console', '~> 3.0' if appium_lib_version.include?('12.')
+gem 'appium_lib', appium_lib_version
 gem 'dotenv'
-gem 'minitest'
+# minitest 6 moved Minitest::Mock into the separate minitest-mock gem.
+gem 'minitest', '~> 5.0'
 gem 'rubocop', require: false
 gem 'tempfile'
 gem 'webmock', '~> 3.18', '>= 3.18.1'

data/percy/metadata/android_metadata.rb

@@ -9,12 +9,11 @@ module Percy
     def initialize(driver)
       super(driver)
       @_bars = nil
-      # Intentionally left as the original lookup: this path already degrades to
-      # driver.get_system_bars consistently across all appium_lib_core versions
-      # (the rect read yields a non-Hash, so the rect arithmetic in
-      # get_system_bars rescues to nil and falls back), so it is out of scope for
-      # the snake_case capability fix.
-      @_viewport_rect = capabilities.to_json['viewportRect']
+      # Read the rect as a Hash. The previous `capabilities.to_json['viewportRect']`
+      # indexed a JSON String, so the rect was never used and every call fell back
+      # to driver.get_system_bars, which appium_lib_core 13+ sends as the
+      # Appium 2-only `mobile: getSystemBars` command.
+      @_viewport_rect = viewport
     end
 
     def device_screen_size
@@ -71,7 +70,8 @@ module Percy
     end
 
     def viewport
-      capabilities.to_json['viewportRect']
+      rect = get_capability_value('viewportRect')
+      rect.is_a?(Hash) ? rect : nil
     end
 
     def scale_factor

data/percy/metadata/driver_metadata.rb

@@ -8,6 +8,23 @@ module Percy
       @driver = driver
     end
 
+    # Returns the hub URL the driver talks to. appium_lib_core <= 10 keeps it in
+    # the HTTP client's @server_url; 11+ (selenium-webdriver ClientConfig) moved it
+    # to client_config.server_url and leaves @server_url unset, which made every
+    # BrowserStack session look like a generic remote (no AppAutomate provider,
+    # no device name, empty executor URL for Percy on Automate).
+    def self.server_url(driver)
+      http = driver.instance_variable_get(:@bridge).instance_variable_get(:@http)
+      url = http.instance_variable_get(:@server_url).to_s
+      return url unless url.empty?
+
+      client_config = http.respond_to?(:client_config) ? http.client_config : nil
+      client_config ||= http.instance_variable_get(:@client_config)
+      return '' unless client_config.respond_to?(:server_url)
+
+      client_config.server_url.to_s.chomp('/')
+    end
+
     def session_id
       @driver.session_id
     end
@@ -15,7 +32,7 @@ module Percy
     def command_executor_url
       url = Percy::Cache.get_cache(session_id, Percy::Cache::COMMAND_EXECUTOR_URL)
       if url.nil?
-        url = @driver.instance_variable_get(:@bridge).instance_variable_get(:@http).instance_variable_get(:@server_url).to_s
+        url = self.class.server_url(@driver)
         Percy::Cache.set_cache(session_id, Percy::Cache::COMMAND_EXECUTOR_URL, url)
       end
       url

data/percy/metadata/metadata.rb

@@ -3,6 +3,7 @@
 require 'json'
 require 'pathname'
 require_relative '../common/common'
+require_relative 'driver_metadata'
 
 DEVICE_INFO_FILE_PATH = File.join(File.dirname(__FILE__), '..', 'configs', 'devices.json')
 DEVICE_INFO = JSON.parse(File.read(DEVICE_INFO_FILE_PATH))
@@ -82,7 +83,7 @@ module Percy
     end
 
     def remote_url
-      driver.instance_variable_get(:@bridge).instance_variable_get(:@http).instance_variable_get(:@server_url).to_s
+      Percy::DriverMetadata.server_url(driver)
     end
 
     def get_orientation(**kwargs)

data/percy/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Percy
-  VERSION = '1.0.1'.freeze
+  VERSION = '1.0.2'.freeze
 end

data/specs/android_metadata.rb

@@ -28,7 +28,7 @@ class TestAndroidMetadata < Minitest::Test
     android_capabilities = get_android_capabilities
     @mock_webdriver.expect(:capabilities, android_capabilities.merge('viewportRect' => viewport))
 
-    assert(viewport, @android_metadata.viewport)
+    assert_equal(viewport, @android_metadata.viewport)
     @mock_webdriver.verify
   end
 
@@ -61,19 +61,30 @@ class TestAndroidMetadata < Minitest::Test
     @mock_webdriver.verify
   end
 
-  def test_get_system_bars
+  def test_get_system_bars_falls_back_to_driver_without_viewport_rect
     system_bars = {
       'statusBar' => { 'height' => 83 },
       'navigationBar' => { 'height' => 44 }
     }
-    android_capabilities = get_android_capabilities
-    session_id = 'session_id_123'
-    @mock_webdriver.expect(:session_id, session_id)
-    @mock_webdriver.expect(:session_id, session_id)
-    @mock_webdriver.expect(:capabilities, android_capabilities.merge('viewportRect' => nil))
-    @mock_webdriver.expect(:get_system_bars, system_bars)
+    driver = Minitest::Mock.new
+    driver.expect(:capabilities, get_android_capabilities.merge('viewportRect' => nil))
+    metadata = Percy::AndroidMetadata.new(driver)
+    driver.expect(:session_id, 'session_id_no_rect')
+    driver.expect(:session_id, 'session_id_no_rect')
+    driver.expect(:get_system_bars, system_bars)
+
+    assert_equal(system_bars, metadata.get_system_bars)
+    driver.verify
+  end
+
+  def test_get_system_bars_uses_viewport_rect
+    # viewportRect top=83, height=2153 on a 1080x2280 screen (see get_android_capabilities)
+    @mock_webdriver.expect(:session_id, 'session_id_rect')
+    @mock_webdriver.expect(:capabilities, get_android_capabilities)
+
+    bars = @android_metadata.get_system_bars
 
-    assert(system_bars, @android_metadata.get_system_bars)
+    assert_equal({ 'statusBar' => { 'height' => 83 }, 'navigationBar' => { 'height' => 44 } }, bars)
     @mock_webdriver.verify
   end
 

data/specs/driver_metadata.rb

@@ -39,6 +39,38 @@ class TestDriverMetadata < Minitest::Test
     assert(url, @metadata.command_executor_url)
   end
 
+  # appium_lib_core 11+ leaves the HTTP client's @server_url unset and keeps the
+  # hub URL in client_config.server_url (a URI with a trailing slash).
+  class ClientConfigHttp
+    attr_reader :client_config
+
+    def initialize(client_config)
+      @client_config = client_config
+    end
+  end
+
+  def driver_with_http(http)
+    bridge = Object.new
+    bridge.instance_variable_set(:@http, http)
+    driver = Object.new
+    driver.instance_variable_set(:@bridge, bridge)
+    driver.define_singleton_method(:session_id) { "session_#{object_id}" }
+    driver
+  end
+
+  def test_command_executor_url_reads_client_config_when_server_url_unset
+    client_config = Struct.new(:server_url).new(URI('https://hub-cloud.browserstack.com/wd/hub/'))
+    driver = driver_with_http(ClientConfigHttp.new(client_config))
+
+    assert_equal('https://hub-cloud.browserstack.com/wd/hub', Percy::DriverMetadata.new(driver).command_executor_url)
+  end
+
+  def test_server_url_is_empty_when_no_known_location_has_it
+    driver = driver_with_http(ClientConfigHttp.new(nil))
+
+    assert_equal('', Percy::DriverMetadata.server_url(driver))
+  end
+
   def test_capabilities
     session_id = 'session_id_123'
     2.times do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: percy-appium-app
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - BroswerStack