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

Rails 6.1: ActionView::Base new now has 3 required arguments #8411

Merged

Conversation

jrafanie
Copy link
Member

@jrafanie jrafanie commented Aug 12, 2022

Fixes this error on 6.1 jasmine specs: TEST_SUITE=spec:javascript bundle exec rake

  #<ArgumentError: wrong number of arguments (given 0, expected 3)>
  /Users/joerafaniello/.gem/ruby/2.7.5/gems/actionview-6.1.6.1/lib/action_view/base.rb:230:in `initialize'
  /Users/joerafaniello/Code/manageiq-ui-classic/spec/javascripts/support/jasmine_helper.rb:18:in `new'
  /Users/joerafaniello/Code/manageiq-ui-classic/spec/javascripts/support/jasmine_helper.rb:18:in `call'
  /Users/joerafaniello/.gem/ruby/2.7.5/gems/rack-2.2.4/lib/rack/urlmap.rb:74:in `block in call'

Note, this is a rails 6.1+ only change discovered by but not caused by #8408. This was broken with rails 6.1 but didn't get detected in cross repo.

I've added this to ManageIQ/manageiq#21516 (comment)

Replaces #8410

Fixes this error on 6.1 jasmine specs: `TEST_SUITE=spec:javascript bundle exec rake`

```
  #<ArgumentError: wrong number of arguments (given 0, expected 3)>
  /Users/joerafaniello/.gem/ruby/2.7.5/gems/actionview-6.1.6.1/lib/action_view/base.rb:230:in `initialize'
  /Users/joerafaniello/Code/manageiq-ui-classic/spec/javascripts/support/jasmine_helper.rb:18:in `new'
  /Users/joerafaniello/Code/manageiq-ui-classic/spec/javascripts/support/jasmine_helper.rb:18:in `call'
  /Users/joerafaniello/.gem/ruby/2.7.5/gems/rack-2.2.4/lib/rack/urlmap.rb:74:in `block in call'
```
@Fryguy Fryguy self-assigned this Aug 12, 2022
@miq-bot
Copy link
Member

miq-bot commented Aug 12, 2022

Checked commit jrafanie@87f0386 with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint
1 file checked, 1 offense detected

spec/javascripts/support/jasmine_helper.rb

@Fryguy Fryguy merged commit be1dd58 into ManageIQ:master Aug 12, 2022
@jrafanie jrafanie deleted the fix_jasmine_headless_chrome_rails_61 branch August 12, 2022 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants