refact: simplify test framework integrations (#141)

* feat: simplify test framework integrations

* docs

* cleanup tests

* cleanup

Author: pftg

.dockerignore

@@ -0,0 +1,12 @@
+.github/
+.ruby-lsp/
+coverage/
+tmp/
+
+# Ignore report files
+*.attempt_*.png
+*.diff.png
+*.base.png
+*.attempt_*.webp
+*.diff.webp
+*.base.webp

.github/actions/setup-ruby-and-dependencies/action.yml

@@ -35,3 +35,6 @@ runs:
     # fallback if cache version is outdated
     - run: sudo apt-get -qq install libvips
       shell: bash
+
+    - run: sudo sed -i 's/true/false/g' /etc/fonts/conf.d/10-yes-antialias.conf
+      shell: bash

.github/workflows/test.yml

@@ -21,7 +21,7 @@ env:
   JAVA_OPTS: -Xmn2g -Xms6g -Xmx6g -XX:MetaspaceSize=256m -XX:MaxMetaspaceSize=512m -Xss1m
     -XX:+UseG1GC -XX:+TieredCompilation -XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=256m
     -XX:+UseCodeCacheFlushing
-  JRUBY_OPTS: --dev
+  JRUBY_OPTS: --dev -J-Djruby.thread.pool.enabled=true
   MALLOC_ARENA_MAX: 2
   RUBY_GC_HEAP_FREE_SLOTS: 600000
   RUBY_GC_HEAP_GROWTH_FACTOR: 1.1
@@ -45,6 +45,8 @@ jobs:
           ruby-version: 3.4
 
       - run: bin/rake test
+        env:
+          SCREENSHOT_DRIVER: vips
 
   functional-test:
     name: Functional Test
@@ -87,11 +89,11 @@ jobs:
       github.event.pull_request.requested_reviewers.length > 0
     needs: [ functional-test ]
     runs-on: ubuntu-latest
-    timeout-minutes: ${{ contains(matrix.ruby-version, 'jruby') && 12 || 8 }}
+    timeout-minutes: ${{ contains(matrix.ruby-version, 'jruby') && 20 || 8 }}
     continue-on-error: ${{ matrix.experimental }}
     strategy:
       matrix:
-        ruby-version: [ 3.4, 3.3, 3.2, jruby ]
+        ruby-version: [ 3.4, 3.3, 3.2, jruby-9.4, jruby-10.0 ]
         gemfile:
           - rails70_gems.rb
           - rails71_gems.rb
@@ -104,7 +106,7 @@ jobs:
             gemfile: rails80_gems.rb
             experimental: false
           # JRuby 9.x is Ruby 3.1 compatible, and Rails 8 requires Ruby 3.2.
-          - ruby-version: jruby
+          - ruby-version: jruby-9.4
             gemfile: rails80_gems.rb
             experimental: false
         include:
@@ -130,7 +132,7 @@ jobs:
       - name: Run tests (with 2 retries)
         uses: nick-fields/retry@v3
         with:
-          timeout_minutes: ${{ contains(matrix.ruby-version, 'jruby') && 3 || 3 }}
+          timeout_minutes: ${{ contains(matrix.ruby-version, 'jruby') && 7 || 3 }}
           max_attempts: 3
           command: bin/rake test
 

.gitignore

@@ -1,6 +1,7 @@
 *~
 /.bundle/
 /.idea
+/.windsurf
 /.yardoc
 /_yardoc/
 /coverage/

.standard.yml

@@ -2,7 +2,7 @@
 fix: true               # default: false
 parallel: true          # default: false
 format: progress        # default: Standard::Formatter
-ruby_version: 3.2       # default: RUBY_VERSION
+ruby_version: 3.1       # to support JRuby 9.4
 default_ignores: false  # default: true
 
 ignore: # default: []

Dockerfile

@@ -3,11 +3,15 @@
 #   $ docker build . -t csd
 #   $ docker run -v $(pwd):/app -ti csd rake test
 
-FROM --platform=linux/amd64 jetthoughts/cimg-ruby:3.4-chrome
+FROM jetthoughts/cimg-ruby:3.4-chrome
 
-# Install dependencies and clean up in one layer to reduce image size
-RUN sudo apt-get update -qq && \
-    DEBIAN_FRONTEND=noninteractive sudo apt-get install -qq \
+ENV DEBIAN_FRONTEND=noninteractive \
+ BUNDLE_PATH=/bundle
+
+RUN --mount=type=cache,target=/var/cache/apt \
+    sudo sed -i 's|http://security.ubuntu.com/ubuntu|http://archive.ubuntu.com/ubuntu|g' /etc/apt/sources.list && \
+    sudo apt-get update -qq && \
+    sudo apt-get install -qq --fix-missing \
       automake \
       build-essential \
       curl \
@@ -34,20 +38,12 @@ RUN sudo apt-get update -qq && \
       libwebp-dev \
       libxml2-dev \
       swig && \
-    sudo apt-get autoremove -y && \
-    sudo apt-get autoclean && \
-    sudo apt-get clean && \
-    sudo rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/*
+    sudo apt-get autoclean
 
-WORKDIR /app
-COPY gems.rb gemfiles capybara-screenshot-diff.gemspec /app/
-COPY lib/capybara/screenshot/diff/version.rb /app/lib/capybara/screenshot/diff/
+RUN sudo sed -i 's/true/false/g' /etc/fonts/conf.d/10-antialias.conf
 
-# Set the location for Bundler to store gems
-ENV BUNDLE_PATH=/bundle
 
-RUN sudo mkdir /bundle && \
-    sudo chmod a+w+r /bundle \
-    sudo mkdir -p /tmp/.X11-unix && \
-    sudo chmod 1777 /tmp/.X11-unix
+RUN sudo mkdir -p /bundle /tmp/.X11-unix && \
+    sudo chmod 1777 /bundle /tmp/.X11-unix
 
+WORKDIR /app

bin/dtest

@@ -1,14 +1,37 @@
 #!/bin/bash
 
-set -eo pipefail
+set -o pipefail
 
 export DOCKER_DEFAULT_PLATFORM=linux/amd64
 
+# Define allowed environment variables to pass to Docker
+ALLOWED_ENV_VARS=(
+  "CI" "DEBUG" "TEST_ENV" "RAILS_ENV" "RACK_ENV" "COVERAGE" "DISABLE_ROLLBACK_COMPARISON_RUNTIME_FILES"
+  "RECORD_SCREENSHOTS" "TEST" "TESTOPTS" "SCREENSHOT_DRIVER"
+)
+
+# Build the Docker env args string
+DOCKER_ENV_ARGS=""
+for var in "${ALLOWED_ENV_VARS[@]}"; do
+  if [[ -n "${!var}" ]]; then
+    DOCKER_ENV_ARGS="$DOCKER_ENV_ARGS -e $var=${!var}"
+  fi
+done
+
+# Build the Docker image
 docker build . -t csd:test
 
-docker run -v ${PWD}:/app -v csd-bundle-cache:/bundle --rm -it csd:test bin/setup
+# Run setup
+(docker run $DOCKER_ENV_ARGS -v ${PWD}:/app -v csd-bundle-cache:/bundle --rm -it csd:test bin/setup) || exit 1
 
+# Run tests with different drivers
 echo "Running tests..."
-docker run -e CAPYBARA_DRIVER=cuprite -v ${PWD}:/app -v csd-bundle-cache:/bundle --rm -it csd:test bin/rake test
-docker run -e CAPYBARA_DRIVER=selenium_chrome_headless -v ${PWD}:/app -v csd-bundle-cache:/bundle --rm -it csd:test bin/rake test
-docker run -e CAPYBARA_DRIVER=selenium_headless -v ${PWD}:/app -v csd-bundle-cache:/bundle --rm -it csd:test bin/rake test
+DRIVERS=("cuprite" "selenium_chrome_headless" "selenium_headless")
+for driver in "${DRIVERS[@]}"; do
+  echo "Running tests with $driver driver..."
+  docker run $DOCKER_ENV_ARGS -e CAPYBARA_DRIVER="$driver" \
+    -v ${PWD}:/app -v csd-bundle-cache:/bundle --rm -it csd:test \
+    bin/rake test "$@"
+
+  CAPYBARA_DRIVER="$driver" bin/rake test "$@"
+done

capybara-screenshot-diff.gemspec

@@ -23,6 +23,7 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_runtime_dependency "actionpack", ">= 7.0", "< 9"
+  spec.add_development_dependency "actionpack", ">= 7.0", "< 9"
+  spec.add_development_dependency "activesupport", ">= 7.0", "< 9"
   spec.add_runtime_dependency "capybara", ">= 2", "< 4"
 end

gemfiles/edge_gems.rb

@@ -4,5 +4,6 @@
 eval File.read(gems), binding, gems
 
 git "https://github.com/rails/rails.git" do
+  gem "activesupport"
   gem "actionpack"
 end

gemfiles/rails70_gems.rb

@@ -4,6 +4,7 @@
 eval File.read(gems), binding, gems
 
 gem "actionpack", "~> 7.0.0"
+gem "activesupport", "~> 7.0.0", require: %w[active_support/deprecator active_support/test_case]
 gem "mutex_m"
 gem "drb"
 gem "bigdecimal"