hammer_cli_foreman 3.1.0 → 3.4.0

data/test/functional/domain/create_test.rb

@@ -0,0 +1,91 @@
+require_relative '../test_helper'
+
+describe 'Domain' do
+  describe 'CreateCommand' do
+    let(:cmd) { %w[domain create] }
+    let(:minimal_params) { %w[--name=dom1.com] }
+
+    def domain_params(additional_params = {})
+      params = {
+        domain: {
+          name: 'dom1.com'
+        }
+      }
+      params[:domain].merge!(additional_params)
+      params
+    end
+
+    it 'should print error on missing --name' do
+      expected_result = missing_args_error_result(cmd, '--name')
+
+      api_expects_no_call
+
+      result = run_cmd(cmd)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'allows minimal options' do
+      api_expects(:domains, :create).with_params(domain_params)
+
+      run_cmd(cmd + minimal_params)
+    end
+
+    it 'allows description' do
+      params = %w[--description=shortdesc]
+      api_expects(:domains, :create).with_params(domain_params(fullname: 'shortdesc'))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+
+    it 'allows dns id' do
+      params = %w[--dns-id=1]
+      api_expects(:domains, :create).with_params(domain_params(dns_id: 1))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+
+    it 'allows dns name' do
+      params = %w[--dns=sp1]
+      api_expects_search(:smart_proxies, { name: 'sp1' }).returns(
+        index_response([{ 'id' => 1 }])
+      )
+      api_expects(:domains, :create).with_params(domain_params(dns_id: 1))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+
+    it 'allows location ids' do
+      params = %w[--location-ids=1,4]
+      api_expects(:domains, :create).with_params(domain_params(location_ids: %w[1 4]))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+
+    it 'allows location names' do
+      params = %w[--locations=loc1,loc2]
+      api_expects(:locations, :index) do |p|
+        p[:search] == 'name = "loc1" or name = "loc2"'
+      end.returns(index_response([{ 'id' => 1 }, { 'id' => 2 }]))
+      api_expects(:domains, :create).with_params(domain_params(location_ids: [1, 2]))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+
+    it 'allows organization ids' do
+      params = %w[--organization-ids=1,4]
+      api_expects(:domains, :create).with_params(domain_params(organization_ids: %w[1 4]))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+
+    it 'allows organization names' do
+      params = %w[--organizations=org1,org2]
+      api_expects(:organizations, :index) do |p|
+        p[:search] == 'name = "org1" or name = "org2"'
+      end.returns(index_response([{ 'id' => 1 }, { 'id' => 2 }]))
+      api_expects(:domains, :create).with_params(domain_params(organization_ids: [1, 2]))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+  end
+end

data/test/functional/domain/update_test.rb

@@ -0,0 +1,90 @@
+require_relative '../test_helper'
+
+describe 'Domain' do
+  describe 'UpdateCommand' do
+    let(:cmd) { %w[domain update] }
+    let(:minimal_params) { %w[--id=1] }
+
+    def domain_params(additional_params = {})
+      params = {
+        id: '1',
+        domain: {}
+      }
+      params[:domain].merge!(additional_params)
+      params
+    end
+
+    it 'should print error on missing --id' do
+      expected_result = missing_args_error_result(cmd, '--id')
+
+      api_expects_no_call
+
+      result = run_cmd(cmd)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'allows minimal options' do
+      api_expects(:domains, :update).with_params(domain_params)
+
+      run_cmd(cmd + minimal_params)
+    end
+
+    it 'allows description' do
+      params = %w[--description=shortdesc]
+      api_expects(:domains, :update).with_params(domain_params(fullname: 'shortdesc'))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+
+    it 'allows dns id' do
+      params = %w[--dns-id=1]
+      api_expects(:domains, :update).with_params(domain_params(dns_id: 1))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+
+    it 'allows dns name' do
+      params = %w[--dns=sp1]
+      api_expects_search(:smart_proxies, { name: 'sp1' }).returns(
+        index_response([{ 'id' => 1 }])
+      )
+      api_expects(:domains, :update).with_params(domain_params(dns_id: 1))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+
+    it 'allows location ids' do
+      params = %w[--location-ids=1,4]
+      api_expects(:domains, :update).with_params(domain_params(location_ids: %w[1 4]))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+
+    it 'allows location names' do
+      params = %w[--locations=loc1,loc2]
+      api_expects(:locations, :index) do |p|
+        p[:search] == 'name = "loc1" or name = "loc2"'
+      end.returns(index_response([{ 'id' => 1 }, { 'id' => 2 }]))
+      api_expects(:domains, :update).with_params(domain_params(location_ids: [1, 2]))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+
+    it 'allows organization ids' do
+      params = %w[--organization-ids=1,4]
+      api_expects(:domains, :update).with_params(domain_params(organization_ids: %w[1 4]))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+
+    it 'allows organization names' do
+      params = %w[--organizations=org1,org2]
+      api_expects(:organizations, :index) do |p|
+        p[:search] == 'name = "org1" or name = "org2"'
+      end.returns(index_response([{ 'id' => 1 }, { 'id' => 2 }]))
+      api_expects(:domains, :update).with_params(domain_params(organization_ids: [1, 2]))
+
+      run_cmd(cmd + minimal_params + params)
+    end
+  end
+end

data/test/functional/partition_table_test.rb

@@ -0,0 +1,63 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'partition-table' do
+  describe 'import' do
+    let(:template) do
+      {
+        'id' => 1,
+        'template' => 'Template content'
+      }
+    end
+    let(:cmd) { %w(partition-table import) }
+    let(:tempfile) { Tempfile.new('template') }
+
+    it 'requires --name and --file' do
+      params = ['--name=test']
+      api_expects_no_call
+      expected_result = usage_error_result(
+        cmd,
+        'Options --name, --file are required.',
+        'Could not import partition table template')
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'import template' do
+      params = ['--name=test', "--file=#{tempfile.path}"]
+      tempfile.write('Template content')
+      tempfile.rewind
+      api_expects(:ptables, :import, 'Import partition table template').with_params(
+        'ptable' => {
+          'name' => 'test',
+          'template' => 'Template content'
+        }).returns(template)
+
+      result = run_cmd(cmd + params)
+      assert_cmd(success_result("Import partition table template succeeded.\n"), result)
+    end
+  end
+
+  describe 'export' do
+    let(:cmd) { %w(partition-table export) }
+    let(:tempfile) { Tempfile.new('template', '/tmp') }
+    let(:params) { ['--id=1', '--path=/tmp'] }
+    let(:template_response) do
+      response = mock('TemplateResponse')
+      response.stubs(:code).returns(200)
+      response.stubs(:body).returns('Template content')
+      response.stubs(:headers).returns({:content_disposition => "filename=\"#{File.basename(tempfile.path)}\""})
+      response
+    end
+
+    it 'download template' do
+      api_expects(:ptables, :export, 'Export partition table template').with_params(
+        'id' => '1').returns(template_response)
+
+      output = OutputMatcher.new("The partition table template has been saved to #{tempfile.path}")
+      expected_result = success_result(output)
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+      assert_equal('Template content', tempfile.read)
+    end
+  end
+end

data/test/functional/report_template_test.rb

@@ -384,6 +384,30 @@ describe 'report-template' do
     end
   end
 
+  describe 'export' do
+    let(:cmd) { %w(report-template export) }
+    let(:tempfile) { Tempfile.new('template', '/tmp') }
+    let(:params) { ['--id=1', '--path=/tmp'] }
+    let(:template_response) do
+      response = mock('TemplateResponse')
+      response.stubs(:code).returns(200)
+      response.stubs(:body).returns('Template content')
+      response.stubs(:headers).returns({:content_disposition => "filename=\"#{File.basename(tempfile.path)}\""})
+      response
+    end
+
+    it 'download template' do
+      api_expects(:report_templates, :export, 'Export report template').with_params(
+        'id' => '1').returns(template_response)
+
+      output = OutputMatcher.new("The report template has been saved to #{tempfile.path}")
+      expected_result = success_result(output)
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+      assert_equal('Template content', tempfile.read)
+    end
+  end
+
   describe 'report-data' do
     let(:cmd) { %w(report-template report-data) }
     let(:tempfile) { Tempfile.new('template', '/tmp') }

data/test/functional/table_preference_test.rb

@@ -0,0 +1,100 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'table_preference' do
+  let(:base_cmd) { ['user', 'table-preference'] }
+  let(:user) do
+    {
+      :id => 1,
+      :name => 'admin'
+    }
+  end
+  let(:table_preference) do
+    {
+      :id => 1,
+      :user_id => user[:id],
+      :name => 'Table Preference',
+      :columns => ['A', 'B', 'C']
+    }
+  end
+
+  describe 'list' do
+    let(:cmd) { base_cmd << 'list' }
+
+    it 'lists table preferences for a given user' do
+      params = ['--user-id', user[:id]]
+      api_expects(:table_preferences, :index, params)
+        .returns(index_response([table_preference]))
+
+      expected_result = success_result(/#{table_preference[:name]}/)
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'create' do
+    let(:cmd) { base_cmd << 'create' }
+
+    it 'creates a table preference for a given table' do
+      params = ['--user-id', user[:id],
+                '--name', table_preference[:name],
+                '--columns', table_preference[:columns]]
+      api_expects(:table_preferences, :create, params)
+        .returns(table_preference)
+
+      expected_result = success_result("Table preference created.\n")
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'update' do
+    let(:cmd) { base_cmd << 'update' }
+
+    it 'updates a table preference for a given table' do
+      params = ['--user-id', user[:id],
+                '--name', table_preference[:name],
+                '--columns', ['A','B','C','D','E']]
+      api_expects(:table_preferences, :update, params)
+        .returns(table_preference)
+
+      expected_result = success_result("Table preference updated.\n")
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'info' do
+    let(:cmd) { base_cmd << 'info' }
+
+    it 'shows table preference details of a given table' do
+      params = ['--user-id', user[:id],
+                '--name', table_preference[:name]]
+      api_expects(:table_preferences, :show, params)
+        .returns(table_preference)
+
+      expected_result = success_result(/#{table_preference[:name]} | #{table_preference[:columns]}/)
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'delete' do
+    let(:cmd) { base_cmd << 'delete' }
+
+    it 'deletes a table preference for a given table' do
+      params = ['--user-id', user[:id],
+                '--name', table_preference[:name]]
+      api_expects(:table_preferences, :destroy, params)
+        .returns({})
+
+      expected_result = success_result("Table preference deleted.\n")
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+end

data/test/functional/template_test.rb

@@ -77,6 +77,66 @@ describe 'template' do
     end
   end
 
+  describe 'import' do
+    let(:template) do
+      {
+        'id' => 1,
+        'template' => 'Template content'
+      }
+    end
+    let(:cmd) { %w(template import) }
+    let(:tempfile) { Tempfile.new('template') }
+
+    it 'requires --name and --file' do
+      params = ['--name=test']
+      api_expects_no_call
+      expected_result = usage_error_result(
+        cmd,
+        'Options --name, --file are required.',
+        'Could not import provisioning template')
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'import template' do
+      params = ['--name=test', "--file=#{tempfile.path}"]
+      tempfile.write('Template content')
+      tempfile.rewind
+      api_expects(:provisioning_templates, :import, 'Import template').with_params(
+        'provisioning_template' => {
+          'name' => 'test',
+          'template' => 'Template content'
+        }).returns(template)
+
+      result = run_cmd(cmd + params)
+      assert_cmd(success_result("Import provisioning template succeeded.\n"), result)
+    end
+  end
+
+  describe 'export' do
+    let(:cmd) { %w(template export) }
+    let(:tempfile) { Tempfile.new('template', '/tmp') }
+    let(:params) { ['--id=1', '--path=/tmp'] }
+    let(:template_response) do
+      response = mock('TemplateResponse')
+      response.stubs(:code).returns(200)
+      response.stubs(:body).returns('Template content')
+      response.stubs(:headers).returns({:content_disposition => "filename=\"#{File.basename(tempfile.path)}\""})
+      response
+    end
+
+    it 'download template' do
+      api_expects(:provisioning_templates, :export, 'Export template').with_params(
+        'id' => '1').returns(template_response)
+
+      output = OutputMatcher.new("The provisioning template has been saved to #{tempfile.path}")
+      expected_result = success_result(output)
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+      assert_equal('Template content', tempfile.read)
+    end
+  end
+
   describe 'update' do
     before do
       @cmd = %w(template update)

data/test/test_helper.rb

@@ -17,7 +17,7 @@ require "mocha/minitest"
 require 'hammer_cli'
 require 'hammer_cli_foreman/testing/api_expectations'
 
-FOREMAN_VERSION = ENV['TEST_API_VERSION'] || '3.1'
+FOREMAN_VERSION = ENV['TEST_API_VERSION'] || '3.4'
 unless Dir.entries('test/data').include? FOREMAN_VERSION
   raise StandardError.new "Version is not correct"
 end