-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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 tests propshaft #51903
Fix tests propshaft #51903
Conversation
f45791c
to
6669fc8
Compare
659e702
to
91216d9
Compare
@@ -156,28 +148,10 @@ class User < ActiveRecord::Base; raise 'should not be reached'; end | |||
assert_no_file_exists("#{app_path}/public/assets/something.else-*.css") | |||
end | |||
|
|||
test "precompile something.js for directory containing index file" do |
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.
Curious what you're planning to do with the sprockets assets tests
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.
My idea is to change tests to be compatible with Propshaft and remove those that are not.
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.
I don't think we should lose the sprockets tests, since we're not deprecating/removing it afaik
91216d9
to
5ea5d07
Compare
000c241
to
069a1b6
Compare
069a1b6
to
617a0c8
Compare
This PR fixes tests blocking Propshaft from being merged to rails.