hammer_cli_foreman 0.12.1 → 0.13.0

data/test/functional/auth_source_test.rb

@@ -0,0 +1,54 @@
+require File.join(File.dirname(__FILE__), '..', 'test_helper')
+
+describe 'AuthSource' do
+  let(:auth_source_ldap) do
+    {
+        :id => 1,
+        :name => 'MyLDAP',
+        :type => 'AuthSourceLdap'
+    }
+  end
+
+  let(:auth_source_external) do
+    {
+        :id => 2,
+        :name => 'MyExternal',
+        :type => 'AuthSourceExternal'
+    }
+  end
+
+  let(:auth_source_internal) do
+    {
+        :id => 3,
+        :name => 'MyInternal',
+        :type => 'AuthSourceInternal'
+    }
+  end
+
+  describe 'list command' do
+    before do
+      @cmd = %w(auth-source list)
+    end
+
+    params = []
+
+    it 'lists all authentication sources' do
+      api_expects(:auth_sources, :index, 'List').with_params(
+          'page' => 1, 'per_page' => 1000
+      ).returns(index_response([auth_source_ldap, auth_source_external, auth_source_internal]))
+
+      output = IndexMatcher.new([
+                                    ['ID', 'NAME',  'TYPE OF AUTH SOURCE'],
+                                    ['1',  'MyLDAP',  'AuthSourceLdap'],
+                                    ['2',  'MyExternal', 'AuthSourceExternal'],
+                                    ['3',  'MyInternal', 'AuthSourceInternal']
+                                ])
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
+
+  end
+
+end

data/test/functional/host_test.rb

@@ -1,5 +1,57 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 
+describe 'host enc-dump' do
+  let(:cmd) { ['host', 'enc-dump'] }
+  let(:params) { ['--id=1'] }
+  let(:context) { { :adapter => :table } }
+  let(:json_dump) do
+    JSON.dump(
+      {
+        :parameters => {
+          :foreman_subnets => [],
+          :location => 'Default Location',
+          :location_title => 'Default Location',
+          :organization => 'Default Organization',
+          :organization_title => 'Default Organization',
+          :domainname => 'testdomain.com',
+        },
+        :classes => []
+      }
+    )
+  end
+
+  it "prints error or missing --id" do
+    expected_result = CommandExpectation.new
+    expected_result.expected_err =
+      ['Could not retrieve ENC values of the host:',
+       "  Missing arguments for 'id'",
+       ''].join("\n")
+    expected_result.expected_exit_code = HammerCLI::EX_USAGE
+
+    api_expects_no_call
+
+    result = run_cmd(cmd)
+    assert_cmd(expected_result, result)
+  end
+
+  it "prints ENC YAML to stdout" do
+    expected_result = success_result(YAML.dump(json_dump))
+
+    api_expects(:hosts, :enc, "Dump host's ENC YAML").with_params('id' => '1').returns(json_dump)
+
+    result = run_cmd(cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "prints ENC YAML even with other adapter" do
+    expected_result = success_result(YAML.dump(json_dump))
+
+    api_expects(:hosts, :enc, "Dump host's ENC YAML").with_params('id' => '1').returns(json_dump)
+
+    result = run_cmd(cmd + params, context)
+    assert_cmd(expected_result, result)
+  end
+end
 
 describe "host create" do
   let(:cmd) { ["host", "create"] }

data/test/functional/hostgroup/create_test.rb

@@ -164,10 +164,12 @@ module HammerCLIForeman
       end
 
       it 'allows parent hostgroup id' do
-        api_expects(:hostgroups, :create) do |p|
-          p['hostgroup']['parent_id'] == 1 &&
-            p['hostgroup']['name'] == 'hg1'
-        end
+        api_expects(:hostgroups, :create).with_params({
+          :hostgroup => {
+            :name => 'hg1',
+            :parent_id => '1'
+          }
+        })
         run_cmd(%w(hostgroup create --name hg1 --parent-id 1))
       end
 
@@ -202,10 +204,12 @@ module HammerCLIForeman
       end
 
       it 'allows puppet ca proxy id' do
-        api_expects(:hostgroups, :create) do |p|
-          p['hostgroup']['puppet_ca_proxy_id'] == 1 &&
-            p['hostgroup']['name'] == 'hg1'
-        end
+        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
 
@@ -243,10 +247,12 @@ module HammerCLIForeman
       end
 
       it 'allows puppet proxy id' do
-        api_expects(:hostgroups, :create) do |p|
-          p['hostgroup']['puppet_proxy_id'] == 1 &&
-            p['hostgroup']['name'] == 'hg1'
-        end
+        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
 

data/test/functional/hostgroup/update_test.rb

@@ -164,10 +164,10 @@ module HammerCLIForeman
       end
 
       it 'allows parent hostgroup id' do
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['parent_id'] == 1 &&
-            p['id'] == '1'
-        end
+        api_expects(:hostgroups, :update).with_params({
+          :id => '1',
+          :hostgroup => { :parent_id => '1' }
+        })
         run_cmd(%w(hostgroup update --id 1 --parent-id 1))
       end
 
@@ -202,10 +202,10 @@ module HammerCLIForeman
       end
 
       it 'allows puppet ca proxy id' do
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppet_ca_proxy_id'] == 1 &&
-            p['id'] == '1'
-        end
+        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
 
@@ -243,10 +243,10 @@ module HammerCLIForeman
       end
 
       it 'allows puppet proxy id' do
-        api_expects(:hostgroups, :update) do |p|
-          p['hostgroup']['puppet_proxy_id'] == 1 &&
-            p['id'] == '1'
-        end
+        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
 

data/test/functional/personal_access_token_test.rb

@@ -0,0 +1,123 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'personal_access_token' do
+  let(:base_cmd) { ['user', 'access-token'] }
+  let(:user) do
+    {
+     :id => 1,
+     :name => 'admin'
+    }
+  end
+  let(:access_token) do
+    {
+      :id => 1,
+      :user_id => user[:id],
+      :name => 'test',
+      :created_at => '01/10/2016',
+      :expires_at => '01/02/2017',
+      :last_used_at => '24/12/2016',
+      :active? => false
+    }
+  end
+  let(:active_access_token) do
+    {
+      :id => 2,
+      :user_id => user[:id],
+      :name => 'test2',
+      :created_at => '01/10/2016',
+      :expires_at => '01/02/2048',
+      :last_used_at => '01/02/2018',
+      :active? => true
+    }
+  end
+
+  describe 'list' do
+    let(:cmd) { base_cmd << 'list' }
+    let(:params) { ['--user-id=1'] }
+
+    it 'lists all access tokens for a given user' do
+      api_expects(:personal_access_tokens, :index, 'List').with_params(
+        'user_id' => 1, 'page' => 1, 'per_page' => 1000
+      ).returns(index_response([access_token, active_access_token]))
+
+      output = IndexMatcher.new([
+        ['ID', 'NAME',  'ACTIVE', 'EXPIRES AT'],
+        ['1',  'test',  'no',     '2017/02/01 00:00:00' ],
+        ['2',  'test2', 'yes',    '2048/02/01 00:00:00' ]
+      ])
+      expected_result = success_result(output)
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'create' do
+    let(:cmd) { base_cmd << 'create' }
+    let(:params) { ['--user-id=1', '--expires-at=01/01/2048', '--name=test'] }
+    let(:access_token) do
+      {
+        :id => 1,
+        :user_id => user[:id],
+        :name => 'test',
+        :expires_at => '01/01/2048',
+        :token_value => 'value'
+      }
+    end
+
+    it 'creates an access token to a given user' do
+      api_expects(:personal_access_tokens, :create).with_params(
+        'user_id' => 1, 'personal_access_token' => {
+          'expires_at' => '01/01/2048', 'name' => 'test'
+        }
+      ).returns(access_token)
+
+      expected_result = success_result(/#{access_token[:value]}/)
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'info' do
+    let(:cmd) { base_cmd << 'info' }
+    let(:params) { ['--id=1', '--user-id=1'] }
+
+    it 'shows the personal access token' do
+      api_expects(:personal_access_tokens, :show, 'Show PAT').with_params(
+        'id' => '1', 'user_id' => 1
+      ).returns(access_token)
+
+      output = OutputMatcher.new([
+        "Id:           1",
+        "Name:         test",
+        "Active:       no",
+        "Expires at:   2017/02/01 00:00:00",
+        "Created at:   2016/10/01 00:00:00",
+        "Last used at: 2016/12/24 00:00:00",
+      ])
+
+      expected_result = success_result(output)
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'revoke' do
+    let(:cmd) { base_cmd << 'revoke' }
+    let(:params) { ['--id=1', '--user-id=1'] }
+    it 'deletes an access token to a given user' do
+      api_expects(:personal_access_tokens, :destroy, 'Revoke PAT').with_params(
+        'id' => '1', 'user_id' => 1
+      ).returns(access_token)
+
+      expected_result = success_result(
+        "Personal access token [#{access_token[:name]}] revoked.\n"
+      )
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    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/test_helper.rb

@@ -17,7 +17,7 @@ require "mocha/setup"
 require 'hammer_cli'
 require 'hammer_cli_foreman/testing/api_expectations'
 
-FOREMAN_VERSION = Gem::Version.new(ENV['TEST_API_VERSION'] || '1.15')
+FOREMAN_VERSION = Gem::Version.new(ENV['TEST_API_VERSION'] || '1.18')
 
 include HammerCLIForeman::Testing::APIExpectations
 HammerCLI.context[:api_connection].create('foreman') do

data/test/unit/audit_test.rb

@@ -0,0 +1,60 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+require 'hammer_cli_foreman/audit'
+
+describe HammerCLIForeman::Audit do
+  include CommandTestHelper
+
+  context "ListCommand" do
+    before do
+      ResourceMocks.mock_action_call(:audits, :index, [])
+    end
+
+    let(:cmd) { HammerCLIForeman::Audit::ListCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "no arguments"
+      it_should_accept_search_params
+    end
+
+    context "output" do
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
+
+      it_should_print_n_records
+      it_should_print_column "Id"
+      it_should_print_column "At"
+      it_should_print_column "IP"
+      it_should_print_column "User"
+      it_should_print_column "Action"
+      it_should_print_column "Audit type"
+      it_should_print_column "Audit record"
+    end
+  end
+
+  context "InfoCommand" do
+    before do
+        cmd.stubs(:extend_data)
+    end
+
+    let(:cmd) { HammerCLIForeman::Audit::InfoCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+    end
+
+    context "output" do
+      with_params ["--id=1"] do
+        it_should_print_n_records 1
+        it_should_print_column "Id"
+        it_should_print_column "At"
+        it_should_print_column "IP"
+        it_should_print_column "User"
+        it_should_print_column "Action"
+        it_should_print_column "Audit type"
+        it_should_print_column "Audit record"
+        it_should_print_column "Audited changes"
+      end
+    end
+  end
+end