Merge pull request #10 from codeclimate/strip-non-css-files-from-incl…

…ude-paths

Strip out non-CSS files directly included in include_paths.

lib/cc/engine/csslint.rb

@@ -58,9 +58,9 @@ def build_files_with_inclusions(inclusions)
           if include_path =~ %r{/$}
             Dir.glob("#{include_path}/**/*.css")
           else
-            include_path
+            include_path if include_path =~ /\.css$/
           end
-        end.flatten
+        end.flatten.compact
       end
 
       def csslint_xml

spec/cc/engine/csslint_spec.rb

@@ -38,7 +38,7 @@ module Engine
 
         describe "with include_paths" do
           let(:engine_config) {
-            {"include_paths" => %w(included.css included_dir/)}
+            {"include_paths" => %w(included.css included_dir/ config.yml)}
           }
 
           before do
@@ -49,6 +49,7 @@ module Engine
             create_source_file(
               "included_dir/sub/sub/subdir/file.css", "img { }"
             )
+            create_source_file("config.yml", "foo:\n  bar: \"baz\"")
             create_source_file("not_included.css", "a { outline: none; }")
           end
 
@@ -72,6 +73,10 @@ module Engine
             expect{ lint.run }.to \
               output(/Don't use IDs in selectors./).to_stdout
           end
+
+          it "only includes CSS files, even when a non-CSS file is directly included" do
+            expect{ lint.run }.not_to output(/config.yml/).to_stdout
+          end
         end
       end