hammer_cli_foreman_puppet 0.0.1

data/test/functional/host/update_test.rb

@@ -0,0 +1,97 @@
+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']['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']['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']['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']['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
+    end
+  end
+end

data/test/functional/hostgroup/create_test.rb

@@ -0,0 +1,149 @@
+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']['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']['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']['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']['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']['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
+    end
+  end
+end

data/test/functional/hostgroup/update_test.rb

@@ -0,0 +1,97 @@
+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']['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']['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']['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']['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
+    end
+  end
+end

data/test/functional/proxy_test.rb

@@ -0,0 +1,86 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'proxy' do
+  describe 'import-classes' do
+    let(:cmd) { ['proxy', 'import-classes'] }
+    let(:params) { ['--id=83'] }
+    let(:report) do
+      {
+        'message' => 'Successfully updated environment and puppetclasses from the on-disk puppet installation',
+        'environments_with_new_puppetclasses' => 2,
+        'environments_updated_puppetclasses' => 0,
+        'environments_obsolete' => 0,
+        'environments_ignored' => 0,
+        'results' => [{
+          'name' => 'development',
+          'actions' => [ 'new', 'update' ],
+          'new_puppetclasses' => [
+            'motd',
+            'hammer'
+          ],
+          'updated_puppetclasses' => [
+            'stdlib',
+            'vim'
+          ]
+        },{
+          'name' => 'production',
+          'actions' => [ 'obsolete', 'ignore' ],
+          'obsolete_puppetclasses' => [
+            'apache'
+          ],
+          'ignored_puppetclasses' => [
+            'hammer'
+          ]
+        }]
+      }
+    end
+    let(:no_change_report) do
+      {
+        'message' => 'No changes to your environments detected'
+      }
+    end
+
+    it 'prints detailed report' do
+      api_expects(:smart_proxies, :import_puppetclasses, 'Import classes')
+        .with_params('id' => '83')
+        .returns(report)
+
+      output = OutputMatcher.new([
+        'Result:',
+        '  Successfully updated environment and puppetclasses from the on-disk puppet installation',
+        'Changed environments:',
+        ' 1) development',
+        '    New classes:',
+        '        motd',
+        '        hammer',
+        '    Updated classes:',
+        '        stdlib',
+        '        vim',
+        ' 2) production',
+        '    Removed classes:',
+        '        apache',
+        '    Ignored classes:',
+        '        hammer'
+      ])
+      expected_result = success_result(output)
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'prints a message when nothig has changed' do
+      api_expects(:smart_proxies, :import_puppetclasses, 'Import classes')
+        .with_params('id' => '83')
+        .returns(no_change_report)
+
+      output = OutputMatcher.new([
+        'Result:',
+        '  No changes to your environments detected'
+      ])
+      expected_result = success_result(output)
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+end

data/test/functional/smart_class_parameter_test.rb

@@ -0,0 +1,97 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'sc-params update' do
+  let(:cmd) { %w[sc-param update] }
+
+  it 'does not allow to update parameter name' do
+    params = ['--id=1', '--new-name=name2']
+
+    expected_result = usage_error_result(
+      cmd,
+      "Unrecognised option '--new-name'.",
+      'Could not update the parameter'
+    )
+
+    api_expects_no_call
+    result = run_cmd(cmd + params)
+    assert_cmd(expected_result, result)
+  end
+end
+
+describe 'sc-params add-matcher' do
+  let(:cmd) { %w[sc-param add-matcher] }
+  let(:override_value) { 'something' }
+  let(:match) { 'domain = example.com' }
+  let(:puppet_class) { { 'name' => 'motd', 'id' => '1' } }
+  let(:parameter) { { 'name' => 'content', 'id' => '2' } }
+  let(:base) do
+    [
+      '--puppet-class', puppet_class['name'],
+      '--smart-class-parameter', parameter['name'],
+      '--match', match
+    ]
+  end
+
+  def api_expects_parameter_search(puppet_class, parameter)
+    api_expects(:puppetclasses, :index, 'Find puppet class') do |par|
+      par[:search] == %Q(name = "#{puppet_class['name']}")
+    end.returns(index_response('motd' => [puppet_class]))
+
+    api_expects(:smart_class_parameters, :index, 'Find smart parameter') do |par|
+      par[:search] == %Q(key = "#{parameter['name']}") &&
+        par[:puppetclass_id] == puppet_class['id']
+    end.returns(index_response([parameter]))
+  end
+
+  it 'allows to set value' do
+    params = ['--value', override_value]
+    expected_result = success_result("Override value created.\n")
+
+    api_expects_parameter_search(puppet_class, parameter)
+
+    api_expects(:override_values, :create, 'Create override value').with_params(
+      smart_class_parameter_id: parameter['id'],
+      override_value: {
+        match: match,
+        value: override_value
+      }
+    )
+
+    result = run_cmd(cmd + base + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it 'allows to set value with disabled puppet default' do
+    params = ['--value', override_value, '--omit', false]
+    expected_result = success_result("Override value created.\n")
+
+    api_expects_parameter_search(puppet_class, parameter)
+
+    api_expects(:override_values, :create, 'Create override value').with_params(
+      smart_class_parameter_id: parameter['id'],
+      override_value: {
+        match: match,
+        value: override_value,
+        omit: false
+      }
+    )
+
+    result = run_cmd(cmd + base + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it 'does not allow to use puppet default and value at the same time' do
+    params = ['--value', 'something', '--omit', true]
+
+    expected_result = usage_error_result(
+      cmd,
+      'Cannot use --value when --omit is true.',
+      'Could not create the override value'
+    )
+
+    api_expects_parameter_search(puppet_class, parameter)
+
+    result = run_cmd(cmd + base + params)
+    assert_cmd(expected_result, result)
+  end
+end