hammer_cli_foreman 2.1.2 → 2.4.0

data/lib/minitest/hammer_coverage_plugin.rb

@@ -0,0 +1,19 @@
+require 'coverage_reporter'
+require "json"
+
+module Minitest
+  def self.plugin_hammer_coverage_options(opts, options)
+    opts.on "-c", "--coverage", "Generate coverage reports for API endpoints" do
+      options[:coverage] = true
+    end
+  end
+
+  def self.plugin_hammer_coverage_init(options)
+    if options[:coverage]
+      Minitest.reporter.reporters.clear
+      Minitest.after_run do
+          Minitest::CoverageRunner.new("test/data/#{ FOREMAN_VERSION }/foreman_api.json").run_tests
+      end
+    end
+  end
+end

data/test/functional/architecture_test.rb

@@ -0,0 +1,49 @@
+# frozen_string_literal: true
+
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'architecture' do
+  describe 'list' do
+    before do
+      @cmd = %w[architecture list]
+      @architectures = [{
+                      id: 1,
+                      name: 'i386',
+                    }]
+    end
+
+    it 'should return a list of architectures' do
+      api_expects(:architectures, :index, 'List architectures').returns(@architectures)
+
+      output = IndexMatcher.new([
+                                  %w[ID NAME],
+                                  %w[1 i386]
+                                ])
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'should run list command with defaults' do
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      defaults = HammerCLI::Defaults.new(
+        {
+          organization_id: {
+            provider: 'foreman'
+          },
+          location_id: {
+            provider: 'foreman'
+          }
+        }
+      )
+      defaults.stubs(:write_to_file).returns(true)
+      defaults.stubs(:providers).returns(providers)
+      api_expects(:architectures, :index, 'List architectures').returns(@architectures)
+
+      result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })
+      _(result.exit_code).must_equal HammerCLI::EX_OK
+    end
+  end
+end
+

data/test/functional/audit_test.rb

@@ -1,28 +1,28 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 
 describe 'audit' do
-  describe 'audit-list' do
+  before do
+    @audit_changed = {
+      'id' => 83,
+      'created_at' => "2017-10-31 11:25:44 UTC",
+      'remote_address' => '::1',
+      'user_name' => "foreman_api_admin",
+      'user_id' => 11,
+      'action' => "update",
+      'auditable_type' => "ProvisioningTemplate",
+      'auditable_name' => "default_location_subscribed_hosts",
+      'auditable_id' => 32,
+      'audited_changes' => {
+        'value' => [
+          nil,
+          "--- false\n..."
+        ]
+      }
+    }
+  end
+  describe 'audit-info' do
     let (:cmd) { ['audit', 'info'] }
     let(:params) { ['--id=83'] }
-    let(:audit_changed) do
-      {
-        'id' => 83,
-        'created_at' => "2017-10-31 11:25:44 UTC",
-        'remote_address' => '::1',
-        'user_name' => "foreman_api_admin",
-        'user_id' => 11,
-        'action' => "update",
-        'auditable_type' => "ProvisioningTemplate",
-        'auditable_name' => "default_location_subscribed_hosts",
-        'auditable_id' => 32,
-        'audited_changes' => {
-          'value' => [
-            nil,
-            "--- false\n..."
-          ]
-        }
-      }
-    end
     let(:audit_new) do
       {
         "user_id" => 1,
@@ -62,24 +62,26 @@ describe 'audit' do
     it 'shows audit information for changed records' do
       api_expects(:audits, :show, 'info')
         .with_params('id' => '83')
-        .returns(audit_changed)
+        .returns(@audit_changed)
 
-      output = OutputMatcher.new([
-        'Id:              83',
-        'At:              2017/10/31 11:25:44',
-        'IP:              ::1',
-        'User:            foreman_api_admin',
-        'Action:          update',
-        'Audit type:      ProvisioningTemplate',
-        'Audit record:    default_location_subscribed_hosts',
-        'Audited changes:',
-        ' 1) Attribute: value',
-        '    Old:',
-        '',
-        '    New:',
-        '      --- false',
-        '      ...',
-      ])
+      output = OutputMatcher.new(
+        [
+          'Id:              83',
+          'At:              2017/10/31 11:25:44',
+          'IP:              ::1',
+          'User:            foreman_api_admin',
+          'Action:          update',
+          'Audit type:      ProvisioningTemplate',
+          'Audit record:    default_location_subscribed_hosts',
+          'Audited changes:',
+          ' 1) Attribute: value',
+          '    Old:',
+          '',
+          '    New:',
+          '      --- false',
+          '      ...',
+        ]
+      )
       expected_result = success_result(output)
 
       result = run_cmd(cmd + params)
@@ -91,36 +93,57 @@ describe 'audit' do
         .with_params('id' => '83')
         .returns(audit_new)
 
-      output = OutputMatcher.new([
-        'Id:              63',
-        'At:              2017/10/09 23:42:44',
-        'IP:              ::1',
-        'User:            foreman_admin',
-        'Action:          create',
-        'Audit type:      User',
-        'Audit record:    John Doe',
-        'Audited changes:',
-        ' 1) Attribute: login',
-        '    Value:     john',
-        ' 2) Attribute: firstname',
-        '    Value:     John',
-        ' 3) Attribute: lastname',
-        '    Value:     Doe',
-        ' 4) Attribute: mail',
-        '    Value:     john@ipa.test',
-        ' 5) Attribute: admin',
-        '    Value:     false',
-        ' 6) Attribute: auth_source_id',
-        '    Value:     3',
-        ' 7) Attribute: lower_login',
-        '    Value:     john',
-        ' 8) Attribute: mail_enabled',
-        '    Value:     true',
-      ])
+      output = OutputMatcher.new(
+        [
+          'Id:              63',
+          'At:              2017/10/09 23:42:44',
+          'IP:              ::1',
+          'User:            foreman_admin',
+          'Action:          create',
+          'Audit type:      User',
+          'Audit record:    John Doe',
+          'Audited changes:',
+          ' 1) Attribute: login',
+          '    Value:     john',
+          ' 2) Attribute: firstname',
+          '    Value:     John',
+          ' 3) Attribute: lastname',
+          '    Value:     Doe',
+          ' 4) Attribute: mail',
+          '    Value:     john@ipa.test',
+          ' 5) Attribute: admin',
+          '    Value:     false',
+          ' 6) Attribute: auth_source_id',
+          '    Value:     3',
+          ' 7) Attribute: lower_login',
+          '    Value:     john',
+          ' 8) Attribute: mail_enabled',
+          '    Value:     true',
+        ]
+      )
       expected_result = success_result(output)
 
       result = run_cmd(cmd + params)
       assert_cmd(expected_result, result)
     end
   end
+
+  describe 'audit-list' do
+    let(:cmd) { ['audit', 'list'] }
+    let(:audits) do
+      [@audit_changed]
+    end
+
+    it "should show index list" do
+      api_expects(:audits, :index).returns(audits)
+
+      output = IndexMatcher.new([
+                                  ['ID', 'AT', 'IP', 'USER', 'ACTION', 'AUDIT TYPE', 'AUDIT RECORD'],
+                                  ['83', '2017/10/31 11:25:44', '::1', 'foreman_api_admin', 'update', 'ProvisioningTemplate', 'default_location_subscribed_hosts']
+                                ])
+
+      result = run_cmd(cmd)
+      assert_cmd(success_result(output), result)
+    end
+  end
 end

data/test/functional/bookmark_test.rb

@@ -27,6 +27,26 @@ describe 'bookmark' do
       result = run_cmd(@cmd)
       assert_cmd(expected_result, result)
     end
+
+    it 'should run list command with defaults' do
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      defaults = HammerCLI::Defaults.new(
+        {
+          organization_id: {
+            provider: 'foreman'
+          },
+          location_id: {
+            provider: 'foreman'
+          }
+        }
+      )
+      defaults.stubs(:write_to_file).returns(true)
+      defaults.stubs(:providers).returns(providers)
+      api_expects(:bookmarks, :index, 'List bookmarks').returns(@bookmarks)
+
+      result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })
+      _(result.exit_code).must_equal HammerCLI::EX_OK
+    end
   end
 
   describe 'info' do

data/test/functional/commands/list_test.rb

@@ -39,12 +39,12 @@ describe HammerCLIForeman::ListCommand do
 
     it 'shows search fields in help' do
       result = run_cmd(cmd + params)
-      result.out.must_match(/.*Search \/ Order fields:\s+(\S+\s+\S+)*\s+name\s+string/)
+      _(result.out).must_match(/.*Search \/ Order fields:\s+(\S+\s+\S+)*\s+name\s+string/)
     end
 
     it 'formats enum values' do
       result = run_cmd(cmd + params)
-      result.out.must_match(/.*Search \/ Order fields:\s+(\S+\s+\S+)*\s+managed\s+Values: true, false/)
+      _(result.out).must_match(/.*Search \/ Order fields:\s+(\S+\s+\S+)*\s+managed\s+Values: true, false/)
     end
   end
 
@@ -53,7 +53,7 @@ describe HammerCLIForeman::ListCommand do
       it "fetches only first page when there's not enough records" do
         expect_paged_call(1, 1000, 10)
         result = run_cmd([], {}, TestList)
-        result.exit_code.must_equal HammerCLI::EX_OK
+        _(result.exit_code).must_equal HammerCLI::EX_OK
       end
 
       it "fetches all records" do
@@ -62,27 +62,27 @@ describe HammerCLIForeman::ListCommand do
         expect_paged_call(3, per_page_all, 10)
 
         result = run_cmd([], {}, TestList)
-        result.exit_code.must_equal HammerCLI::EX_OK
+        _(result.exit_code).must_equal HammerCLI::EX_OK
       end
 
       it "uses --per-page value" do
         per_page = 10
         expect_paged_call(1, per_page, 10)
         result = run_cmd(["--per-page=#{per_page}"], {}, TestList)
-        result.exit_code.must_equal HammerCLI::EX_OK
+        _(result.exit_code).must_equal HammerCLI::EX_OK
       end
 
       it "uses both --per-page and --page value" do
         per_page = 10
         expect_paged_call(2, per_page, 10)
         result = run_cmd(["--per-page=#{per_page}", '--page=2'], {}, TestList)
-        result.exit_code.must_equal HammerCLI::EX_OK
+        _(result.exit_code).must_equal HammerCLI::EX_OK
       end
 
       it "sets per_page to 20 when only --page is used" do
         expect_paged_call(2, 20, 10)
         result = run_cmd(['--page=2'], {}, TestList)
-        result.exit_code.must_equal HammerCLI::EX_OK
+        _(result.exit_code).must_equal HammerCLI::EX_OK
       end
     end
 
@@ -96,20 +96,20 @@ describe HammerCLIForeman::ListCommand do
         per_page = 10
         expect_paged_call(1, per_page, 10)
         result = run_cmd(["--per-page=#{per_page}"], {}, TestList)
-        result.exit_code.must_equal HammerCLI::EX_OK
+        _(result.exit_code).must_equal HammerCLI::EX_OK
       end
 
       it "respects per_page setting when the adapter allows pagination by default" do
         expect_paged_call(1, per_page_in_settings, 30)
         result = run_cmd([], { :adapter => :base, :interactive => false }, TestList)
-        result.exit_code.must_equal HammerCLI::EX_OK
+        _(result.exit_code).must_equal HammerCLI::EX_OK
       end
 
       it "fetches all records when the adapter doesn't allow pagination by default" do
         expect_paged_call(1, per_page_all, 1000)
         expect_paged_call(2, per_page_all, 10)
         result = run_cmd([], { :adapter => :csv, :interactive => false }, TestList)
-        result.exit_code.must_equal HammerCLI::EX_OK
+        _(result.exit_code).must_equal HammerCLI::EX_OK
       end
     end
   end
@@ -128,7 +128,7 @@ describe HammerCLIForeman::ListCommand do
 
       result = run_cmd([], {}, TestListWithOutput)
       assert_cmd(expected_result, result)
-      result.out.wont_match pagination_line_re
+      _(result.out).wont_match pagination_line_re
     end
 
     it 'prints one page when --per-page is used' do

data/test/functional/compute_profile_test.rb

@@ -33,8 +33,10 @@ describe "parameters" do
           }
     end
     it 'update compute profile name' do
-      params = ['--id=1' ,'--new-name=profile2']
-      api_expects(:compute_profiles, :update, 'Update the compute profile').with_params({"name" =>"profile2"}).returns(@compute_profile)
+      params = ['--id=1', '--new-name=profile2']
+      api_expects(:compute_profiles, :update, 'Update the compute profile').with_params(
+        { 'compute_profile' => { 'name' => 'profile2' } }
+      ).returns(@compute_profile)
       result = run_cmd(@cmd + params)
       assert_cmd(success_result("Compute profile updated.\n"), result)
     end
@@ -96,4 +98,47 @@ describe "parameters" do
     end
   end
 
+  describe 'list' do
+    before do
+      @cmd = %w[compute-profile list]
+      @compute_profiles = [{
+                          id: 1,
+                          name: '1-Small',
+                        }]
+    end
+
+    it 'should return a list of compute profiles' do
+      api_expects(:compute_profiles, :index, 'List compute profiles').returns(@compute_profiles)
+
+      output = IndexMatcher.new([
+                                  %w[ID NAME],
+                                  %w[1 1-Small]
+                                ])
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'should run list command with defaults' do
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      defaults = HammerCLI::Defaults.new(
+        {
+          organization_id: {
+            provider: 'foreman'
+          },
+          location_id: {
+            provider: 'foreman'
+          }
+        }
+      )
+      defaults.stubs(:write_to_file).returns(true)
+      defaults.stubs(:providers).returns(providers)
+      api_expects(:compute_profiles, :index, 'List compute profiles').returns(@compute_profiles)
+
+      result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })
+      _(result.exit_code).must_equal HammerCLI::EX_OK
+    end
+  end
+
 end