foreman_bootdisk 18.1.0 → 19.0.3

data/test/models/host/managed_test.rb

@@ -1,26 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-module Host
-  class ManagedTest < ActiveSupport::TestCase
-    include ForemanBootdiskTestHelper
-
-    setup do
-      User.current = users(:admin)
-      setup_bootdisk
-    end
-
-    context 'with host' do
-      let(:host) { FactoryBot.create(:host, :managed, :with_subnet, build: true) }
-
-      test 'finds the bootdisk_template specified in settings' do
-        assert_kind_of ProvisioningTemplate, host.bootdisk_template
-      end
-
-      test 'renders the host bootdisk template' do
-        assert_includes host.bootdisk_template_render, 'loop_success'
-      end
-    end
-  end
-end

data/test/test_plugin_helper.rb

@@ -1,72 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_helper'
-require 'webmock/minitest'
-require 'webmock'
-
-module ForemanBootdiskTestHelper
-  def create_tempfile
-    file = Tempfile.new('bootdisk-test', '/tmp')
-    file.close
-    file
-  end
-
-  def setup_bootdisk
-    ForemanBootdisk::Scope::Bootdisk.any_instance.stubs(:bootdisk_chain_url).returns('http://smart-proxy.example.com/unattended/iPXE?mac=')
-    setup_routes
-    setup_settings
-    setup_templates
-  end
-
-  # TODO remove me
-  def setup_host_env
-    setup_org_loc
-    setup_subnet_with_tftp
-    setup_host
-  end
-
-  def setup_routes
-    @routes = ForemanBootdisk::Engine.routes
-  end
-
-  def setup_settings
-    Setting::Bootdisk.load_defaults
-  end
-
-  def setup_templates
-    load File.join(File.dirname(__FILE__), '..', 'db', 'seeds.d', '50-bootdisk_templates.rb')
-  end
-
-  def setup_referer
-    request.env['HTTP_REFERER'] = '/history'
-  end
-
-  def setup_org_loc
-    disable_orchestration
-    @org = FactoryBot.create(:organization)
-    @loc = FactoryBot.create(:location)
-  end
-
-  def setup_subnet_with_tftp_httpboot_template
-    ProxyAPI::V2::Features.any_instance.stubs(:features).returns(:httpboot => { :settings => { :https_port => 1234 }, :state => 'running' }, :templates => {}, :tftp => {})
-    SmartProxy.any_instance.stubs(:httpboot_http_port).returns('8888')
-    SmartProxy.any_instance.stubs(:httpboot_https_port).returns('9999')
-
-    proxy = FactoryBot.create(:smart_proxy, features: [FactoryBot.create(:tftp_feature), FactoryBot.create(:feature, :templates), FactoryBot.create(:feature, :httpboot)])
-    proxy.reload
-    setup_subnet_no_tftp.update! tftp: proxy, httpboot: proxy, template: proxy
-  end
-
-  def setup_subnet_with_tftp
-    tftp_proxy = FactoryBot.create(:smart_proxy, features: [FactoryBot.create(:tftp_feature)])
-    setup_subnet_no_tftp.update! tftp: tftp_proxy
-  end
-
-  def setup_subnet_no_tftp
-    @subnet = FactoryBot.create(:subnet_ipv4, gateway: '10.0.1.254', dns_primary: '8.8.8.8', organizations: [@org], locations: [@loc])
-  end
-
-  def setup_host
-    @host = FactoryBot.create(:host, :managed, subnet: @subnet, ip: @subnet.network.sub(/0$/, '4'), organization: @org, location: @loc, build: true)
-  end
-end

data/test/unit/access_permissions_test.rb

@@ -1,10 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_helper'
-require 'unit/shared/access_permissions_test_base'
-
-class AccessPermissionsTest < ActiveSupport::TestCase
-  include AccessPermissionsTestBase
-
-  check_routes(ForemanBootdisk::Engine.routes, [])
-end

data/test/unit/concerns/compute_resources/vmware_test.rb

@@ -1,83 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-module ForemanBootdisk
-  class VmwareTest < ActiveSupport::TestCase
-    describe '#create_vm' do
-      setup do
-        @cr = FactoryBot.build(:vmware_cr)
-        @cr.stubs(:test_connection)
-      end
-
-      test 'does not call clone_vm when bootdisk provisioning' do
-        args = { 'provision_method' => 'bootdisk' }
-        mock_vm = mock('vm')
-        mock_vm.stubs(:firmware)
-        mock_vm.expects(:save).returns(mock_vm)
-        @cr.stubs(:parse_networks).returns(args)
-        @cr.expects(:clone_vm).times(0)
-        @cr.expects(:new_vm).returns(mock_vm)
-        @cr.create_vm(args)
-      end
-    end
-
-    describe '#new_vm' do
-      setup do
-        @cr = FactoryBot.build(:vmware_cr)
-      end
-
-      test 'calls client with cdrom drive and correct boot order when bootdisk provisioning' do
-        args = { 'provision_method' => 'bootdisk' }
-        mock_client = mock('client')
-        mock_servers = mock('servers')
-        mock_cdrom = mock('cdrom')
-        mock_client.expects(:servers).returns(mock_servers)
-        mock_servers.expects(:new).with do |opts|
-          assert_equal opts[:boot_order], %w[cdrom disk]
-          assert_equal opts[:boot_retry], 10
-          assert_includes opts[:cdroms], mock_cdrom
-        end
-        @cr.expects(:new_cdrom).returns(mock_cdrom)
-        @cr.expects(:new_interface)
-        @cr.expects(:new_volume)
-        @cr.expects(:datacenter)
-        @cr.expects(:client).returns(mock_client)
-        @cr.new_vm(args)
-      end
-    end
-
-    describe '#parse_args' do
-      setup do
-        @cr = FactoryBot.build(:vmware_cr)
-      end
-
-      test 'should add a cdrom drive while keeping other parameters when provision_method is bootdisk' do
-        mock_cdrom = mock('cdrom')
-        @cr.expects(:new_cdrom).returns(mock_cdrom)
-        attrs_in = HashWithIndifferentAccess.new(
-          'cpus' => '1',
-          :provision_method => 'bootdisk'
-        )
-        attrs_out = {
-          cpus: '1',
-          provision_method: 'bootdisk',
-          cdroms: [mock_cdrom],
-          boot_order: %w[cdrom disk],
-          boot_retry: 10
-        }
-        assert_equal attrs_out, @cr.parse_args(attrs_in)
-      end
-    end
-
-    describe '#capabilities' do
-      setup do
-        @cr = FactoryBot.build(:vmware_cr)
-      end
-
-      test 'should include bootdisk' do
-        assert_includes @cr.capabilities, :bootdisk
-      end
-    end
-  end
-end

data/test/unit/concerns/host_test.rb

@@ -1,84 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-module ForemanBootdisk
-  class HostTest < ActiveSupport::TestCase
-    test '#bootdisk_build? must be true when provision_method is bootdisk' do
-      host = FactoryBot.build(:host, :managed)
-      host.provision_method = 'bootdisk'
-      assert host.bootdisk_build?
-      assert_not host.image_build?
-      assert_not host.pxe_build?
-    end
-
-    test '#validate_media? must be true when provision_method is bootdisk' do
-      host = FactoryBot.build(:host, :managed,
-                              provision_method: 'bootdisk',
-                              build: true)
-      assert host.validate_media?
-    end
-
-    test '#can_be_built? must be true when provision_method is bootdisk' do
-      host = FactoryBot.build(:host, :managed,
-                              provision_method: 'bootdisk')
-      assert host.can_be_built?
-    end
-
-    test 'host should have bootdisk' do
-      if unattended?
-        h = FactoryBot.build(:host, :managed,
-                             provision_method: 'bootdisk')
-        assert h.bootdisk?
-      end
-    end
-
-    test 'host should not have bootdisk' do
-      if unattended?
-        h = FactoryBot.create(:host)
-        assert_not(h.bootdisk?)
-      end
-    end
-
-    context '#bootdisk_downloadable?' do
-      test 'should be true for 64 bit architecture' do
-        architecture = Architecture.where(name: 'x86_64').first
-        host = FactoryBot.build(:host, :managed, architecture: architecture)
-
-        assert host.bootdisk_downloadable?
-      end
-
-      test 'should be true for 32 bit architecture' do
-        architecture = FactoryBot.create(:architecture, name: 'i386')
-        host = FactoryBot.build(:host, :managed, architecture: architecture)
-
-        assert host.bootdisk_downloadable?
-      end
-
-      test 'should be false for non-intel architecture' do
-        architecture = Architecture.where(name: 's390').first
-        host = FactoryBot.build(:host, :managed, architecture: architecture)
-
-        assert_not host.bootdisk_downloadable?
-      end
-
-      test 'should be true if architecture is absent' do
-        host = FactoryBot.build(:host, :managed, architecture: nil)
-
-        assert_nil host.architecture
-        assert host.bootdisk_downloadable?
-      end
-    end
-
-    context '#bootdisk_build?' do
-      test 'should be false for hosts without bootdisk' do
-        host = FactoryBot.create(:host)
-        assert_not host.bootdisk_build?
-      end
-
-      test 'should be available in safe mode' do
-        assert Host::Managed::Jail.allowed?(:bootdisk_build?)
-      end
-    end
-  end
-end

data/test/unit/concerns/orchestration/compute_test.rb

@@ -1,69 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-module ForemanBootdisk
-  class OrchestrationComputeTest < ActiveSupport::TestCase
-    setup do
-      disable_orchestration
-      @cr = FactoryBot.build(:vmware_cr)
-      @host = FactoryBot.build(:host, :managed,
-                               compute_resource: @cr,
-                               provision_method: 'bootdisk')
-    end
-
-    test 'provisioning a host with provision method bootdisk should upload iso' do
-      @cr.expects(:iso_upload)
-      @host.send(:setIsoImage)
-    end
-
-    test 'provisioning a host with provision method bootdisk should attach iso' do
-      @cr.expects(:iso_attach)
-      @host.send(:setAttachIsoImage)
-    end
-
-    test 'provisioning a host with provision method bootdisk should detach iso' do
-      @cr.expects(:iso_detach)
-      @host.send(:setDetachIsoImage)
-    end
-
-    test 'provisioning a new host with provision method bootdisk should queue bootdisk tasks' do
-      @host.stubs(:compute?).returns(true)
-      @host.stubs(:build?).returns(true)
-      @host.send(:queue_bootdisk_compute)
-      tasks = @host.queue.all.map(&:name)
-      assert_includes tasks, "Generating ISO image for #{@host.name}"
-      assert_includes tasks, "Upload ISO image to datastore for #{@host.name}"
-      assert_includes tasks, "Attach ISO image to CDROM drive for #{@host.name}"
-      assert_not_includes tasks, "Detach ISO image from CDROM drive for #{@host.name}"
-    end
-
-    test 'rebuilding a host with provision method bootdisk should queue bootdisk tasks' do
-      @host.stubs(:compute?).returns(true)
-      old = stub()
-      old.stubs(:build?).returns(false)
-      @host.stubs(:old).returns(old)
-      @host.stubs(:build?).returns(true)
-      @host.send(:queue_bootdisk_compute)
-      tasks = @host.queue.all.map(&:name)
-      assert_includes tasks, "Generating ISO image for #{@host.name}"
-      assert_includes tasks, "Upload ISO image to datastore for #{@host.name}"
-      assert_includes tasks, "Attach ISO image to CDROM drive for #{@host.name}"
-      assert_not_includes tasks, "Detach ISO image from CDROM drive for #{@host.name}"
-    end
-
-    test 'the iso should be detached when the host leaves build mode' do
-      @host.stubs(:compute?).returns(true)
-      old = stub()
-      old.stubs(:build?).returns(true)
-      @host.stubs(:old).returns(old)
-      @host.stubs(:build?).returns(false)
-      @host.send(:queue_bootdisk_compute)
-      tasks = @host.queue.all.map(&:name)
-      assert_not_includes tasks, "Generating ISO image for #{@host.name}"
-      assert_not_includes tasks, "Upload ISO image to datastore for #{@host.name}"
-      assert_not_includes tasks, "Attach ISO image to CDROM drive for #{@host.name}"
-      assert_includes tasks, "Detach ISO image from CDROM drive for #{@host.name}"
-    end
-  end
-end

data/test/unit/foreman_bootdisk/renderer_test.rb

@@ -1,36 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-module ForemanBootdisk
-  class RendererTest < ActiveSupport::TestCase
-    include ForemanBootdiskTestHelper
-
-    setup do
-      User.current = users(:admin)
-      setup_bootdisk
-    end
-
-    let(:renderer) { ForemanBootdisk::Renderer.new }
-
-    describe 'a generic template' do
-      test 'can be rendered' do
-        rendered_template = renderer.generic_template_render
-        assert_includes rendered_template, 'ifstat'
-      end
-
-      test 'does not include a host token' do
-        rendered_template = renderer.generic_template_render
-        assert_includes rendered_template, 'http://smart-proxy.example.com/unattended/iPXE?mac=${net0/mac}'
-        assert_not_includes rendered_template, 'token'
-      end
-    end
-
-    context 'with a subnet' do
-      let(:subnet) { FactoryBot.create(:subnet_ipv4) }
-      test 'renders a subnet template' do
-        assert_includes renderer.generic_template_render(subnet), 'ifstat'
-      end
-    end
-  end
-end

data/test/unit/foreman_bootdisk/scope/bootdisk_test.rb

@@ -1,31 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-module ForemanBootdisk
-  module Scope
-    class BootdiskTest < ActiveSupport::TestCase
-      let(:host) { FactoryBot.build(:host, :managed) }
-      let(:source) { Foreman::Renderer::Source::String.new(content: 'Test') }
-      let(:scope) { ForemanBootdisk::Scope::Bootdisk.new(host: host, source: source) }
-
-      describe '#bootdisk_chain_url' do
-        test 'should render bootdisk chain url' do
-          assert_equal "http://foreman.some.host.fqdn/unattended/iPXE?mac=#{URI.encode_www_form_component(host.mac)}", scope.bootdisk_chain_url
-        end
-
-        test 'should render bootdisk chain url with custom mac' do
-          assert_equal 'http://foreman.some.host.fqdn/unattended/iPXE?mac=00%3A11%3A22%3A33%3A44%3A55', scope.bootdisk_chain_url('00:11:22:33:44:55')
-        end
-      end
-
-      describe '#bootdisk_raise' do
-        test 'should raise a Foreman::Exception' do
-          assert_raise ::Foreman::Exception do
-            scope.bootdisk_raise('Some error.')
-          end
-        end
-      end
-    end
-  end
-end

data/test/unit/foreman_bootdisk/scope/full_host_bootdisk_test.rb

@@ -1,30 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-module ForemanBootdisk
-  module Scope
-    class BootdiskTest < ActiveSupport::TestCase
-      let(:operatingsystem) { FactoryBot.create(:ubuntu14_10, :with_media, :with_archs) }
-      let(:host) { FactoryBot.build(:host, :managed, operatingsystem: operatingsystem) }
-      let(:source) { Foreman::Renderer::Source::String.new(content: 'Test') }
-      let(:scope) { ForemanBootdisk::Scope::FullHostBootdisk.new(host: host, source: source) }
-
-      setup do
-        MediumProviders::Default.any_instance.stubs(:unique_id).returns('MyMedium01-ZYHBD6OPET')
-      end
-
-      describe '@kernel' do
-        test 'should match filename on bootdisk' do
-          assert_equal 'BOOT/MYMEDIUM01_ZYHBD6OPET_LINUX', scope.instance_variable_get('@kernel')
-        end
-      end
-
-      describe '@initrd' do
-        test 'should match filename on bootdisk' do
-          assert_equal 'BOOT/EDIUM01_ZYHBD6OPET_INITRD_GZ', scope.instance_variable_get('@initrd')
-        end
-      end
-    end
-  end
-end

data/test/unit/iso_generator_test.rb

@@ -1,74 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-module ForemanBootdisk
-  class IsoGeneratorTest < ActiveSupport::TestCase
-    include ForemanBootdiskTestHelper
-    setup :setup_bootdisk
-
-    describe '#generate_full_host' do
-      let(:medium) { FactoryBot.create(:medium, name: 'Red Hat Enterprise Linux Atomic Mirror') }
-      let(:operatingsystem) { FactoryBot.create(:ubuntu14_10, :with_archs, :with_ptables, media: [medium]) }
-      let(:host) { FactoryBot.create(:host, :managed, operatingsystem: operatingsystem, build: true) }
-      let(:pxelinux_template) { FactoryBot.create(:provisioning_template, template: 'Fake kernel line <%= @kernel %> - <%= @initrd %>') }
-      let(:pxegrub2_template) { FactoryBot.create(:provisioning_template, template: 'Fake kernel line <%= @kernel %> - <%= @initrd %>') }
-
-      setup do
-        host.stubs(:provisioning_template).with(kind: :PXELinux).returns(pxelinux_template)
-        host.stubs(:provisioning_template).with(kind: :PXEGrub2).returns(pxegrub2_template)
-      end
-
-      test 'fetch handles redirect' do
-        Dir.mktmpdir do |dir|
-          url = 'http://example.com/request'
-          redirection = 'http://example.com/redirect'
-          stub_request(:get, url).to_return(status: 301, headers: { 'Location' => redirection })
-          stub_request(:get, redirection)
-          ForemanBootdisk::ISOGenerator.fetch(File.join(dir, 'test'), url)
-        end
-      end
-
-      test 'generate_full_host creates with ISO-compatible file names' do
-        urls = host.operatingsystem.boot_file_sources(host.medium_provider)
-
-        kernel = ForemanBootdisk::ISOGenerator.iso9660_filename(
-          host.operatingsystem.kernel(host.medium_provider)
-        )
-        kernel_url = urls[:kernel]
-
-        initrd = ForemanBootdisk::ISOGenerator.iso9660_filename(
-          host.operatingsystem.initrd(host.medium_provider)
-        )
-        initrd_url = urls[:initrd]
-
-        ForemanBootdisk::ISOGenerator.expects(:generate)
-                                     .with({ isolinux: "Fake kernel line #{kernel} - #{initrd}", grub: "Fake kernel line /#{kernel} - /#{initrd}", files: { kernel => kernel_url, initrd => initrd_url } }, anything)
-
-        ForemanBootdisk::ISOGenerator.generate_full_host(host)
-      end
-    end
-
-    describe '#generate' do
-      test 'generates an iso image' do
-        ForemanBootdisk::ISOGenerator.expects(:system).with(
-          regexp_matches(/genisoimage -o .*output.iso -iso-level 2 -b isolinux.bin -c boot.cat -no-emul-boot -boot-load-size 4 -boot-info-table .*build/)
-        ).returns(true)
-        ForemanBootdisk::ISOGenerator.expects(:system).with('isohybrid', anything).returns(true)
-        ForemanBootdisk::ISOGenerator.generate do |iso|
-          assert_not_nil iso
-        end
-      end
-    end
-
-    describe '#iso9660_filename' do
-      test 'converts path to iso9660' do
-        assert_equal 'BOOT/SOME_FILE_N_A_M_E123_', ForemanBootdisk::ISOGenerator.iso9660_filename('boot/some-File-n_a_m_e123Ä')
-      end
-
-      test 'shortens long filenames' do
-        assert_equal 'BOOT/RPRISELINUXATOMIC_7_3_X86_64', ForemanBootdisk::ISOGenerator.iso9660_filename('boot/RedHatEnterpriseLinuxAtomic-7.3-x86_64')
-      end
-    end
-  end
-end