hammer_cli_foreman 0.7.0 → 0.8.0

data/test/functional/smart_variable_test.rb

@@ -0,0 +1,150 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'smart-variable update' do
+  let(:cmd) { %w(smart-variable update) }
+
+  it 'allows to save new name' do
+    params = ['--id=1', '--new-variable=name2']
+
+    expected_result = success_result("Smart variable [name2] updated\n")
+
+    api_expects(:smart_variables, :update, 'Update the variable') do |par|
+      par['smart_variable']['variable'] == 'name2'
+    end.returns(
+      'description' => '',
+      'parameter_type' => 'string',
+      'default_value' => '',
+      'hidden_value?' => false,
+      'hidden_value' => '*****',
+      'validator_type' => '',
+      'validator_rule' => nil,
+      'override_value_order' => 'fqdn\nhostgroup\nos\ndomain',
+      'merge_overrides' => false,
+      'merge_default' => false,
+      'avoid_duplicates' => false,
+      'puppetclass_id' => 68,
+      'puppetclass_name' => 'apache',
+      'created_at' => '2016-08-16 07:48:29 UTC',
+      'updated_at' => '2016-08-16 07:54:05 UTC',
+      'variable' => 'name2',
+      'id' => 1,
+      'override_values_count' => 0,
+      'override_values' => []
+    )
+
+    result = run_cmd(cmd + params)
+    assert_cmd(expected_result, result)
+  end
+end
+
+describe 'smart-variable info' do
+  let(:cmd) { %w(smart-variable info) }
+  let(:variable) do
+    {
+      'description' => '',
+      'parameter_type' => 'string',
+      'default_value' => "1.2.3.4     aa.con\r\n1.2.3.5     bb.com",
+      'hidden_value?' => false,
+      'hidden_value' => '*****',
+      'validator_type' => '',
+      'validator_rule' => nil,
+      'override_value_order' => "fqdn\nhostgroup\nos\ndomain",
+      'merge_overrides' => false,
+      'merge_default' => false,
+      'avoid_duplicates' => false,
+      'puppetclass_id' => 68,
+      'puppetclass_name' => 'apache',
+      'created_at' => '2016-08-01 13:05:24 UTC',
+      'updated_at' => '2016-08-08 12:36:08 UTC',
+      'variable' => 'abc',
+      'id' => 1,
+      'override_values_count' => 0
+    }
+  end
+
+  it 'searches by --variable parameter' do
+    params = ['--variable=abc']
+
+    api_expects(:smart_variables, :index, 'Search for variable') do |par|
+      par[:search] == %(key = "#{variable['variable']}")
+    end.returns(index_response([variable]))
+
+    api_expects(:smart_variables, :show, 'Variable details') do |par|
+      par['id'] == 1
+    end.returns(variable)
+
+    result = run_cmd(cmd + params)
+    assert_match(/abc/, result.out)
+  end
+
+  it 'searches by deprecated --name parameter' do
+    params = ['--name=abc']
+
+    api_expects(:smart_variables, :index, 'Search for variable') do |par|
+      par[:search] == %(key = "#{variable['variable']}")
+    end.returns(index_response([variable]))
+
+    api_expects(:smart_variables, :show, 'Variable details') do |par|
+      par['id'] == 1
+    end.returns(variable)
+
+    result = run_cmd(cmd + params)
+    assert_match(/abc/, result.out)
+  end
+end
+
+describe 'smart-variable delete' do
+  let(:cmd) { %w(smart-variable delete) }
+  let(:variable) do
+    {
+      'description' => '',
+      'parameter_type' => 'string',
+      'default_value' => "1.2.3.4     aa.con\r\n1.2.3.5     bb.com",
+      'hidden_value?' => false,
+      'hidden_value' => '*****',
+      'validator_type' => '',
+      'validator_rule' => nil,
+      'override_value_order' => "fqdn\nhostgroup\nos\ndomain",
+      'merge_overrides' => false,
+      'merge_default' => false,
+      'avoid_duplicates' => false,
+      'puppetclass_id' => 68,
+      'puppetclass_name' => 'apache',
+      'created_at' => '2016-08-01 13:05:24 UTC',
+      'updated_at' => '2016-08-08 12:36:08 UTC',
+      'variable' => 'abc',
+      'id' => 1,
+      'override_values_count' => 0
+    }
+  end
+
+  it 'deletes by --variable parameter' do
+    params = ['--variable=abc']
+
+    api_expects(:smart_variables, :index, 'Search for variable') do |par|
+      par[:search] == %(key = "#{variable['variable']}")
+    end.returns(index_response([variable]))
+
+    api_expects(:smart_variables, :destroy, 'Variable details') do |par|
+      par['id'] == 1
+    end.returns(variable)
+
+    result = run_cmd(cmd + params)
+    assert_match(/abc/, result.out)
+  end
+
+  it 'deletes by deprecated --name parameter' do
+    params = ['--name=abc']
+
+    api_expects(:smart_variables, :index, 'Search for variable') do |par|
+      par[:search] == %(key = "#{variable['variable']}")
+    end.returns(index_response([variable]))
+
+    api_expects(:smart_variables, :destroy, 'Variable details') do |par|
+      par['id'] == 1
+    end.returns(variable)
+
+    result = run_cmd(cmd + params)
+    assert_match(/abc/, result.out)
+  end
+end

data/test/unit/host_test.rb

@@ -29,10 +29,6 @@ describe HammerCLIForeman::Host do
 
     let(:cmd) { HammerCLIForeman::Host::InfoCommand.new("", ctx) }
 
-    before :each do
-      cmd.stubs(:get_parameters).returns([])
-    end
-
     context "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=host"]

data/test/unit/location_test.rb

@@ -25,6 +25,7 @@ describe HammerCLIForeman::Location do
       let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
+      it_should_print_column "Description"
       it_should_print_column "Name"
       it_should_print_column "Id"
     end

data/test/unit/organization_test.rb

@@ -26,6 +26,7 @@ describe HammerCLIForeman::Organization do
       let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
+      it_should_print_column "Description"
       it_should_print_column "Name"
       it_should_print_column "Id"
     end

data/test/unit/realm_test.rb

@@ -0,0 +1,87 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+require 'hammer_cli_foreman/realm'
+
+describe HammerCLIForeman::Realm do
+
+  include CommandTestHelper
+
+  context "ListCommand" do
+
+    let(:cmd) { HammerCLIForeman::Realm::ListCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "no arguments"
+      it_should_accept_search_params
+    end
+
+    context "output" do
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
+
+      it_should_print_n_records
+      it_should_print_columns ["Id", "Name"]
+    end
+
+  end
+
+
+  context "InfoCommand" do
+
+    let(:cmd) { HammerCLIForeman::Realm::InfoCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name", ["--name=arch"]
+      # it_should_fail_with "no arguments" # TODO: temporarily disabled, parameters are checked in the id resolver
+    end
+
+    context "output" do
+      with_params ["--id=1"] do
+        it_should_print_n_records 1
+        it_should_print_columns ["Id", "Name", "Created at", "Updated at"]
+        it_should_print_columns ["Realm proxy id", "Realm type"]
+      end
+    end
+
+  end
+
+  context "CreateCommand" do
+
+    let(:cmd) { HammerCLIForeman::Realm::CreateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name, type, proxy", ["--name=realm", "--realm-type=FreeIPA", "--realm-proxy-id=1"]
+    end
+
+  end
+
+
+  context "DeleteCommand" do
+
+    let(:cmd) { HammerCLIForeman::Realm::DeleteCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=realm"]
+      it_should_accept "id", ["--id=1"]
+      # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
+    end
+
+  end
+
+
+  context "UpdateCommand" do
+
+    let(:cmd) { HammerCLIForeman::Realm::UpdateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=realm", "--new-name=realm2"]
+      it_should_accept "id", ["--id=1", "--new-name=realm2"]
+      # it_should_fail_with "no params", []
+      # it_should_fail_with "name or id missing", ["--new-name=arch2", "--description=full_realm_name"]
+      # TODO: temporarily disabled, parameters are checked in the id resolver
+    end
+
+  end
+
+end

data/test/unit/smart_class_parameter_test.rb

@@ -54,6 +54,13 @@ describe HammerCLIForeman::SmartClassParameter do
       # it_should_fail_with "no arguments"
       # TODO: temporarily disabled, parameters are checked in the id resolver
     end
+
+    context "output" do
+      with_params ["--id=1"] do
+        it_should_print_n_records 1
+        it_should_print_columns ["Description", "Type", "Use puppet default", "Required"]
+      end
+    end
   end
 
 

data/test/unit/smart_variable_test.rb

@@ -28,7 +28,7 @@ describe HammerCLIForeman::SmartVariable do
 
       it_should_print_n_records
       it_should_print_column "Id"
-      it_should_print_column "Name"
+      it_should_print_column "Variable"
       it_should_print_column "Default Value"
       it_should_print_column "Type"
       it_should_print_column "Class Id"

data/test/unit/usergroup_test.rb

@@ -20,6 +20,7 @@ describe HammerCLIForeman::Usergroup do
     context "output" do
       it_should_print_column "Id"
       it_should_print_column "Name"
+      it_should_print_column "Admin"
     end
 
   end
@@ -37,6 +38,7 @@ describe HammerCLIForeman::Usergroup do
       with_params ["--id=1"] do
         it_should_print_column "Id"
         it_should_print_column "Name"
+        it_should_print_column "Admin"
         it_should_print_column "Users"
         it_should_print_column "User groups"
         it_should_print_column "Roles"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_foreman
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.8.0
 platform: ruby
 authors:
 - Tomáš Strachota
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-14 00:00:00.000000000 Z
+date: 2016-09-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hammer_cli
@@ -17,14 +17,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: 0.8.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: 0.8.0
 - !ruby/object:Gem::Dependency
   name: apipie-bindings
   requirement: !ruby/object:Gem::Requirement
@@ -101,6 +101,7 @@ files:
 - lib/hammer_cli_foreman/parameter.rb
 - lib/hammer_cli_foreman/partition_table.rb
 - lib/hammer_cli_foreman/puppet_class.rb
+- lib/hammer_cli_foreman/realm.rb
 - lib/hammer_cli_foreman/references.rb
 - lib/hammer_cli_foreman/report.rb
 - lib/hammer_cli_foreman/resource_supported_test.rb
@@ -176,9 +177,12 @@ files:
 - test/data/README.md
 - test/functional/commands/list_test.rb
 - test/functional/host_test.rb
+- test/functional/hostgroup/create_test.rb
+- test/functional/hostgroup/update_test.rb
 - test/functional/location_test.rb
 - test/functional/organization_test.rb
 - test/functional/smart_class_parameter_test.rb
+- test/functional/smart_variable_test.rb
 - test/functional/template_test.rb
 - test/functional/test_helper.rb
 - test/reports/TEST-HammerCLIForeman-Architecture-CreateCommand-parameters.xml
@@ -457,6 +461,7 @@ files:
 - test/unit/param_filters_test.rb
 - test/unit/partition_table_test.rb
 - test/unit/puppet_class_test.rb
+- test/unit/realm_test.rb
 - test/unit/report_test.rb
 - test/unit/role_test.rb
 - test/unit/smart_class_parameter_test.rb
@@ -735,10 +740,13 @@ test_files:
 - test/reports/TEST-HammerCLIForeman-PartitionTable-ListCommand-output.xml
 - test/reports/TEST-HammerCLIForeman-Hostgroup-UpdateCommand-parameters.xml
 - test/functional/host_test.rb
+- test/functional/smart_variable_test.rb
 - test/functional/organization_test.rb
 - test/functional/location_test.rb
 - test/functional/template_test.rb
 - test/functional/smart_class_parameter_test.rb
+- test/functional/hostgroup/create_test.rb
+- test/functional/hostgroup/update_test.rb
 - test/functional/commands/list_test.rb
 - test/functional/test_helper.rb
 - test/unit/image_test.rb
@@ -754,6 +762,7 @@ test_files:
 - test/unit/smart_proxy_test.rb
 - test/unit/subnet_test.rb
 - test/unit/location_test.rb
+- test/unit/realm_test.rb
 - test/unit/dependency_resolver_test.rb
 - test/unit/helpers/command.rb
 - test/unit/helpers/resource_disabled.rb