playwright-ruby-client 0.2.1 → 0.5.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 140f77362280a6874d89ed73fde1e80fa47e2b83f075b6207835f02ab62cb97f
-  data.tar.gz: e715b39c620668eeea1a0dbb022865f759d02f59d2f3d23948c7c20536ca0136
+  metadata.gz: cd2cd0b1ca23f5807c139c717c91341d2279c98cf6a8b513560221f9da9701cf
+  data.tar.gz: 8a059997574ce31fb065b1dbc3dae5c94c3660ada06fc0eaaff5cb29cc6efa6b
 SHA512:
-  metadata.gz: 0315d44031788794eca7f506ba9a20a81dbb6bf77c15efda873a1efc3e25074d46e1629ae07551f2dd8368f8a7609568cd719fd1470fdd3bf2717373168655a2
-  data.tar.gz: a3b6a02b60059ac3ef14c0ca1222e3b88a3aaee588b8510c49ef748fab5f27c354417b1e38cc6adfe98e6b1f38805ed1c0cead8cfb465f4968e449c98dd5a9a5
+  metadata.gz: 6eb023ebb94129a006dcdacb383b43221dbcae61b1d1f474e382f84c2dd4d2b39d6c9cc878673e193f71c2409860bd69d0a8c865508a5930188aeb607325ed00
+  data.tar.gz: a3bd488faa4ef02f7afe9e50bd9e001429440b0bfd3a96a00ddf074aac809db828d06562985083b8573c31be966e64394253d6d5d26593cf4ad8434bc5660c50

data/README.md

@@ -9,22 +9,34 @@ Note: Currently, this Gem is just a PoC (Proof of Concept). If you want to devel
 At this point, playwright-ruby-client doesn't include the downloader of playwright driver, so **we have to install [playwright](https://github.com/microsoft/playwright) in advance**.
 
 ```sh
+npx playwright install
+```
+
+and then, set `playwright_cli_executable_path: "npx playwright"` at `Playwright.create`.
+
+**Prefer npm install instead of npx?**
+
+Actually `npx playwright` is a bit slow. We can also use `npm install` to setup.
+
+Instead of `npx playwright install`:
+
+```
 npm install playwright
 ./node_modules/.bin/playwright install
 ```
 
-and then, set `playwright_cli_executable_path: ./node_modules/.bin/playwright` at `Playwright.create`.
+And set `playwright_cli_executable_path: './node_modules/.bin/playwright'`
 
 **Prefer playwrighting without Node.js?**
 
-Instead of npm install, you can also directly download playwright driver from playwright.azureedge.net/builds/. The URL can be easily detected from [here](https://github.com/microsoft/playwright-python/blob/79f6ce0a6a69c480573372706df84af5ef99c4a4/setup.py#L56-L61)
+Instead of npm, you can also directly download playwright driver from playwright.azureedge.net/builds/. The URL can be easily detected from [here](https://github.com/microsoft/playwright-python/blob/79f6ce0a6a69c480573372706df84af5ef99c4a4/setup.py#L56-L61)
 
 ### Capture a site
 
 ```ruby
 require 'playwright'
 
-Playwright.create(playwright_cli_executable_path: '/path/to/playwright') do |playwright|
+Playwright.create(playwright_cli_executable_path: 'npx playwright') do |playwright|
   playwright.chromium.launch(headless: false) do |browser|
     page = browser.new_page
     page.goto('https://github.com/YusukeIwaki')
@@ -40,7 +52,7 @@ end
 ```ruby
 require 'playwright'
 
-Playwright.create(playwright_cli_executable_path: './node_modules/.bin/playwright') do |playwright|
+Playwright.create(playwright_cli_executable_path: 'npx playwright') do |playwright|
   playwright.chromium.launch(headless: false) do |browser|
     page = browser.new_page
     page.goto('https://github.com/')
@@ -82,7 +94,7 @@ $ bundle exec ruby main.rb
 ```ruby
 require 'playwright'
 
-Playwright.create(playwright_cli_executable_path: './node_modules/.bin/playwright') do |playwright|
+Playwright.create(playwright_cli_executable_path: 'npx playwright') do |playwright|
   devices = playwright.android.devices
   unless devices.empty?
     device = devices.last

data/docs/api_coverage.md

@@ -19,24 +19,24 @@
 
 ## Response
 
-* ~~body~~
-* ~~finished~~
-* ~~frame~~
-* ~~headers~~
-* ~~json~~
-* ~~ok~~
-* ~~request~~
-* ~~status~~
-* ~~status_text~~
-* ~~text~~
-* ~~url~~
+* body
+* finished
+* frame
+* headers
+* json
+* ok
+* request
+* status
+* status_text
+* text
+* url
 
-## ~~Route~~
+## Route
 
-* ~~abort~~
-* ~~continue_~~
-* ~~fulfill~~
-* ~~request~~
+* abort
+* continue
+* fulfill
+* request
 
 ## ~~WebSocket~~
 
@@ -55,11 +55,11 @@
 
 ## Mouse
 
-* ~~click~~
-* ~~dblclick~~
-* ~~down~~
-* ~~move~~
-* ~~up~~
+* click
+* dblclick
+* down
+* move
+* up
 
 ## Touchscreen
 
@@ -173,6 +173,7 @@
 * expect_navigation
 * wait_for_selector
 * ~~wait_for_timeout~~
+* wait_for_url
 
 ## ~~Worker~~
 
@@ -182,7 +183,7 @@
 
 ## Selectors
 
-* ~~register~~
+* register
 
 ## ConsoleMessage
 
@@ -191,26 +192,13 @@
 * text
 * type
 
-## ~~Dialog~~
-
-* ~~accept~~
-* ~~default_value~~
-* ~~dismiss~~
-* ~~message~~
-* ~~type~~
-
-## Download
-
-* delete
-* failure
-* path
-* save_as
-* suggested_filename
-* url
+## Dialog
 
-## ~~Video~~
-
-* ~~path~~
+* accept
+* default_value
+* dismiss
+* message
+* type
 
 ## Page
 
@@ -258,7 +246,7 @@
 * query_selector
 * query_selector_all
 * reload
-* ~~route~~
+* route
 * screenshot
 * select_option
 * set_content
@@ -272,24 +260,25 @@
 * title
 * type
 * uncheck
-* ~~unroute~~
+* unroute
 * url
-* ~~video~~
+* video
 * viewport_size
+* expect_console_message
+* expect_download
 * expect_event
+* expect_file_chooser
 * wait_for_function
 * wait_for_load_state
 * expect_navigation
+* expect_popup
 * expect_request
 * expect_response
 * wait_for_selector
 * ~~wait_for_timeout~~
-* ~~workers~~
-* expect_download
-* expect_popup
+* wait_for_url
 * ~~expect_worker~~
-* expect_console_message
-* expect_file_chooser
+* ~~workers~~
 * ~~wait_for_event~~
 * accessibility
 * keyboard
@@ -298,28 +287,31 @@
 
 ## BrowserContext
 
-* ~~add_cookies~~
-* ~~add_init_script~~
-* ~~browser~~
-* ~~clear_cookies~~
-* ~~clear_permissions~~
+* add_cookies
+* add_init_script
+* ~~background_pages~~
+* browser
+* clear_cookies
+* clear_permissions
 * close
-* ~~cookies~~
-* ~~expose_binding~~
-* ~~expose_function~~
-* ~~grant_permissions~~
+* cookies
+* expose_binding
+* expose_function
+* grant_permissions
+* ~~new_cdp_session~~
 * new_page
 * pages
-* ~~route~~
-* ~~set_default_navigation_timeout~~
-* ~~set_default_timeout~~
-* ~~set_extra_http_headers~~
-* ~~set_geolocation~~
-* ~~set_offline~~
+* route
+* ~~service_workers~~
+* set_default_navigation_timeout
+* set_default_timeout
+* set_extra_http_headers
+* set_geolocation
+* set_offline
 * ~~storage_state~~
-* ~~unroute~~
-* ~~expect_event~~
-* ~~expect_page~~
+* unroute
+* expect_event
+* expect_page
 * ~~wait_for_event~~
 
 ## ~~CDPSession~~
@@ -327,19 +319,16 @@
 * ~~detach~~
 * ~~send_message~~
 
-## ChromiumBrowserContext
-
-* ~~background_pages~~
-* ~~new_cdp_session~~
-* ~~service_workers~~
-
 ## Browser
 
 * close
 * contexts
 * connected?
+* ~~new_browser_cdp_session~~
 * new_context
 * new_page
+* start_tracing
+* stop_tracing
 * version
 
 ## BrowserType
@@ -355,5 +344,58 @@
 * chromium
 * devices
 * firefox
-* ~~selectors~~
+* selectors
 * webkit
+
+## Android
+
+* devices
+* ~~set_default_timeout~~
+
+## AndroidDevice
+
+* close
+* ~~drag~~
+* ~~fill~~
+* ~~fling~~
+* info
+* ~~install_apk~~
+* launch_browser
+* ~~long_tap~~
+* model
+* ~~open~~
+* ~~pinch_close~~
+* ~~pinch_open~~
+* ~~press~~
+* ~~push~~
+* screenshot
+* ~~scroll~~
+* serial
+* ~~set_default_timeout~~
+* shell
+* ~~swipe~~
+* ~~tap_point~~
+* ~~wait~~
+* ~~wait_for_event~~
+* ~~web_view~~
+* ~~web_views~~
+* input
+
+## AndroidInput
+
+* drag
+* press
+* ~~swipe~~
+* tap_point
+* type
+
+## ~~AndroidSocket~~
+
+* ~~close~~
+* ~~write~~
+
+## ~~AndroidWebView~~
+
+* ~~page~~
+* ~~pid~~
+* ~~pkg~~

data/lib/playwright.rb

@@ -17,14 +17,17 @@ require 'playwright/utils'
 require 'playwright/api_implementation'
 require 'playwright/channel'
 require 'playwright/channel_owner'
+require 'playwright/download'
 require 'playwright/http_headers'
 require 'playwright/input_files'
 require 'playwright/connection'
+require 'playwright/route_handler_entry'
 require 'playwright/select_option_values'
 require 'playwright/timeout_settings'
 require 'playwright/transport'
 require 'playwright/url_matcher'
 require 'playwright/version'
+require 'playwright/video'
 require 'playwright/wait_helper'
 
 require 'playwright/playwright_api'
@@ -32,18 +35,54 @@ require 'playwright/playwright_api'
 Dir[File.join(__dir__, 'playwright_api', '*.rb')].each { |f| require f }
 
 module Playwright
-  module_function def create(playwright_cli_executable_path:, &block)
-    raise ArgumentError.new("block must be provided") unless block
+  class Execution
+    def initialize(connection, playwright)
+      @connection = connection
+      @playwright = playwright
+    end
+
+    def stop
+      @connection.stop
+    end
+
+    attr_reader :playwright
+  end
 
+  # Recommended to call this method with block.
+  #
+  # Playwright.create(...) do |playwright|
+  #   browser = playwright.chromium.launch
+  #   ...
+  # end
+  #
+  # When we use this method without block, an instance of Puppeteer::Execution is returned
+  # and we *must* call execution.stop on the end.
+  # The instance of playwright is available by calling execution.playwright
+  module_function def create(playwright_cli_executable_path:, &block)
     connection = Connection.new(playwright_cli_executable_path: playwright_cli_executable_path)
+    connection.async_run
+
+    execution =
+      begin
+        playwright = connection.wait_for_object_with_known_name('Playwright')
+        Execution.new(connection, PlaywrightApi.wrap(playwright))
+      rescue
+        connection.stop
+        raise
+      end
 
-    playwright_promise = connection.async_wait_for_object_with_known_name('Playwright')
-    Thread.new { connection.run }
-    playwright = PlaywrightApi.wrap(playwright_promise.value!)
-    begin
-      block.call(playwright)
-    ensure
-      connection.stop
+    if block
+      begin
+        block.call(execution.playwright)
+      ensure
+        execution.stop
+      end
+    else
+      execution
     end
   end
+
+  module_function def instance
+    @playwright_instance
+  end
 end

data/lib/playwright/channel.rb

@@ -15,8 +15,9 @@ module Playwright
 
     # @param method [String]
     # @param params [Hash]
+    # @return [Playwright::ChannelOwner|nil]
     def send_message_to_server(method, params = {})
-      result = @connection.send_message_to_server(@guid, method, params)
+      result = send_message_to_server_result(method, params)
       if result.is_a?(Hash)
         _type, channel_owner = result.first
         channel_owner
@@ -27,8 +28,17 @@ module Playwright
 
     # @param method [String]
     # @param params [Hash]
-    def send_no_reply(method, params)
+    # @return [Hash]
+    def send_message_to_server_result(method, params)
+      @connection.send_message_to_server(@guid, method, params)
+    end
+
+    # @param method [String]
+    # @param params [Hash]
+    # @returns nil
+    def async_send_message_to_server(method, params = {})
       @connection.async_send_message_to_server(@guid, method, params)
+
       nil
     end
   end