Merge pull request #54 from anynines/ds_563_parse_credhub

[DS-564] CredHub client returning JSON instead of string (2)
anynines · Mar 11, 2021 · 3790835 · 3790835
2 parents a134696 + c08d1ca
commit 3790835
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/unit_tests_utils/credhub.rb b/lib/unit_tests_utils/credhub.rb
@@ -7,7 +7,8 @@ class UnitTestsUtils::CredHub
   COMMAND_FAIL_PREFIX_MESSAGE = 'CredHub command failed:'
 
   def self.get_by_name(name)
-    execute(GET_COMMAND, "--name='#{name}'")
+    stdout, stderr = execute(GET_COMMAND, "--name='#{name}'")
+    return JSON.parse(stdout), stderr
   end
 
   private

diff --git a/spec/lib/unit_tests_utils/credhub_spec.rb b/spec/lib/unit_tests_utils/credhub_spec.rb
@@ -10,7 +10,7 @@
       msg_expected = "#{UnitTestsUtils::CredHub::COMMAND_FAIL_PREFIX_MESSAGE} #{command_expected}"
       allow(UnitTestsUtils::Cmd).to receive(:exec)
         .with(command_expected, msg_expected)
-        .and_return({})
+        .and_return('{}')
 
       get_by_name
     end