hammer_cli_foreman 2.0.2 → 2.1.0

data/test/functional/bookmark_test.rb

@@ -0,0 +1,193 @@
+# frozen_string_literal: true
+
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'bookmark' do
+  describe 'list' do
+    before do
+      @cmd = %w[bookmark list]
+      @bookmarks = [{
+        id: 1,
+        name: 'active',
+        controller: 'hosts',
+        owner_id: 1,
+        owner_type: 'User'
+      }]
+    end
+
+    it 'should return a list of bookmarks' do
+      api_expects(:bookmarks, :index, 'List bookmarks').returns(@bookmarks)
+
+      output = IndexMatcher.new([
+                                  %w[ID NAME CONTROLLER OWNER\ ID OWNER\ TYPE],
+                                  %w[1 active hosts 1 User]
+                                ])
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'info' do
+    before do
+      @cmd = %w[bookmark info]
+      @bookmark = {
+        id: 1,
+        name: 'active',
+        controller: 'hosts',
+        owner_id: 1,
+        owner_type: 'User'
+      }
+    end
+
+    it 'should return the info of a bookmark' do
+      params = ['--id=1']
+      api_expects(:bookmarks, :show, 'Info bookmark').returns(@bookmark)
+      output = OutputMatcher.new([
+                                   'Id:         1',
+                                   'Name:       active',
+                                   'Controller: hosts',
+                                   'Owner Id:   1',
+                                   'Owner Type: User'
+                                 ])
+
+      expected_result = success_result(output)
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
+
+    describe 'create bookmark' do
+      before do
+        @cmd = %w[bookmark create]
+      end
+
+      it 'should create a bookmark' do
+        params = ['--name=newBookmark', '--controller=hosts', '--query=status = failed']
+        api_expects(:bookmarks, :create, 'Create bookmark') do |params|
+          (params['bookmark']['name'] == 'newBookmark')
+          (params['bookmark']['controller'] == 'hosts')
+          (params['bookmark']['query'] == 'status = failed')
+        end
+        result = run_cmd(@cmd + params)
+        assert_cmd(success_result("Bookmark %<name>s created.\n"), result)
+      end
+
+      it 'should print error on missing --name' do
+        params = ['--controller=hosts', '--query=status = failed']
+        expected_result = CommandExpectation.new
+        expected_result.expected_err =
+          ['Failed to create  bookmark:',
+           "  Missing arguments for '--name'.",
+           ''].join("\n")
+        expected_result.expected_exit_code = HammerCLI::EX_USAGE
+
+        api_expects_no_call
+
+        result = run_cmd(@cmd + params)
+        assert_cmd(expected_result, result)
+      end
+
+      it 'should print error on missing --controller' do
+        params = ['--name=newBookmark', '--query=status = failed']
+        expected_result = CommandExpectation.new
+        expected_result.expected_err =
+          ['Failed to create  bookmark:',
+           "  Missing arguments for '--controller'.",
+           ''].join("\n")
+        expected_result.expected_exit_code = HammerCLI::EX_USAGE
+        api_expects_no_call
+
+        result = run_cmd(@cmd + params)
+
+        assert_cmd(expected_result, result)
+      end
+
+      it 'should print error on missing --query' do
+        params = ['--name=newBookmark', '--controller=hosts']
+
+        expected_result = CommandExpectation.new
+        expected_result.expected_err =
+          ['Failed to create  bookmark:',
+           "  Missing arguments for '--query'.",
+           ''].join("\n")
+        expected_result.expected_exit_code = HammerCLI::EX_USAGE
+
+        api_expects_no_call
+
+        result = run_cmd(@cmd + params)
+
+        assert_cmd(expected_result, result)
+      end
+    end
+
+    describe 'update bookmark' do
+      before do
+        @cmd = %w[bookmark update]
+        @bookmark = {
+          "id": 1,
+          'name': 'bookmark-2',
+          'controller': 'hosts',
+          'query': 'status = failed',
+          'owner_id': 1,
+          'owner_type': 'User'
+        }
+      end
+
+      it 'allows minimal options' do
+        api_expects(:bookmarks, :update) do |par|
+          par['id'] == '1'
+        end
+        run_cmd(%w[bookmark update --id 1])
+      end
+
+      it 'update bookmark' do
+        params = ['--id=1', '--new-name=bookmark-2', '--controller=hosts', '--query=status != failed']
+        api_expects(:bookmarks, :update, 'Update the bookmark') do |params|
+          (params['bookmark']['id'] = 1)
+          (params['bookmark']['name'] = 'bookmark-2')
+          (params['bookmark']['controller'] = 'hosts')
+          (params['bookmark']['query'] = 'status != failed')
+        end
+        result = run_cmd(@cmd + params)
+        assert_cmd(success_result("Bookmark %<name>s updated successfully.\n"), result)
+      end
+    end
+
+    describe 'delete bookmark' do
+      before do
+        @cmd = %w[bookmark delete]
+
+        @bookmark = {
+          "id": 1,
+          'name': 'bookmark-2',
+          'controller': 'hosts',
+          'query': 'status = failed',
+          'owner_id': 1,
+          'owner_type': 'User'
+        }
+      end
+
+      it 'delete bookmark' do
+        params = ['--id=1']
+        api_expects(:bookmarks, :destroy, id: 1).returns({})
+        result = run_cmd(@cmd + params)
+        assert_cmd(success_result("Bookmark deleted successfully.\n"), result)
+      end
+
+      it 'should print error on missing --id' do
+        params = []
+
+        expected_result = CommandExpectation.new
+        expected_result.expected_err =
+          ['Failed to delete bookmark:',
+           "  Missing arguments for '--id'.",
+           ''].join("\n")
+        expected_result.expected_exit_code = HammerCLI::EX_USAGE
+        api_expects_no_call
+        result = run_cmd(@cmd + params)
+        assert_cmd(expected_result, result)
+      end
+    end
+  end
+end

data/test/functional/host_test.rb

@@ -308,7 +308,10 @@ describe 'host update' do
       'organization_id' => '5',
       'location_id' => '5',
       'compute_attributes' => {},
-      'puppetclass_ids' => []
+      'puppetclass_ids' => [],
+      'owner_id' => '1',
+      'owner_name' => 'adminGroup',
+      'owner_type' => 'Usergroup'
     }
   end
 
@@ -384,6 +387,29 @@ describe 'host update' do
 
     assert_cmd(expected_result, result)
   end
+
+  it 'should update the host owner' do
+    params = ['--owner-type=Usergroup', '--owner=adminGroup']
+
+    api_expects_search(:usergroups, name: 'adminGroup').returns(
+        index_response([{ 'id' => '1' }])
+    )
+    api_expects(:hosts, :update, 'Update host with new owner').with_params(
+        'id' => '1', 'location_id' => 1, 'organization_id' => 1, 'host' => {
+        'owner_id' => '1', 'compute_attributes' => {}
+    }
+    ) do |par|
+      par['id'] == '1' &&
+          par['host']['owner_id'] == '1' &&
+          par['host']['compute_attributes'] == {}
+    end.returns(updated_host)
+
+    expected_result = success_result("Host updated.\n")
+
+    result = run_cmd(cmd + minimal_params + params)
+
+    assert_cmd(expected_result, result)
+  end
 end
 
 describe 'host config reports' do

data/test/functional/mail_notification_test.rb

@@ -0,0 +1,57 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'mail_notification' do
+  describe 'list' do
+    before do
+      @cmd = %w[mail-notification list]
+      @mail_notifications = [{
+        id: 1,
+        name: 'config_summary',
+        description: 'A summary of eventful configuration management reports',
+        subscription_type: 'report'
+      }]
+    end
+
+    it 'should return a list of mail notifications' do
+      api_expects(:mail_notifications, :index, 'List mail notifications').returns(@mail_notifications)
+
+      output = IndexMatcher.new([
+                                  %w[ID NAME],
+                                  %w[1 config_summary]
+                                ])
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'info' do
+    before do
+      @cmd = %w[mail-notification info]
+      @mail_notification = {
+        id: 1,
+        name: 'config_summary',
+        description: 'A summary of eventful configuration management reports',
+        subscription_type: 'report'
+      }
+    end
+
+    it 'should return the info of a mail notification' do
+      params = ['--id=1']
+      api_expects(:mail_notifications, :show, 'Info mail notification').returns(@mail_notification)
+
+      output = OutputMatcher.new([
+                                   'Id:                1',
+                                   'Name:              config_summary',
+                                   'Description:       A summary of eventful configuration management reports',
+                                   'Subscription type: report'
+                                 ])
+
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+end

data/test/functional/subnet/create_test.rb

@@ -14,13 +14,36 @@ module HammerCLIForeman
         params
       end
 
-      it 'allows minimal options' do
-        api_expects(:subnets, :create) do |par|
-          par['subnet']['name'] == 'net1'
-        end
+      it 'should print error on missing --mask or --prefix' do
+        params = ['--name=net1']
+        cmd = %w[subnet create]
+
+        expected_result = usage_error_result(
+          cmd,
+          'At least one of options --mask, --prefix is required.',
+          'Could not create the subnet'
+        )
+
+        api_expects_no_call
+        result = run_cmd(cmd + params)
+        assert_cmd(expected_result, result)
+      end
+
+      it 'allows minimal options with mask for IPv4' do
+        api_expects(:subnets, :create).with_params(subnet_params(mask: '255.255.255.0'))
         run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0))
       end
 
+      it 'allows minimal options with prefix for IPv4' do
+        api_expects(:subnets, :create).with_params(subnet_params(mask: '255.255.255.0'))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --prefix=24))
+      end
+
+      it 'allows minimal options with prefix for IPv6' do
+        api_expects(:subnets, :create).with_params(subnet_params(cidr: '96'))
+        run_cmd(%w(subnet create --name net1 --network-type=IPv6 --network='::ffff:c0a8:7a00' --prefix=96))
+      end
+
       it 'allows dhcp id' do
         api_expects(:subnets, :create).with_params(subnet_params(:dhcp_id => 1))
         run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --dhcp-id 1))
@@ -91,7 +114,7 @@ module HammerCLIForeman
       it 'allows organization names' do
         api_expects(:organizations, :index) do |p|
           p[:search] == "name = \"org1\" or name = \"org2\""
-        end.returns(index_response([{'id' => 1}, {'id' => 2}])) 
+        end.returns(index_response([{'id' => 1}, {'id' => 2}]))
         api_expects(:subnets, :create).with_params(subnet_params(:organization_ids => [1, 2]))
         run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --organizations org1,org2))
       end

data/test/functional/template_test.rb

@@ -131,7 +131,7 @@ describe 'template' do
     it 'should create new combination with warning' do
       params = ['create','--provisioning-template-id=10', '--hostgroup-id=1', '--environment-id=1']
       expected_result = success_result("Template combination created.\n")
-      expected_result.expected_err = "Warning: Option --environment-id is deprecated. Use --puppet-environment-id instead\n"
+      expected_result.expected_err = "Warning: Option --environment-id is deprecated. Use --puppet-environment[-id] instead\n"
       api_expects(:template_combinations, :create, 'Create template combination') do |params|
         params['provisioning_template_id'] == 10 &&
           params['hostgroup_id'] == 1 &&
@@ -160,7 +160,7 @@ describe 'template' do
     it 'should update combination with warning' do
       params = ['update', '--id=3', '--provisioning-template-id=10', '--hostgroup-id=1', '--environment-id=1']
       expected_result = success_result("Template combination updated.\n")
-      expected_result.expected_err = "Warning: Option --environment-id is deprecated. Use --puppet-environment-id instead\n"
+      expected_result.expected_err = "Warning: Option --environment-id is deprecated. Use --puppet-environment[-id] instead\n"
       api_expects(:template_combinations, :update, 'Update template combination') do |params|
         params['id'] == '3' &&
           params['provisioning_template_id'] == 10 &&

data/test/functional/virtual_machine_test.rb

@@ -0,0 +1,129 @@
+
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+require 'hammer_cli_foreman/compute_resource'
+
+describe HammerCLIForeman::VirtualMachine do
+
+  context "InfoCommand" do
+    before do
+      @cmd = ["compute-resource", "virtual-machine", "info"]
+      @vm = {
+          'id' => 1,
+          'name' => 'vm1',
+          'provider' => 'Ovirt',
+          'cpu' => 1,
+          'memory' => 1,
+          'status' => 'down'
+      }
+    end
+
+    it "should print error on missing --id and --vm-id" do
+      api_expects_no_call
+      result = run_cmd(@cmd)
+      assert_match("Missing arguments for '--id', '--vm-id'.\n", result.err)
+    end
+
+    it "should print error on missing --vm-id" do
+      params = ['--id=1']
+
+      api_expects_no_call
+      result = run_cmd(@cmd + params)
+      assert_match("Missing arguments for '--vm-id'.\n", result.err)
+    end
+
+    it "should return vm info" do
+      params = ['--id=1', '--vm-id=1']
+      api_expects(:compute_resources, :show_vm) do |par|
+        par['id'] == '1' && par['vm_id'] == '1'
+      end.returns(@vm)
+      result = run_cmd(@cmd + params)
+      assert_match("Id:     1\nName:   vm1\nCPUs:   1\nMemory: 1\nStatus: down\n\n", result.out)
+    end
+
+  end
+
+  context "ListCommand" do
+    before do
+      @cmd = ["compute-resource", "virtual-machines"]
+      @available_virtual_machines = [
+        { id: 1, name: 'vm1' },
+        { id: 2, name: 'vm2' }
+      ]
+    end
+
+    it "should print error on missing --id" do
+      api_expects_no_call
+      result = run_cmd(@cmd)
+      assert_match("Missing arguments for '--id'.\n", result.err)
+    end
+
+    it "should return vms list" do
+      params = ['--id=1']
+      api_expects(:compute_resources, :available_virtual_machines) do |par|
+        par['id'] == '1'
+      end.returns(@available_virtual_machines)
+      result = run_cmd(@cmd + params)
+      assert_match(/-----|---\nNAME | ID\n-----|---\nvm1  | 1 \nvm2  | 2 \n-----|---\n/, result.out)
+    end
+  end
+
+  context "PowerCommand" do
+    before do
+      @cmd = ["compute-resource", "virtual-machine", "power"]
+    end
+
+    it "should print error on missing --id and --vm-id" do
+      api_expects_no_call
+      result = run_cmd(@cmd)
+      assert_match("Could not power the virtual machine:\n  Missing arguments for '--id', '--vm-id'.\n", result.err)
+    end
+
+    it "should print error on missing --vm-id" do
+      params = ['--id=1']
+
+      api_expects_no_call
+      result = run_cmd(@cmd + params)
+      assert_match("Could not power the virtual machine:\n  Missing arguments for '--vm-id'.\n", result.err)
+    end
+
+    it 'should power a virtual machine' do
+      params = ['--id=1', '--vm-id=1']
+      api_expects(:compute_resources, :power_vm) do |par|
+        par['id'] == '1' && par['vm_id'] == '1'
+      end.returns({})
+      result = run_cmd(@cmd + params)
+      assert_cmd(success_result("Virtual machine is powering.\n"), result)
+    end
+  end
+
+  context "DeleteCommand" do
+    before do
+      @cmd = ["compute-resource", "virtual-machine", "delete"]
+    end
+
+    it "should print error on missing --id and --vm-id" do
+      api_expects_no_call
+      result = run_cmd(@cmd)
+      assert_match("Could not delete the virtual machine:\n  Missing arguments for '--id', '--vm-id'.\n", result.err)
+    end
+
+    it "should print error on missing --vm-id" do
+      params = ['--id=1']
+
+      api_expects_no_call
+      result = run_cmd(@cmd + params)
+      assert_match("Could not delete the virtual machine:\n  Missing arguments for '--vm-id'.\n", result.err)
+    end
+
+    it 'delete virtual machine' do
+      params = ['--id=1', '--vm-id=1']
+      api_expects(:compute_resources, :destroy_vm) do |par|
+        par['id'] == '1' && par['vm_id'] == '1'
+      end.returns({})
+      result = run_cmd(@cmd + params)
+      assert_cmd(success_result("Virtual machine deleted.\n"), result)
+    end
+  end
+
+end