playwright-ruby-client 0.3.0 → 0.5.6

data/lib/playwright/keyboard_impl.rb

@@ -10,7 +10,7 @@ module Playwright
     end
 
     def up(key)
-      @channel.send_message_to_server('keyboardDown', key: key)
+      @channel.send_message_to_server('keyboardUp', key: key)
     end
 
     def insert_text(text)

data/lib/playwright/mouse_impl.rb

@@ -3,5 +3,46 @@ module Playwright
     def initialize(channel)
       @channel = channel
     end
+
+    def move(x, y, steps: nil)
+      params = { x: x, y: y, steps: steps }.compact
+      @channel.send_message_to_server('mouseMove', params)
+      nil
+    end
+
+    def down(button: nil, clickCount: nil)
+      params = { button: button, clickCount: clickCount }.compact
+      @channel.send_message_to_server('mouseDown', params)
+      nil
+    end
+
+    def up(button: nil, clickCount: nil)
+      params = { button: button, clickCount: clickCount }.compact
+      @channel.send_message_to_server('mouseUp', params)
+      nil
+    end
+
+    def click(
+      x,
+      y,
+      button: nil,
+      clickCount: nil,
+      delay: nil)
+
+      params = {
+        x: x,
+        y: y,
+        button: button,
+        clickCount: clickCount,
+        delay: delay,
+      }.compact
+      @channel.send_message_to_server('mouseClick', params)
+
+      nil
+    end
+
+    def dblclick(x, y, button: nil, delay: nil)
+      click(x, y, button: button, clickCount: 2, delay: delay)
+    end
   end
 end

data/lib/playwright/playwright_api.rb

@@ -14,7 +14,7 @@ module Playwright
       when ApiImplementation
         ApiImplementationWrapper.new(channel_owner_or_api_implementation).wrap
       else
-        nil
+        channel_owner_or_api_implementation
       end
     end
 
@@ -119,12 +119,14 @@ module Playwright
       if object.is_a?(Array)
         object.map { |obj| wrap_impl(obj) }
       else
-        ::Playwright::PlaywrightApi.wrap(object) || object
+        ::Playwright::PlaywrightApi.wrap(object)
       end
     end
 
     private def unwrap_impl(object)
-      if object.is_a?(PlaywrightApi)
+      if object.is_a?(Array)
+        object.map { |obj| unwrap_impl(obj) }
+      elsif object.is_a?(PlaywrightApi)
         object.instance_variable_get(:@impl)
       else
         object

data/lib/playwright/route_handler_entry.rb

@@ -9,7 +9,7 @@ module Playwright
     end
 
     def handle(route, request)
-      if url_match?(request.url)
+      if @url_matcher.match?(request.url)
         @handler.call(route, request)
         true
       else
@@ -24,13 +24,5 @@ module Playwright
         @url_value == url
       end
     end
-
-    private def url_match?(request_url)
-      if @url_value.is_a?(Regexp)
-        @url_matcher.match?(request_url)
-      else
-        @url_matcher.match?(request_url) || File.fnmatch?(@url_value, request_url)
-      end
-    end
   end
 end

data/lib/playwright/select_option_values.rb

@@ -1,18 +1,30 @@
 module Playwright
   class SelectOptionValues
     def initialize(element: nil, index: nil, value: nil, label: nil)
-      @params =
-        if element
-          convert(elmeent)
-        elsif index
-          convert(index)
-        elsif value
-          convert(value)
-        elsif label
-          convert(label)
-        else
-          {}
-        end
+      params = {}
+
+      options = []
+      if value
+        options.concat(convert(:value, value))
+      end
+
+      if index
+        options.concat(convert(:index, index))
+      end
+
+      if label
+        options.concat(convert(:label, label))
+      end
+
+      unless options.empty?
+        params[:options] = options
+      end
+
+      if element
+        params[:elements] = convert(:element, element)
+      end
+
+      @params = params
     end
 
     # @return [Hash]
@@ -20,22 +32,19 @@ module Playwright
       @params
     end
 
-    private def convert(values)
-      return convert([values]) unless values.is_a?(Enumerable)
-      return {} if values.empty?
+    private def convert(key, values)
+      return convert(key, [values]) unless values.is_a?(Enumerable)
+      return [] if values.empty?
       values.each_with_index do |value, index|
-        unless values
+        unless value
           raise ArgumentError.new("options[#{index}]: expected object, got null")
         end
       end
 
-      case values.first
-      when ElementHandle
-        { elements: values.map(&:channel) }
-      when String
-        { options: values.map { |value| { value: value } } }
+      if key == :element
+        values.map(&:channel)
       else
-        { options: values }
+        values.map { |value| { key => value } }
       end
     end
   end

data/lib/playwright/transport.rb

@@ -12,39 +12,46 @@ module Playwright
     def initialize(playwright_cli_executable_path:)
       @driver_executable_path = playwright_cli_executable_path
       @debug = ENV['DEBUG'].to_s == 'true' || ENV['DEBUG'].to_s == '1'
+      @mutex = Mutex.new
     end
 
     def on_message_received(&block)
       @on_message = block
     end
 
+    def on_driver_crashed(&block)
+      @on_driver_crashed = block
+    end
+
     class AlreadyDisconnectedError < StandardError ; end
 
     # @param message [Hash]
     def send_message(message)
       debug_send_message(message) if @debug
       msg = JSON.dump(message)
-      @stdin.write([msg.size].pack('V')) # unsigned 32bit, little endian
-      @stdin.write(msg)
-    rescue Errno::EPIPE
+      @mutex.synchronize {
+        @stdin.write([msg.bytes.length].pack('V')) # unsigned 32bit, little endian, real byte size instead of chars
+        @stdin.write(msg) # write UTF-8 in binary mode as byte stream
+      }
+    rescue Errno::EPIPE, IOError
       raise AlreadyDisconnectedError.new('send_message failed')
     end
 
     # Terminate playwright-cli driver.
     def stop
       [@stdin, @stdout, @stderr].each { |io| io.close unless io.closed? }
+      @thread&.terminate
     end
 
     # Start `playwright-cli run-driver`
     #
     # @note This method blocks until playwright-cli exited. Consider using Thread or Future.
-    def run
-      @stdin, @stdout, @stderr, @thread = Open3.popen3(@driver_executable_path, 'run-driver')
+    def async_run
+      @stdin, @stdout, @stderr, @thread = Open3.popen3("#{@driver_executable_path} run-driver")
+      @stdin.binmode  # Ensure Strings are written 1:1 without encoding conversion, necessary for integer values
 
       Thread.new { handle_stdout }
       Thread.new { handle_stderr }
-
-      @thread.join
     end
 
     private
@@ -69,6 +76,21 @@ module Playwright
 
     def handle_stderr
       while err = @stderr.read
+        # sometimed driver crashes with the error below.
+        # --------
+        # undefined:1
+        # �
+        # ^
+
+        # SyntaxError: Unexpected token � in JSON at position 0
+        #     at JSON.parse (<anonymous>)
+        #     at Transport.transport.onmessage (/home/runner/work/playwright-ruby-client/playwright-ruby-client/node_modules/playwright/lib/cli/driver.js:42:73)
+        #     at Immediate.<anonymous> (/home/runner/work/playwright-ruby-client/playwright-ruby-client/node_modules/playwright/lib/protocol/transport.js:74:26)
+        #     at processImmediate (internal/timers.js:461:21)
+        if err.include?('undefined:1')
+          @on_driver_crashed&.call
+          break
+        end
         $stderr.write(err)
       end
     rescue IOError

data/lib/playwright/url_matcher.rb

@@ -8,7 +8,7 @@ module Playwright
     def match?(target_url)
       case @url
       when String
-        @url == target_url
+        @url == target_url || File.fnmatch?(@url, target_url)
       when Regexp
         @url.match?(target_url)
       else

data/lib/playwright/utils.rb

@@ -11,6 +11,14 @@ module Playwright
         if params[:extraHTTPHeaders]
           params[:extraHTTPHeaders] = ::Playwright::HttpHeaders.new(params[:extraHTTPHeaders]).as_serialized
         end
+        if params[:record_video_dir]
+          params[:recordVideo] = {
+            dir: params.delete(:record_video_dir)
+          }
+          if params[:record_video_size]
+            params[:recordVideo][:size] = params.delete(:record_video_size)
+          end
+        end
         if params[:storageState].is_a?(String)
           params[:storageState] = JSON.parse(File.read(params[:storageState]))
         end

data/lib/playwright/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Playwright
-  VERSION = '0.3.0'
+  VERSION = '0.5.6'
 end

data/lib/playwright/video.rb

@@ -0,0 +1,51 @@
+module Playwright
+  class Video
+    def initialize(page)
+      @page = page
+      @artifact = Concurrent::Promises.resolvable_future
+      if @page.closed?
+        on_page_closed
+      else
+        page.once('close', -> { on_page_closed })
+      end
+    end
+
+    private def on_page_closed
+      unless @artifact.resolved?
+        @artifact.reject('Page closed')
+      end
+    end
+
+    # called only from Page#on_video via send(:set_artifact, artifact)
+    private def set_artifact(artifact)
+      @artifact.fulfill(artifact)
+    end
+
+    def path
+      wait_for_artifact_and do |artifact|
+        artifact.absolute_path
+      end
+    end
+
+    def save_as(path)
+      wait_for_artifact_and do |artifact|
+        artifact.save_as(path)
+      end
+    end
+
+    def delete
+      wait_for_artifact_and do |artifact|
+        artifact.delete
+      end
+    end
+
+    private def wait_for_artifact_and(&block)
+      artifact = @artifact.value!
+      unless artifact
+        raise 'Page did not produce any video frames'
+      end
+
+      block.call(artifact)
+    end
+  end
+end

data/lib/playwright/wait_helper.rb

@@ -22,9 +22,9 @@ module Playwright
     def reject_on_timeout(timeout_ms, message)
       return if timeout_ms <= 0
 
-      Concurrent::Promises.schedule(timeout_ms / 1000.0) {
+      Concurrent::Promises.schedule(timeout_ms / 1000.0) do
         reject(TimeoutError.new(message: message))
-      }
+      end
 
       self
     end

data/lib/playwright_api/accessibility.rb

@@ -6,7 +6,7 @@ module Playwright
   # Accessibility is a very platform-specific thing. On different platforms, there are different screen readers that might
   # have wildly different output.
   # 
-  # Rendering engines of Chromium, Firefox and Webkit have a concept of "accessibility tree", which is then translated into
+  # Rendering engines of Chromium, Firefox and WebKit have a concept of "accessibility tree", which is then translated into
   # different platform-specific APIs. Accessibility namespace gives access to this Accessibility Tree.
   # 
   # Most of the accessibility tree gets filtered out when converting from internal browser AX Tree to Platform-specific
@@ -28,6 +28,11 @@ module Playwright
     # console.log(snapshot);
     # ```
     # 
+    # ```java
+    # String snapshot = page.accessibility().snapshot();
+    # System.out.println(snapshot);
+    # ```
+    # 
     # ```python async
     # snapshot = await page.accessibility.snapshot()
     # print(snapshot)
@@ -38,6 +43,11 @@ module Playwright
     # print(snapshot)
     # ```
     # 
+    # ```csharp
+    # var accessibilitySnapshot = await Page.Accessibility.SnapshotAsync();
+    # Console.WriteLine(accessibilitySnapshot);
+    # ```
+    # 
     # An example of logging the focused node's name:
     # 
     #
@@ -57,6 +67,34 @@ module Playwright
     # }
     # ```
     # 
+    # ```csharp
+    # Func<AccessibilitySnapshotResult, AccessibilitySnapshotResult> findFocusedNode = root =>
+    # {
+    #     var nodes = new Stack<AccessibilitySnapshotResult>(new[] { root });
+    #     while (nodes.Count > 0)
+    #     {
+    #         var node = nodes.Pop();
+    #         if (node.Focused) return node;
+    #         foreach (var innerNode in node.Children)
+    #         {
+    #             nodes.Push(innerNode);
+    #         }
+    #     }
+    # 
+    #     return null;
+    # };
+    # 
+    # var accessibilitySnapshot = await Page.Accessibility.SnapshotAsync();
+    # var focusedNode = findFocusedNode(accessibilitySnapshot);
+    # if(focusedNode != null)
+    #   Console.WriteLine(focusedNode.Name);
+    # ```
+    # 
+    # ```java
+    # // FIXME
+    # String snapshot = page.accessibility().snapshot();
+    # ```
+    # 
     # ```python async
     # def find_focused_node(node):
     #     if (node.get("focused"))

data/lib/playwright_api/android.rb

@@ -3,18 +3,18 @@ module Playwright
   # 
   #
   # ```js
-  # const { _android } = require('playwright');
+  # const { _android: android } = require('playwright');
   # ```
   # 
   # An example of the Android automation script would be:
   # 
   #
   # ```js
-  # const { _android } = require('playwright');
+  # const { _android: android } = require('playwright');
   # 
   # (async () => {
   #   // Connect to the device.
-  #   const [device] = await playwright._android.devices();
+  #   const [device] = await android.devices();
   #   console.log(`Model: ${device.model()}`);
   #   console.log(`Serial: ${device.serial()}`);
   #   // Take screenshot of the whole device.
@@ -35,7 +35,7 @@ module Playwright
   # 
   #     // Work with WebView's page as usual.
   #     const page = await webview.page();
-  #     await page.page.waitForNavigation({ url: /.*microsoft\/playwright.*/ });
+  #     await page.waitForNavigation({ url: /.*microsoft\/playwright.*/ });
   #     console.log(await page.title());
   #   }
   # 
@@ -79,12 +79,6 @@ module Playwright
     end
     alias_method :default_timeout=, :set_default_timeout
 
-    # -- inherited from EventEmitter --
-    # @nodoc
-    def off(event, callback)
-      event_emitter_proxy.off(event, callback)
-    end
-
     # -- inherited from EventEmitter --
     # @nodoc
     def once(event, callback)
@@ -97,6 +91,12 @@ module Playwright
       event_emitter_proxy.on(event, callback)
     end
 
+    # -- inherited from EventEmitter --
+    # @nodoc
+    def off(event, callback)
+      event_emitter_proxy.off(event, callback)
+    end
+
     private def event_emitter_proxy
       @event_emitter_proxy ||= EventEmitterProxy.new(self, @impl)
     end