hammer_cli_foreman 2.3.1 → 2.4.0

data/test/unit/compute_profile_test.rb

@@ -0,0 +1,87 @@
+# frozen_string_literal: true
+
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+require 'hammer_cli_foreman/compute_profile'
+
+describe HammerCLIForeman::ComputeProfile do
+  include CommandTestHelper
+
+  context 'ListCommand' do
+    before :each do
+      ResourceMocks.compute_profiles
+    end
+
+    let(:cmd) { HammerCLIForeman::ComputeProfile::ListCommand.new('', ctx) }
+
+    context 'parameters' do
+      it_should_accept 'no arguments'
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
+    end
+
+    context 'output' do
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
+
+      it_should_print_n_records
+      it_should_print_column 'Id'
+      it_should_print_column 'Name'
+    end
+  end
+
+  context 'InfoCommand' do
+    let(:cmd) { HammerCLIForeman::ComputeProfile::InfoCommand.new('', ctx) }
+
+    context 'parameters' do
+      it_should_accept 'id', ['--id=1']
+      it_should_accept 'name', ['--name=test']
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
+    end
+
+    context 'output' do
+      with_params ['--id=1'] do
+        it_should_print_n_records 1
+        it_should_print_column 'Id'
+        it_should_print_column 'Name'
+        it_should_print_column 'Created at'
+        it_should_print_column 'Updated at'
+        it_should_print_column 'Compute attributes'
+      end
+    end
+  end
+
+  context 'CreateCommand' do
+    let(:cmd) { HammerCLIForeman::ComputeProfile::CreateCommand.new('', ctx) }
+
+    context 'parameters' do
+      it_should_accept 'name', ['--name=test']
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
+    end
+
+  end
+
+  context 'DeleteCommand' do
+    let(:cmd) { HammerCLIForeman::ComputeProfile::DeleteCommand.new('', ctx) }
+
+    context 'parameters' do
+      it_should_accept 'id', ['--id=1']
+      it_should_accept 'name', ['--name=test']
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
+    end
+  end
+
+  context 'UpdateCommand' do
+    let(:cmd) { HammerCLIForeman::ComputeProfile::UpdateCommand.new('', ctx) }
+
+    context 'parameters' do
+      it_should_accept 'id', ['--id=1']
+      it_should_accept 'name', ['--name=test']
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
+    end
+  end
+end

data/test/unit/config_group_test.rb

@@ -16,6 +16,8 @@ describe HammerCLIForeman::ConfigGroup do
     context "parameters" do
       it_should_accept "no arguments"
       it_should_accept_search_params
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
 
     context "output" do
@@ -31,6 +33,8 @@ describe HammerCLIForeman::ConfigGroup do
     context "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=group_x"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
 
     context "output" do
@@ -48,6 +52,8 @@ describe HammerCLIForeman::ConfigGroup do
 
     context "parameters" do
       it_should_accept "name, puppetclass ids", ["--name=first_group", "--puppet-class-ids=1,2"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
   end
 
@@ -57,6 +63,8 @@ describe HammerCLIForeman::ConfigGroup do
     context "parameters" do
       it_should_accept "name", ["--name=group_x"]
       it_should_accept "id", ["--id=1"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
   end
 
@@ -67,6 +75,8 @@ describe HammerCLIForeman::ConfigGroup do
     context "parameters" do
       it_should_accept "name", ["--name=group_x"]
       it_should_accept "id", ["--id=1"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
   end
 end

data/test/unit/mail_notification_test.rb

@@ -0,0 +1,53 @@
+# frozen_string_literal: true
+
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+require 'hammer_cli_foreman/mail_notification'
+
+describe HammerCLIForeman::MailNotification do
+  include CommandTestHelper
+
+  context 'ListCommand' do
+    before :each do
+      ResourceMocks.mail_notifications
+    end
+
+    let(:cmd) { HammerCLIForeman::MailNotification::ListCommand.new('', ctx) }
+
+    context 'parameters' do
+      it_should_accept 'no arguments'
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
+    end
+
+    context 'output' do
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
+
+      it_should_print_n_records
+      it_should_print_column 'Id'
+      it_should_print_column 'Name'
+    end
+  end
+
+  context 'InfoCommand' do
+    let(:cmd) { HammerCLIForeman::MailNotification::InfoCommand.new('', ctx) }
+
+    context 'parameters' do
+      it_should_accept 'id', ['--id=1']
+      it_should_accept 'name', ['--name=test']
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
+    end
+
+    context 'output' do
+      with_params ['--id=1'] do
+        it_should_print_n_records 1
+        it_should_print_column 'Id'
+        it_should_print_column 'Name'
+        it_should_print_column 'Description'
+        it_should_print_column 'Subscription type'
+      end
+    end
+  end
+end

data/test/unit/model_test.rb

@@ -17,6 +17,8 @@ describe HammerCLIForeman::Model do
     context "parameters" do
       it_should_accept "no arguments"
       it_should_accept_search_params
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
 
     context "output" do
@@ -38,6 +40,8 @@ describe HammerCLIForeman::Model do
     context "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=model"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "no arguments" # TODO: temporarily disabled, parameters are checked in the id resolver
     end
 
@@ -63,6 +67,8 @@ describe HammerCLIForeman::Model do
 
     context "parameters" do
       it_should_accept "name", ["--name=model", "--info=description", "--vendor-class=class", "--hardware-model=model"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "name missing", ["--info=description", "--vendor-class=class", "--hardware-model=model"]
       # TODO: temporarily disabled, parameters are checked in the api
     end
@@ -77,6 +83,8 @@ describe HammerCLIForeman::Model do
     context "parameters" do
       it_should_accept "name", ["--name=model"]
       it_should_accept "id", ["--id=1"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
     end
 
@@ -90,6 +98,8 @@ describe HammerCLIForeman::Model do
     context "parameters" do
       it_should_accept "name", ["--name=model", "--new-name=model2", "--info=description", "--vendor-class=class", "--hardware-model=model"]
       it_should_accept "id", ["--id=1", "--new-name=model2", "--info=description", "--vendor-class=class", "--hardware-model=model"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "no params", []
       # it_should_fail_with "name or id missing", ["--new-name=model2", "--info=description", "--vendor-class=class", "--hardware-model=model"]
       # TODO: temporarily disabled, parameters are checked in the id resolver

data/test/unit/operating_system_test.rb

@@ -18,6 +18,8 @@ describe HammerCLIForeman::OperatingSystem do
     context "parameters" do
       it_should_accept "no arguments"
       it_should_accept_search_params
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
 
     context "output" do
@@ -40,6 +42,8 @@ describe HammerCLIForeman::OperatingSystem do
     context "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "title", ["--title=Rhel 6.5"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "no arguments" # TODO: temporarily disabled, parameters are checked in the id resolver
     end
 
@@ -70,6 +74,8 @@ describe HammerCLIForeman::OperatingSystem do
 
     context "parameters" do
       it_should_accept "name, major, minor, family, release name", ["--name=media", "--major=1", "--minor=2", "--family=Red Hat", "--release-name=awesome"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "name missing", ["--major=1", "--minor=2", "--family=Red Hat", "--release-name=awesome"]
       # TODO: temporarily disabled, parameters are checked in the api
     end
@@ -87,6 +93,8 @@ describe HammerCLIForeman::OperatingSystem do
     context "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "title", ["--title=Rhel 6.5"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
       # TODO: temporarily disabled, parameters are checked in the id resolver
     end
@@ -102,6 +110,8 @@ describe HammerCLIForeman::OperatingSystem do
       it_should_accept "id", ["--id=1"]
       it_should_accept "title", ["--title=Rhel 6.5"]
       it_should_accept "name, major, minor, family, release name", ["--id=83", "--name=os", "--major=1", "--minor=2", "--family=Red Hat", "--release-name=awesome"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "no params", []
       # it_should_fail_with "label or id missing", ["--name=os", "--major=1", "--minor=2", "--family=Red Hat", "--release-name=awesome"]
       # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -126,6 +136,8 @@ describe HammerCLIForeman::OperatingSystem do
       it_should_accept "name, value and os id", ["--name=domain", "--value=val", "--operatingsystem-id=1"]
       it_should_accept "name, value and os title", ["--name=domain", "--value=val", "--operatingsystem=Rhel 6.5"]
       it_should_accept "name, value, type and os id", ["--name=domain", "--value=val", "--parameter-type=string", "--operatingsystem-id=1"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "name missing", ["--value=val", "--operatingsystem-id=id"]
       # it_should_fail_with "value missing", ["--name=name", "--operatingsystem-id=id"]
       # it_should_fail_with "os id missing", ["--name=name", "--value=val"]
@@ -142,11 +154,12 @@ describe HammerCLIForeman::OperatingSystem do
     context "parameters" do
       it_should_accept "name and os id", ["--name=domain", "--operatingsystem-id=1"]
       it_should_accept "name and os title", ["--name=domain", "--operatingsystem=Rhel 6.5"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
       # it_should_fail_with "name missing", ["--operatingsystem-id=id"]
       # it_should_fail_with "os id missing", ["--name=name"]
       # TODO: temporarily disabled, parameters are checked in the id resolver
     end
 
   end
-
 end

data/test/unit/settings_test.rb

@@ -18,6 +18,8 @@ describe HammerCLIForeman::Settings do
     context "parameters" do
       it_should_accept "no arguments"
       it_should_accept_search_params
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
 
     context "output" do
@@ -38,6 +40,8 @@ describe HammerCLIForeman::Settings do
     context "parameters" do
       it_should_accept "name", ["--name=setting1", "--value=setting2"]
       it_should_accept "id", ["--id=1", "--value=setting2"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
 
   end

data/test/unit/usergroup_test.rb

@@ -18,6 +18,8 @@ describe HammerCLIForeman::Usergroup do
     context "parameters" do
       it_should_accept "no arguments"
       it_should_accept_search_params
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
 
     context "output" do
@@ -35,6 +37,8 @@ describe HammerCLIForeman::Usergroup do
     context "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=ug"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
 
     context "output" do
@@ -58,6 +62,8 @@ describe HammerCLIForeman::Usergroup do
     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"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
   end
 
@@ -68,6 +74,8 @@ describe HammerCLIForeman::Usergroup do
     context "parameters" do
       it_should_accept "name", ["--name=ug"]
       it_should_accept "id", ["--id=1"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
   end
 
@@ -80,6 +88,8 @@ describe HammerCLIForeman::Usergroup do
       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"]
+      it_should_fail_with 'organization param', ['--organization-id=1']
+      it_should_fail_with 'location param', ['--location-id=1']
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_foreman
 version: !ruby/object:Gem::Version
-  version: 2.3.1
+  version: 2.4.0
 platform: ruby
 authors:
 - Tomáš Strachota
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-27 00:00:00.000000000 Z
+date: 2021-02-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hammer_cli
@@ -17,14 +17,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.3.0
+        version: 2.4.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.3.0
+        version: 2.4.0
 - !ruby/object:Gem::Dependency
   name: apipie-bindings
   requirement: !ruby/object:Gem::Requirement
@@ -246,6 +246,7 @@ files:
 - test/data/2.0/foreman_api.json
 - test/data/2.1/foreman_api.json
 - test/data/README.md
+- test/functional/architecture_test.rb
 - test/functional/associating_commands_test.rb
 - test/functional/audit_test.rb
 - test/functional/auth_source_test.rb
@@ -254,6 +255,7 @@ files:
 - test/functional/compute_attribute_test.rb
 - test/functional/compute_profile_test.rb
 - test/functional/compute_resource_test.rb
+- test/functional/config_group_test.rb
 - test/functional/filter_test.rb
 - test/functional/host_test.rb
 - test/functional/hostgroup/create_test.rb
@@ -262,6 +264,8 @@ files:
 - test/functional/location_test.rb
 - test/functional/mail_notification_test.rb
 - test/functional/media_test.rb
+- test/functional/model_test.rb
+- test/functional/operating_system_test.rb
 - test/functional/organization_test.rb
 - test/functional/personal_access_token_test.rb
 - test/functional/ping_test.rb
@@ -279,6 +283,7 @@ files:
 - test/functional/test_helper.rb
 - test/functional/user_mail_notification_test.rb
 - test/functional/user_test.rb
+- test/functional/usergroup_test.rb
 - test/functional/virtual_machine_test.rb
 - test/reports/TEST-Minitest-Result.xml
 - test/test_helper.rb
@@ -293,8 +298,10 @@ files:
 - test/unit/audit_test.rb
 - test/unit/auth_source_external.rb
 - test/unit/auth_source_ldap_test.rb
+- test/unit/bookmark_test.rb
 - test/unit/commands_test.rb
 - test/unit/common_parameter_test.rb
+- test/unit/compute_profile_test.rb
 - test/unit/compute_resource_test.rb
 - test/unit/config_group_test.rb
 - test/unit/config_report_test.rb
@@ -314,6 +321,7 @@ files:
 - test/unit/id_resolver_test.rb
 - test/unit/image_test.rb
 - test/unit/location_test.rb
+- test/unit/mail_notification_test.rb
 - test/unit/media_test.rb
 - test/unit/messages_test.rb
 - test/unit/model_test.rb
@@ -385,6 +393,7 @@ test_files:
 - test/unit/auth_source_ldap_test.rb
 - test/unit/audit_test.rb
 - test/unit/test_helper.rb
+- test/unit/mail_notification_test.rb
 - test/unit/smart_class_parameter_test.rb
 - test/unit/partition_table_test.rb
 - test/unit/operating_system_test.rb
@@ -413,6 +422,7 @@ test_files:
 - test/unit/filter_test.rb
 - test/unit/puppet_environment_test.rb
 - test/unit/output/formatters_test.rb
+- test/unit/compute_profile_test.rb
 - test/unit/host_test.rb
 - test/unit/config_report_test.rb
 - test/unit/fact_test.rb
@@ -420,6 +430,7 @@ test_files:
 - test/unit/commands_test.rb
 - test/unit/common_parameter_test.rb
 - test/unit/subnet_test.rb
+- test/unit/bookmark_test.rb
 - test/data/2.1/foreman_api.json
 - test/data/2.0/foreman_api.json
 - test/data/1.24/foreman_api.json
@@ -438,6 +449,7 @@ test_files:
 - test/functional/http_proxy_test.rb
 - test/functional/commands/list_test.rb
 - test/functional/user_mail_notification_test.rb
+- test/functional/config_group_test.rb
 - test/functional/user_test.rb
 - test/functional/realm_test.rb
 - test/functional/personal_access_token_test.rb
@@ -446,7 +458,10 @@ test_files:
 - test/functional/mail_notification_test.rb
 - test/functional/ping_test.rb
 - test/functional/smart_class_parameter_test.rb
+- test/functional/operating_system_test.rb
 - test/functional/media_test.rb
+- test/functional/architecture_test.rb
+- test/functional/usergroup_test.rb
 - test/functional/role_test.rb
 - test/functional/location_test.rb
 - test/functional/hostgroup/create_test.rb
@@ -457,6 +472,7 @@ test_files:
 - test/functional/settings_test.rb
 - test/functional/template_test.rb
 - test/functional/proxy_test.rb
+- test/functional/model_test.rb
 - test/functional/compute_resource_test.rb
 - test/functional/compute_attribute_test.rb
 - test/functional/filter_test.rb