hammer_cli_foreman_puppet 0.0.7 → 0.1.1

data/test/data/README.md

@@ -1,27 +0,0 @@
-# Data for tests
-
-## How to include API tests for new Foreman release
-
- 1. on the **foreman instance** run the tests with recording of examples turned on
-
-    ```bash
-    $ APIPIE_RECORD=examples bundle exec rake test:functionals
-    ```
- 1. on the **Foreman instance** generate API documentation cache. It will land in `/usr/share/foreman/public/apipie-cache/apidoc/v2.en.json`
-
-    ```bash
-    $ FOREMAN_APIPIE_LANGS=en foreman-rake apipie:cache
-    ```
- 1. when running from **git**, run a similar command and it will land in `$FOREMAN_CHECKOUT/public/apipie-cache/apidoc/v2.en.json`
-
-    ```bash
-    $ FOREMAN_APIPIE_LANGS=en bundle exec rake apipie:cache
-    ```
- 1. in **hammer-cli-foreman** in `test/data/` Create directory with name matching the Foreman version in `test/data/` (e.g. `test/data/1.10`)
- 1. copy the API cache from the Foreman instance into the newly created directory and name it as `foreman_api.json`
- 1. update the following line in `test/test_helper.rb` to match the new default Foreman version
-
-    ```ruby
-    FOREMAN_VERSION = Gem::Version.new(ENV['TEST_API_VERSION'] || '3.0')
-    ```
- 1. make sure the tests are green

data/test/functional/config_group_test.rb

@@ -1,28 +0,0 @@
-# frozen_string_literal: true
-
-require File.join(File.dirname(__FILE__), 'test_helper')
-
-describe 'config_group' do
-  describe 'list' do
-    before do
-      @cmd = %w[config-group list]
-      @config_groups = [{
-                          id: 1,
-                          name: 'config-group-test',
-                        }]
-    end
-
-    it 'should return a list of config groups' do
-      api_expects(:config_groups, :index, 'List config groups').returns(@config_groups)
-
-      output = IndexMatcher.new([
-                                  %w[ID NAME],
-                                  %w[1 config-group-test]
-                                ])
-      expected_result = success_result(output)
-
-      result = run_cmd(@cmd)
-      assert_cmd(expected_result, result)
-    end
-  end
-end

data/test/functional/host/create_test.rb

@@ -1,195 +0,0 @@
-require_relative '../test_helper'
-
-module HammerCLIForemanPuppet
-  describe Host do
-    describe CreateCommand do
-      let(:cmd) { %w[host create] }
-      let(:minimal_params_without_hostgroup) { %w[--location-id=1 --organization-id=1 --name=hst1] }
-
-      it 'allows puppet environment id' do
-        params = %w[--puppet-environment-id=1]
-        api_expects(:hosts, :create) do |p|
-          p['host']['environment_id'] == 1 &&
-            p['host']['name'] == 'hst1'
-        end
-        run_cmd(cmd + minimal_params_without_hostgroup + params)
-      end
-
-      it 'allows puppet environment name' do
-        params = %w[--puppet-environment=env1]
-        api_expects(:environments, :index) do |p|
-          p[:search] == 'name = "env1"'
-        end.returns(index_response([{ 'id' => 1 }]))
-        # FIXME: Called twice because of environment_id being mentioned twice in the docs
-        api_expects(:environments, :index) do |p|
-          p[:search] == 'name = "env1"'
-        end.returns(index_response([{ 'id' => 1 }]))
-        api_expects(:hosts, :create) do |p|
-          p['host']['puppet_attributes']['environment_id'] == 1 &&
-            p['host']['name'] == 'hst1'
-        end
-        run_cmd(cmd + minimal_params_without_hostgroup + params)
-      end
-
-      it 'allows puppet ca proxy id' do
-        params = %w[--puppet-ca-proxy-id=1]
-        api_expects(:hosts, :create).with_params(
-          {
-            host: {
-              name: 'hst1',
-              puppet_ca_proxy_id: 1
-            }
-          }
-        )
-        run_cmd(cmd + minimal_params_without_hostgroup + params)
-      end
-
-      it 'allows puppet ca proxy name' do
-        params = %w[--puppet-ca-proxy=sp1]
-        api_expects(:smart_proxies, :index) do |p|
-          p[:search] = 'name = "sp1"'
-        end.returns(index_response([{ 'id' => 1 }]))
-        api_expects(:hosts, :create) do |p|
-          p['host']['puppet_ca_proxy_id'] == 1 &&
-            p['host']['name'] == 'hst1'
-        end
-        run_cmd(cmd + minimal_params_without_hostgroup + params)
-      end
-
-      it 'allows puppet class ids' do
-        params = %w[--puppet-class-ids=1,2]
-        api_expects(:hosts, :create) do |p|
-          p['host']['puppet_attributes']['puppetclass_ids'] == %w[1 2] &&
-            p['host']['name'] == 'hst1'
-        end
-        run_cmd(cmd + minimal_params_without_hostgroup + params)
-      end
-
-      it 'allows puppet class names' do
-        params = %w[--puppet-classes=pc1,pc2]
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] = 'name = "pc1" or name = "pc2"'
-        end.returns(index_response('puppetclasses' => [
-          {'id' => 1, 'name' => 'pc1'},
-          {'id' => 2, 'name' => 'pc2'}
-        ]))
-        # FIXME: Called twice because of puppetclass_ids being mentioned twice in the docs
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] = 'name = "pc1" or name = "pc2"'
-        end.returns(index_response('puppetclasses' => [
-          {'id' => 1, 'name' => 'pc1'},
-          {'id' => 2, 'name' => 'pc2'}
-        ]))
-        api_expects(:hosts, :create) do |p|
-          p['host']['puppet_attributes']['puppetclass_ids'] == [1,2] &&
-            p['host']['name'] == 'hst1'
-        end
-        run_cmd(cmd + minimal_params_without_hostgroup + params)
-      end
-
-      it 'allows puppet class names that exceeds entries_per_page' do
-        search_objects = []
-        response_objects = []
-        ids = []
-        names = []
-        1100.times.with_object('pc').each do |id, name|
-          next_name = "\"#{name}#{id}\""
-          search_objects << "name = #{next_name}"
-          response_objects << { 'id' => id, 'name' => next_name }
-          ids << id
-          names << next_name
-        end
-        params = ["--puppet-classes=#{names.join(',').tr('"', '')}"]
-
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] == search_objects.join(' or ') &&
-            p[:page].to_i == 1 &&
-            p[:per_page].to_i == HammerCLIForeman::IdResolver::ALL_PER_PAGE
-        end.returns(
-          index_response('puppetclasses' => response_objects[0...1000]))
-
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] == search_objects.join(' or ') &&
-            p[:page].to_i == 2 &&
-            p[:per_page].to_i == HammerCLIForeman::IdResolver::ALL_PER_PAGE
-        end.returns(index_response('puppetclasses' => response_objects[1000...1100]))
-
-        # FIXME: Called twice because of puppetclass_ids being mentioned twice in the docs
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] == search_objects.join(' or ') &&
-            p[:page].to_i == 1 &&
-            p[:per_page].to_i == HammerCLIForeman::IdResolver::ALL_PER_PAGE
-        end.returns(
-          index_response('puppetclasses' => response_objects[0...1000]))
-
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] == search_objects.join(' or ') &&
-            p[:page].to_i == 2 &&
-            p[:per_page].to_i == HammerCLIForeman::IdResolver::ALL_PER_PAGE
-        end.returns(index_response('puppetclasses' => response_objects[1000...1100]))
-
-        api_expects(:hosts, :create) do |p|
-          p['host']['name'] == 'hst1' &&
-            p['host']['puppet_attributes']['puppetclass_ids'] == ids
-        end
-        run_cmd(cmd + minimal_params_without_hostgroup + params)
-      end
-
-      it 'allows puppet proxy id' do
-        params = %w[--puppet-proxy-id=1]
-        api_expects(:hosts, :create).with_params(
-          {
-            host: {
-              name: 'hst1',
-              puppet_proxy_id: 1
-            }
-          }
-        )
-        run_cmd(cmd + minimal_params_without_hostgroup + params)
-      end
-
-      it 'allows puppet proxy name' do
-        params = %w[--puppet-proxy=sp1]
-        api_expects(:smart_proxies, :index) do |p|
-          p[:search] = 'name = "sp1"'
-        end.returns(index_response([{ 'id' => 1 }]))
-        api_expects(:hosts, :create) do |p|
-          p['host']['puppet_proxy_id'] == 1 &&
-            p['host']['name'] == 'hst1'
-        end
-        run_cmd(cmd + minimal_params_without_hostgroup + params)
-      end
-
-      it 'allows config group ids' do
-        params = %w[--config-group-ids=1,2]
-        api_expects(:hosts, :create) do |p|
-          p['host']['puppet_attributes']['config_group_ids'] == %w[1 2] &&
-            p['host']['name'] == 'hst1'
-        end
-        run_cmd(cmd + minimal_params_without_hostgroup + params)
-      end
-
-      it 'allows config group names' do
-        params = %w[--config-groups=cg1,cg2]
-        api_expects(:config_groups, :index) do |p|
-          p[:search] = 'name = "cg1" or name = "cg2"'
-        end.returns(index_response([
-          {'id' => 1, 'name' => 'cg1'},
-          {'id' => 2, 'name' => 'cg2'}
-        ]))
-        # FIXME: Called twice because of config_group_ids being mentioned twice in the docs
-        api_expects(:config_groups, :index) do |p|
-          p[:search] = 'name = "cg1" or name = "cg2"'
-        end.returns(index_response([
-          {'id' => 1, 'name' => 'cg1'},
-          {'id' => 2, 'name' => 'cg2'}
-        ]))
-        api_expects(:hosts, :create) do |p|
-          p['host']['puppet_attributes']['config_group_ids'] == [1, 2] &&
-            p['host']['name'] == 'hst1'
-        end
-        run_cmd(cmd + minimal_params_without_hostgroup + params)
-      end
-    end
-  end
-end

data/test/functional/host/update_test.rb

@@ -1,126 +0,0 @@
-require_relative '../test_helper'
-
-module HammerCLIForemanPuppet
-  describe Host do
-    describe UpdateCommand do
-      it 'allows environment id' do
-        api_expects(:hosts, :update) do |p|
-          p['host']['puppet_attributes']['environment_id'] == 1 &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(host update --id 1 --puppet-environment-id 1))
-      end
-
-      it 'allows environment name' do
-        api_expects(:environments, :index) do |p|
-          p[:search] = "name = \"env1\""
-        end.returns(index_response([{'id' => 1}]))
-        # FIXME: Called twice because of environment_id being mentioned twice in the docs
-        api_expects(:environments, :index) do |p|
-          p[:search] = "name = \"env1\""
-        end.returns(index_response([{'id' => 1}]))
-        api_expects(:hosts, :update) do |p|
-          p['host']['puppet_attributes']['environment_id'] == 1 &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(host update --id 1 --puppet-environment env1))
-      end
-
-      it 'allows puppet ca proxy id' do
-        api_expects(:hosts, :update).with_params({
-          :id => '1',
-          :host => { :puppet_ca_proxy_id => 1 }
-        })
-        run_cmd(%w(host update --id 1 --puppet-ca-proxy-id 1))
-      end
-
-      it 'allows puppet ca proxy name' do
-        api_expects(:smart_proxies, :index) do |p|
-          p[:search] = "name = \"sp1\""
-        end.returns(index_response([{'id' => 1}]))
-        api_expects(:hosts, :update) do |p|
-          p['host']['puppet_ca_proxy_id'] == 1 &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(host update --id 1 --puppet-ca-proxy sp1))
-      end
-
-      it 'allows puppet class ids' do
-        api_expects(:hosts, :update) do |p|
-          p['host']['puppet_attributes']['puppetclass_ids'] == ['1','2'] &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(host update --id 1 --puppet-class-ids 1,2))
-      end
-
-      it 'allows puppet class names' do
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] = "name = \"pc1\" or name = \"pc2\""
-        end.returns(index_response('puppetclasses' => [
-          {'id' => 1, 'name' => 'pc1'},
-          {'id' => 2, 'name' => 'pc2'}
-        ]))
-        # FIXME: Called twice because of puppetclass_ids being mentioned twice in the docs
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] = "name = \"pc1\" or name = \"pc2\""
-        end.returns(index_response('puppetclasses' => [
-          {'id' => 1, 'name' => 'pc1'},
-          {'id' => 2, 'name' => 'pc2'}
-        ]))
-        api_expects(:hosts, :update) do |p|
-          p['host']['puppet_attributes']['puppetclass_ids'] == [1,2] &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(host update --id 1 --puppet-classes pc1,pc2))
-      end
-
-      it 'allows puppet proxy id' do
-        api_expects(:hosts, :update).with_params({
-          :id => '1',
-          :host => { :puppet_proxy_id => 1 }
-        })
-        run_cmd(%w(host update --id 1 --puppet-proxy-id 1))
-      end
-
-      it 'allows puppet proxy name' do
-        api_expects(:smart_proxies, :index) do |p|
-          p[:search] = "name = \"sp1\""
-        end.returns(index_response([{'id' => 1}]))
-        api_expects(:hosts, :update) do |p|
-          p['host']['puppet_proxy_id'] == 1 &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(host update --id 1 --puppet-proxy sp1))
-      end
-
-      it 'allows config group ids' do
-        api_expects(:hosts, :update) do |p|
-          p['host']['puppet_attributes']['config_group_ids'] == %w[1 2] &&
-            p['id'] == '1'
-        end
-        run_cmd(%w[host update --id 1 --config-group-ids=1,2])
-      end
-
-      it 'allows config group names' do
-        api_expects(:config_groups, :index) do |p|
-          p[:search] = 'name = "cg1" or name = "cg2"'
-        end.returns(index_response([
-          {'id' => 1, 'name' => 'cg1'},
-          {'id' => 2, 'name' => 'cg2'}
-        ]))
-        # FIXME: Called twice because of config_group_ids being mentioned twice in the docs
-        api_expects(:config_groups, :index) do |p|
-          p[:search] = 'name = "cg1" or name = "cg2"'
-        end.returns(index_response([
-          {'id' => 1, 'name' => 'cg1'},
-          {'id' => 2, 'name' => 'cg2'}
-        ]))
-        api_expects(:hosts, :update) do |p|
-          p['host']['puppet_attributes']['config_group_ids'] == [1, 2] &&
-            p['id'] == '1'
-        end
-        run_cmd(%w[host update --id 1 --config-groups=cg1,cg2])
-      end
-    end
-  end
-end

data/test/functional/hostgroup/create_test.rb

@@ -1,178 +0,0 @@
-require_relative '../test_helper'
-
-module HammerCLIForemanPuppet
-  describe Hostgroup do
-    describe CreateCommand do
-      it 'allows puppet environment id' do
-        api_expects(:hostgroups, :create) do |p|
-          p['hostgroup']['puppet_attributes']['environment_id'] == 1 &&
-            p['hostgroup']['name'] == 'hg1'
-        end
-        run_cmd(%w[hostgroup create --name hg1 --puppet-environment-id 1])
-      end
-
-      it 'allows puppet environment name' do
-        api_expects(:environments, :index) do |p|
-          p[:search] == 'name = "env1"'
-        end.returns(index_response([{ 'id' => 1 }]))
-        # FIXME: Called twice because of environment_id being mentioned twice in the docs
-        api_expects(:environments, :index) do |p|
-          p[:search] == 'name = "env1"'
-        end.returns(index_response([{ 'id' => 1 }]))
-        api_expects(:hostgroups, :create) do |p|
-          p['hostgroup']['puppet_attributes']['environment_id'] == 1 &&
-            p['hostgroup']['name'] == 'hg1'
-        end
-        run_cmd(%w[hostgroup create --name hg1 --puppet-environment env1])
-      end
-
-      it 'allows puppet ca proxy id' do
-        api_expects(:hostgroups, :create).with_params({
-          :hostgroup => {
-            :name => 'hg1',
-            :puppet_ca_proxy_id => 1
-          }
-        })
-        run_cmd(%w(hostgroup create --name hg1 --puppet-ca-proxy-id 1))
-      end
-
-      it 'allows puppet ca proxy name' do
-        api_expects(:smart_proxies, :index) do |p|
-          p[:search] = "name = \"sp1\""
-        end.returns(index_response([{'id' => 1}]))
-        api_expects(:hostgroups, :create) do |p|
-          p['hostgroup']['puppet_ca_proxy_id'] == 1 &&
-            p['hostgroup']['name'] == 'hg1'
-        end
-        run_cmd(%w(hostgroup create --name hg1 --puppet-ca-proxy sp1))
-      end
-
-      it 'allows puppet class ids' do
-        api_expects(:hostgroups, :create) do |p|
-          p['hostgroup']['puppet_attributes']['puppetclass_ids'] == ['1','2'] &&
-            p['hostgroup']['name'] == 'hg1'
-        end
-        run_cmd(%w(hostgroup create --name hg1 --puppet-class-ids 1,2))
-      end
-
-      it 'allows puppet class names' do
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] = "name = \"pc1\" or name = \"pc2\""
-        end.returns(index_response('puppetclasses' => [
-          {'id' => 1, 'name' => 'pc1'},
-          {'id' => 2, 'name' => 'pc2'}
-        ]))
-        # FIXME: Called twice because of puppetclass_ids being mentioned twice in the docs
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] = "name = \"pc1\" or name = \"pc2\""
-        end.returns(index_response('puppetclasses' => [
-          {'id' => 1, 'name' => 'pc1'},
-          {'id' => 2, 'name' => 'pc2'}
-        ]))
-        api_expects(:hostgroups, :create) do |p|
-          p['hostgroup']['puppet_attributes']['puppetclass_ids'] == [1,2] &&
-            p['hostgroup']['name'] == 'hg1'
-        end
-        run_cmd(%w(hostgroup create --name hg1 --puppet-classes pc1,pc2))
-      end
-
-      it 'allows puppet class names that exceeds entries_per_page' do
-        search_objects = []
-        response_objects = []
-        ids = []
-        names = []
-        1100.times.with_object('pc').each do |id, name|
-          next_name = "\"#{name}#{id}\""
-          search_objects << "name = #{next_name}"
-          response_objects << { 'id' => id, 'name' => next_name }
-          ids << id
-          names << next_name
-        end
-
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] == search_objects.join(' or ') &&
-            p[:page].to_i == 1 &&
-            p[:per_page].to_i == HammerCLIForeman::IdResolver::ALL_PER_PAGE
-        end.returns(
-          index_response('puppetclasses' => response_objects[0...1000]))
-
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] == search_objects.join(' or ') &&
-            p[:page].to_i == 2 &&
-            p[:per_page].to_i == HammerCLIForeman::IdResolver::ALL_PER_PAGE
-        end.returns(index_response('puppetclasses' => response_objects[1000...1100]))
-
-        # FIXME: Called twice because of puppetclass_ids being mentioned twice in the docs
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] == search_objects.join(' or ') &&
-            p[:page].to_i == 1 &&
-            p[:per_page].to_i == HammerCLIForeman::IdResolver::ALL_PER_PAGE
-        end.returns(
-          index_response('puppetclasses' => response_objects[0...1000]))
-
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] == search_objects.join(' or ') &&
-            p[:page].to_i == 2 &&
-            p[:per_page].to_i == HammerCLIForeman::IdResolver::ALL_PER_PAGE
-        end.returns(index_response('puppetclasses' => response_objects[1000...1100]))
-
-        api_expects(:hostgroups, :create) do |p|
-          p['hostgroup']['name'] == 'hg1' &&
-            p['hostgroup']['puppet_attributes']['puppetclass_ids'] == ids
-        end
-
-        run_cmd(%w[hostgroup create --name hg1 --puppet-classes] << names.join(',').tr('"', ''))
-      end
-
-      it 'allows puppet proxy id' do
-        api_expects(:hostgroups, :create).with_params({
-          :hostgroup => {
-            :name => 'hg1',
-            :puppet_proxy_id => 1
-          }
-        })
-        run_cmd(%w(hostgroup create --name hg1 --puppet-proxy-id 1))
-      end
-
-      it 'allows puppet proxy name' do
-        api_expects(:smart_proxies, :index) do |p|
-          p[:search] = "name = \"sp1\""
-        end.returns(index_response([{'id' => 1}]))
-        api_expects(:hostgroups, :create) do |p|
-          p['hostgroup']['puppet_proxy_id'] == 1 &&
-            p['hostgroup']['name'] == 'hg1'
-        end
-        run_cmd(%w(hostgroup create --name hg1 --puppet-proxy sp1))
-      end
-
-      it 'allows config group ids' do
-        api_expects(:hostgroups, :create) do |p|
-          p['hostgroup']['puppet_attributes']['config_group_ids'] == %w[1 2] &&
-            p['hostgroup']['name'] == 'hg1'
-        end
-        run_cmd(%w[hostgroup create --name hg1 --config-group-ids 1,2])
-      end
-
-      it 'allows config group names' do
-        api_expects(:config_groups, :index) do |p|
-          p[:search] = 'name = "cg1" or name = "cg2"'
-        end.returns(index_response([
-          {'id' => 1, 'name' => 'cg1'},
-          {'id' => 2, 'name' => 'cg2'}
-        ]))
-        # FIXME: Called twice because of config_group_ids being mentioned twice in the docs
-        api_expects(:config_groups, :index) do |p|
-          p[:search] = 'name = "cg1" or name = "cg2"'
-        end.returns(index_response([
-          {'id' => 1, 'name' => 'cg1'},
-          {'id' => 2, 'name' => 'cg2'}
-        ]))
-        api_expects(:hostgroups, :create) do |p|
-          p['hostgroup']['puppet_attributes']['config_group_ids'] == [1, 2] &&
-            p['hostgroup']['name'] == 'hg1'
-        end
-        run_cmd(%w[hostgroup create --name hg1 --config-groups cg1,cg2])
-      end
-    end
-  end
-end

data/test/functional/hostgroup/update_test.rb

@@ -1,126 +0,0 @@
-require_relative '../test_helper'
-
-module HammerCLIForemanPuppet
-  describe Hostgroup do
-    describe UpdateCommand do
-      it 'allows environment id' do
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppet_attributes']['environment_id'] == 1 &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(hostgroup update --id 1 --puppet-environment-id 1))
-      end
-
-      it 'allows environment name' do
-        api_expects(:environments, :index) do |p|
-          p[:search] = "name = \"env1\""
-        end.returns(index_response([{'id' => 1}]))
-        # FIXME: Called twice because of environment_id being mentioned twice in the docs
-        api_expects(:environments, :index) do |p|
-          p[:search] = "name = \"env1\""
-        end.returns(index_response([{'id' => 1}]))
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppet_attributes']['environment_id'] == 1 &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(hostgroup update --id 1 --puppet-environment env1))
-      end
-
-      it 'allows puppet ca proxy id' do
-        api_expects(:hostgroups, :update).with_params({
-          :id => '1',
-          :hostgroup => { :puppet_ca_proxy_id => 1 }
-        })
-        run_cmd(%w(hostgroup update --id 1 --puppet-ca-proxy-id 1))
-      end
-
-      it 'allows puppet ca proxy name' do
-        api_expects(:smart_proxies, :index) do |p|
-          p[:search] = "name = \"sp1\""
-        end.returns(index_response([{'id' => 1}]))
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppet_ca_proxy_id'] == 1 &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(hostgroup update --id 1 --puppet-ca-proxy sp1))
-      end
-
-      it 'allows puppet class ids' do
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppet_attributes']['puppetclass_ids'] == ['1','2'] &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(hostgroup update --id 1 --puppet-class-ids 1,2))
-      end
-
-      it 'allows puppet class names' do
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] = "name = \"pc1\" or name = \"pc2\""
-        end.returns(index_response('puppetclasses' => [
-          {'id' => 1, 'name' => 'pc1'},
-          {'id' => 2, 'name' => 'pc2'}
-        ]))
-        # FIXME: Called twice because of puppetclass_ids being mentioned twice in the docs
-        api_expects(:puppetclasses, :index) do |p|
-          p[:search] = "name = \"pc1\" or name = \"pc2\""
-        end.returns(index_response('puppetclasses' => [
-          {'id' => 1, 'name' => 'pc1'},
-          {'id' => 2, 'name' => 'pc2'}
-        ]))
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppet_attributes']['puppetclass_ids'] == [1,2] &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(hostgroup update --id 1 --puppet-classes pc1,pc2))
-      end
-
-      it 'allows puppet proxy id' do
-        api_expects(:hostgroups, :update).with_params({
-          :id => '1',
-          :hostgroup => { :puppet_proxy_id => 1 }
-        })
-        run_cmd(%w(hostgroup update --id 1 --puppet-proxy-id 1))
-      end
-
-      it 'allows puppet proxy name' do
-        api_expects(:smart_proxies, :index) do |p|
-          p[:search] = "name = \"sp1\""
-        end.returns(index_response([{'id' => 1}]))
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppet_proxy_id'] == 1 &&
-            p['id'] == '1'
-        end
-        run_cmd(%w(hostgroup update --id 1 --puppet-proxy sp1))
-      end
-
-      it 'allows config group ids' do
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppet_attributes']['config_group_ids'] == %w[1 2] &&
-            p['id'] == '1'
-        end
-        run_cmd(%w[hostgroup update --id 1 --config-group-ids 1,2])
-      end
-
-      it 'allows config group names' do
-        api_expects(:config_groups, :index) do |p|
-          p[:search] = 'name = "cg1" or name = "cg2"'
-        end.returns(index_response([
-          {'id' => 1, 'name' => 'cg1'},
-          {'id' => 2, 'name' => 'cg2'}
-        ]))
-        # FIXME: Called twice because of config_group_ids being mentioned twice in the docs
-        api_expects(:config_groups, :index) do |p|
-          p[:search] = 'name = "cg1" or name = "cg2"'
-        end.returns(index_response([
-          {'id' => 1, 'name' => 'cg1'},
-          {'id' => 2, 'name' => 'cg2'}
-        ]))
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppet_attributes']['config_group_ids'] == [1, 2] &&
-            p['id'] == '1'
-        end
-        run_cmd(%w[hostgroup update --id 1 --config-groups cg1,cg2])
-      end
-    end
-  end
-end