Merge pull request #1258 from seanpdoyle/rails-engine-test-coverage

`action_text-trix`: Add test coverage for Engine

Author: flavorjones

.github/workflows/ci.yml

@@ -53,6 +53,59 @@ jobs:
         run: |
           cd rails/actiontext
           bundle exec rake test test:system
+  action_text-trix:
+    name: Action Text tests
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+          - ruby: "3.2"
+            rails_branch: 7-2-stable
+          - ruby: "3.3"
+            rails_branch: 7-2-stable
+          - ruby: "3.4"
+            rails_branch: 7-2-stable
+
+          - ruby: "3.2"
+            rails_branch: 8-0-stable
+          - ruby: "3.3"
+            rails_branch: 8-0-stable
+          - ruby: "3.4"
+            rails_branch: 8-0-stable
+
+          - ruby: "3.2"
+            rails_branch: main
+            experimental: true
+          - ruby: "3.3"
+            rails_branch: main
+            experimental: true
+          - ruby: "3.4"
+            rails_branch: main
+            experimental: true
+
+          - ruby: head
+            rails_branch: main
+            experimental: true
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 18
+          cache: "yarn"
+      - uses: ruby/setup-ruby@v1
+        env:
+          RAILS_BRANCH: ${{ matrix.rails_branch }}
+        with:
+          working-directory: "./action_text-trix"
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+      - name: Run tests
+        env:
+          RAILS_BRANCH: ${{ matrix.rails_branch }}
+        continue-on-error: ${{ matrix.experimental }}
+        working-directory: "./action_text-trix"
+        run: bin/rails test:all
 
 env:
   SAUCE_USERNAME: ${{ secrets.SAUCE_USERNAME }}

action_text-trix/.gitignore

@@ -1,2 +1,6 @@
 Gemfile.lock
 pkg
+tmp/
+log/
+storage/
+vendor/

action_text-trix/Gemfile

@@ -2,3 +2,14 @@ source "https://rubygems.org"
 
 # Specify your gem's dependencies in trix.gemspec.
 gemspec
+
+branch = ENV.fetch("RAILS_BRANCH", "main")
+gem "rails", github: "rails/rails", branch: branch
+gem "importmap-rails"
+gem "propshaft"
+gem "puma"
+gem "sqlite3"
+
+group :test do
+  gem "cuprite", require: "capybara/cuprite"
+end

action_text-trix/Rakefile

@@ -1,6 +1,11 @@
 require "bundler/gem_tasks"
 require "rake/clean"
 
+if defined?(Rails)
+  APP_RAKEFILE = File.expand_path("test/dummy/Rakefile", __dir__)
+  load "rails/tasks/engine.rake"
+end
+
 task :sync do
   require "json"
 

action_text-trix/bin/rails

@@ -0,0 +1,15 @@
+#!/usr/bin/env ruby
+# This command will automatically be run when you run "rails" with Rails gems
+# installed from the root of your application.
+
+ENGINE_ROOT = File.expand_path('..', __dir__)
+ENGINE_PATH = File.expand_path('../lib/action_text/trix/engine.rb', __dir__)
+APP_PATH = File.expand_path('../test/dummy/config/application', __dir__)
+
+# Set up gems listed in the Gemfile.
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../Gemfile', __dir__)
+require "bundler/setup" if File.exist?(ENV["BUNDLE_GEMFILE"])
+
+require "rails/all"
+require "rails/test_unit/railtie"
+require "rails/engine/commands"

action_text-trix/test/application_system_test_case.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+class ApplicationSystemTestCase < ActionDispatch::SystemTestCase
+  driven_by :cuprite, using: :chrome, options: {
+    js_errors: true,
+    headless: ENV["HEADLESS"] != "0"
+  }
+end
+
+Capybara.server = :puma, { Silent: true }

action_text-trix/test/dummy/Rakefile

@@ -0,0 +1,6 @@
+# Add your own tasks in files placed in lib/tasks ending in .rake,
+# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
+
+require_relative "config/application"
+
+Rails.application.load_tasks

action_text-trix/test/dummy/app/assets/images/.keep

@@ -0,0 +1,15 @@
+/*
+ * This is a manifest file that'll be compiled into application.css, which will include all the files
+ * listed below.
+ *
+ * Any CSS and SCSS file within this directory, lib/assets/stylesheets, vendor/assets/stylesheets,
+ * or any plugin's vendor/assets/stylesheets directory can be referenced here using a relative path.
+ *
+ * You're free to add application-wide styles to this file and they'll appear at the bottom of the
+ * compiled file so the styles you add here take precedence over styles defined in any other CSS/SCSS
+ * files in this directory. Styles in this file should be added after the last require_* statement.
+ * It is generally better to create a new file per style scope.
+ *
+ *= require_tree .
+ *= require_self
+ */