hammer_cli_foreman 0.4.0 → 0.5.0

data/test/unit/defaults_test.rb

@@ -0,0 +1,32 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+describe HammerCLIForeman::Defaults do
+
+  context "Defaults" do
+
+    defaults_provider = HammerCLIForeman::Defaults.new
+    user = {"results" => ["default_organization" => {"id" => 2}, "default_location" => {"id" => 1}]}
+    empty_user = {"results" => ["default_organization" => nil, "default_location" => nil]}
+
+    it "returns defaults organization when exisits " do
+      defaults_provider.stubs(:get_user).returns user
+      assert_equal 2, defaults_provider.get_defaults(:organization_id)
+    end
+
+    it "returns nil when defaults organization doesn't exisits " do
+      defaults_provider.stubs(:get_user).returns empty_user
+      assert_nil defaults_provider.get_defaults(:organization_id)
+    end
+
+    it "returns defaults location when exisits " do
+      defaults_provider.stubs(:get_user).returns user
+      assert_equal 1, defaults_provider.get_defaults(:location_id)
+    end
+
+    it "returns nil when defaults location doesn't exisits " do
+      defaults_provider.stubs(:get_user).returns empty_user
+      assert_nil defaults_provider.get_defaults(:location_id)
+    end
+
+  end
+
+end

data/test/unit/dependency_resolver_test.rb

@@ -4,41 +4,46 @@ require File.join(File.dirname(__FILE__), 'helpers/fake_searchables')
 
 describe HammerCLIForeman::DependencyResolver do
 
+  let(:api) do ApipieBindings::API.new({
+    :apidoc_cache_dir => 'test/unit/data',
+    :apidoc_cache_name => 'test_api',
+    :dry_run => true})
+  end
+
+  before :each do
+    HammerCLIForeman.stubs(:foreman_api).returns(api)
+  end
+
   let(:resolver) { HammerCLIForeman::DependencyResolver.new }
 
   describe "for resource" do
 
     it "returns empty array for an independent resource" do
-      resource = HammerCLIForeman.foreman_resource!(:architectures)
+      resource = api.resource(:users)
       resolver.resource_dependencies(resource).must_equal []
     end
 
     it "returns list of dependent resources" do
-      resource = HammerCLIForeman.foreman_resource!(:images)
-      resolver.resource_dependencies(resource).map(&:name).sort_by{|sym| sym.to_s}.must_equal [
-        :compute_resources, :organizations, :locations
-      ].sort_by{|sym| sym.to_s}
-    end
+      resource = api.resource(:comments)
 
+      resources = resolver.resource_dependencies(resource).map(&:name).sort_by{ |sym| sym.to_s }
+      expected = [:posts, :users]
+      resources.must_equal expected.sort_by{ |sym| sym.to_s }
+    end
   end
 
   describe "for action" do
 
     it "returns empty array for an independent action" do
-      action = HammerCLIForeman.foreman_resource!(:organizations).action(:index)
+      action = HammerCLIForeman.foreman_resource!(:users).action(:index)
       resolver.action_dependencies(action).must_equal []
     end
 
     it "returns list of dependent resources" do
-      action = HammerCLIForeman.foreman_resource!(:hostgroups).action(:create)
-      resolver.action_dependencies(action).map(&:name).sort_by{|sym| sym.to_s}.must_equal [
-        :environments, :operatingsystems, :architectures, :media,
-        :ptables, :subnets, :domains, :realms, :organizations, :locations
-      ].sort_by{|sym| sym.to_s}
+      action = HammerCLIForeman.foreman_resource!(:comments).action(:create)
+      resources = resolver.action_dependencies(action).map(&:name).sort_by{ |sym| sym.to_s }
+      expected = [:posts, :users]
+      resources.must_equal expected.sort_by{|sym| sym.to_s}
     end
-
   end
-
 end
-
-

data/test/unit/domain_test.rb

@@ -17,7 +17,7 @@ describe HammerCLIForeman::Domain do
     end
 
     context "output" do
-      let(:expected_record_count) { cmd.resource.call(:index).length }
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
       it_should_print_columns ["Id", "Name"]

data/test/unit/environment_test.rb

@@ -17,7 +17,7 @@ describe HammerCLIForeman::Environment do
     end
 
     context "output" do
-      let(:expected_record_count) { cmd.resource.call(:index).length }
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
       it_should_print_column "Name"

data/test/unit/external_usergroup_test.rb

@@ -20,7 +20,7 @@ describe HammerCLIForeman::ExternalUsergroup do
     end
 
     context "output" do
-      let(:expected_record_count) { cmd.resource.call(:index, :usergroup_id => 1).length }
+      let(:expected_record_count) { count_records(cmd.resource.call(:index, :usergroup_id => 1)) }
 
       with_params ["--user-group-id=1"] do
         it_should_print_n_records

data/test/unit/helpers/command.rb

@@ -24,7 +24,7 @@ class IdResolverTestProxy
 
       method_name = "#{resource.singular_name}_ids"
       self.class.send(:define_method, method_name) do |options|
-        [1]
+        options["option_#{resource.singular_name}_ids"].nil? ? nil : [1]
       end
     end
   end
@@ -43,6 +43,10 @@ module CommandTestHelper
     end
   end
 
+  def count_records(data)
+    HammerCLIForeman.collection_to_common_format(data['results']).count
+  end
+
   module ClassMethods
 
     def with_params(params, &block)

data/test/unit/host_test.rb

@@ -17,7 +17,7 @@ describe HammerCLIForeman::Host do
     end
 
     context "output" do
-      let(:expected_record_count) { cmd.resource.call(:index).length }
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
       it_should_print_columns ["Id", "Name", "Operating System", "Host Group", "IP", "MAC"]
@@ -73,7 +73,7 @@ describe HammerCLIForeman::Host do
 
         it "should output status" do
           cmd.stubs(:context).returns(ctx.update(:adapter => :test))
-          proc { cmd.run(with_params) }.must_output "#Status#Power#\n#missing#running#\n"
+          proc { cmd.run(with_params) }.must_output "#Status#Power#\n#No reports#running#\n"
         end
       end
     end
@@ -160,7 +160,7 @@ describe HammerCLIForeman::Host do
 
     context "output" do
       with_params ["--id=1"] do
-        let(:expected_record_count) { cmd.resource.call(:index)[0].length }
+        let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
         it_should_print_n_records
         it_should_print_column "Id"
@@ -202,7 +202,7 @@ describe HammerCLIForeman::Host do
       it_should_accept "name, environment_id, architecture_id, domain_id, puppet_proxy_id, operatingsystem_id and more",
           ["--name=host", "--environment-id=1", "--architecture-id=1", "--domain-id=1", "--puppet-proxy-id=1", "--operatingsystem-id=1",
             "--ip=1.2.3.4", "--mac=11:22:33:44:55:66", "--medium-id=1", "--partition-table-id=1", "--subnet-id=1",
-            "--sp-subnet-id=1", "--model-id=1", "--hostgroup-id=1", "--owner-id=1", '--puppet-ca-proxy-id=1', '--puppet-class-ids',
+            "--model-id=1", "--hostgroup-id=1", "--owner-id=1", '--puppet-ca-proxy-id=1', '--puppet-class-ids',
             "--root-password=pwd", "--ask-root-password=true", "--provision-method=build", "--interface=primary=true,provision=true"]
       it_should_fail_with "name or id missing",
           ["--environment-id=1", "--architecture-id=1", "--domain-id=1", "--puppet-proxy-id=1", "--operatingsystem-id=1", "--interface=primary=true,provision=true"]
@@ -221,7 +221,7 @@ describe HammerCLIForeman::Host do
 
       with_params ["--name=host", "--environment-id=1", "--architecture-id=1", "--domain-id=1", "--puppet-proxy-id=1", "--operatingsystem-id=1",
             "--ip=1.2.3.4", "--mac=11:22:33:44:55:66", "--medium-id=1", "--partition-table-id=1", "--subnet-id=1",
-            "--sp-subnet-id=1", "--model-id=1", "--hostgroup-id=1", "--owner-id=1", '--puppet-ca-proxy-id=1', '--puppet-class-ids',
+            "--model-id=1", "--hostgroup-id=1", "--owner-id=1", '--puppet-ca-proxy-id=1', '--puppet-class-ids',
             "--root-password=pwd", "--ask-root-password=true", "--provision-method=build", "--interface=primary=true,provision=true"] do
         it_should_call_action_and_test_params(:create) { |par| par["host"]["managed"] == true }
         it_should_call_action_and_test_params(:create) { |par| par["host"]["build"] == true }
@@ -231,8 +231,11 @@ describe HammerCLIForeman::Host do
         it_should_call_action_and_test_params(:create) { |par| par["host"]["interfaces_attributes"]["0"]["provision"] == "true" }
       end
 
-      with_params ["--name=host", "--hostgroup-id=1", "--interface=primary=true,provision=true", "--parameters=servers=[pool.ntp.org,ntp.time.org]"] do
-        it_should_call_action_and_test_params(:create) { |par| par["host"]["host_parameters_attributes"][0]["value"] == "[\"pool.ntp.org\", \"ntp.time.org\"]" }
+      with_params ["--name=host", "--hostgroup-id=1", "--interface=primary=true,provision=true", "--parameters=servers=[pool.ntp.org,ntp.time.org],password=secret"] do
+        it_should_call_action_and_test_params(:create) do |par|
+          par["host"]["host_parameters_attributes"][0]["value"] == "[\"pool.ntp.org\", \"ntp.time.org\"]" &&
+          par["host"]["host_parameters_attributes"][1]["value"] == "secret"
+        end
       end
 
       it_should_fail_with "primary interface missing", ["--hostgroup-id=example", "--interface=primary=true"]
@@ -253,7 +256,7 @@ describe HammerCLIForeman::Host do
       it_should_accept "id and more", ["--id=1", "--new-name=host2", "--environment-id=1", "--architecture-id=1",
             "--domain-id=1", "--puppet-proxy-id=1", "--operatingsystem-id=1",
             "--ip=1.2.3.4", "--mac=11:22:33:44:55:66", "--medium-id=1", "--partition-table-id=1", "--subnet-id=1",
-            "--sp-subnet-id=1", "--model-id=1", "--hostgroup-id=1", "--owner-id=1", '--puppet-ca-proxy-id=1',
+            "--model-id=1", "--hostgroup-id=1", "--owner-id=1", '--puppet-ca-proxy-id=1',
             "--root-password=pwd", "--ask-root-password=true", "--provision-method=build"]
       # it_should_fail_with "no params", []
       # it_should_fail_with "name or id missing", ["--new-name=host2"]
@@ -263,6 +266,7 @@ describe HammerCLIForeman::Host do
         it_should_call_action_and_test_params(:update) { |par| par["host"].key?("managed") != true }
         it_should_call_action_and_test_params(:update) { |par| par["host"].key?("build") != true }
         it_should_call_action_and_test_params(:update) { |par| par["host"].key?("enabled") != true }
+        it_should_call_action_and_test_params(:update) { |par| par["host"].key?("overwrite") != true }
       end
 
       with_params ["--id=1", "--enabled=true"] do
@@ -273,6 +277,10 @@ describe HammerCLIForeman::Host do
         it_should_call_action_and_test_params(:update) { |par| par["host"]["enabled"] == false }
       end
 
+      with_params ["--id=1", "--overwrite=true"] do
+        it_should_call_action_and_test_params(:update) { |par| par["host"]["overwrite"] == true }
+      end
+
       with_params ["--id=1","--provision-method=build"] do
         it_should_call_action_and_test_params(:update) { |par| par["host"]["provision_method"] == "build" }
       end

data/test/unit/hostgroup_test.rb

@@ -17,7 +17,7 @@ describe HammerCLIForeman::Hostgroup do
     end
 
     context "output" do
-      let(:expected_record_count) { cmd.resource.call(:index).length }
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
       it_should_print_columns ["Id", "Name", "Title", "Operating System"]

data/test/unit/id_resolver_test.rb

@@ -4,7 +4,11 @@ require File.join(File.dirname(__FILE__), 'helpers/fake_searchables')
 
 describe HammerCLIForeman::IdResolver do
 
-  let(:api) { HammerCLIForeman.foreman_api_connection.api }
+  let(:api) do ApipieBindings::API.new({
+    :apidoc_cache_dir => 'test/unit/data',
+    :apidoc_cache_name => 'test_api',
+    :dry_run => true})
+  end
   let(:searchables) { FakeSearchables.new(['name','label']) }
   let(:resolver) { HammerCLIForeman::IdResolver.new(api, searchables) }
 
@@ -66,7 +70,7 @@ describe HammerCLIForeman::IdResolver do
     end
 
     describe "when no search options are found" do
-      let(:resolver_run) { proc { resolver.architecture_id({"option_unknown" => "value"}) } }
+      let(:resolver_run) { proc { resolver.comment_id({"option_unknown" => "value"}) } }
 
       it "raises exception" do
         err = resolver_run.must_raise HammerCLIForeman::MissingSeachOptions
@@ -74,49 +78,49 @@ describe HammerCLIForeman::IdResolver do
 
       it "builds correct error message" do
         err = resolver_run.must_raise HammerCLIForeman::MissingSeachOptions
-        err.message.must_equal "Missing options to search architecture"
+        err.message.must_equal "Missing options to search comment"
       end
     end
 
     describe "searching independent resource" do
-      let(:resolver_run) { proc { resolver.architecture_id({"option_name" => "arch"}) } }
+      let(:resolver_run) { proc { resolver.user_id({"option_name" => "John Doe"}) } }
 
       it "raises exception when no resource is found" do
-        ResourceMocks.mock_action_call(:architectures, :index, [])
+        ResourceMocks.mock_action_call(:users, :index, [])
 
         err = resolver_run.must_raise HammerCLIForeman::ResolverError
-        err.message.must_equal "architecture not found"
+        err.message.must_equal "user not found"
       end
 
       it "raises exception when multiple resources are found" do
-        ResourceMocks.mock_action_call(:architectures, :index, [
-          {"id" => 11, "name" => "arch1"},
-          {"id" => 22, "name" => "arch2"}
+        ResourceMocks.mock_action_call(:users, :index, [
+          {"id" => 11, "name" => "user11"},
+          {"id" => 22, "name" => "user22"}
         ])
 
         err = resolver_run.must_raise HammerCLIForeman::ResolverError
-        err.message.must_equal "architecture found more than once"
+        err.message.must_equal "found more than one user"
       end
 
       it "calls index action with appropriate search params" do
         ApipieBindings::API.any_instance.expects(:call).with() do |resource, action, params, headers, opts|
-          ( resource == :architectures &&
+          ( resource == :users &&
             action == :index &&
-            params[:search] == "name = \"arch\"")
-        end.returns({"id" => 11, "name" => "arch1"})
+            params[:search] == "name = \"John Doe\"")
+        end.returns({"id" => 1, "name" => "John Doe"})
 
         resolver_run.call
       end
 
       it "uses option id when it's available" do
-        ResourceMocks.mock_action_call(:architectures, :index, [])
+        ResourceMocks.mock_action_call(:users, :index, [])
 
-        resolver.architecture_id({"option_id" => 83, "option_name" => "arch"}).must_equal 83
+        resolver.user_id({"option_id" => 83, "option_name" => "John Doe"}).must_equal 83
       end
 
       it "returns id of the resource" do
-        ResourceMocks.mock_action_call(:architectures, :index, [
-          {"id" => 11, "name" => "arch1"}
+        ResourceMocks.mock_action_call(:users, :index, [
+          {"id" => 11, "name" => "John Doe"}
         ])
 
         resolver_run.call.must_equal 11
@@ -125,51 +129,50 @@ describe HammerCLIForeman::IdResolver do
     end
 
     describe "searching dependent resources" do
-      let(:resolver_run) { proc { resolver.image_id({"option_name" => "img", "option_compute_resource_name" => "cr"}) } }
+      let(:resolver_run) { proc { resolver.post_id({"option_name" => "Post 11", "option_user_name" => "User 22"}) } }
 
       it "raises exception when no resource is found" do
-        ResourceMocks.mock_action_call(:images, :index, [])
-        ResourceMocks.mock_action_call(:compute_resources, :index, [])
+        ResourceMocks.mock_action_call(:posts, :index, [])
+        ResourceMocks.mock_action_call(:users, :index, [])
 
         err = resolver_run.must_raise HammerCLIForeman::ResolverError
-        err.message.must_equal "compute_resource not found"
+        err.message.must_equal "user not found"
       end
 
       it "raises exception when multiple resources are found" do
-        ResourceMocks.mock_action_call(:images, :index, [])
-        ResourceMocks.mock_action_call(:compute_resources, :index, [
-          {"id" => 11, "name" => "cr1"},
-          {"id" => 22, "name" => "cr2"}
+        ResourceMocks.mock_action_call(:posts, :index, [])
+        ResourceMocks.mock_action_call(:users, :index, [
+          {"id" => 11, "name" => "user1"},
+          {"id" => 22, "name" => "user2"}
         ])
 
         err = resolver_run.must_raise HammerCLIForeman::ResolverError
-        err.message.must_equal "compute_resource found more than once"
+        err.message.must_equal "found more than one user"
       end
 
       it "calls index action with appropriate search params" do
         ApipieBindings::API.any_instance.expects(:call).with() do |resource, action, params, headers, opts|
-          ( resource == :compute_resources &&
+          ( resource == :users &&
             action == :index &&
-            params[:search] == "name = \"cr\"")
-        end.returns({"id" => 11, "name" => "cr"})
+            params[:search] == "name = \"User 22\"")
+        end.returns({"id" => 22, "name" => "User 22"})
 
         ApipieBindings::API.any_instance.expects(:call).with() do |resource, action, params, headers, opts|
-          ( resource == :images &&
+          ( resource == :posts &&
             action == :index &&
-            params[:search] == "name = \"img\"")
-        end.returns({"id" => 11, "name" => "img"})
+            params[:search] == "name = \"Post 11\"")
+        end.returns({"id" => 11, "name" => "Post 11"})
 
         resolver_run.call
       end
 
       it "returns id of the resource" do
-        ResourceMocks.mock_action_call(:images, :index, [
-          {"id" => 11, "name" => "img1"}
+        ResourceMocks.mock_action_call(:posts, :index, [
+          {"id" => 11, "name" => "Post 11"}
         ])
-        ResourceMocks.mock_action_call(:compute_resources, :index, [
-          {"id" => 22, "name" => "cr2"}
+        ResourceMocks.mock_action_call(:users, :index, [
+          {"id" => 22, "name" => "User 22"}
         ])
-
         resolver_run.call.must_equal 11
       end
     end

data/test/unit/image_test.rb

@@ -20,7 +20,7 @@ describe HammerCLIForeman::Image do
     end
 
     context "output" do
-      let(:expected_record_count) { cmd.resource.call(:index, :compute_resource_id=>1).length }
+      let(:expected_record_count) { count_records(cmd.resource.call(:index, :compute_resource_id=>1)) }
 
       with_params ["--compute-resource-id=1"] do
         it_should_print_n_records
@@ -130,5 +130,3 @@ describe HammerCLIForeman::Image do
   end
 
 end
-
-

data/test/unit/location_test.rb

@@ -22,7 +22,7 @@ describe HammerCLIForeman::Location do
     end
 
     context "output" do
-      let(:expected_record_count) { cmd.resource.call(:index).length }
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
       it_should_print_column "Name"

data/test/unit/media_test.rb

@@ -17,7 +17,7 @@ describe HammerCLIForeman::Medium do
     end
 
     context "output" do
-      let(:expected_record_count) { cmd.resource.call(:index).length }
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
       it_should_print_column "Name"
@@ -63,7 +63,7 @@ describe HammerCLIForeman::Medium do
     end
 
     with_params ["--name=medium_x", "--path=http://some.path/", "--operatingsystem-ids=1,2"] do
-      it_should_call_action :create, {'medium' => {'name' => 'medium_x', 'path' => 'http://some.path/', 'operatingsystem_ids' => [1], 'location_ids' => [1], 'organization_ids' => [1]}}
+      it_should_call_action :create, {'medium' => {'name' => 'medium_x', 'path' => 'http://some.path/', 'operatingsystem_ids' => [1]}}
     end
   end
 
@@ -92,7 +92,7 @@ describe HammerCLIForeman::Medium do
     end
 
     with_params ["--id=1", "--new-name=medium_x", "--path=http://some.path/", "--operatingsystem-ids=1,2"] do
-      it_should_call_action :update, {'id' => '1', 'name' => 'medium_x', 'medium' => {'name' => 'medium_x', 'path' => 'http://some.path/', 'operatingsystem_ids' => [1], 'location_ids' => [1], 'organization_ids' => [1]}}
+      it_should_call_action :update, {'id' => '1', 'name' => 'medium_x', 'medium' => {'name' => 'medium_x', 'path' => 'http://some.path/', 'operatingsystem_ids' => [1]}}
     end
 
   end