hammer_cli_foreman 2.2.0 → 2.3.0

data/test/functional/settings_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '..', 'test_helper')
+require File.join(File.dirname(__FILE__), 'test_helper')
 
 describe 'Settings' do
   let(:setting) do
@@ -33,5 +33,61 @@ describe 'Settings' do
 
   end
 
+  describe 'info' do
+    before do
+      @cmd = ['setting', 'info']
+      @setting = {
+        description: 'Fix DB cache on next Foreman restart',
+        category: 'Setting::General',
+        settings_type: 'boolean',
+        default: false,
+        created_at: "2020-05-24 09:02:11 UTC",
+        updated_at: "2020-07-01 07:42:14 UTC",
+        id: 1,
+        name: "fix_db_cache",
+        full_name: "Fix DB cache",
+        value: false,
+        category_name: "General"
+      }
+    end
+
+    it 'should return a setting' do
+      params = ['--id=1']
+      api_expects(:settings, :show, 'Show a setting').returns(@setting)
+
+      output = OutputMatcher.new([
+                                   'Id:            1',
+                                   'Name:          fix_db_cache',
+                                   'Description:   Fix DB cache on next Foreman restart',
+                                   'Category:      General',
+                                   'Settings type: boolean',
+                                   'Value:         false'
+                                 ])
+
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'update' do
+    before do
+      @cmd = ['setting', 'set']
+    end
+
+    it 'should update a setting' do
+      params = ['--id=1', '--value=true']
+
+      api_expects(:settings, :update, 'Update a setting') do |params|
+        (params['setting']['id'] == '1')
+        (params['setting']['value'] == 'true')
+      end
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(success_result("Setting [%{name}] updated to [%{value}].\n"), result)
+    end
+  end
+
 end
 

data/test/functional/user_test.rb

@@ -162,4 +162,43 @@ describe "user" do
       end
     end
   end
+
+  describe 'list' do
+    let(:cmd) { ['user', 'list'] }
+    let(:users) do
+      [{
+           id: 1,
+           login: 'admin',
+           full_name: 'Admin User',
+           mail: 'root@localadmin.net',
+           admin: true,
+           disabled: false
+       }] end
+
+    it "should show index list" do
+      api_expects(:users, :index).returns(users)
+
+      output = IndexMatcher.new([
+                                    ['ID', 'LOGIN', 'NAME', 'EMAIL', 'ADMIN', 'DISABLED', 'AUTHORIZED BY'],
+                                    ['1',  'admin', 'Admin User', 'root@localadmin.net', 'yes', 'no', ''],
+                                ])
+
+      result = run_cmd(cmd)
+      assert_cmd(success_result(output), result)
+    end
+  end
+
+  describe 'delete' do
+    let(:cmd) { ['user', 'delete']  }
+
+    it 'deletes a user' do
+      params = ['--id', user['id']]
+      api_expects(:users, :destroy).returns(user)
+
+      expected_result = success_result("User [#{user['login']}] deleted.\n")
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
 end

data/test/unit/media_test.rb

@@ -95,7 +95,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','2']}}
+      it_should_call_action :update, {'id' => '1', 'medium' => {'name' => 'medium_x', 'path' => 'http://some.path/', 'operatingsystem_ids' => ['1','2']}}
     end
 
   end

data/test/unit/partition_table_test.rb

@@ -119,8 +119,8 @@ describe HammerCLIForeman::PartitionTable do
       # TODO: temporarily disabled, parameters are checked in the id resolver
     end
 
-    with_params ["--id=83", "--new-name=ptable","--file=~/table.sh", "--os-family=RedHat"] do
-      it_should_call_action :update, {'id' => '83', 'name' => 'ptable', 'ptable' => {'name' => 'ptable', 'layout' => 'FILE_CONTENT', 'os_family' => 'RedHat'}}
+    with_params ['--id=83', '--new-name=ptable', '--file=~/table.sh', '--os-family=RedHat'] do
+      it_should_call_action :update, { 'id' => '83', 'ptable' => { 'name' => 'ptable', 'layout' => 'FILE_CONTENT', 'os_family' => 'RedHat' } }
     end
 
   end

data/test/unit/role_test.rb

@@ -77,8 +77,8 @@ describe HammerCLIForeman::Role do
       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'}}
+    with_params ['--id=1', '--new-name=role2'] do
+      it_should_call_action :update, { 'id' => '1', 'role' => { 'name' => 'role2' } }
     end
 
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_foreman
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 2.3.0
 platform: ruby
 authors:
 - Tomáš Strachota
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-08-11 00:00:00.000000000 Z
+date: 2020-11-03 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.1.0
+        version: 2.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.1.0
+        version: 2.3.0
 - !ruby/object:Gem::Dependency
   name: apipie-bindings
   requirement: !ruby/object:Gem::Requirement
@@ -211,7 +211,6 @@ files:
 - lib/hammer_cli_foreman/task_helper.rb
 - lib/hammer_cli_foreman/template.rb
 - lib/hammer_cli_foreman/testing/api_expectations.rb
-- lib/hammer_cli_foreman/trend.rb
 - lib/hammer_cli_foreman/user.rb
 - lib/hammer_cli_foreman/user_mail_notification.rb
 - lib/hammer_cli_foreman/usergroup.rb
@@ -262,6 +261,7 @@ files:
 - test/functional/http_proxy_test.rb
 - test/functional/location_test.rb
 - test/functional/mail_notification_test.rb
+- test/functional/media_test.rb
 - test/functional/organization_test.rb
 - test/functional/personal_access_token_test.rb
 - test/functional/ping_test.rb
@@ -277,7 +277,6 @@ files:
 - test/functional/subnet/update_test.rb
 - test/functional/template_test.rb
 - test/functional/test_helper.rb
-- test/functional/trend_test.rb
 - test/functional/user_mail_notification_test.rb
 - test/functional/user_test.rb
 - test/functional/virtual_machine_test.rb
@@ -447,6 +446,7 @@ test_files:
 - test/functional/mail_notification_test.rb
 - test/functional/ping_test.rb
 - test/functional/smart_class_parameter_test.rb
+- test/functional/media_test.rb
 - test/functional/role_test.rb
 - test/functional/location_test.rb
 - test/functional/hostgroup/create_test.rb
@@ -460,7 +460,6 @@ test_files:
 - test/functional/compute_resource_test.rb
 - test/functional/compute_attribute_test.rb
 - test/functional/filter_test.rb
-- test/functional/trend_test.rb
 - test/functional/report_template_test.rb
 - test/functional/compute_profile_test.rb
 - test/functional/host_test.rb

data/lib/hammer_cli_foreman/trend.rb

@@ -1,47 +0,0 @@
-module HammerCLIForeman
-
-  class Trend < HammerCLIForeman::Command
-
-    resource :trends
-
-    class ListCommand < HammerCLIForeman::ListCommand
-      output do
-        field :id, _("Id")
-        field :name, _("Name")
-        field :label, _("Label")
-      end
-
-      build_options
-    end
-
-    class CreateCommand < HammerCLIForeman::CreateCommand
-      success_message _("Trend for %<trendable_type>s created.")
-      failure_message _("Could not create the trend")
-
-      build_options
-    end
-
-    class DeleteCommand < HammerCLIForeman::DeleteCommand
-      success_message _("Trend for %<trendable_type>s deleted.")
-      failure_message _("Could not delete the trend")
-
-      build_options
-    end
-
-    class InfoCommand < HammerCLIForeman::InfoCommand
-      output do
-        field :id, _('Id')
-        field :name, _('Name')
-        field :label, _("Label")
-        field :trendable_type, _('Type')
-        HammerCLIForeman::References.taxonomies(self)
-        HammerCLIForeman::References.timestamps(self)
-      end
-
-      build_options
-    end
-
-    autoload_subcommands
-  end
-end
-

data/test/functional/trend_test.rb

@@ -1,83 +0,0 @@
-require File.join(File.dirname(__FILE__), 'test_helper')
-
-describe 'trend' do
-  describe 'list' do
-    before do
-      @cmd = %w[trend list]
-      @trends =
-        [{
-           id: 1,
-           label: "Operatingsystem",
-           trendable_type: "Operatingsystem",
-           name: "os_trend" }]
-    end
-
-    it 'should return a list of trends' do
-      api_expects(:trends, :index, 'List trends').returns(@trends)
-
-      result = run_cmd(@cmd)
-      _(result.exit_code).must_equal HammerCLI::EX_OK
-    end
-  end
-
-  describe 'info' do
-    before do
-      @cmd = %w[trend info]
-      @trend = {
-        id: 1,
-        label: "Operatingsystem",
-        trendable_type: "Operatingsystem",
-        name: "os_trend"
-      }
-    end
-
-    it 'should return a trend' do
-      params = ['--id=1']
-      api_expects(:trends, :show, 'Show trend').returns(@trend)
-
-      result = run_cmd(@cmd + params)
-      _(result.exit_code).must_equal HammerCLI::EX_OK
-    end
-  end
-
-  describe 'create' do
-    before do
-      @cmd = %w[trend create]
-    end
-
-    it 'should print error on missing --trendable-type' do
-      expected_result = "Could not create the trend:\n  Missing arguments for '--trendable-type'.\n"
-
-      api_expects_no_call
-      result = run_cmd(@cmd)
-      assert_match(expected_result, result.err)
-    end
-
-    it 'should create a trend' do
-      params = ['--trendable-type=ComputeResource']
-
-      api_expects(:trends, :create, 'Create a trend') do |params|
-        (params['trendable_type'] == 'ComputeResource')
-      end
-
-      result = run_cmd(@cmd + params)
-
-      assert_cmd(success_result("Trend for %<trendable_type>s created.\n"), result)
-    end
-  end
-
-  describe 'delete' do
-    before do
-      @cmd = %w[trend delete]
-    end
-
-    it 'should delete a trend' do
-      params = ['--id=1']
-
-      api_expects(:trends, :destroy, 'Delete trend').with_params(:id => '1')
-
-      result = run_cmd(@cmd + params)
-      assert_cmd(success_result("Trend for %<trendable_type>s deleted.\n"), result)
-    end
-  end
-end