hammer_cli_foreman 0.9.0 → 0.10.0

data/test/unit/apipie_resource_mock.rb

@@ -54,6 +54,146 @@ module ResourceMocks
       ]})
   end
 
+  def self.users_show
+    ResourceMocks.mock_action_call(:users, :show, {
+      "firstname" => "Admin",
+      "lastname" => "User",
+      "mail" => "root@example.com",
+      "admin" => true,
+      "auth_source_id" => 1,
+      "auth_source_name" => "Internal",
+      "timezone" => nil,
+      "locale" => nil,
+      "last_login_on" => "2016-12-20 17:30:13 UTC",
+      "created_at" => "2016-10-24 08:11:24 UTC",
+      "updated_at" => "2016-10-24 08:11:38 UTC",
+      "id" => 3,
+      "login" => "admin",
+      "description" => nil,
+      "default_location" => nil,
+      "locations" => [],
+      "default_organization" => {
+        "id" => 1,
+        "name" => "Default Organization",
+        "title" => "Default Organization",
+        "description" => nil
+      },
+      "organizations" => [],
+      "effective_admin" => true,
+      "cached_usergroups" => [],
+      "auth_source_internal" => {
+        "id" => 1,
+        "type" => "AuthSourceInternal",
+        "name" => "Internal"
+      },
+      "mail_notifications" => [],
+      "roles" => [{
+        "name" => "Default role",
+        "id" => 9,
+        "description" => nil
+      }],
+      "usergroups" => []
+    })
+  end
+
+  def self.hosts_show
+    ResourceMocks.mock_action_call(:hosts, :show, {
+      "ip" => "192.168.122.51",
+      "ip6" => nil,
+      "environment_id" => 1,
+      "environment_name" => "production",
+      "last_report" => "2016-10-24 12:06:31 UTC",
+      "mac" => "52:54:00:ce:b2:b9",
+      "realm_id" => nil,
+      "realm_name" => nil,
+      "sp_mac" => nil,
+      "sp_ip" => nil,
+      "sp_name" => nil,
+      "domain_id" => 1,
+      "domain_name" => "tstrachota.usersys.redhat.com",
+      "architecture_id" => 1,
+      "architecture_name" => "x86_64",
+      "operatingsystem_id" => 1,
+      "operatingsystem_name" => "CentOS 7.2.1511",
+      "build" => false,
+      "model_id" => 1,
+      "hostgroup_id" => nil,
+      "owner_id" => nil,
+      "owner_type" => nil,
+      "enabled" => true,
+      "managed" => false,
+      "use_image" => nil,
+      "image_file" => "",
+      "uuid" => nil,
+      "compute_resource_id" => nil,
+      "compute_resource_name" => nil,
+      "compute_profile_id" => nil,
+      "compute_profile_name" => nil,
+      "capabilities" => ["build"],
+      "provision_method" => "build",
+      "certname" => "foreman.example.com",
+      "image_id" => nil,
+      "image_name" => nil,
+      "created_at" => "2016-10-24 08:36:43 UTC",
+      "updated_at" => "2016-10-24 12:06:46 UTC",
+      "last_compile" => "2016-10-24 12:06:41 UTC",
+      "global_status" => 0,
+      "global_status_label" => "Warning",
+      "organization_id" => nil,
+      "organization_name" => nil,
+      "location_id" => nil,
+      "location_name" => nil,
+      "puppet_status" => 0,
+      "model_name" => "KVM",
+      "configuration_status" => 0,
+      "configuration_status_label" => "No reports",
+      "name" => "foreman.example.com",
+      "id" => 1,
+      "puppet_proxy_id" => 1,
+      "puppet_proxy_name" => "foreman.example.com",
+      "puppet_ca_proxy_id" => 1,
+      "puppet_ca_proxy_name" => "foreman.example.com",
+      "puppet_proxy" => {
+        "name" => "foreman.example.com",
+        "id" => 1,
+        "url" => "https://foreman.example.com:9090"
+      },
+      "puppet_ca_proxy" => {
+        "name" => "foreman.example.com",
+        "id" => 1,
+        "url" => "https://foreman.example.com:9090"
+      },
+      "hostgroup_name" => nil,
+      "hostgroup_title" => nil,
+      "parameters" => [],
+      "all_parameters" => [],
+      "interfaces" => [{
+        "id" => 1,
+        "name" => "foreman.example.com",
+        "ip" => "192.168.122.51",
+        "mac" => "52:54:00:ce:b2:b9",
+        "identifier" => "eth0",
+        "primary" => true,
+        "provision" => true,
+        "type" => "interface"
+      },
+      {
+        "id" => 2,
+        "name" => nil,
+        "ip" => "10.34.130.105",
+        "mac" => "52:54:00:f5:1b:57",
+        "identifier" => "eth1",
+        "primary" => false,
+        "provision" => false,
+        "type" => "interface"
+      }],
+      "puppetclasses" => [],
+      "config_groups" => [],
+      "all_puppetclasses" => []
+    })
+  end
+
+
   def self.organizations_show
     ResourceMocks.mock_action_calls(
       [:organizations, :index, [{ "id" => 2, "name" => "ACME" }]],
@@ -85,6 +225,7 @@ module ResourceMocks
   def self.operatingsystems
     ResourceMocks.mock_action_calls(
       [:parameters, :index, []],
+      [:operatingsystems, :index, []],
       [:operatingsystems, :show, {}]
       )
   end
@@ -114,4 +255,43 @@ module ResourceMocks
       }]
     })
   end
+
+  def self.auth_source_ldap_index
+    ResourceMocks.mock_action_call(:auth_source_ldaps, :index, {
+      "results" => [{
+        "name" => "my LDAP",
+        "id" => 1,
+        "tls" => false,
+        "port" => 389,
+        "server_type" => "POSIX"
+      }]
+    })
+  end
+
+  def self.compute_resource_show
+    cr = {
+      "name" => "My compute resource",
+      "id" => 42,
+      "provider" => "Libvirt"
+    }
+    ResourceMocks.mock_action_calls(
+      [:compute_resources, :index, [ "results" => cr ]],
+      [:compute_resources, :show, cr]
+    )
+  end
+
+  def self.common_parameter_show
+    ResourceMocks.mock_action_call(:common_parameters, :show, {
+      "name" => "my param",
+      "value" => "random value"
+    })
+  end
+
+  def self.config_groups_index
+    ResourceMocks.mock_action_call(:config_groups, :index, [{
+      :id => 15,
+      :name => "test config group",
+      :puppetclasses => [ { :name => "My puppetclass" } ]
+    }])
+  end
 end

data/test/unit/architecture_test.rb

@@ -8,6 +8,9 @@ describe HammerCLIForeman::Architecture do
   include CommandTestHelper
 
   context "ListCommand" do
+    before do
+      ResourceMocks.mock_action_call(:architectures, :index, [])
+    end
 
     let(:cmd) { HammerCLIForeman::Architecture::ListCommand.new("", ctx) }
 

data/test/unit/auth_source_ldap_test.rb

@@ -8,6 +8,9 @@ describe HammerCLIForeman::AuthSourceLdap do
   include CommandTestHelper
 
   context "ListCommand" do
+    before :each do
+      ResourceMocks.auth_source_ldap_index
+    end
 
     let(:cmd) { HammerCLIForeman::AuthSourceLdap::ListCommand.new("", ctx) }
 

data/test/unit/commands_test.rb

@@ -179,6 +179,8 @@ describe HammerCLIForeman::Command do
       )
     )
 
+    ResourceMocks.mock_action_call(:domains, :index, [])
+
     out, err = capture_io do
       com.run([]).must_equal HammerCLI::EX_OK
     end

data/test/unit/common_parameter_test.rb

@@ -8,6 +8,9 @@ describe HammerCLIForeman::CommonParameter do
   include CommandTestHelper
 
   context "ListCommand" do
+    before do
+      ResourceMocks.mock_action_call(:common_parameters, :index, [])
+    end
 
     let(:cmd) { HammerCLIForeman::CommonParameter::ListCommand.new("", ctx) }
 
@@ -27,11 +30,14 @@ describe HammerCLIForeman::CommonParameter do
 
 
   context "SetCommand" do
+    before do
+      ResourceMocks.common_parameter_show
+    end
 
     let(:cmd) { HammerCLIForeman::CommonParameter::SetCommand.new("", ctx) }
 
     context "parameters" do
-      it_should_accept "name and value", ["--name=param", "--value=val"]
+      it_should_accept "name, value and hidden-value", ["--name=param", "--value=val", "--hidden-value=true"]
       # it_should_fail_with "name missing", ["--value=val"]
       # it_should_fail_with "value missing", ["--name=param"]
       # TODO: temporarily disabled, parameters are checked by the api
@@ -42,10 +48,10 @@ describe HammerCLIForeman::CommonParameter do
         ResourceMocks.mock_action_calls(
           [:common_parameters, :index, []],
           [:common_parameters, :create,
-            {"id" => 1, "name" => "param", "value" => "val"},
-            {'common_parameter' => {'name' => 'param', 'value' => 'val'}, 'id' => 'param'}])
+            {"id" => 1, "name" => "param", "value" => "val", "hidden-value" => false},
+            {'common_parameter' => {'name' => 'param', 'value' => 'val', 'hidden_value' => false}, 'id' => 'param'}])
       end
-      with_params ["--name=param", "--value=val"] do
+      with_params ["--name=param", "--value=val", "--hidden-value=false"] do
         it_should_output "Created parameter [param] with value [val].,1,param", :csv
       end
     end
@@ -55,11 +61,11 @@ describe HammerCLIForeman::CommonParameter do
         ResourceMocks.mock_action_calls(
           [:common_parameters, :index, [{'name' => 'param', 'value' => 'test'}]],
           [:common_parameters, :update,
-            {"id" => 1, "name" => "param", "value" => "val"},
-            {'common_parameter' => {'name' => 'param', 'value' => 'val'}, 'id' => 'param'}])
+            {"id" => 1, "name" => "param", "value" => "val", "hidden-value" => false},
+            {'common_parameter' => {'name' => 'param', 'value' => 'val', 'hidden_value' => false}, 'id' => 'param'}])
       end
 
-      with_params ["--name=param", "--value=val"] do
+      with_params ["--name=param", "--value=val", "--hidden-value=false"] do
         it_should_output "Parameter [param] updated to [val].,1,param", :csv
       end
     end

data/test/unit/compute_resource_test.rb

@@ -9,6 +9,9 @@ describe HammerCLIForeman::ComputeResource do
   include CommandTestHelper
 
   context "ListCommand" do
+    before do
+      ResourceMocks.mock_action_call(:compute_resources, :index, [])
+    end
 
     let(:cmd) { HammerCLIForeman::ComputeResource::ListCommand.new("", ctx) }
 
@@ -28,6 +31,9 @@ describe HammerCLIForeman::ComputeResource do
 
 
   context "InfoCommand" do
+    before do
+      ResourceMocks.compute_resource_show
+    end
 
     let(:cmd) { HammerCLIForeman::ComputeResource::InfoCommand.new("", ctx) }
 

data/test/unit/config_group_test.rb

@@ -0,0 +1,72 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+require 'hammer_cli_foreman/config_group'
+
+describe HammerCLIForeman::ConfigGroup do
+  include CommandTestHelper
+
+  context "ListCommand" do
+    let(:cmd) { HammerCLIForeman::ConfigGroup::ListCommand.new("", ctx) }
+
+    before :each do
+      ResourceMocks.config_groups_index
+    end
+
+    context "parameters" do
+      it_should_accept "no arguments"
+      it_should_accept_search_params
+    end
+
+    context "output" do
+      let(:expected_record_count) { cmd.resource.call(:index).length }
+      it_should_print_n_records
+      it_should_print_columns ["ID", "Name"]
+    end
+  end
+
+  context "InfoCommand" do
+    let(:cmd) { HammerCLIForeman::ConfigGroup::InfoCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name", ["--name=group_x"]
+    end
+
+    context "output" do
+      with_params ["--id=1"] do
+        it_should_print_n_records 1
+        it_should_print_column "Name"
+        it_should_print_column "ID"
+        it_should_print_column "Puppetclasses"
+      end
+    end
+  end
+
+  context "CreateCommand" do
+    let(:cmd) { HammerCLIForeman::ConfigGroup::CreateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name, puppetclass ids", ["--name=first_group", "--puppet-class-ids=1,2"]
+    end
+  end
+
+  context "DeleteCommand" do
+    let(:cmd) { HammerCLIForeman::ConfigGroup::DeleteCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=group_x"]
+      it_should_accept "id", ["--id=1"]
+    end
+  end
+
+  context "UpdateCommand" do
+
+    let(:cmd) { HammerCLIForeman::ConfigGroup::UpdateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=group_x"]
+      it_should_accept "id", ["--id=1"]
+    end
+  end
+end

data/test/unit/defaults_test.rb

@@ -1,32 +1,36 @@
 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
+describe HammerCLIForeman::Defaults do
+  let(:user) {
+    {"default_organization" => {"id" => 2}, "default_location" => {"id" => 1}}
+  }
+  let(:empty_user) {
+    {"default_organization" => nil, "default_location" => nil}
+  }
+
+  before do
+    connection = api_connection
+    @api = APIExpectationsDecorator.new(connection.api)
+    @defaults_provider = HammerCLIForeman::Defaults.new(connection)
+  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 defaults organization when exisits " do
+    @api.expects_search(:users, 'login=admin').returns(index_response([user]))
+    assert_equal 2, @defaults_provider.get_defaults(:organization_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
+  it "returns nil when defaults organization doesn't exisits " do
+    @api.expects_search(:users, 'login=admin').returns(index_response([empty_user]))
+    assert_nil @defaults_provider.get_defaults(:organization_id)
+  end
 
+  it "returns defaults location when exisits " do
+    @api.expects_search(:users, 'login=admin').returns(index_response([user]))
+    assert_equal 1, @defaults_provider.get_defaults(:location_id)
   end
 
+  it "returns nil when defaults location doesn't exisits " do
+    @api.expects_search(:users, 'login=admin').returns(index_response([empty_user]))
+    assert_nil @defaults_provider.get_defaults(:location_id)
+  end
 end

data/test/unit/domain_test.rb

@@ -8,6 +8,9 @@ describe HammerCLIForeman::Domain do
   include CommandTestHelper
 
   context "ListCommand" do
+    before do
+      ResourceMocks.mock_action_call(:domains, :index, [])
+    end
 
     let(:cmd) { HammerCLIForeman::Domain::ListCommand.new("", ctx) }