Merge pull request #586 from nthanes/command-stdout-as-json

[WIP] command stdout_as_json feature request
mizzy · Jan 16, 2017 · 5e518e1 · 5e518e1
2 parents 8ddea2b + 08a5378
commit 5e518e1
Show file tree

Hide file tree

Showing 2 changed files with 45 additions and 0 deletions.
diff --git a/lib/serverspec/type/command.rb b/lib/serverspec/type/command.rb
@@ -1,9 +1,15 @@
+require 'multi_json'
+
 module Serverspec::Type
   class Command < Base
     def stdout
       command_result.stdout
     end
 
+    def stdout_as_json
+      MultiJson.load(command_result.stdout)
+    end
+
     def stderr
       command_result.stderr
     end

diff --git a/spec/type/base/command_spec.rb b/spec/type/base/command_spec.rb
@@ -65,3 +65,42 @@
   its(:stdout) { should contain('4260').before('home') }
   its(:stdout) { should_not contain('4260').before('bin') }
 end
+
+describe command('curl http://localhost:8080/info') do
+  let(:stdout) { <<EOF
+{
+   "sensu":{
+      "version":"0.26.5"
+   },
+   "transport":{
+      "keepalives":{
+         "messages":0,
+         "consumers":1
+      },
+      "results":{
+         "messages":0,
+         "consumers":1
+      },
+      "connected":true
+   },
+   "redis":{
+      "connected":true
+   },
+   "array":[
+      {
+         "title":"array 1"
+      },
+      {
+         "title":"array 2"
+      }
+   ]
+}
+EOF
+  }
+
+  its(:stdout_as_json) { should include('sensu') }
+  its(:stdout_as_json) { should include('sensu' => include('version' => '0.26.5')) }
+  its(:stdout_as_json) { should include('transport' => include('keepalives' => include('consumers' => 1))) }
+  its(:stdout_as_json) { should include('transport' => include('connected' => true)) }
+  its(:stdout_as_json) { should include('array' => include('title' => 'array 2')) }
+end