hammer_cli_foreman 0.11.0 → 0.12.0

data/test/functional/host_test.rb

@@ -3,8 +3,8 @@ require File.join(File.dirname(__FILE__), 'test_helper')
 
 describe "host create" do
   let(:cmd) { ["host", "create"] }
-  let(:minimal_params_without_hosgroup) { ['--location-id=1', '--organization-id=1', '--name=test'] }
-  let(:minimal_params) { ['--hostgroup-id=1'] + minimal_params_without_hosgroup }
+  let(:minimal_params_without_hostgroup) { ['--location-id=1', '--organization-id=1', '--name=test'] }
+  let(:minimal_params) { ['--hostgroup-id=1'] + minimal_params_without_hostgroup }
 
   it "accepts hostgroup title" do
     api_expects_search(:hostgroups, { :title => 'test/hg1' }).returns(index_response([{ 'id' => '83' }]))
@@ -12,9 +12,9 @@ describe "host create" do
       par['host']['hostgroup_id'] == '83'
     end.returns({})
 
-    expected_result = success_result("Host created\n")
+    expected_result = success_result("Host created.\n")
 
-    result = run_cmd(cmd + minimal_params_without_hosgroup + ['--hostgroup-title=test/hg1'])
+    result = run_cmd(cmd + minimal_params_without_hostgroup + ['--hostgroup-title=test/hg1'])
     assert_cmd(expected_result, result)
   end
 
@@ -29,7 +29,7 @@ describe "host create" do
       ifces[0]['identifier'] == 'eth0'
     end.returns({})
 
-    expected_result = success_result("Host created\n")
+    expected_result = success_result("Host created.\n")
 
     result = run_cmd(cmd + minimal_params + params)
     assert_cmd(expected_result, result)
@@ -43,7 +43,7 @@ describe "host create" do
       par['host']['interfaces_attributes'] == []
     end.returns({})
 
-    expected_result = success_result("Host created\n")
+    expected_result = success_result("Host created.\n")
 
     result = run_cmd(cmd + minimal_params)
     assert_cmd(expected_result, result)
@@ -66,7 +66,7 @@ describe "host create" do
       true
     end.returns({})
 
-    expected_result = success_result("Host created\n")
+    expected_result = success_result("Host created.\n")
 
     result = run_cmd(cmd + minimal_params + params)
     assert_cmd(expected_result, result)
@@ -88,7 +88,7 @@ describe "host create" do
       ifces[0]['domain_id'] == 32
     end.returns({})
 
-    expected_result = success_result("Host created\n")
+    expected_result = success_result("Host created.\n")
 
     result = run_cmd(cmd + minimal_params + params)
     assert_cmd(expected_result, result)
@@ -110,9 +110,60 @@ describe "host create" do
     api_expects_search(:ptables,          { :name => 'default' }).returns(index_response([mock_item]))
     api_expects(:hosts, :create, 'Create host with interfaces params').returns({})
 
-    expected_result = success_result("Host created\n")
+    expected_result = success_result("Host created.\n")
 
-    result = run_cmd(cmd + minimal_params_without_hosgroup + params)
+    result = run_cmd(cmd + minimal_params_without_hostgroup + params)
     assert_cmd(expected_result, result)
   end
 end
+
+describe 'host reports' do
+  let(:report15) do
+    {
+        "id" => 15,
+        "host_id" => 1,
+        "host_name" => "host.example.com",
+        "reported_at" => "2017-11-13 03:04:53 UTC",
+    }
+  end
+
+  it 'filters reports by --id' do
+    api_expects(:reports, :index, 'Filter the reports') do |params|
+      params['search'] == 'host_id=1'
+    end.returns(index_response([report15]))
+
+    result = run_cmd(['host', 'reports', '--id=1'])
+    result.exit_code.must_equal HammerCLI::EX_OK
+  end
+
+  it 'filters reports by --name' do
+    api_expects(:reports, :index, 'Filter the reports') do |params|
+      params['search'] == 'host="host.example.com"'
+    end.returns(index_response([report15]))
+
+    result = run_cmd(['host', 'reports', '--name=host.example.com'])
+    result.exit_code.must_equal HammerCLI::EX_OK
+  end
+
+  it 'prints error or missing --id and --name' do
+      expected_result = CommandExpectation.new
+      expected_result.expected_err =
+          ['Error: At least one of options --name, --id is required.', '',
+              "See: 'hammer host reports --help'.", ''].join("\n")
+      expected_result.expected_exit_code = HammerCLI::EX_USAGE
+
+      api_expects_no_call
+
+      result = run_cmd(['host', 'reports'])
+      assert_cmd(expected_result, result)
+  end
+
+  it 'filters reports by --name together with search' do
+    api_expects(:reports, :index, 'Filter the reports') do |params|
+      params['search'] == 'reported > "2 hour ago" and host="host.example.com"'
+    end.returns(index_response([report15]))
+
+    result = run_cmd(['host', 'reports', '--name=host.example.com', %Q(--search=reported > "2 hour ago")])
+    result.exit_code.must_equal HammerCLI::EX_OK
+  end
+end

data/test/functional/location_test.rb

@@ -25,7 +25,7 @@ describe "parameters" do
 
       expected_result = usage_error_result(
         @cmd,
-        "option '--name' is required",
+        "Option '--name' is required.",
         "Could not set location parameter"
       )
 
@@ -39,7 +39,7 @@ describe "parameters" do
 
       expected_result = usage_error_result(
         @cmd,
-        "option '--value' is required",
+        "Option '--value' is required.",
         "Could not set location parameter"
       )
 
@@ -93,7 +93,7 @@ describe "parameters" do
 
       expected_result = usage_error_result(
         @cmd,
-        "option '--name' is required",
+        "Option '--name' is required.",
         "Could not delete location parameter"
       )
 
@@ -111,7 +111,7 @@ describe "parameters" do
 
       expected_result = common_error_result(
         @cmd,
-        "parameter not found",
+        "parameter not found.",
         "Could not delete location parameter"
       )
 
@@ -130,7 +130,7 @@ describe "parameters" do
       end.returns({:name => 'A', :value => '1'})
 
       expected_result = success_result(
-        "Parameter [A] deleted\n"
+        "Parameter [A] deleted.\n"
       )
 
       result = run_cmd(@cmd + params)

data/test/functional/organization_test.rb

@@ -25,7 +25,7 @@ describe "parameters" do
 
       expected_result = usage_error_result(
         @cmd,
-        "option '--name' is required",
+        "Option '--name' is required.",
         "Could not set organization parameter"
       )
 
@@ -39,7 +39,7 @@ describe "parameters" do
 
       expected_result = usage_error_result(
         @cmd,
-        "option '--value' is required",
+        "Option '--value' is required.",
         "Could not set organization parameter"
       )
 
@@ -93,7 +93,7 @@ describe "parameters" do
 
       expected_result = usage_error_result(
         @cmd,
-        "option '--name' is required",
+        "Option '--name' is required.",
         "Could not delete organization parameter"
       )
 
@@ -111,7 +111,7 @@ describe "parameters" do
 
       expected_result = common_error_result(
         @cmd,
-        "parameter not found",
+        "parameter not found.",
         "Could not delete organization parameter"
       )
 
@@ -130,7 +130,7 @@ describe "parameters" do
       end.returns({:name => 'A', :value => '1'})
 
       expected_result = success_result(
-        "Parameter [A] deleted\n"
+        "Parameter [A] deleted.\n"
       )
 
       result = run_cmd(@cmd + params)
@@ -167,7 +167,7 @@ describe 'associating commands' do
           par['organization']['domain_ids'] == ['1', '5']
       end
       result = run_cmd(@cmd + @params)
-      expected = success_result("The domain has been associated\n")
+      expected = success_result("The domain has been associated.\n")
       assert_cmd(expected, result)
     end
 
@@ -196,7 +196,7 @@ describe 'associating commands' do
           par['organization']['domain_ids'] == ['1']
       end
       result = run_cmd(@cmd + @params)
-      expected = success_result("The domain has been disassociated\n")
+      expected = success_result("The domain has been disassociated.\n")
       assert_cmd(expected, result)
     end
 

data/test/functional/role_test.rb

@@ -27,7 +27,7 @@ describe 'role' do
 
       expected_result = usage_error_result(
         @cmd,
-        'Option --new-name is required',
+        'Option --new-name is required.',
         'Could not clone the user role')
 
       api_expects_no_call
@@ -45,7 +45,7 @@ describe 'role' do
       })
 
       result = run_cmd(@cmd + params)
-      assert_cmd(success_result("User role cloned\n"), result)
+      assert_cmd(success_result("User role cloned.\n"), result)
     end
 
     it 'should clone a role by name' do
@@ -63,7 +63,7 @@ describe 'role' do
       })
 
       result = run_cmd(@cmd + params)
-      assert_cmd(success_result("User role cloned\n"), result)
+      assert_cmd(success_result("User role cloned.\n"), result)
     end
   end
 end

data/test/functional/smart_class_parameter_test.rb

@@ -9,7 +9,7 @@ describe 'sc-params update' do
 
     expected_result = usage_error_result(
       cmd,
-      "Unrecognised option '--new-name'",
+      "Unrecognised option '--new-name'.",
       "Could not update the parameter"
     )
 
@@ -20,8 +20,8 @@ describe 'sc-params update' do
 
 end
 
-describe 'sc-params add-override-value' do
-  let(:cmd) { %w(sc-param add-override-value) }
+describe 'sc-params add-matcher' do
+  let(:cmd) { %w(sc-param add-matcher) }
   let(:override_value) { 'something' }
   let(:match) { 'domain = example.com' }
   let(:puppet_class) { { 'name' => 'motd', 'id' => '1' } }
@@ -34,7 +34,7 @@ describe 'sc-params add-override-value' do
 
   it 'allows to set value' do
     params = ['--value', override_value]
-    expected_result = success_result("Override value created\n")
+    expected_result = success_result("Override value created.\n")
 
     prepare_api_expectations = proc do
       expectations = []
@@ -71,7 +71,7 @@ describe 'sc-params add-override-value' do
 
     expected_result = usage_error_result(
       cmd,
-      "Cannot use --value when --use-puppet-default is true",
+      "Cannot use --value when --use-puppet-default is true.",
       'Could not create the override value'
     )
 

data/test/functional/smart_variable_test.rb

@@ -6,7 +6,7 @@ describe 'smart-variable update' do
   it 'allows to save new name and override value order' do
     params = ['--id=1', '--new-variable=name2', '--override-value-order=\'fqdn\',\'hostgroup\',\'domain\',\'os\'']
 
-    expected_result = success_result("Smart variable [name2] updated\n")
+    expected_result = success_result("Smart variable [name2] updated.\n")
 
     api_expects(:smart_variables, :update, 'Update the variable') do |par|
      (par['smart_variable']['variable'] == 'name2') && (par['smart_variable']['override_value_order'] == "fqdn\nhostgroup\ndomain\nos")

data/test/functional/ssh_keys_test.rb

@@ -0,0 +1,105 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'ssh_keys' do
+  let(:base_cmd) { ['user', 'ssh-keys'] }
+  let(:user) do
+    {
+     :id => 1,
+     :name => 'admin'
+    }
+  end
+  let(:ssh_key) do
+    {
+      :id => 1,
+      :user_id => user[:id],
+      :name => 'SSH Key',
+      :fingerprint => 'FINGERPRINT',
+      :key => 'KEY',
+      :lenght => 256
+    }
+  end
+
+  describe 'list' do
+    let(:cmd) { base_cmd << 'list' }
+
+    it 'lists all ssh-keys for a given user' do
+      params = ['--user-id', user[:id]]
+      api_expects(:ssh_keys, :index, :user_id => user[:id])
+        .returns(index_response([ssh_key]))
+
+      expected_result = success_result(/#{ssh_key[:name]}/)
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'add' do
+    let(:cmd) { base_cmd << 'add' }
+
+    it 'adds an ssh-key to a given user' do
+      params = ['--user-id', user[:id],
+                '--key', ssh_key[:key],
+                '--name', ssh_key[:name]]
+      api_expects(:ssh_keys, :create, params)
+        .returns(ssh_key)
+
+      expected_result = success_result("SSH Key #{ssh_key[:name]} added.\n")
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'adds an ssh-key to a given user with a key-file' do
+      params = ['--user-id', user[:id],
+                '--key-file', '~/.ssh/id_rsa.pub',
+                '--name', ssh_key[:name]]
+      File.stubs(:read).returns(ssh_key[:key])
+      api_expects(:ssh_keys, :create, {
+        :user_id => user[:id],
+        :ssh_key => {
+          :user_id => user[:id],
+          :key => ssh_key[:key],
+          :name => ssh_key[:name]
+        }
+      }).returns(ssh_key)
+
+      expected_result = success_result("SSH Key #{ssh_key[:name]} added.\n")
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'info' do
+    let(:cmd) { base_cmd << 'info' }
+
+    it 'shows the public key' do
+      params = ['--id', ssh_key[:id],
+                '--user-id', user[:id]]
+      api_expects(:ssh_keys, :show, :id => ssh_key[:id])
+        .returns(ssh_key)
+
+      expected_result = success_result(/#{ssh_key[:key]}/)
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'delete' do
+    let(:cmd) { base_cmd << 'delete' }
+
+    it 'deletes an ssh-key to a given user' do
+      params = ['--id', ssh_key[:id],
+                '--user-id', user[:id]]
+      api_expects(:ssh_keys, :destroy, :id => ssh_key[:id])
+        .returns({})
+
+      expected_result = success_result("SSH Key deleted.\n")
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+end

data/test/functional/template_test.rb

@@ -28,7 +28,7 @@ describe 'template' do
 
       expected_result = usage_error_result(
         @cmd,
-        'Option --new-name is required',
+        'Option --new-name is required.',
         'Could not clone the provisioning template')
 
       api_expects_no_call
@@ -47,7 +47,7 @@ describe 'template' do
 
       result = run_cmd(@cmd + params)
 
-      assert_cmd(success_result("Provisioning template cloned\n"), result)
+      assert_cmd(success_result("Provisioning template cloned.\n"), result)
     end
 
     it 'should clone a template by name' do
@@ -74,7 +74,7 @@ describe 'template' do
 
       result = run_cmd(@cmd + params)
 
-      assert_cmd(success_result("Provisioning template cloned\n"), result)
+      assert_cmd(success_result("Provisioning template cloned.\n"), result)
     end
   end
 
@@ -94,7 +94,33 @@ describe 'template' do
 
       result = run_cmd(@cmd + params)
 
-      assert_cmd(success_result("Provisioning template updated\n"), result)
+      assert_cmd(success_result("Provisioning template updated.\n"), result)
+    end
+  end
+
+  describe 'create' do
+    before do
+      @cmd = %w(template create)
+    end
+
+    it 'should print correct error on providing unknown template kind' do
+      params = ['--name=tpl', '--file=Gemfile', '--type=unknown']
+
+      expected_result = CommandExpectation.new
+      expected_result.expected_err =
+        ['Could not create the provisioning template:',
+         "  Error: unknown template kind",
+         '  ',
+         "  See: 'hammer template create --help'.",
+         ''].join("\n")
+      expected_result.expected_exit_code = HammerCLI::EX_USAGE
+
+      HammerCLIForeman::Template::CreateCommand.any_instance.stubs(:kinds).returns(["PXELinux"])
+
+      api_expects_no_call
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
     end
   end
 end