-
Notifications
You must be signed in to change notification settings - Fork 143
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 assert_select wanting something to call .css on #39
base: 4.2
Are you sure you want to change the base?
Conversation
otherwise
fails with nasty undefined method css for Array :( |
Yes please |
@rafaelfranca this is some old 💩 but we still need that :D |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add a test case?
@@ -126,9 +126,15 @@ def assert_select_rjs(*args, &block) | |||
if matches | |||
assert true # to count the assertion | |||
if block_given? && !([:remove, :show, :hide, :toggle].include? rjs_type) | |||
to_select = if ActionPack::VERSION::STRING > "4.2.0" | |||
Nokogiri::HTML(matches.map(&:to_s).join) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this gem depends on Nokogiri? I don't think so. If we want to use nokogiri we need to make it a dependency
yeah I guess you are right ... this is too dirty |
What I did in Shopify was to keep the prototype JavaScript code and just remove all ruby methods calls that we had. |
I think we are ready to remove prototype ... lots of old routes gone now
... so maybe worth a new try :)
…On Thu, Jun 15, 2017 at 2:52 PM, Rafael França ***@***.***> wrote:
What I did in Shopify was to keep the prototype JavaScript code and just
remove all ruby methods calls that we had.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#39 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAAsZ5lCjHaPYZQn5_FIER9dyLO7eSkVks5sEaewgaJpZM4HwuGW>
.
|
@pschambacher