hammer_cli_foreman 2.3.1 → 2.4.0

data/test/functional/mail_notification_test.rb

@@ -24,6 +24,26 @@ describe 'mail_notification' do
       result = run_cmd(@cmd)
       assert_cmd(expected_result, result)
     end
+
+    it 'should run list command with defaults' do
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      defaults = HammerCLI::Defaults.new(
+        {
+          organization_id: {
+            provider: 'foreman'
+          },
+          location_id: {
+            provider: 'foreman'
+          }
+        }
+      )
+      defaults.stubs(:write_to_file).returns(true)
+      defaults.stubs(:providers).returns(providers)
+      api_expects(:mail_notifications, :index, 'List mail notifications').returns(@mail_notifications)
+
+      result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })
+      _(result.exit_code).must_equal HammerCLI::EX_OK
+    end
   end
 
   describe 'info' do

data/test/functional/model_test.rb

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'model' do
+  describe 'list' do
+    before do
+      @cmd = %w[model list]
+      @models = [{
+                          id: 1,
+                          name: 'model',
+                        }]
+    end
+
+    it 'should return a list of models' do
+      api_expects(:models, :index, 'List models').returns(@models)
+
+      output = IndexMatcher.new([
+                                  %w[ID NAME],
+                                  %w[1 model]
+                                ])
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'should run list command with defaults' do
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      defaults = HammerCLI::Defaults.new(
+        {
+          organization_id: {
+            provider: 'foreman'
+          },
+          location_id: {
+            provider: 'foreman'
+          }
+        }
+      )
+      defaults.stubs(:write_to_file).returns(true)
+      defaults.stubs(:providers).returns(providers)
+      api_expects(:models, :index, 'List models').returns(@models)
+
+      result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })
+      _(result.exit_code).must_equal HammerCLI::EX_OK
+    end
+  end
+end
+
+

data/test/functional/operating_system_test.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'operating_system' do
+  describe 'list' do
+    before do
+      @cmd = %w[os list]
+      @operating_system = [{
+                          id: 1,
+                          release_name: 'Redhat 7',
+                          family: 'Redhat',
+                        }]
+    end
+
+    it 'should return a list of operating system' do
+      api_expects(:operatingsystems, :index, 'List operating systems').returns(@operating_system)
+
+      output = IndexMatcher.new([
+                                  ['ID', 'RELEASE NAME', 'FAMILY'],
+                                  ['1', 'Redhat 7', 'Redhat']
+                                ])
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'should run list command with defaults' do
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      defaults = HammerCLI::Defaults.new(
+        {
+          organization_id: {
+            provider: 'foreman'
+          },
+          location_id: {
+            provider: 'foreman'
+          }
+        }
+      )
+      defaults.stubs(:write_to_file).returns(true)
+      defaults.stubs(:providers).returns(providers)
+      api_expects(:operatingsystems, :index, 'List operating systems').returns(@operating_system)
+
+      result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })
+      _(result.exit_code).must_equal HammerCLI::EX_OK
+    end
+  end
+end
+
+

data/test/functional/settings_test.rb

@@ -31,6 +31,27 @@ describe 'Settings' do
       assert_cmd(expected_result, result)
     end
 
+    it 'should run list command with defaults' do
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      defaults = HammerCLI::Defaults.new(
+        {
+          organization_id: {
+            provider: 'foreman'
+          },
+          location_id: {
+            provider: 'foreman'
+          }
+        }
+      )
+      defaults.stubs(:write_to_file).returns(true)
+      defaults.stubs(:providers).returns(providers)
+      api_expects(:settings, :index, 'List').with_params(
+        'page' => 1, 'per_page' => 1000
+      ).returns(index_response([setting]))
+
+      result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })
+      _(result.exit_code).must_equal HammerCLI::EX_OK
+    end
   end
 
   describe 'info' do

data/test/functional/status_test.rb

@@ -1,24 +1,60 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 
+BASIC_STATUS = {
+  'database' => { 'active' => true, 'duration_ms' => 0 },
+  'version' => '1.24.0-develop',
+  'api' => { 'version' => 'v2' },
+  'plugins' => [],
+  'smart_proxies' => [],
+  'compute_resources' => []
+}
+
+STRING_STATUS = Marshal.load(Marshal.dump(BASIC_STATUS)).merge({
+  'plugins' => [
+    'Foreman plugin: foreman_ansible, 6.0.1, Daniel Lobato Garcia, Ansible integration with Foreman'
+  ]
+})
+
+STRUCTURED_STATUS = Marshal.load(Marshal.dump(BASIC_STATUS)).merge({
+  'plugins' => [
+    {'name': 'foreman_ansible', 'version': '6.0.1'}
+  ]
+})
+
 describe 'status' do
   let(:base_cmd) { %w[status] }
 
   describe 'foreman' do
     let(:cmd) { base_cmd << 'foreman' }
-    let(:status_results) do
-      {
-        'results' => {
-          'foreman' => {
-            'database' => { 'active' => true, 'duration_ms' => 0 },
-            'version' => '1.24.0-develop',
-            'api' => { 'version' => 'v2' },
-            'plugins' => [],
-            'smart_proxies' => [],
-            'compute_resources' => []
-          }
-        }
-      }
+    let(:status_results) { {'results' => {'foreman' => BASIC_STATUS }} }
+
+    it 'checks status of the foreman system' do
+      api_expects(:ping, :statuses, 'Status').returns(status_results)
+
+      output = OutputMatcher.new(
+        [
+          'Version:           1.24.0-develop',
+          'API Version:       v2',
+          'Database:',
+          '    Status:          ok',
+          '    Server Response: Duration: 0ms',
+          'Plugins:',
+          '',
+          'Smart Proxies:',
+          '',
+          'Compute Resources:'
+        ]
+      )
+
+      expected_result = success_result(output)
+      result = run_cmd(cmd)
+      assert_cmd(expected_result, result)
     end
+  end
+
+  describe 'foreman with string plugins' do
+    let(:cmd) { base_cmd << 'foreman' }
+    let(:status_results) { {'results' => {'foreman' => STRING_STATUS }} }
 
     it 'checks status of the foreman system' do
       api_expects(:ping, :statuses, 'Status').returns(status_results)
@@ -31,7 +67,37 @@ describe 'status' do
           '    Status:          ok',
           '    Server Response: Duration: 0ms',
           'Plugins:',
+          ' 1) Name:    foreman_ansible',
+          '    Version: 6.0.1',
+          'Smart Proxies:',
           '',
+          'Compute Resources:'
+        ]
+      )
+
+      expected_result = success_result(output)
+      result = run_cmd(cmd)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'foreman with structured plugins' do
+    let(:cmd) { base_cmd << 'foreman' }
+    let(:status_results) { {'results' => {'foreman' => STRUCTURED_STATUS }} }
+
+    it 'checks status of the foreman system' do
+      api_expects(:ping, :statuses, 'Status').returns(status_results)
+
+      output = OutputMatcher.new(
+        [
+          'Version:           1.24.0-develop',
+          'API Version:       v2',
+          'Database:',
+          '    Status:          ok',
+          '    Server Response: Duration: 0ms',
+          'Plugins:',
+          ' 1) Name:    foreman_ansible',
+          '    Version: 6.0.1',
           'Smart Proxies:',
           '',
           'Compute Resources:'

data/test/functional/usergroup_test.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'user-group' do
+  describe 'list' do
+    before do
+      @cmd = %w[user-group list]
+      @user_groups = [{
+                          id: 1,
+                          name: 'test-user-group',
+                          admin: 'yes',
+                        }]
+    end
+
+    it 'should return a list of user groups' do
+      api_expects(:usergroups, :index, 'List user groups').returns(@user_groups)
+
+      output = IndexMatcher.new([
+                                  %w[ID NAME ADMIN],
+                                  %w[1 test-user-group yes]
+                                ])
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'should run list command with defaults' do
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      defaults = HammerCLI::Defaults.new(
+        {
+          organization_id: {
+            provider: 'foreman'
+          },
+          location_id: {
+            provider: 'foreman'
+          }
+        }
+      )
+      defaults.stubs(:write_to_file).returns(true)
+      defaults.stubs(:providers).returns(providers)
+      api_expects(:usergroups, :index, 'List user groups').returns(@user_groups)
+
+      result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })
+      _(result.exit_code).must_equal HammerCLI::EX_OK
+    end
+  end
+end
+
+

data/test/unit/apipie_resource_mock.rb

@@ -237,6 +237,27 @@ module ResourceMocks
       )
   end
 
+  def self.bookmarks
+    ResourceMocks.mock_action_calls(
+      [:bookmarks, :index, []],
+      [:bookmarks, :show, {}]
+    )
+  end
+
+  def self.mail_notifications
+    ResourceMocks.mock_action_calls(
+      [:mail_notifications, :index, []],
+      [:mail_notifications, :show, {}]
+    )
+  end
+
+  def self.compute_profiles
+    ResourceMocks.mock_action_calls(
+      [:compute_profiles, :index, []],
+      [:compute_profiles, :show, {}]
+    )
+  end
+
   def self.parameters_index
     ResourceMocks.mock_action_call(:parameters, :index, [])
   end

data/test/unit/architecture_test.rb

@@ -17,6 +17,8 @@ describe HammerCLIForeman::Architecture do
     context "parameters" do
       it_should_accept "no arguments"
       it_should_accept_search_params
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
 
     context "output" do
@@ -37,6 +39,8 @@ describe HammerCLIForeman::Architecture do
     context "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=arch"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "no arguments" # TODO: temporarily disabled, parameters are checked in the id resolver
     end
 
@@ -57,6 +61,8 @@ describe HammerCLIForeman::Architecture do
 
     context "parameters" do
       it_should_accept "name", ["--name=arch"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "name missing", []
       # TODO: temporarily disabled, parameters are checked in the api
     end
@@ -71,6 +77,8 @@ describe HammerCLIForeman::Architecture do
     context "parameters" do
       it_should_accept "name", ["--name=arch"]
       it_should_accept "id", ["--id=1"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
     end
 
@@ -84,9 +92,10 @@ describe HammerCLIForeman::Architecture do
     context "parameters" do
       it_should_accept "name", ["--name=arch", "--new-name=arch2"]
       it_should_accept "id", ["--id=1", "--new-name=arch2"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "no params", [] # TODO: temporarily disabled, parameters are checked in the id resolver
       # it_should_fail_with "name or id missing", ["--new-name=arch2"] # TODO: temporarily disabled, parameters are checked in the id resolver
     end
-
   end
 end

data/test/unit/bookmark_test.rb

@@ -0,0 +1,99 @@
+# frozen_string_literal: true
+
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+require 'hammer_cli_foreman/bookmark'
+
+describe HammerCLIForeman::Bookmark do
+  include CommandTestHelper
+
+  context 'ListCommand' do
+    before :each do
+      ResourceMocks.bookmarks
+    end
+
+    let(:cmd) { HammerCLIForeman::Bookmark::ListCommand.new('', ctx) }
+
+    context 'parameters' do
+      it_should_accept 'no arguments'
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
+    end
+
+    context 'output' do
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
+
+      it_should_print_n_records
+      it_should_print_column 'Id'
+      it_should_print_column 'Name'
+      it_should_print_column 'Controller'
+      it_should_print_column 'Search Query'
+      it_should_print_column 'Public'
+      it_should_print_column 'Owner Id'
+      it_should_print_column 'Owner Type'
+    end
+  end
+
+  context 'InfoCommand' do
+    let(:cmd) { HammerCLIForeman::Bookmark::InfoCommand.new('', ctx) }
+
+    context 'parameters' do
+      it_should_accept 'id', ['--id=1']
+      it_should_accept 'name', ['--name=active']
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
+    end
+
+    context 'output' do
+      with_params ['--id=1'] do
+        it_should_print_n_records 1
+        it_should_print_column 'Id'
+        it_should_print_column 'Name'
+        it_should_print_column 'Controller'
+        it_should_print_column 'Search Query'
+        it_should_print_column 'Public'
+        it_should_print_column 'Owner Id'
+        it_should_print_column 'Owner Type'
+      end
+    end
+  end
+
+  context 'CreateCommand' do
+    let(:cmd) { HammerCLIForeman::Bookmark::CreateCommand.new('', ctx) }
+
+    context 'parameters' do
+      it_should_accept 'name, public, controller, query',
+                       ['--name=active', '--public=1', '--controller=hosts',
+                        '--query=last_report > "35 minutes ago" and (status.applied > 0 or status.restarted > 0)']
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
+    end
+
+  end
+
+  context 'DeleteCommand' do
+    let(:cmd) { HammerCLIForeman::Bookmark::DeleteCommand.new('', ctx) }
+
+    context 'parameters' do
+      it_should_accept 'id', ['--id=1']
+      it_should_accept 'name', ['--name=active']
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
+    end
+  end
+
+  context 'UpdateCommand' do
+    let(:cmd) { HammerCLIForeman::Bookmark::UpdateCommand.new('', ctx) }
+
+    context 'parameters' do
+      it_should_accept 'id', ['--id=1']
+      it_should_accept 'name', ['--name=active']
+      it_should_accept 'name, public, controller, query',
+                       ['--name=active', '--public=1', '--controller=hosts',
+                        '--query=last_report > "35 minutes ago" and (status.applied > 0 or status.restarted > 0)']
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
+    end
+  end
+end