foodcritic-junit 0.1.5 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a0f7a26777b21411625583b1757dac376ae70e6a
-  data.tar.gz: 1ddb79448fe00760f8faaef81d005fd10fdfce61
+  metadata.gz: a3c5023726931552afcfb5475468cbb5b99fd78b
+  data.tar.gz: 82910a0f5e7bf374c0fc6b9e35877c5f63b21413
 SHA512:
-  metadata.gz: 032578005442d3f7d530868df65dfa39551e910a193ab37450d089f808a4c7ccbfafa776fff9bbb0a0c19ad6d3c133d9a766c27ec16578b250011d8fb851aa85
-  data.tar.gz: 81678487786ce326b1eb59ccc28d1c29f8ff3133b3bded557bad9fe9ad9e0be487e013375023fd1fb3948a6dd9c23251e0c58f739131fc58932d6806e2cabc5f
+  metadata.gz: c5c0c319a0f5e5f0c2164ca3d46f0f369254c31b360a7407ed1f7084e8e961d2c39fb227f05f613224465d73f12c84f001dbac7244a1acc171a45f022fdd386b
+  data.tar.gz: 70d41c66aa38dafa600ce94ecc62c363fc19dd289c23e568815f2e1412d30030ace0cb1c695bb32525730d7cc9f38784d39135661b93c5e89534828ccc0e26fd

data/lib/foodcritic/junit/outputter.rb

@@ -45,7 +45,7 @@ module Foodcritic
             @current_violation = violation
             @current_violation_lines = []
           else
-            current_violation_lines << line.encode(xml: :attr) if current_violation_lines
+            current_violation_lines << line if current_violation_lines
           end
         end
       end
@@ -64,7 +64,7 @@ module Foodcritic
         <<-EOS
 <?xml version="1.0" encoding="UTF-8"?>
 <testsuites>
-  <testsuite name="#{cookbook_name}" timestamp="#{Time.now.utc.iso8601}">
+  <testsuite name=#{cookbook_name.encode(xml: :attr)} timestamp=#{Time.now.utc.iso8601.to_s.encode(xml: :attr)}>
     #{violations_as_xml}
   </testsuite>
 </testsuites>
@@ -80,12 +80,13 @@ module Foodcritic
       end
 
       def xml_for_violation(violation)
-        name = "#{violation[:rule]}: #{violation[:message]}"
-        file_name = violation[:file_name]
+        name = "#{violation[:rule]}: #{violation[:message]}".encode(xml: :attr)
+        file_name = violation[:file_name].encode(xml: :attr)
+        location = "Located in #{violation[:file_name]}".encode(xml: :attr)
         <<-EOS
-      <testcase name="#{name}" classname="#{file_name}" assertions="0" time="0">
-        <error type="#{name}" message="Located in #{file_name}">
-#{violation[:lines]}
+      <testcase name=#{name} classname=#{file_name} assertions="0" time="0">
+        <error type=#{name} message=#{location}>
+          #{violation[:lines].encode(xml: :text)}
         </error>
       </testcase>
         EOS

data/lib/foodcritic/junit/version.rb

@@ -1,5 +1,5 @@
 module Foodcritic
   module Junit
-    VERSION = "0.1.5"
+    VERSION = "0.1.6"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foodcritic-junit
 version: !ruby/object:Gem::Version
-  version: 0.1.5
+  version: 0.1.6
 platform: ruby
 authors:
 - Thomas Vendetta