From fe84d243a6eccec58ce8de61d27c0272c2aee224 Mon Sep 17 00:00:00 2001 From: Jean Boussier Date: Wed, 22 Nov 2023 11:17:40 +0100 Subject: [PATCH] Update the CI matrix and fix test suite on Rails 7.1 --- .github/workflows/ci.yml | 8 ++------ Gemfile | 2 -- test/support/acceptance_test.rb | 6 +++++- test/support/application_generator.rb | 6 +++++- 4 files changed, 12 insertions(+), 10 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index ba3eda24..44bc360e 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -6,15 +6,11 @@ jobs: strategy: fail-fast: false matrix: - ruby: [ '2.7', '3.0.2', '3.1', '3.2', 'head' ] - rails: [ '6.0', '6.1', '7.0', 'edge' ] + ruby: [ '2.7', '3.0', '3.1', '3.2', 'head' ] + rails: [ '6.1', '7.0', '7.1', 'edge' ] exclude: - - ruby: '3.1' - rails: '6.0' - ruby: '3.1' rails: '6.1' - - ruby: '3.2' - rails: '6.0' - ruby: '3.2' rails: '6.1' diff --git a/Gemfile b/Gemfile index 25d1bc10..9515d42d 100644 --- a/Gemfile +++ b/Gemfile @@ -5,8 +5,6 @@ gemspec if ENV["RAILS_VERSION"] == "edge" gem "activesupport", github: "rails/rails", branch: "main" -elsif ENV['RAILS_VERSION'] == "7.0" - gem "activesupport", ">= 7.0.0.alpha" elsif ENV["RAILS_VERSION"] gem "activesupport", "~> #{ENV["RAILS_VERSION"]}.0" end diff --git a/test/support/acceptance_test.rb b/test/support/acceptance_test.rb index af2c36a0..34f829a2 100644 --- a/test/support/acceptance_test.rb +++ b/test/support/acceptance_test.rb @@ -139,7 +139,11 @@ def without_gem(name) test "raises if config.cache_classes is true" do config_path = app.path("config/environments/development.rb") config = File.read(config_path) - config.sub!(/config.cache_classes\s*=\s*false/, "config.cache_classes = true") + if config.include?("config.cache_classes") + config.sub!(/config\.cache_classes\s*=\s*false/, "config.cache_classes = true") + else # 7.1+ doesn't have config.cache_classes in the config at all + config.sub!(/config.enable_reloading = true/, "config.enable_reloading = true\nconfig.cache_classes = true") + end File.write(config_path, config) assert_failure "bin/rails runner 1", stderr: "Please, set config.cache_classes to false" diff --git a/test/support/application_generator.rb b/test/support/application_generator.rb index 8b813495..84a3403a 100644 --- a/test/support/application_generator.rb +++ b/test/support/application_generator.rb @@ -64,7 +64,11 @@ def generate_files end rewrite_file(application.path("config/environments/test.rb")) do |c| - c.sub!(/config\.cache_classes\s*=\s*true/, "config.cache_classes = false") + if c.include?("config.cache_classes") + c.sub!(/config\.cache_classes\s*=\s*true/, "config.cache_classes = false") + else # 7.1+ doesn't have config.cache_classes in the config at all + c.sub!(/config.enable_reloading = false/, "config.enable_reloading = false\nconfig.cache_classes = false") + end c end