The Wayback Machine - https://web.archive.org/web/20200919085437/https://github.com/rails/rails-controller-testing/pull/56
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix travis failures #56

Merged
merged 4 commits into from May 5, 2020
Merged

Conversation

@eileencodes
Copy link
Member

eileencodes commented Apr 16, 2020

  • Removes EOL'd Ruby and Rails versions
  • Adds a Gemfile for 6.0
  • Fixes the test failures we were seeing on new PRs on relative file renders
eileencodes added 3 commits Apr 16, 2020
Master is on 6.1 so we should add a Gemfile for 6.0
Rails 5.0 and 5.1 as well as Ruby 2.2, 2.3, and 2.4 are EOL. We can
remove them.

Adds 6.0 Gemfile for testing.
rails/rails#35688 deprecated calling `render file` and letting it lookup
the file from anywhere. This was deprecated in 6.0 and removed in 6.1,
which is why travis was failing for new PRs.
@eileencodes eileencodes force-pushed the eileencodes:fix-travis-failures branch from b5c1e5c to 951c784 May 4, 2020
These tests are now failing because of changes made in
rails/rails#35688. This causes the instrumentation for `@_files` to not
get run, but also don't think this should be supported since that
instrumentation is for templates and now files are just rendered as
files not as templates.
@eileencodes eileencodes force-pushed the eileencodes:fix-travis-failures branch from 951c784 to 9acf9b7 May 4, 2020
@eileencodes eileencodes requested a review from jhawthorn May 4, 2020
@lukebooth
Copy link

lukebooth commented May 5, 2020

🎉 This will help the Ruby 2.7 PRs along. Thanks!!

@eileencodes eileencodes merged commit 6f07f82 into rails:master May 5, 2020
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@eileencodes eileencodes deleted the eileencodes:fix-travis-failures branch May 5, 2020
@jhawthorn jhawthorn mentioned this pull request May 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.