hammer_cli_foreman 2.4.0 → 3.1.0

data/test/functional/hostgroup/update_test.rb

@@ -68,25 +68,6 @@ module HammerCLIForeman
         run_cmd(%w(hostgroup update --id 1 --domain d1))
       end
 
-      it 'allows environment id' do
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['environment_id'] == 1 &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(hostgroup update --id 1 --environment-id 1))
-      end
-
-      it 'allows environment name' do
-        api_expects(:environments, :index) do |p|
-          p[:search] = "name = \"env1\""
-        end.returns(index_response([{'id' => 1}]))
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['environment_id'] == 1 &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(hostgroup update --id 1 --environment env1))
-      end
-
       it 'allows location ids' do
         api_expects(:hostgroups, :update) do |p|
           p['hostgroup']['location_ids'] == ['1','4'] &&
@@ -182,6 +163,17 @@ module HammerCLIForeman
         run_cmd(%w(hostgroup update --id 1 --parent parent_hg))
       end
 
+      it 'allows parent hostgroup title' do
+        api_expects(:hostgroups, :index) do |p|
+          p[:search] = 'title = "parent_hg"'
+        end.returns(index_response([{ 'id' => 1 }]))
+        api_expects(:hostgroups, :update) do |p|
+          p['hostgroup']['parent_id'] == 1 &&
+            p['id'] == '1'
+        end
+        run_cmd(%w[hostgroup update --id 1 --parent parent_hg])
+      end
+
       it 'allows partition table id' do
         api_expects(:hostgroups, :update) do |p|
           p['hostgroup']['ptable_id'] == 1 &&
@@ -201,66 +193,6 @@ module HammerCLIForeman
         run_cmd(%w(hostgroup update --id 1 --partition-table pt1))
       end
 
-      it 'allows puppet ca proxy id' do
-        api_expects(:hostgroups, :update).with_params({
-          :id => '1',
-          :hostgroup => { :puppet_ca_proxy_id => 1 }
-        })
-        run_cmd(%w(hostgroup update --id 1 --puppet-ca-proxy-id 1))
-      end
-
-      it 'allows puppet ca proxy name' do
-        api_expects(:smart_proxies, :index) do |p|
-          p[:search] = "name = \"sp1\""
-        end.returns(index_response([{'id' => 1}]))
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppet_ca_proxy_id'] == 1 &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(hostgroup update --id 1 --puppet-ca-proxy sp1))
-      end
-
-      it 'allows puppet class ids' do
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppetclass_ids'] == ['1','2'] &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(hostgroup update --id 1 --puppet-class-ids 1,2))
-      end
-
-      it 'allows puppet class names' do
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] = "name = \"pc1\" or name = \"pc2\""
-        end.returns(index_response('puppetclasses' => [
-          {'id' => 1, 'name' => 'pc1'},
-          {'id' => 2, 'name' => 'pc2'}
-        ]))
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppetclass_ids'] == [1,2] &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(hostgroup update --id 1 --puppet-classes pc1,pc2))
-      end
-
-      it 'allows puppet proxy id' do
-        api_expects(:hostgroups, :update).with_params({
-          :id => '1',
-          :hostgroup => { :puppet_proxy_id => 1 }
-        })
-        run_cmd(%w(hostgroup update --id 1 --puppet-proxy-id 1))
-      end
-
-      it 'allows puppet proxy name' do
-        api_expects(:smart_proxies, :index) do |p|
-          p[:search] = "name = \"sp1\""
-        end.returns(index_response([{'id' => 1}]))
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppet_proxy_id'] == 1 &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(hostgroup update --id 1 --puppet-proxy sp1))
-      end
-
       it 'allows realm id' do
         api_expects(:hostgroups, :update) do |p|
           p['hostgroup']['realm_id'] == 1 &&

data/test/functional/http_proxy_test.rb

@@ -61,9 +61,21 @@ describe 'httpproxy' do
   end
 
   it 'deletes an http proxy' do
+    api_expects(:http_proxies, :destroy, 'Delete proxy').returns(http_proxy)
+
     expected_result = success_result("Http proxy deleted.\n")
 
     result = run_cmd(%w(http-proxy delete --id 1))
     assert_cmd(expected_result, result)
   end
+
+  it 'updates nothing without related parameters' do
+    api_expects(:http_proxies, :update, 'Update proxy with no params').returns({})
+
+    expected_result = success_result("Nothing to update.\n")
+
+    result = run_cmd(%w(http-proxy update --id 1))
+    assert_cmd(expected_result, result)
+  end
+
 end

data/test/functional/location_test.rb

@@ -208,7 +208,6 @@ describe 'info' do
         provisioning_templates: [],
         domains: [],
         realms: [],
-        environments: [],
         hostgroups: [],
         organizations: [],
         parameters: []
@@ -242,8 +241,6 @@ describe 'info' do
                                  '',
                                  'Realms:',
                                  '',
-                                 'Environments:',
-                                 '',
                                  'Hostgroups:',
                                  '',
                                  'Parameters:',
@@ -272,4 +269,17 @@ describe 'update' do
     result = run_cmd(@cmd + params)
     assert_cmd(success_result("Location updated.\n"), result)
   end
+
+  it 'updates nothing without related parameters' do
+    params = %w[--id=1]
+
+    api_expects(:locations, :index)
+    api_expects(:locations, :update, 'Update location with no params').returns({})
+
+    expected_result = success_result("Nothing to update.\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
 end

data/test/functional/mail_notification_test.rb

@@ -26,7 +26,7 @@ describe 'mail_notification' do
     end
 
     it 'should run list command with defaults' do
-      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.5')) }
       defaults = HammerCLI::Defaults.new(
         {
           organization_id: {
@@ -39,6 +39,8 @@ describe 'mail_notification' do
       )
       defaults.stubs(:write_to_file).returns(true)
       defaults.stubs(:providers).returns(providers)
+      api_expects(:users, :index, 'Find user').with_params(search: 'login=admin').returns(index_response([{ 'default_organization' => { 'id' => 2 } }]))
+      api_expects(:users, :index, 'Find user').with_params(search: 'login=admin').returns(index_response([{ 'default_location' => { 'id' => 1 } }]))
       api_expects(:mail_notifications, :index, 'List mail notifications').returns(@mail_notifications)
 
       result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })

data/test/functional/media_test.rb

@@ -126,5 +126,16 @@ describe 'medium' do
       result = run_cmd(@cmd + params)
       assert_cmd(success_result("Installation medium updated.\n"), result)
     end
+
+    it 'updates nothing without medium related parameters' do
+      params = %w[--id=1]
+
+      api_expects(:media, :update, 'Update medium with no params').returns({})
+
+      expected_result = success_result("Nothing to update.\n")
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
   end
 end

data/test/functional/model_test.rb

@@ -26,7 +26,7 @@ describe 'model' do
     end
 
     it 'should run list command with defaults' do
-      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.5')) }
       defaults = HammerCLI::Defaults.new(
         {
           organization_id: {
@@ -39,6 +39,8 @@ describe 'model' do
       )
       defaults.stubs(:write_to_file).returns(true)
       defaults.stubs(:providers).returns(providers)
+      api_expects(:users, :index, 'Find user').with_params(search: 'login=admin').returns(index_response([{ 'default_organization' => { 'id' => 2 } }]))
+      api_expects(:users, :index, 'Find user').with_params(search: 'login=admin').returns(index_response([{ 'default_location' => { 'id' => 1 } }]))
       api_expects(:models, :index, 'List models').returns(@models)
 
       result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })

data/test/functional/operating_system_test.rb

@@ -27,7 +27,7 @@ describe 'operating_system' do
     end
 
     it 'should run list command with defaults' do
-      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.5')) }
       defaults = HammerCLI::Defaults.new(
         {
           organization_id: {
@@ -40,6 +40,8 @@ describe 'operating_system' do
       )
       defaults.stubs(:write_to_file).returns(true)
       defaults.stubs(:providers).returns(providers)
+      api_expects(:users, :index, 'Find user').with_params(search: 'login=admin').returns(index_response([{ 'default_organization' => { 'id' => 2 } }]))
+      api_expects(:users, :index, 'Find user').with_params(search: 'login=admin').returns(index_response([{ 'default_location' => { 'id' => 1 } }]))
       api_expects(:operatingsystems, :index, 'List operating systems').returns(@operating_system)
 
       result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })

data/test/functional/personal_access_token_test.rb

@@ -37,7 +37,7 @@ describe 'personal_access_token' do
 
     it 'lists all access tokens for a given user' do
       api_expects(:personal_access_tokens, :index, 'List').with_params(
-        'user_id' => 1, 'page' => 1, 'per_page' => 1000
+        'user_id' => '1', 'page' => 1, 'per_page' => 1000
       ).returns(index_response([access_token, active_access_token]))
 
       output = IndexMatcher.new([
@@ -67,7 +67,7 @@ describe 'personal_access_token' do
 
     it 'creates an access token to a given user' do
       api_expects(:personal_access_tokens, :create).with_params(
-        'user_id' => 1, 'personal_access_token' => {
+        'user_id' => '1', 'personal_access_token' => {
           'expires_at' => '01/01/2048', 'name' => 'test'
         }
       ).returns(access_token)
@@ -85,7 +85,7 @@ describe 'personal_access_token' do
 
     it 'shows the personal access token' do
       api_expects(:personal_access_tokens, :show, 'Show PAT').with_params(
-        'id' => '1', 'user_id' => 1
+        'id' => '1', 'user_id' => '1'
       ).returns(access_token)
 
       output = OutputMatcher.new([
@@ -109,7 +109,7 @@ describe 'personal_access_token' do
     let(:params) { ['--id=1', '--user-id=1'] }
     it 'deletes an access token to a given user' do
       api_expects(:personal_access_tokens, :destroy, 'Revoke PAT').with_params(
-        'id' => '1', 'user_id' => 1
+        'id' => '1', 'user_id' => '1'
       ).returns(access_token)
 
       expected_result = success_result(

data/test/functional/ping_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require File.join(File.dirname(__FILE__), 'test_helper')
 
 describe 'ping' do
@@ -30,5 +32,36 @@ describe 'ping' do
       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' => {
+          'first' => {
+            'status' => 'FAIL'
+          },
+          'second' => {
+            'status' => 'ok'
+          }
+        },
+        'status' => 'FAIL'
+      }
+      api_expects(:ping, :ping, 'Ping').returns(ping_results)
+
+      expected_result = CommandExpectation.new
+      expected_result.expected_out = OutputMatcher.new(
+        [
+          'database:',
+          '    Status:          ok',
+          '    Server Response: Duration: 0ms'
+        ]
+      )
+      expected_result.expected_err =
+        ['1 more service(s) failed, but not shown:',
+         'first',
+         ''].join("\n")
+      expected_result.expected_exit_code = 1
+      result = run_cmd(cmd)
+      assert_cmd(expected_result, result)
+    end
   end
 end

data/test/functional/realm_test.rb

@@ -99,5 +99,16 @@ describe 'realm' do
       result = run_cmd(@cmd + params)
       assert_cmd(success_result("Realm [%{name}] updated.\n"), result)
     end
+
+    it 'updates nothing without realm related parameters' do
+      params = %w[--id=1]
+
+      api_expects(:realms, :update, 'Update realm with no params').returns({})
+
+      expected_result = success_result("Nothing to update.\n")
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
   end
 end

data/test/functional/registration_test.rb

@@ -0,0 +1,8 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'registration_commands' do
+  it 'create' do
+    api_expects(:registration_commands, :create)
+    run_cmd(%w(host-registration generate-command))
+  end
+end

data/test/functional/report_template_test.rb

@@ -195,6 +195,17 @@ describe 'report-template' do
       result = run_cmd(cmd + params)
       assert_cmd(success_result("Report template updated.\n"), result)
     end
+
+    it 'updates nothing without template related parameters' do
+      params = %w[--id=1]
+
+      api_expects(:report_templates, :update, 'Update template with no params').returns({})
+
+      expected_result = success_result("Nothing to update.\n")
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
   end
 
   describe 'dump' do

data/test/functional/settings_test.rb

@@ -32,7 +32,7 @@ describe 'Settings' do
     end
 
     it 'should run list command with defaults' do
-      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.5')) }
       defaults = HammerCLI::Defaults.new(
         {
           organization_id: {
@@ -45,9 +45,9 @@ describe 'Settings' do
       )
       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]))
+      api_expects(:users, :index, 'Find user').with_params(search: 'login=admin').returns(index_response([{ 'default_organization' => { 'id' => 2 } }]))
+      api_expects(:users, :index, 'Find user').with_params(search: 'login=admin').returns(index_response([{ 'default_location' => { 'id' => 1 } }]))
+      api_expects(:settings, :index, 'List settings').returns(setting)
 
       result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })
       _(result.exit_code).must_equal HammerCLI::EX_OK

data/test/functional/template_test.rb

@@ -85,7 +85,6 @@ describe 'template' do
     it "doesn't send snippet flag when --type is undefined" do
       params = ['--id=1', '--locked=true']
 
-      api_expects(:template_kinds, :index, 'Get list of template kinds').returns(index_response([]))
       api_expects(:provisioning_templates, :update, 'Update the template') do |par|
         par['id'] == '1' &&
         par['provisioning_template']['locked'] == true
@@ -95,6 +94,59 @@ describe 'template' do
 
       assert_cmd(success_result("Provisioning template updated.\n"), result)
     end
+
+    it 'updates nothing without template related parameters' do
+      params = %w[--id=1 --organization-id=1 --location-id=1]
+
+      api_expects(:provisioning_templates, :update, 'Update template with no params').returns({})
+
+      expected_result = success_result("Nothing to update.\n")
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'should update a template' do
+      params = ['--id=1', '--type=snippet']
+
+      expected_result = CommandExpectation.new
+      expected_result.expected_out =
+        ['Provisioning template updated.',
+         ''].join("\n")
+      expected_result.expected_exit_code = HammerCLI::EX_OK
+
+      api_expects(:provisioning_templates, :update, 'Update template with params') do |p|
+        p['id'] == '1' && p['provisioning_template']['snippet'] == true
+      end
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'should update a template by name' do
+      params = ['--name=tpl', '--type=provision']
+
+      expected_result = CommandExpectation.new
+      expected_result.expected_out =
+        ['Provisioning template updated.',
+         ''].join("\n")
+      expected_result.expected_exit_code = HammerCLI::EX_OK
+
+      api_expects_search(:provisioning_templates, name: 'tpl').returns(
+        index_response([{ 'id' => '1' }])
+      )
+      api_expects_search(:template_kinds, name: 'provision').returns(
+        index_response([{ 'id' => '1' }])
+      )
+      api_expects(:provisioning_templates, :update, 'Update template with params') do |p|
+        p['id'] == '1' &&
+          p['provisioning_template']['snippet'] == false &&
+          p['provisioning_template']['template_kind_id'] == '1'
+      end
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
   end
 
   describe 'create' do
@@ -108,65 +160,53 @@ describe 'template' do
       expected_result = CommandExpectation.new
       expected_result.expected_err =
         ['Could not create the provisioning template:',
-         "  Error: unknown template kind",
-         '  ',
-         "  See: 'hammer template create --help'.",
+         '  Error: template_kind not found.',
          ''].join("\n")
-      expected_result.expected_exit_code = HammerCLI::EX_USAGE
+      expected_result.expected_exit_code = HammerCLI::EX_SOFTWARE
 
-      HammerCLIForeman::Template::CreateCommand.any_instance.stubs(:kinds).returns(["PXELinux"])
-
-      api_expects_no_call
+      api_expects_search(:template_kinds, name: 'unknown').returns(
+        index_response([])
+      )
 
       result = run_cmd(@cmd + params)
       assert_cmd(expected_result, result)
     end
-  end
 
-  describe 'combinations' do
-    before do
-      @cmd = %w(template combination)
-    end
+    it 'should create a template' do
+      params = ['--name=tpl', '--file=Gemfile', '--type=provision']
 
-    it 'should create new combination with warning' do
-      params = ['create','--provisioning-template-id=10', '--hostgroup-id=1', '--environment-id=1']
-      expected_result = success_result("Template combination created.\n")
-      expected_result.expected_err = "Warning: Option --environment-id is deprecated. Use --puppet-environment[-id] instead\n"
-      api_expects(:template_combinations, :create, 'Create template combination') do |params|
-        params['provisioning_template_id'] == 10 &&
-          params['hostgroup_id'] == 1 &&
-          params['environment_id'] == 1 &&
-          params['template_combination'] == {'environment_id' => 1, 'hostgroup_id' => 1}
+      expected_result = CommandExpectation.new
+      expected_result.expected_out =
+        ['Provisioning template created.',
+         ''].join("\n")
+      expected_result.expected_exit_code = HammerCLI::EX_OK
+
+      api_expects_search(:template_kinds, name: 'provision').returns(
+        index_response([{ 'id' => '1' }])
+      )
+      api_expects(:provisioning_templates, :create, 'Create template with params') do |p|
+        p['provisioning_template']['name'] == 'tpl' &&
+          p['provisioning_template']['snippet'] == false &&
+          p['provisioning_template']['template_kind_id'] = '1'
       end
 
       result = run_cmd(@cmd + params)
       assert_cmd(expected_result, result)
     end
+  end
+
+  describe 'combinations' do
+    before do
+      @cmd = %w(template combination)
+    end
 
     it 'should create new combination' do
-      params = ['create','--provisioning-template-id=10', '--hostgroup-id=1', '--puppet-environment-id=1']
+      params = ['create','--provisioning-template-id=10', '--hostgroup-id=1']
       expected_result = success_result("Template combination created.\n")
       api_expects(:template_combinations, :create, 'Create template combination') do |params|
-        params['provisioning_template_id'] == 10 &&
-          params['hostgroup_id'] == 1 &&
-          params['environment_id'] == 1 &&
-          params['template_combination'] == {'environment_id' => 1, 'hostgroup_id' => 1}
-      end
-
-      result = run_cmd(@cmd + params)
-      assert_cmd(expected_result, result)
-    end
-
-    it 'should update combination with warning' do
-      params = ['update', '--id=3', '--provisioning-template-id=10', '--hostgroup-id=1', '--environment-id=1']
-      expected_result = success_result("Template combination updated.\n")
-      expected_result.expected_err = "Warning: Option --environment-id is deprecated. Use --puppet-environment[-id] instead\n"
-      api_expects(:template_combinations, :update, 'Update template combination') do |params|
-        params['id'] == '3' &&
-          params['provisioning_template_id'] == 10 &&
-          params['hostgroup_id'] == 1 &&
-          params['environment_id'] == 1 &&
-          params['template_combination'] == { 'environment_id' => 1, 'hostgroup_id' => 1 }
+        params['provisioning_template_id'] == '10' &&
+          params['hostgroup_id'] == '1' &&
+          params['template_combination'] == { 'hostgroup_id' => '1' }
       end
 
       result = run_cmd(@cmd + params)
@@ -174,14 +214,13 @@ describe 'template' do
     end
 
     it 'should update combination' do
-      params = ['update', '--id=3', '--provisioning-template-id=10', '--hostgroup-id=1', '--puppet-environment-id=1']
+      params = ['update', '--id=3', '--provisioning-template-id=10', '--hostgroup-id=1']
       expected_result = success_result("Template combination updated.\n")
       api_expects(:template_combinations, :update, 'Update template combination') do |params|
         params['id'] == '3' &&
-          params['provisioning_template_id'] == 10 &&
-          params['hostgroup_id'] == 1 &&
-          params['environment_id'] == 1 &&
-          params['template_combination'] == { 'environment_id' => 1, 'hostgroup_id' => 1 }
+          params['provisioning_template_id'] == '10' &&
+          params['hostgroup_id'] == '1' &&
+          params['template_combination'] == { 'hostgroup_id' => '1' }
       end
 
       result = run_cmd(@cmd + params)

data/test/functional/user_mail_notification_test.rb

@@ -55,9 +55,9 @@ describe 'user_mail_notification' do
       params = ['--user-id=2']
 
       expected_result = common_error_result(
-          @cmd,
-          "Could not find mail_notification, please set one of options --mail-notification, --mail-notification-id.",
-          "Could not add user mail notification"
+        @cmd,
+        'Could not find mail_notification, please set one of options --mail-notification-id, --mail-notification.',
+        'Could not add user mail notification'
       )
 
       result = run_cmd(@cmd + params)

data/test/functional/user_test.rb

@@ -82,6 +82,17 @@ describe "user" do
       assert_cmd(expected_result, result)
     end
 
+    it 'updates nothing without user related parameters' do
+      params = %w[--id=1]
+
+      api_expects(:users, :update, 'Update user with no params').returns({})
+
+      expected_result = success_result("Nothing to update.\n")
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+
     describe "update password" do
       def replace_foreman_connection(connection)
         HammerCLI.context[:api_connection].drop('foreman')

data/test/functional/usergroup_test.rb

@@ -27,7 +27,7 @@ describe 'user-group' do
     end
 
     it 'should run list command with defaults' do
-      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.5')) }
       defaults = HammerCLI::Defaults.new(
         {
           organization_id: {
@@ -40,6 +40,8 @@ describe 'user-group' do
       )
       defaults.stubs(:write_to_file).returns(true)
       defaults.stubs(:providers).returns(providers)
+      api_expects(:users, :index, 'Find user').with_params(search: 'login=admin').returns(index_response([{ 'default_organization' => { 'id' => 2 } }]))
+      api_expects(:users, :index, 'Find user').with_params(search: 'login=admin').returns(index_response([{ 'default_location' => { 'id' => 1 } }]))
       api_expects(:usergroups, :index, 'List user groups').returns(@user_groups)
 
       result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })

data/test/test_helper.rb

@@ -17,7 +17,7 @@ require "mocha/minitest"
 require 'hammer_cli'
 require 'hammer_cli_foreman/testing/api_expectations'
 
-FOREMAN_VERSION = ENV['TEST_API_VERSION'] || '2.1'
+FOREMAN_VERSION = ENV['TEST_API_VERSION'] || '3.1'
 unless Dir.entries('test/data').include? FOREMAN_VERSION
   raise StandardError.new "Version is not correct"
 end