Merge pull request #2778 from rspec/add-rails-7.2-to-matrix

Add Rails 7.2 to CI matrix

Author: pirj

.github/workflows/ci.yml

@@ -36,15 +36,29 @@ jobs:
       fail-fast: false
       matrix:
        include:
-         # Edge Rails (?) builds >= 2.7
+         # Edge Rails (?) builds >= 3.1
          - ruby: 3.2
            env:
              RAILS_VERSION: 'main'
          - ruby: 3.1
            env:
              RAILS_VERSION: 'main'
 
+         # Rails 7.2 builds >= 3.1
+         - ruby: 3.3
+           env:
+             RAILS_VERSION: '~> 7.2.0'
+         - ruby: 3.2
+           env:
+             RAILS_VERSION: '~> 7.2.0'
+         - ruby: 3.1
+           env:
+             RAILS_VERSION: '~> 7.2.0'
+
          # Rails 7.1 builds >= 2.7
+         - ruby: 3.3
+           env:
+             RAILS_VERSION: '~> 7.1.0'
          - ruby: 3.2
            env:
              RAILS_VERSION: '~> 7.1.0'

example_app_generator/generate_app.rb

@@ -67,7 +67,7 @@
 
   copy_file capybara_backport_path, 'spec/support/capybara.rb'
 
-  if Rails::VERSION::STRING > '7'
+  if Rails::VERSION::STRING > '7' && Rails::VERSION::STRING < '7.2'
     create_file 'app/assets/config/manifest.js' do
       "//= link application.css"
     end

example_app_generator/spec/support/default_preview_path

@@ -69,7 +69,7 @@ exit(0) if ENV['NO_ACTION_MAILER']
 if ENV['DEFAULT_URL']
   puts ActionMailer::Base.default_url_options[:host]
 elsif defined?(::ActionMailer::Preview)
-  if Rails::VERSION::STRING.start_with?('7.1')
+  if Rails::VERSION::STRING.to_f >= 7.1
     puts Rails.application.config.action_mailer.preview_paths
   else
     puts Rails.application.config.action_mailer.preview_path

spec/sanity_check_spec.rb

@@ -28,6 +28,7 @@ def with_clean_env
       expect(`bundle exec #{script} 2>&1`)
         .to match(/uninitialized constant RSpec::Support/)
         .or match(/undefined method `require_rspec_core' for RSpec::Support:Module/)
+        .or match(/undefined method `require_rspec_core' for module RSpec::Support/)
 
       expect($?.exitstatus).to eq(1)
     end