rubysamurai / material_design_lite-sass

Google's Material Design Lite with Material Icons and Roboto font for Ruby applications
MIT License
98 stars 10 forks source link

Rspec Tests Failing after adding @import 'material' to application.css.scss #12

Closed disc0ninja closed 7 years ago

disc0ninja commented 7 years ago

After adding

@import 'material';

to /application.css.scss all of my feature tests fail with the following error message

Failure/Error: <%= stylesheet_link_tag    'application', media: 'all', 'data-turbolinks-track': 'reload' %>

     ActionView::Template::Error:
       wrong number of arguments (1 for 3) for `rgb'
     # (sass):3706
     # ./app/views/layouts/application.html.erb:7:in `_app_views_layouts_application_html_erb___504158236062439453_47019994749700'
     # ./spec/features/creating_article_spec.rb:6:in `block (2 levels) in <top (required)>'
     # ------------------
     # --- Caused by: ---
     # ArgumentError:
     #   wrong number of arguments (given 1, expected 3)
     #   ./app/views/layouts/application.html.erb:7:in `_app_views_layouts_application_html_erb___504158236062439453_47019994749700'

I removed the following lines before importing as per the readme instructions

 *= require_tree .
 *= require_self

And added the turbolinks fix described in the readme to application.js

I'm using rspec, and capybara for feature tests and using Rails 5.0.1 and ruby 2.4.0

Also commenting out

//@import 'material';

Results in passing tests again.

disc0ninja commented 7 years ago

Nevermind. It was an issue with one of the overrides that I had written.