hammer_cli_foreman 0.1.1 → 0.1.2

data/test/unit/organization_test.rb

@@ -2,6 +2,7 @@ require File.join(File.dirname(__FILE__), 'test_helper')
 require File.join(File.dirname(__FILE__), 'helpers/resource_disabled')
 require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
 
+require 'hammer_cli_foreman/organization'
 
 describe HammerCLIForeman::Organization do
 

data/test/unit/param_filters_test.rb

@@ -0,0 +1,143 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+require File.join(File.dirname(__FILE__), 'helpers/fake_searchables')
+
+describe HammerCLIForeman::ParamsFlattener do
+
+  let(:flattener) { HammerCLIForeman::ParamsFlattener.new }
+
+  let(:param1) {
+    stub(:name => "param1", :params => [])
+  }
+  let(:param2) {
+    stub(:name => "param1", :params => [])
+  }
+  let(:hash_param) {
+    stub(:name => "object", :params => (
+      [param1, param2]
+    ))
+  }
+  let(:hash_param2) {
+    stub(:name => "object", :params => (
+      [param1, hash_param]
+    ))
+  }
+
+  it "flatten params" do
+    flattener.filter([param1, param2, hash_param2]).must_equal [param1, param2, hash_param2, param1, hash_param, param1, param2]
+  end
+
+  it "returns empty array for no params" do
+    flattener.filter([]).must_equal []
+  end
+
+end
+
+
+describe HammerCLIForeman::IdParamsFilter do
+
+  let(:filter) { HammerCLIForeman::IdParamsFilter.new }
+  let(:nonrequired_filter) { HammerCLIForeman::IdParamsFilter.new(:only_required => false) }
+  let(:required_filter) { HammerCLIForeman::IdParamsFilter.new(:only_required => true) }
+
+  describe "params for action" do
+    let(:required_params) {[
+      stub(:name => "architecture_id", :required? => true, :params => []),
+      stub(:name => "organization_id", :required? => true, :params => [])
+    ]}
+    let(:id_params) {[
+      stub(:name => "domain_id", :required? => false, :params => [])
+    ]}
+    let(:other_params) {[
+      stub(:name => "location", :required? => true, :params => []),
+      stub(:name => "param", :required? => false, :params => [])
+    ]}
+    let(:hash_param) {
+      stub(:name => "object", :required? => false, :params => (
+        required_params+id_params+other_params
+      ))
+    }
+
+    let(:action) {
+      stub(:params => (required_params+id_params+other_params))
+    }
+
+    it "returns only required params ending with _id" do
+      required_filter.for_action(action).must_equal required_params
+    end
+
+    it "returns only ending with _id when :only_required is set to false" do
+      nonrequired_filter.for_action(action).must_equal (required_params+id_params)
+    end
+
+    it "returns required params by default" do
+      filter.for_action(action).must_equal required_params
+    end
+
+    context "with hash params" do
+
+      let(:action) {
+        stub(:params => [hash_param])
+      }
+
+      it "finds params inside a hash" do
+        filter.for_action(action).must_equal required_params
+      end
+
+      it "returns only required params ending with _id" do
+        required_filter.for_action(action).must_equal required_params
+      end
+
+      it "returns only ending with _id when :only_required is set to false" do
+        nonrequired_filter.for_action(action).must_equal (required_params+id_params)
+      end
+
+    end
+  end
+
+end
+
+
+
+
+describe HammerCLIForeman::ParamsNameFilter do
+
+  describe "params for action" do
+    let(:the_param) {
+      stub(:name => "xxx", :params => [])
+    }
+    let(:other_params) {[
+      stub(:name => "param1", :params => []),
+      stub(:name => "param2", :params => [])
+    ]}
+    let(:hash_param) {
+      stub(:name => "object", :params => (
+        [the_param]+other_params
+      ))
+    }
+
+    let(:action) {
+      stub(:params => ([the_param]+other_params))
+    }
+
+    it "finds parameter by name" do
+      action = stub(:params => ([the_param]+other_params))
+      filter = HammerCLIForeman::ParamsNameFilter.new("xxx")
+      filter.for_action(action).must_equal [the_param]
+    end
+
+    it "finds parameter by name nested in another hash parameter" do
+      action = stub(:params => ([hash_param]+other_params))
+      filter = HammerCLIForeman::ParamsNameFilter.new("xxx")
+      filter.for_action(action).must_equal [the_param]
+    end
+
+    it "returns empty array when the parameter is not found" do
+      action = stub(:params => ([the_param]+other_params))
+      filter = HammerCLIForeman::ParamsNameFilter.new("???")
+      filter.for_action(action).must_equal []
+    end
+
+  end
+
+end

data/test/unit/partition_table_test.rb

@@ -1,6 +1,7 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
 
+require 'hammer_cli_foreman/partition_table'
 
 describe HammerCLIForeman::PartitionTable do
 

data/test/unit/puppet_class_test.rb

@@ -1,6 +1,7 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
 
+require 'hammer_cli_foreman/puppet_class'
 
 describe HammerCLIForeman::PuppetClass do
 

data/test/unit/report_test.rb

@@ -1,6 +1,7 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
 
+require 'hammer_cli_foreman/report'
 
 describe HammerCLIForeman::Report do
 

data/test/unit/role_test.rb

@@ -0,0 +1,94 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+require 'hammer_cli_foreman/role'
+
+describe HammerCLIForeman::Role do
+
+  include CommandTestHelper
+
+
+  context "ListCommand" do
+
+    let(:cmd) { HammerCLIForeman::Role::ListCommand.new("", ctx) }
+
+    before :each do
+      ResourceMocks.mock_action_call(:roles, :index, [])
+    end
+
+    context "parameters" do
+      it_should_accept "no arguments"
+      it_should_accept_search_params
+    end
+
+    context "output" do
+      it_should_print_column "Id"
+      it_should_print_column "Name"
+    end
+
+  end
+
+
+  context "CreateCommand" do
+
+    let(:cmd) { HammerCLIForeman::Role::CreateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=role"]
+    end
+
+    with_params ["--name=role"] do
+      it_should_call_action :create, {'role' => {'name' => 'role'}}
+    end
+  end
+
+  context "DeleteCommand" do
+
+    let(:cmd) { HammerCLIForeman::Role::DeleteCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=role"]
+      it_should_accept "id", ["--id=1"]
+    end
+  end
+
+  context "UpdateCommand" do
+
+    let(:cmd) { HammerCLIForeman::Role::UpdateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=role"]
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name and new name", ["--name=role", "--new-name=role2"]
+    end
+
+    with_params ["--id=1", "--new-name=role2"] do
+      it_should_call_action :update, {'id' => '1', 'name' => 'role2', 'role' => {'name' => 'role2'}}
+    end
+
+  end
+
+  context "FiltersCommand" do
+
+    let(:cmd) { HammerCLIForeman::Role::FiltersCommand.new("", ctx) }
+
+    before :each do
+      ResourceMocks.mock_action_call(:filters, :index, [])
+    end
+
+    context "parameters" do
+      it_should_accept "name", ["--name=role"]
+      it_should_accept "id", ["--id=1"]
+    end
+
+    context "output" do
+      it_should_print_column "Id"
+      it_should_print_column "Resource type"
+      it_should_print_column "Search"
+      it_should_print_column "Role"
+      it_should_print_column "Permissions"
+    end
+
+  end
+
+end

data/test/unit/smart_class_parameter_test.rb

@@ -1,6 +1,7 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
 
+require 'hammer_cli_foreman/smart_class_parameter'
 
 describe HammerCLIForeman::SmartClassParameter do
 

data/test/unit/smart_proxy_test.rb

@@ -1,6 +1,7 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
 
+require 'hammer_cli_foreman/smart_proxy'
 
 describe HammerCLIForeman::SmartProxy do
 
@@ -12,7 +13,7 @@ describe HammerCLIForeman::SmartProxy do
 
     context "parameters" do
       it_should_accept "no arguments"
-      it_should_accept "type, page, per_page", ["--type=tftp", "--page=1", "--per-page=2"]
+      it_should_accept_search_params
     end
 
     context "output" do
@@ -98,4 +99,16 @@ describe HammerCLIForeman::SmartProxy do
 
   end
 
+  context "RefreshFeaturesCommand" do
+
+    let(:cmd) { HammerCLIForeman::SmartProxy::RefreshFeaturesCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name", ["--name=proxy"]
+      # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
+    end
+
+  end
+
 end

data/test/unit/subnet_test.rb

@@ -1,6 +1,7 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
 
+require 'hammer_cli_foreman/subnet'
 
 describe HammerCLIForeman::Subnet do
 

data/test/unit/template_test.rb

@@ -1,6 +1,7 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
 
+require 'hammer_cli_foreman/template'
 
 describe HammerCLIForeman::Template do
 

data/test/unit/test_helper.rb

@@ -19,7 +19,7 @@ require 'hammer_cli'
 require 'hammer_cli_foreman/commands'
 
 HammerCLIForeman.stubs(:resource_config).returns({
-  :apidoc_cache_dir => 'test/unit/data/' + (ENV['TEST_API_VERSION'] || '1.5'),
+  :apidoc_cache_dir => 'test/unit/data/' + (ENV['TEST_API_VERSION'] || '1.6'),
   :apidoc_cache_name => 'foreman_api',
   :dry_run => true})
 

data/test/unit/user_test.rb

@@ -1,6 +1,7 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
 
+require 'hammer_cli_foreman/user'
 
 describe HammerCLIForeman::User do
 

data/test/unit/usergroup_test.rb

@@ -0,0 +1,80 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+require 'hammer_cli_foreman/usergroup'
+
+describe HammerCLIForeman::Usergroup do
+
+  include CommandTestHelper
+
+
+  context "ListCommand" do
+
+    let(:cmd) { HammerCLIForeman::Usergroup::ListCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "no arguments"
+      it_should_accept_search_params
+    end
+
+    context "output" do
+      it_should_print_column "Id"
+      it_should_print_column "Name"
+    end
+
+  end
+
+  context "InfoCommand" do
+
+    let(:cmd) { HammerCLIForeman::Usergroup::InfoCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name", ["--name=ug"]
+    end
+
+    context "output" do
+      with_params ["--id=1"] do
+        it_should_print_column "Id"
+        it_should_print_column "Name"
+        it_should_print_column "Users"
+        it_should_print_column "User groups"
+        it_should_print_column "Roles"
+        it_should_print_column "Created at"
+        it_should_print_column "Updated at"
+      end
+    end
+  end
+
+  context "CreateCommand" do
+
+    let(:cmd) { HammerCLIForeman::Usergroup::CreateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=ug"]
+      it_should_accept "name, role ids, user group ids and user ids", ["--name=ug", "--role-ids=1,2,3", "--user-group-ids=1,2,3", "--user-ids=1,2,3"]
+    end
+  end
+
+  context "DeleteCommand" do
+
+    let(:cmd) { HammerCLIForeman::Usergroup::DeleteCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=ug"]
+      it_should_accept "id", ["--id=1"]
+    end
+  end
+
+  context "UpdateCommand" do
+
+    let(:cmd) { HammerCLIForeman::Usergroup::UpdateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=ug"]
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name and new name", ["--name=ug", "--new-name=ug2"]
+      it_should_accept "id, new name, role ids, user group ids and user ids", ["--id=1", "--new-name=ug", "--role-ids=1,2,3", "--user-group-ids=1,2,3", "--user-ids=1,2,3"]
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_foreman
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Tomáš Strachota
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-20 00:00:00.000000000 Z
+date: 2014-08-13 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.1.1
+        version: 0.1.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: 0.1.1
+        version: 0.1.2
 - !ruby/object:Gem::Dependency
   name: apipie-bindings
   requirement: !ruby/object:Gem::Requirement
@@ -46,6 +46,9 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: 1.6.5
+    - - <
+      - !ruby/object:Gem::Version
+        version: '1.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -53,6 +56,9 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: 1.6.5
+    - - <
+      - !ruby/object:Gem::Version
+        version: '1.7'
 description: |
   Foreman commands for Hammer CLI
 email: tstracho@redhat.com
@@ -60,7 +66,11 @@ executables: []
 extensions: []
 extra_rdoc_files:
 - doc/configuration.md
+- doc/developer_docs.md
+- doc/option_builder.md
 - doc/host_create.md
+- doc/name_id_resolution.md
+- doc/release_notes.md
 - README.md
 files:
 - lib/hammer_cli_foreman/report.rb
@@ -68,12 +78,14 @@ files:
 - lib/hammer_cli_foreman/common_parameter.rb
 - lib/hammer_cli_foreman/subnet.rb
 - lib/hammer_cli_foreman/model.rb
+- lib/hammer_cli_foreman/role.rb
 - lib/hammer_cli_foreman/fact.rb
 - lib/hammer_cli_foreman/media.rb
 - lib/hammer_cli_foreman/exception_handler.rb
 - lib/hammer_cli_foreman/compute_resource.rb
 - lib/hammer_cli_foreman/i18n.rb
 - lib/hammer_cli_foreman/associating_commands.rb
+- lib/hammer_cli_foreman/smart_variables.rb
 - lib/hammer_cli_foreman/hostgroup.rb
 - lib/hammer_cli_foreman/template.rb
 - lib/hammer_cli_foreman/version.rb
@@ -84,6 +96,7 @@ files:
 - lib/hammer_cli_foreman/references.rb
 - lib/hammer_cli_foreman/resource_supported_test.rb
 - lib/hammer_cli_foreman/operating_system.rb
+- lib/hammer_cli_foreman/dependency_resolver.rb
 - lib/hammer_cli_foreman/smart_proxy.rb
 - lib/hammer_cli_foreman/environment.rb
 - lib/hammer_cli_foreman/puppet_class.rb
@@ -92,18 +105,25 @@ files:
 - lib/hammer_cli_foreman/domain.rb
 - lib/hammer_cli_foreman/smart_class_parameter.rb
 - lib/hammer_cli_foreman/auth.rb
-- lib/hammer_cli_foreman/searchables_option_builder.rb
+- lib/hammer_cli_foreman/usergroup.rb
 - lib/hammer_cli_foreman/image.rb
 - lib/hammer_cli_foreman/output.rb
 - lib/hammer_cli_foreman/partition_table.rb
 - lib/hammer_cli_foreman/credentials.rb
 - lib/hammer_cli_foreman/id_resolver.rb
 - lib/hammer_cli_foreman/user.rb
+- lib/hammer_cli_foreman/filter.rb
+- lib/hammer_cli_foreman/option_builders.rb
 - lib/hammer_cli_foreman/location.rb
+- lib/hammer_cli_foreman/param_filters.rb
 - lib/hammer_cli_foreman/architecture.rb
 - lib/hammer_cli_foreman.rb
 - doc/configuration.md
+- doc/developer_docs.md
+- doc/option_builder.md
 - doc/host_create.md
+- doc/name_id_resolution.md
+- doc/release_notes.md
 - test/reports/TEST-HammerCLIForeman-Domain-CreateCommand-parameters.xml
 - test/reports/TEST-HammerCLIForeman-Organization-UpdateCommand.xml
 - test/reports/TEST-HammerCLIForeman-Template-DeleteCommand.xml
@@ -346,18 +366,20 @@ files:
 - test/reports/TEST-HammerCLIForeman-PartitionTable-ListCommand-output.xml
 - test/reports/TEST-HammerCLIForeman-Hostgroup-UpdateCommand-parameters.xml
 - test/unit/image_test.rb
+- test/unit/filter_test.rb
 - test/unit/domain_test.rb
 - test/unit/exception_handler_test.rb
 - test/unit/media_test.rb
+- test/unit/api_test.rb
 - test/unit/host_test.rb
 - test/unit/organization_test.rb
 - test/unit/smart_proxy_test.rb
 - test/unit/subnet_test.rb
 - test/unit/location_test.rb
+- test/unit/dependency_resolver_test.rb
 - test/unit/helpers/command.rb
 - test/unit/helpers/resource_disabled.rb
 - test/unit/helpers/fake_searchables.rb
-- test/unit/searchables_option_builder_test.rb
 - test/unit/credentials_test.rb
 - test/unit/operating_system_test.rb
 - test/unit/template_test.rb
@@ -366,24 +388,38 @@ files:
 - test/unit/partition_table_test.rb
 - test/unit/hostgroup_test.rb
 - test/unit/smart_class_parameter_test.rb
+- test/unit/param_filters_test.rb
 - test/unit/report_test.rb
+- test/unit/role_test.rb
 - test/unit/output/formatters_test.rb
 - test/unit/id_resolver_test.rb
 - test/unit/test_output_adapter.rb
+- test/unit/option_builders_test.rb
 - test/unit/fact_test.rb
 - test/unit/model_test.rb
 - test/unit/compute_resource_test.rb
 - test/unit/puppet_class_test.rb
+- test/unit/usergroup_test.rb
 - test/unit/apipie_resource_mock.rb
 - test/unit/common_parameter_test.rb
 - test/unit/test_helper.rb
 - test/unit/architecture_test.rb
 - test/unit/user_test.rb
+- test/unit/data/1.6/foreman_api.json
 - test/unit/data/1.5/foreman_api.json
 - test/unit/data/1.4/foreman_api.json
+- locale/README.md
+- locale/en_GB/hammer-cli-foreman.po
+- locale/en_GB/LC_MESSAGES/hammer-cli-foreman.mo
+- locale/es/hammer-cli-foreman.po
+- locale/es/LC_MESSAGES/hammer-cli-foreman.mo
+- locale/en/hammer-cli-foreman.po
+- locale/en/LC_MESSAGES/hammer-cli-foreman.mo
 - locale/hammer-cli-foreman.pot
 - locale/Makefile
 - locale/zanata.xml
+- locale/fr/hammer-cli-foreman.po
+- locale/fr/LC_MESSAGES/hammer-cli-foreman.mo
 - README.md
 homepage: http://github.com/theforeman/hammer-cli-foreman
 licenses:
@@ -652,18 +688,20 @@ test_files:
 - test/reports/TEST-HammerCLIForeman-PartitionTable-ListCommand-output.xml
 - test/reports/TEST-HammerCLIForeman-Hostgroup-UpdateCommand-parameters.xml
 - test/unit/image_test.rb
+- test/unit/filter_test.rb
 - test/unit/domain_test.rb
 - test/unit/exception_handler_test.rb
 - test/unit/media_test.rb
+- test/unit/api_test.rb
 - test/unit/host_test.rb
 - test/unit/organization_test.rb
 - test/unit/smart_proxy_test.rb
 - test/unit/subnet_test.rb
 - test/unit/location_test.rb
+- test/unit/dependency_resolver_test.rb
 - test/unit/helpers/command.rb
 - test/unit/helpers/resource_disabled.rb
 - test/unit/helpers/fake_searchables.rb
-- test/unit/searchables_option_builder_test.rb
 - test/unit/credentials_test.rb
 - test/unit/operating_system_test.rb
 - test/unit/template_test.rb
@@ -672,19 +710,24 @@ test_files:
 - test/unit/partition_table_test.rb
 - test/unit/hostgroup_test.rb
 - test/unit/smart_class_parameter_test.rb
+- test/unit/param_filters_test.rb
 - test/unit/report_test.rb
+- test/unit/role_test.rb
 - test/unit/output/formatters_test.rb
 - test/unit/id_resolver_test.rb
 - test/unit/test_output_adapter.rb
+- test/unit/option_builders_test.rb
 - test/unit/fact_test.rb
 - test/unit/model_test.rb
 - test/unit/compute_resource_test.rb
 - test/unit/puppet_class_test.rb
+- test/unit/usergroup_test.rb
 - test/unit/apipie_resource_mock.rb
 - test/unit/common_parameter_test.rb
 - test/unit/test_helper.rb
 - test/unit/architecture_test.rb
 - test/unit/user_test.rb
+- test/unit/data/1.6/foreman_api.json
 - test/unit/data/1.5/foreman_api.json
 - test/unit/data/1.4/foreman_api.json
 has_rdoc: