Skip to content

Commit

Permalink
Test with Rails 7.1 (#79)
Browse files Browse the repository at this point in the history
* Rails 7.1: call `to_s` on `rendered` proxy

Probably stemming from `rendered` having been converted to a proxy object in rails/rails#49194

* CI with Rails 7.1

* Don't eager_load on CI

* Needs to be moved up further
  • Loading branch information
kaspth authored Oct 9, 2023
1 parent c07bc81 commit b4eb299
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
fail-fast: false
matrix:
rails-version:
- "7.0"
- "7.1"
- "6.1"
- "main"
ruby-version:
Expand Down
2 changes: 1 addition & 1 deletion test/capybara_extensions/assertions.rb
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,6 @@ def assert_no_#{selector}(...)
include CapybaraExtensions::Assertions

def page
@page ||= Capybara.string(rendered)
@page ||= Capybara.string(rendered.to_s)
end
end
3 changes: 3 additions & 0 deletions test/test_helper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,10 @@

require_relative "../test/dummy/config/environment"
ActiveRecord::Migrator.migrations_paths = [File.expand_path("../test/dummy/db/migrate", __dir__)]
# Don't eager load, since that expects Action Mailbox tables to be present in database.
Rails.configuration.eager_load = false
require "rails/test_help"

require "capybara_extensions"
require "template_helpers"

Expand Down

0 comments on commit b4eb299

Please sign in to comment.