bidi2pdf 0.1.11 → 0.1.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f566dbc37d780cfb8596d673639ab1ed944cf9306107a7ff65d0712d74fd9276
-  data.tar.gz: 58273f6e806ea14372ae37f942c05582b80effe6a16cf172b965ea4f45e18108
+  metadata.gz: 71faf59892553b22a889f4502396d7c1670269b6223ce6aadf855a2651102e5d
+  data.tar.gz: 4af6722525e182f381430750e28ced8516fd6e3febbdd007560e4b5bf1e177a9
 SHA512:
-  metadata.gz: 1ec28d088c92fb1d48b42a68f51613045c13ad3d115c8fcd208ea37268ccc1520bf33af1070df513eaa50001259438222af0413f378195539b3eb3e2c95ef292
-  data.tar.gz: e52a3b58e025f726d46c1a512a54e932e744181a021e3e2b3c02b6f12f5d13b8a3d3d285568ce09e98f61476c37eb2ba13cb2b1920f75f443a076931f6dd067c
+  metadata.gz: 18ce08f28f70148e88c09f26238d4b840dacb75318cb36ef14b6d8d1e12d9fd475d734bf495ea30eb7bd4d41684b9da15d008b9d8f73f3e64a6881a8b09cdd9d
+  data.tar.gz: 62f78f1130380da20ffd72f5450e87db66bb7f21ae2c947d8f6b1ab1a3bc738690801c2b42117789c89d9a4566f1343f9e3d7b72fe5e5cd55c9e1f1e66d32c8c

data/.rubocop.yml

@@ -1,6 +1,6 @@
 AllCops:
   NewCops: enable
-  TargetRubyVersion: 3.3
+  TargetRubyVersion: 3.4
 
 Style/StringLiterals:
   EnforcedStyle: double_quotes

data/.ruby-version

@@ -1 +1 @@
-ruby-3.3.4
+ruby-3.4.3

data/CHANGELOG.md

@@ -6,10 +6,107 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-[unreleased]: https://github.com///compare/v0.1.11..HEAD
+## [Unreleased]
+
+[unreleased]: https://github.com/dieter-medium/bidi2pdf/compare/v0.1.12..HEAD
 
 <!-- generated by git-cliff end -->
 
+## [0.1.13] - 2026-03-28
+
+### 🔄 Changed
+
+- Merge pull request #77 from dieter-medium/enhance-chromedriver-params by @dieter-medium
+  in [#77](https://github.com/dieter-medium/bidi2pdf/pull/77)
+- Merge pull request #76 from dieter-medium/chore by @dieter-medium
+  in [#76](https://github.com/dieter-medium/bidi2pdf/pull/76)
+- Merge pull request #73 from dieter-medium/dependabot/bundler/bundler-3a1337c73c by @dieter-medium
+  in [#73](https://github.com/dieter-medium/bidi2pdf/pull/73)
+- Merge pull request #68 from dieter-medium/dependabot/bundler/main/json-2.18.1 by @dieter-medium
+  in [#68](https://github.com/dieter-medium/bidi2pdf/pull/68)
+- Merge pull request #60 from dieter-medium/dependabot/bundler/main/pdf-reader-2.15.1 by @dieter-medium
+  in [#60](https://github.com/dieter-medium/bidi2pdf/pull/60)
+- Merge pull request #66 from dieter-medium/dependabot/bundler/main/rubocop-1.84.1 by @dieter-medium
+  in [#66](https://github.com/dieter-medium/bidi2pdf/pull/66)
+- Merge pull request #63 from dieter-medium/dependabot/bundler/main/rbs-3.10.2 by @dependabot[bot]
+  in [#63](https://github.com/dieter-medium/bidi2pdf/pull/63)
+- Merge pull request #65 from dieter-medium/dependabot/bundler/main/thor-1.5.0 by @dependabot[bot]
+  in [#65](https://github.com/dieter-medium/bidi2pdf/pull/65)
+- Merge pull request #58 from dieter-medium/dependabot/bundler/main/ruby-vips-2.3.0 by @dieter-medium
+  in [#58](https://github.com/dieter-medium/bidi2pdf/pull/58)
+- Merge pull request #52 from dieter-medium/dependabot/bundler/main/rubocop-1.81.7 by @dependabot[bot]
+  in [#52](https://github.com/dieter-medium/bidi2pdf/pull/52)
+- Merge pull request #51 from dieter-medium/dependabot/bundler/main/rake-13.3.1 by @dieter-medium
+  in [#51](https://github.com/dieter-medium/bidi2pdf/pull/51)
+- Merge pull request #49 from dieter-medium/dependabot/bundler/main/rubocop-1.81.6 by @dependabot[bot]
+  in [#49](https://github.com/dieter-medium/bidi2pdf/pull/49)
+- Merge pull request #48 from dieter-medium/dependabot/bundler/main/rspec-3.13.2 by @dependabot[bot]
+  in [#48](https://github.com/dieter-medium/bidi2pdf/pull/48)
+- Merge pull request #47 from dieter-medium/dependabot/bundler/main/json-2.15.1 by @dieter-medium
+  in [#47](https://github.com/dieter-medium/bidi2pdf/pull/47)
+- Merge pull request #46 from dieter-medium/dependabot/bundler/main/rubocop-1.81.1 by @dieter-medium
+  in [#46](https://github.com/dieter-medium/bidi2pdf/pull/46)
+- Merge pull request #45 from dieter-medium/dependabot/bundler/main/json-2.15.0 by @dieter-medium
+  in [#45](https://github.com/dieter-medium/bidi2pdf/pull/45)
+- Merge pull request #44 from dieter-medium/dependabot/bundler/main/rubocop-1.80.2 by @dependabot[bot]
+  in [#44](https://github.com/dieter-medium/bidi2pdf/pull/44)
+- Merge pull request #43 from dieter-medium/dependabot/bundler/main/rbs-3.9.5 by @dieter-medium
+  in [#43](https://github.com/dieter-medium/bidi2pdf/pull/43)
+- Merge pull request #41 from dieter-medium/dependabot/bundler/main/rubocop-1.80.1 by @dieter-medium
+  in [#41](https://github.com/dieter-medium/bidi2pdf/pull/41)
+
+### 🔧 Build
+
+- Bump json in the bundler group across 1 directory by @dependabot[bot]
+- Bump json from 2.18.0 to 2.18.1 by @dependabot[bot]
+- Bump pdf-reader from 2.15.0 to 2.15.1 by @dependabot[bot]
+- Bump rubocop from 1.81.7 to 1.84.1 by @dependabot[bot]
+- Bump rbs from 3.9.5 to 3.10.2 by @dependabot[bot]
+- Bump thor from 1.4.0 to 1.5.0 by @dependabot[bot]
+- Bump ruby-vips from 2.2.5 to 2.3.0 by @dependabot[bot]
+- Bump rubocop from 1.81.6 to 1.81.7 by @dependabot[bot]
+- Bump rake from 13.3.0 to 13.3.1 by @dependabot[bot]
+- Bump rubocop from 1.81.1 to 1.81.6 by @dependabot[bot]
+- Bump rspec from 3.13.1 to 3.13.2 by @dependabot[bot]
+- Bump json from 2.15.0 to 2.15.1 by @dependabot[bot]
+- Bump rubocop from 1.80.2 to 1.81.1 by @dependabot[bot]
+- Bump json from 2.13.2 to 2.15.0 by @dependabot[bot]
+- Bump rubocop from 1.80.1 to 1.80.2 by @dependabot[bot]
+- Bump rbs from 3.9.4 to 3.9.5 by @dependabot[bot]
+- Bump rubocop from 1.79.2 to 1.80.1 by @dependabot[bot]
+
+### 🚀 Added
+
+- Enhance chromedriver parameters by @dieter-medium
+
+## [0.1.12] - 2025-09-02
+
+### 🔄 Changed
+
+- Merge pull request #37 from dieter-medium/dependabot/bundler/main/ruby-vips-2.2.5
+- Merge pull request #34 from dieter-medium/enhance-timeout-error-handling-during-navigate
+- Merge pull request #33 from dieter-medium/dependabot/bundler/main/rubocop-1.79.2
+- Merge pull request #32 from dieter-medium/dependabot/bundler/main/rubocop-1.79.1
+- Merge pull request #31 from dieter-medium/dependabot/bundler/main/json-2.13.2
+- Merge branch 'main' into dependabot/bundler/main/json-2.13.2
+- Merge pull request #30 from dieter-medium/dependabot/bundler/main/rubocop-1.79.0
+- Merge pull request #29 from dieter-medium/remove-codeclimate
+- Merge pull request #27 from dieter-medium/dependabot/bundler/main/json-2.13.0
+- Merge branch 'main' into dependabot/bundler/main/json-2.13.0
+- Merge pull request #28 from dieter-medium/dependabot/bundler/main/thor-1.4.0
+- Merge pull request #26 from dieter-medium/dependabot/bundler/main/rubocop-1.78.0
+
+### 🔧 Build
+
+- Bump ruby-vips from 2.2.4 to 2.2.5
+- Bump rubocop from 1.77.0 to 1.79.2
+- Bump json from 2.12.2 to 2.13.2
+- Bump thor from 1.3.2 to 1.4.0
+
+### 🚀 Added
+
+- Improve navigation timeout error handling
+
 ## [0.1.11] - 2025-07-09
 
 ### 🐛 Fixed
@@ -271,7 +368,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### 🔄 Released
 
-- [unreleased](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.11..HEAD)
+- [unreleased](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.13..HEAD)
+- [0.1.13](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.12..V0.1.13)
+- [0.1.12](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.11..V0.1.12)
 - [0.1.11](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.10..V0.1.11)
 - [0.1.10](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.9..V0.1.10)
 - [0.1.9](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.8..v0.1.9)

data/README.md

@@ -1,7 +1,6 @@
 [![Build Status](https://github.com/dieter-medium/bidi2pdf/actions/workflows/ruby.yml/badge.svg)](https://github.com/dieter-medium/bidi2pdf/blob/main/.github/workflows/ruby.yml)
 [![Maintainability](https://api.codeclimate.com/v1/badges/6425d9893aa3a9ca243e/maintainability)](https://codeclimate.com/github/dieter-medium/bidi2pdf/maintainability)
 [![Gem Version](https://badge.fury.io/rb/bidi2pdf.svg)](https://badge.fury.io/rb/bidi2pdf)
-[![Test Coverage](https://api.codeclimate.com/v1/badges/6425d9893aa3a9ca243e/test_coverage)](https://codeclimate.com/github/dieter-medium/bidi2pdf/test_coverage)
 [![Open Source Helpers](https://www.codetriage.com/dieter-medium/bidi2pdf/badges/users.svg)](https://www.codetriage.com/dieter-medium/bidi2pdf)
 
 ---

data/docker/Dockerfile

@@ -1,4 +1,4 @@
-FROM ruby:3.3
+FROM ruby:3.4
 
 ENV DEBIAN_FRONTEND=noninteractive
 

data/docker/entrypoint.sh

@@ -45,10 +45,10 @@ fi
 
 # DISPLAY=:99 /home/appuser/.webdrivers/chromedriver --port=33259 --whitelisted-ips=""  --allowed-origins="*" --disable-dev-shm-usage --disable-gpu  --verbose
 /home/appuser/.webdrivers/chromedriver --port=${CHROMEDRIVER_PORT} \
-                                       --headless \
-                                       --whitelisted-ips="" \
+                                       --allowed-ips="" \
                                        --allowed-origins="*" \
                                        --disable-dev-shm-usage \
                                        --disable-gpu \
                                        --user-data-dir=${USER_DATA_DIR} \
-                                       --verbose
+                                       --log-level=${CHROMEDRIVER_LOG_LEVEL:-INFO} \
+                                       --readable-timestamp

data/lib/bidi2pdf/bidi/browser_tab.rb

@@ -172,7 +172,7 @@ module Bidi2pdf
       # @raise [NavigationError] If the URL is invalid or improperly formatted.
       # @example
       #   browser_tab.navigate_to("https://example.com")
-      def navigate_to(url)
+      def navigate_to(url, wait: "complete")
         begin
           URI.parse(url)
         rescue URI::InvalidURIError => e
@@ -180,14 +180,15 @@ module Bidi2pdf
         end
 
         Bidi2pdf.notification_service.instrument("navigate_to.bidi2pdf", url: url) do
-          navigate_with_listeners url
+          navigate_with_listeners url, wait: wait
         end
       end
 
       # Renders HTML content in the browser tab.
       #
       # @param [String] html_content The HTML content to render.
-      def render_html_content(html_content)
+      # rubocop:disable  Metrics/BlockLength
+      def render_html_content(html_content, wait: "complete")
         Bidi2pdf.notification_service.instrument("render_html_content.bidi2pdf", url: "data:text/html") do |instrumentation_payload|
           base64_encoded = Base64.strict_encode64(html_content)
 
@@ -195,10 +196,51 @@ module Bidi2pdf
 
           data_url = "data:text/html;charset=utf-8;base64,#{base64_encoded}"
 
-          navigate_with_listeners data_url
+          begin
+            navigate_with_listeners data_url, wait: wait
+          rescue Bidi2pdf::CmdTimeoutError
+            Bidi2pdf.logger.info "Waiting for page to load seemed to have timed out. Checking if page is ready via javascript..."
+
+            # check if the page is still loading
+            execute_script <<~JS, wrap_in_promise: false
+              const desiredState = "interactive";#{" "}
+              const timeoutMs = 30000;#{"        "}
+              const intervalMs = 50;
+
+              result = new Promise((resolveReady, rejectReady) => {
+                const start = Date.now();
+
+                const reachedDesired = (state) =>
+                  desiredState === "interactive"
+                    ? state === "interactive" || state === "complete"
+                    : state === "complete";
+
+                const check = () => {
+                  const state = document.readyState;
+                  if (reachedDesired(state)) {
+                    resolveReady(state); // resolves to "interactive" or "complete"
+                    return;
+                  }
+                  if (Date.now() - start >= timeoutMs) {
+                    rejectReady(new Error(`Timeout waiting for document.readyState="${desiredState}". Last state="${state}".`));
+                    return;
+                  }
+
+                  console.warn(`Waiting for document.readyState="${desiredState}". Last state="${state}".`);
+
+                  setTimeout(check, intervalMs);
+                };
+
+                // Immediate check in case the page is already loaded
+                check();
+              });
+            JS
+          end
         end
       end
 
+      # rubocop:enable  Metrics/BlockLength
+
       # Executes a script in the browser tab.
       #
       # This method allows you to execute JavaScript code within the context of the browser tab.
@@ -434,10 +476,10 @@ module Bidi2pdf
                          end
       end
 
-      def navigate_with_listeners(url)
+      def navigate_with_listeners(url, wait: "complete")
         register_event_listeners
 
-        cmd = Bidi2pdf::Bidi::Commands::BrowsingContextNavigate.new url: url, context: browsing_context_id
+        cmd = Bidi2pdf::Bidi::Commands::BrowsingContextNavigate.new url: url, context: browsing_context_id, wait: wait
 
         client.send_cmd_and_wait(cmd) do |response|
           Bidi2pdf.logger.debug "Navigated to page url: #{url} response: #{response}"

data/lib/bidi2pdf/bidi/event_manager.rb

@@ -9,8 +9,8 @@ module Bidi2pdf
           self.source_location = block.source_location
         end
 
-        def call(*args)
-          block.call(*args)
+        def call(*)
+          block.call(*)
         end
 
         def ==(other)

data/lib/bidi2pdf/chromedriver_manager.rb

@@ -185,6 +185,8 @@ module Bidi2pdf
       cmd = [bin]
       cmd << "--port=#{@port}" unless @port.zero?
       cmd << "--headless" if @headless
+      cmd << "--log-level=#{chromedriver_log_level}"
+      cmd << "--readable-timestamp"
 
       cmd << user_data_dir_arg
       cmd.concat(process_chrome_args)
@@ -192,6 +194,15 @@ module Bidi2pdf
       cmd.join(" ")
     end
 
+    def chromedriver_log_level
+      case Bidi2pdf.logger.level
+      when Logger::DEBUG then "ALL"
+      when Logger::INFO then "INFO"
+      when Logger::WARN then "WARNING"
+      else "SEVERE"
+      end
+    end
+
     def user_data_dir_arg
       provided = @chrome_args&.find { |arg| arg.include?("--user-data-dir") }
       return provided if provided

data/lib/bidi2pdf/process_tree.rb

@@ -36,8 +36,8 @@ module Bidi2pdf
     end
 
     def build_process_map
-      ProcTable.ps.each_with_object({}) do |process, map|
-        map[process.pid] = { info: process, children: [] }
+      ProcTable.ps.to_h do |process|
+        [process.pid, { info: process, children: [] }]
       end
     end
 

data/lib/bidi2pdf/test_helpers/pdf_file_helper.rb

@@ -5,6 +5,10 @@ module Bidi2pdf
     # This module provides helper methods for handling PDF files in tests.
     # It includes methods for debugging, storing, and managing PDF files.
     module PdfFileHelper
+      def reporter
+        RSpec.configuration.reporter
+      end
+
       # Executes a block with the given PDF data and handles debugging in case of test failures.
       # If an expectation fails, the PDF data is saved to a file for debugging purposes.
       # @param [String] pdf_data the PDF data to debug
@@ -14,7 +18,7 @@ module Bidi2pdf
         yield pdf_data
       rescue RSpec::Expectations::ExpectationNotMetError => e
         failure_output = store_pdf_file pdf_data, "test-failure"
-        puts "Test failed! PDF saved to: #{failure_output}"
+        reporter.message("Test failed! PDF saved to: #{failure_output}")
         raise e
       end
 

data/lib/bidi2pdf/test_helpers/pdf_text_sanitizer.rb

@@ -7,187 +7,109 @@ require "diff/lcs/hunk"
 module Bidi2pdf
   module TestHelpers
     # rubocop: disable Metrics/ModuleLength
-    # Provides utilities for sanitizing and comparing PDF text content.
-    # This module includes methods for cleaning text, comparing PDF content,
-    # and reporting differences between actual and expected PDF outputs.
-    #
-    # The sanitization process includes normalizing whitespace, replacing
-    # typographic ligatures, and handling other common text formatting issues.
-    #
-    # @example Cleaning text
-    #   sanitized_text = Bidi2pdf::TestHelpers::PDFTextSanitizer.clean("Some text")
-    #
-    # @example Comparing PDF content
-    #   match = Bidi2pdf::TestHelpers::PDFTextSanitizer.match?(actual_pdf, expected_pdf)
     module PDFTextSanitizer
       class << self
-        # Cleans the given text by replacing common typographic ligatures,
-        # normalizing whitespace, and removing unnecessary characters.
-        #
-        # @param [String] text The text to clean.
-        # @return [String] The cleaned text.
         def clean(text)
           text = UnicodeUtils.nfkd(text)
 
           text.gsub("\uFB01", "fi")
               .gsub("\uFB02", "fl")
               .gsub("-\n", "")
-              .gsub(/["]/, '"')
-              .gsub(/[']/, "'")
+              .gsub('"', '"')
+              .gsub("'", "'")
               .gsub("…", "...")
               .gsub("—", "--")
               .gsub("–", "-")
-              .gsub(/\s+/, " ") # Replace all whitespace sequences with a single space
+              .gsub(/\s+/, " ")
               .strip
         end
 
-        # Cleans an array of PDF page texts by applying the `clean` method
-        # to each page's content.
-        #
-        # @param [Object] actual_pdf_thingy The PDF object to clean.
-        # @return [Array<String>] An array of cleaned page texts.
-        def clean_pages(actual_pdf_thingy)
-          Bidi2pdf::TestHelpers::PDFReaderUtils.pdf_text(actual_pdf_thingy).map { |text| clean(text) }
+        def clean_pages(pdf)
+          Bidi2pdf::TestHelpers::PDFReaderUtils.pdf_text(pdf).map { |text| clean(text) }
         end
 
-        # Cleans the given text and removes all whitespace for comparison purposes.
-        #
-        # @param [String] text The text to clean and normalize.
-        # @return [String] The cleaned text without whitespace.
         def normalize(text)
           clean(text).gsub(/\s+/, "")
         end
 
-        # Checks if the given PDF contains the expected text or pattern.
-        #
-        # @param [Object] actual_pdf_thingy The PDF object to search.
-        # @param [String, Regexp] expected The expected text or pattern.
-        # @param [Integer, nil] page_number The specific page to search (optional).
-        # @return [Boolean] `true` if the expected text is found, `false` otherwise.
         def contains?(actual_pdf_thingy, expected, page_number = nil)
-          pages = Bidi2pdf::TestHelpers::PDFReaderUtils.pdf_text(actual_pdf_thingy)
-          cleaned_pages = clean_pages(pages)
+          cleaned_pages = clean_pages(actual_pdf_thingy)
 
           return false if page_number && page_number > cleaned_pages.size
 
-          # Narrow to specific page if requested
           if page_number
             text = cleaned_pages[page_number - 1]
             return match_expected?(text, expected)
           end
 
-          # Search all pages
           cleaned_pages.any? { |page| match_expected?(page, expected) }
         end
 
-        # Matches the given text against the expected text or pattern.
-        #
-        # @param [String] text The text to match.
-        # @param [String, Regexp] expected The expected text or pattern.
-        # @return [Boolean] `true` if the text matches, `false` otherwise.
         def match_expected?(text, expected)
           return false unless text
 
           expected.is_a?(Regexp) ? text.match?(expected) : text.include?(expected.to_s)
         end
 
-        # Compares the content of two PDF objects for equality.
-        #
-        # @param [Object] actual_pdf_thingy The actual PDF object.
-        # @param [Object] expected_pdf_thingy The expected PDF object.
-        # @return [Boolean] `true` if the content matches, `false` otherwise.
         def match?(actual_pdf_thingy, expected_pdf_thingy)
-          actual = Bidi2pdf::TestHelpers::PDFReaderUtils.pdf_text actual_pdf_thingy
-          expected = Bidi2pdf::TestHelpers::PDFReaderUtils.pdf_text expected_pdf_thingy
+          cleaned_actual = clean_pages(actual_pdf_thingy)
+          cleaned_expected = clean_pages(expected_pdf_thingy)
 
-          cleaned_actual = clean_pages(actual)
-          cleaned_expected = clean_pages(expected)
-
-          # Compare without whitespace for equality check
           actual_for_comparison = cleaned_actual.map { |text| normalize(text) }
           expected_for_comparison = cleaned_expected.map { |text| normalize(text) }
 
-          if actual_for_comparison == expected_for_comparison
-            true
-          else
-            report_content_mismatch(cleaned_actual, cleaned_expected)
-            false
-          end
+          return true if actual_for_comparison == expected_for_comparison
+
+          report_content_mismatch(cleaned_actual, cleaned_expected)
+          false
         end
 
-        # Reports differences between actual and expected PDF content.
-        #
-        # @param [Array<String>] actual The actual PDF content.
-        # @param [Array<String>] expected The expected PDF content.
-        # @return [void]
         def report_content_mismatch(actual, expected)
-          puts "--- PDF content mismatch ---"
-          print_differences(actual, expected)
+          reporter.message(content_mismatch_message(actual, expected))
         end
 
-        # Prints detailed differences between actual and expected PDF content.
-        #
-        # @param [Array<String>] actual The actual PDF content.
-        # @param [Array<String>] expected The expected PDF content.
-        # @return [void]
-        def print_differences(actual, expected)
+        def content_mismatch_message(actual, expected)
+          sections = ["--- PDF content mismatch ---"]
           max_pages = [actual.length, expected.length].max
 
           (0...max_pages).each do |page_idx|
             actual_page = actual[page_idx] || "(missing page)"
             expected_page = expected[page_idx] || "(missing page)"
 
-            print_differences_for_page(actual_page, expected_page, page_idx)
+            page_message = page_difference_message(actual_page, expected_page, page_idx)
+            sections << page_message if page_message
           end
+
+          sections.join("\n")
         end
 
-        # Prints the differences between actual and expected content for a specific page.
-        # This method compares the content ignoring whitespace and, if differences are found,
-        # outputs a formatted representation of those differences.
-        #
-        # @param [String] actual_page The actual page content.
-        # @param [String] expected_page The expected page content.
-        # @param [Integer] page_idx The zero-based index of the page being compared.
-        # @return [void]
-        def print_differences_for_page(actual_page, expected_page, page_idx)
-          # Compare without whitespace
+        def page_difference_message(actual_page, expected_page, page_idx)
           actual_no_space = normalize(actual_page.to_s)
           expected_no_space = normalize(expected_page.to_s)
 
-          return if actual_no_space == expected_no_space
-
-          puts "\nPage #{page_idx + 1} differences (ignoring whitespace):"
+          return nil if actual_no_space == expected_no_space
 
-          # Create diffs between the two pages
           diffs = Diff::LCS.sdiff(expected_page, actual_page)
 
-          # Format and display the differences
-          puts format_diff_output(diffs, expected_page, actual_page)
+          [
+            "",
+            "Page #{page_idx + 1} differences (ignoring whitespace):",
+            format_diff_output(diffs, expected_page, actual_page)
+          ].join("\n")
         end
 
-        # Formats the output of differences for display.
-        #
-        # @param [Array<Diff::LCS::ContextChange>] diffs The list of differences.
-        # @param [String] expected The expected text.
-        # @param [String] actual The actual text.
-        # @return [String] The formatted differences.
         def format_diff_output(diffs, expected, actual)
-          output = []
-
           changes = group_changed_diffs(diffs)
 
-          # Output each change with context
-          changes.each do |change|
-            output += format_change expected, actual, change
-          end
+          changes.flat_map { |change| format_change(expected, actual, change) }.join("\n")
+        end
 
-          output.join("\n")
+        def reporter
+          RSpec.configuration.reporter
         end
 
         private
 
-        # Groups contiguous “real” diffs (added/removed/changed) into blocks,
-        # splitting whenever you hit an unchanged (“=”) diff.
         def group_changed_diffs(diffs)
           diffs
             .chunk_while { |_prev, curr| curr.action != "=" }
@@ -212,7 +134,6 @@ module Bidi2pdf
           }
         end
 
-        # 3. Build the final lines of output
         def build_output(snip_data, pos)
           start = snip_data[:context_start]
           ctx = snip_data[:context]

data/lib/bidi2pdf/test_helpers/testcontainers/chromedriver_test_helper.rb

@@ -16,6 +16,10 @@ module Bidi2pdf
       end
 
       module SessionTestHelper
+        def reporter
+          RSpec.configuration.reporter
+        end
+
         def chrome_args
           chrome_args = Bidi2pdf::Bidi::Session::DEFAULT_CHROME_ARGS.dup
 
@@ -24,7 +28,7 @@ module Bidi2pdf
           if ENV["DISABLE_CHROME_SANDBOX"]
             chrome_args << "--no-sandbox"
 
-            puts "🚨 Chrome sandbox disabled"
+            reporter.message("🚨 Chrome sandbox disabled")
           end
           chrome_args
         end
@@ -51,7 +55,7 @@ RSpec.configure do |config|
         shared_network: config.shared_network
       )
 
-      puts "🚀 chromedriver container started for tests"
+      reporter.message("🚀 chromedriver container started for tests")
     end
   end
 
@@ -61,20 +65,31 @@ RSpec.configure do |config|
 end
 
 def stop_container(container)
-  if container&.running?
+  return unless container
 
-    if ENV["SHOW_CONTAINER_LOGS"]
-      puts "Container logs:"
-      logs_std, logs_error = container.logs
+  dump_container_logs(container) if container.running?
+  stop_running_container(container)
+  container.remove
+end
 
-      puts logs_error
-      puts logs_std
-    end
+def dump_container_logs(container)
+  return unless ENV["SHOW_CONTAINER_LOGS"]
 
-    puts "🧹 #{container.image} stopping container..."
-    container.stop
-  end
-  container&.remove
+  stdout, stderr = container.logs
+  reporter.message("Container logs:")
+  reporter.message(stderr.to_s) unless stderr.to_s.empty?
+  reporter.message(stdout.to_s) unless stdout.to_s.empty?
+end
+
+def stop_running_container(container)
+  return unless container.running?
+
+  reporter.message("🧹 #{container.image} stopping container...")
+  container.stop
+end
+
+def reporter
+  RSpec.configuration.reporter
 end
 
 def chromedriver_tests_present?

data/lib/bidi2pdf/test_helpers/testcontainers/shared_docker_network.rb

@@ -11,7 +11,7 @@ RSpec.configure do |config|
 
     if uses_containers
       config.shared_network = Docker::Network.create("bidi2pdf-test-net-#{SecureRandom.hex(4)}")
-      puts "🕸️  started shared network #{config.shared_network}"
+      reporter.message("🕸️  started shared network #{config.shared_network}")
     end
   end
 
@@ -19,3 +19,7 @@ RSpec.configure do |config|
     config.shared_network&.remove
   end
 end
+
+def reporter
+  RSpec.configuration.reporter
+end

data/lib/bidi2pdf/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Bidi2pdf
-  VERSION = "0.1.11"
+  VERSION = "0.1.13"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bidi2pdf
 version: !ruby/object:Gem::Version
-  version: 0.1.11
+  version: 0.1.13
 platform: ruby
 authors:
 - Dieter S.
@@ -195,14 +195,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '4.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -538,14 +538,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 3.3.0
+      version: 3.4.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.8
+rubygems_version: 3.6.7
 specification_version: 4
 summary: A Ruby gem that generates PDFs from web pages using Chrome's BiDi protocol,
   providing high-quality PDF documents from any URL with full support for modern web