Merge pull request #421 from mozilla/fix_419_v2

Fix 419 v2
mozilla · Oct 2, 2017 · 126d509 · 126d509
2 parents 85d637f + 2925900
commit 126d509
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/bin/ssh_scan b/bin/ssh_scan
@@ -261,7 +261,7 @@ puts JSON.pretty_generate(results)
 
 if options["unit_test"] == true
   results.each do |result|
-    if result.compliant == false
+    if result["compliance"] && result["compliance"]["compliant"] == false
       exit 1 #non-zero means a false
     else
       exit 0 #non-zero means pass

diff --git a/lib/ssh_scan/policy_manager.rb b/lib/ssh_scan/policy_manager.rb
@@ -212,10 +212,10 @@ def recommendations
 
     def compliance_results
       {
-        :policy => @policy.name,
-        :compliant => compliant?,
-        :recommendations => recommendations,
-        :references => @policy.references,
+        "policy" => @policy.name,
+        "compliant" => compliant?,
+        "recommendations" => recommendations,
+        "references" => @policy.references,
       }
     end
   end

diff --git a/lib/ssh_scan/version.rb b/lib/ssh_scan/version.rb
@@ -1,3 +1,3 @@
 module SSHScan
-  VERSION = '0.0.28'
+  VERSION = '0.0.29'
 end