Fixes #36954 - add cache status to ping output

theforeman · Nov 29, 2023 · 4cf8232 · 4cf8232
1 parent c5e56c7
commit 4cf8232
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 0 deletions.
diff --git a/lib/hammer_cli_foreman/ping.rb b/lib/hammer_cli_foreman/ping.rb
@@ -12,6 +12,12 @@ class ForemanCommand < HammerCLIForeman::Command
             field :active, _('Status')
             field :duration_ms, _('Server Response')
           end
+          field :cache, _('cache'), Fields::Label do
+            field :servers, _('servers'), Fields::Collection do
+              field :status, _('Status')
+              field :duration_ms, _('Server Response')
+            end
+          end
         end
       end
 

diff --git a/test/functional/ping_test.rb b/test/functional/ping_test.rb
@@ -33,6 +33,31 @@
       assert_cmd(expected_result, result)
     end
 
+    it 'includes cache status if foreman reports it' do
+      ping_results['results']['foreman']['cache'] = {
+        'servers' => [
+          {
+            'status' => 'ok',
+            'duration_ms' => 5
+          }
+        ]
+      }
+      api_expects(:ping, :ping, 'Ping').returns(ping_results)
+
+      output = OutputMatcher.new(
+        [
+          'cache:',
+          '    servers:',
+          '     1) Status:          ok',
+          '        Server Response: 5'
+        ]
+      )
+
+      expected_result = success_result(output)
+      result = run_cmd(cmd)
+      assert_cmd(expected_result, result)
+    end
+
     it 'returns 1 if one of the services failed and shows unrecognized services' do
       ping_results['results']['new_plugin'] = {
         'services' => {