Fix printing empty testcase if no errors

florianeckerstorfer · Jun 9, 2015 · b40bcbb · b40bcbb
1 parent 16cb19e
commit b40bcbb
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -43,6 +43,10 @@ scss-lint --require=scss_lint_reporter_junit --format=JUnit foo.scss
 Changelog
 ---------
 
+### Version 0.1.2 (9 June 2015)
+
+- Fix printing empty testcase if no errors
+
 ### Version 0.1.1 (9 June 2015)
 
 - Add missing line break between error messages

diff --git a/lib/scss_lint_reporter_junit.rb b/lib/scss_lint_reporter_junit.rb
@@ -5,7 +5,7 @@ def report_lints
       output = "<?xml version=\"1.0\" encoding=\"utf-8\"?>\n"
 
       output << "<testsuite name=\"scsslint\" failures=\"#{results.length}\">\n"
-      if !results.length
+      if results.length == 0
         output << "  <testcase name=\"Empty\"/>\n"
       end
       results.each do |file_name, errors|

diff --git a/scss_lint_reporter_junit.gemspec b/scss_lint_reporter_junit.gemspec
@@ -4,7 +4,7 @@ Gem::Specification.new do |s|
   s.platform          = Gem::Platform::RUBY
   s.license           = "MIT"
   s.name              = "scss_lint_reporter_junit"
-  s.version           = "0.1.1"
+  s.version           = "0.1.2"
   s.summary           = "Extend the scss-lint with a jUnit formatter"
   s.description       = "This gem add a new formatter to the scss-lint which is compatible with the jUnit standard."
   s.homepage          = "https://github.com/florianeckerstorfer/scss_lint_reporter_junit"