Add test for detecting sass variables in compiled css

primer · Dec 8, 2022 · a3ea629 · a3ea629
1 parent 0b5f100
commit a3ea629
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/test/components/component_css_test.rb b/test/components/component_css_test.rb
@@ -13,8 +13,12 @@ def test_css_is_compiled_correctly
     Dir["app/components/**/*.css"].each do |file|
       css = File.read(file)
 
+      # remove comments
+      css.gsub!(/\/\*((?!\*\/).)*\*\//m, "")
+
       refute(css.include?("@import"), "CSS files should not import other CSS files:\n#{file} contains @import")
       refute(css.include?("&"), "CSS Nesting wasn't compiled correctly:\n#{file} contains &")
+      refute(css.include?("$"), "Sass variable(s) detected:\n#{file} contains $")
     end
   end
 end