foreman_bootdisk 18.1.0 → 19.0.3

data/webpack/src/extensions/host/HostBootdiskButtonsSelectors.js

@@ -0,0 +1,27 @@
+import { STATUS } from 'foremanReact/constants';
+import {
+  selectAPIStatus,
+  selectAPIResponse,
+} from 'foremanReact/redux/API/APISelectors';
+
+import {
+  HOST_BOOTDISK_BUTTONS_REQUEST_KEY,
+  HOST_DETAILS_REQUEST_KEY,
+} from '../constants';
+
+const selectHostButtonsResponse = state =>
+  selectAPIResponse(state, HOST_BOOTDISK_BUTTONS_REQUEST_KEY) || {};
+
+export const selectBootdiskOptions = state =>
+  selectHostButtonsResponse(state).bootdiskOptions || {};
+
+const selectHostButtonsStatus = state =>
+  selectAPIStatus(state, HOST_BOOTDISK_BUTTONS_REQUEST_KEY);
+
+export const selectIsLoading = state => {
+  const status = selectHostButtonsStatus(state);
+  return !status || status === STATUS.PENDING;
+};
+
+export const selectHostId = state =>
+  selectAPIResponse(state, HOST_DETAILS_REQUEST_KEY).id;

data/webpack/src/extensions/host/__tests__/HostBootdiskButtonsSelectors.test.js

@@ -0,0 +1,66 @@
+import { testSelectorsSnapshotWithFixtures } from '@theforeman/test';
+
+import {
+  selectHostId,
+  selectIsLoading,
+  selectBootdiskOptions,
+} from '../HostBootdiskButtonsSelectors';
+
+import {
+  HOST_DETAILS_REQUEST_KEY,
+  HOST_BOOTDISK_BUTTONS_REQUEST_KEY,
+} from '../../constants';
+
+const stateFactory = (bootdiskState, hostState) => ({
+  API: {
+    [HOST_DETAILS_REQUEST_KEY]: {
+      response: {
+        ...hostState,
+      },
+    },
+    [HOST_BOOTDISK_BUTTONS_REQUEST_KEY]: {
+      response: {
+        ...bootdiskState,
+      },
+    },
+  },
+});
+
+const state = stateFactory(
+  {
+    bootdiskOptions: {
+      bootdiskDownloadable: true,
+      architectureName: 'x86_64',
+      actions: [
+        {
+          title: "Host 'sppup' image",
+          link: '/bootdisk/disks/hosts/8',
+          disabled: false,
+          description: null,
+        },
+        {
+          title: "Full host 'sppup' image",
+          link: '/bootdisk/disks/full_hosts/8',
+          disabled: true,
+          description: 'Host is not in build mode',
+        },
+        {
+          title: 'Boot disk help',
+          link: '/bootdisk/disks/help',
+        },
+      ],
+    },
+  },
+  {
+    id: 8,
+  }
+);
+
+const fixtures = {
+  'should return options': () => selectBootdiskOptions(state),
+  'should return isLoading': () => selectIsLoading(state),
+  'should return hostId': () => selectHostId(state),
+};
+
+describe('HostBootdiskButtons selectors', () =>
+  testSelectorsSnapshotWithFixtures(fixtures));

data/webpack/src/extensions/host/__tests__/__snapshots__/HostBootdiskButtonsSelectors.test.js.snap

@@ -0,0 +1,30 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`HostBootdiskButtons selectors should return hostId 1`] = `8`;
+
+exports[`HostBootdiskButtons selectors should return isLoading 1`] = `true`;
+
+exports[`HostBootdiskButtons selectors should return options 1`] = `
+Object {
+  "actions": Array [
+    Object {
+      "description": null,
+      "disabled": false,
+      "link": "/bootdisk/disks/hosts/8",
+      "title": "Host 'sppup' image",
+    },
+    Object {
+      "description": "Host is not in build mode",
+      "disabled": true,
+      "link": "/bootdisk/disks/full_hosts/8",
+      "title": "Full host 'sppup' image",
+    },
+    Object {
+      "link": "/bootdisk/disks/help",
+      "title": "Boot disk help",
+    },
+  ],
+  "architectureName": "x86_64",
+  "bootdiskDownloadable": true,
+}
+`;

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_bootdisk
 version: !ruby/object:Gem::Version
-  version: 18.1.0
+  version: 19.0.3
 platform: ruby
 authors:
 - Dominic Cleal
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-11-09 00:00:00.000000000 Z
+date: 2022-01-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: theforeman-rubocop
@@ -48,9 +48,6 @@ extra_rdoc_files:
 - LICENSE
 - README.md
 files:
-- ".github/workflows/rubocop.yaml"
-- ".tx/config"
-- AUTHORS
 - CHANGES.md
 - LICENSE
 - README.md
@@ -99,52 +96,81 @@ files:
 - locale/Makefile
 - locale/action_names.rb
 - locale/ca/LC_MESSAGES/foreman_bootdisk.mo
+- locale/ca/foreman_bootdisk.edit.po
 - locale/ca/foreman_bootdisk.po
+- locale/ca/foreman_bootdisk.po.time_stamp
 - locale/de/LC_MESSAGES/foreman_bootdisk.mo
+- locale/de/foreman_bootdisk.edit.po
 - locale/de/foreman_bootdisk.po
+- locale/de/foreman_bootdisk.po.time_stamp
 - locale/en/LC_MESSAGES/foreman_bootdisk.mo
+- locale/en/foreman_bootdisk.edit.po
 - locale/en/foreman_bootdisk.po
+- locale/en/foreman_bootdisk.po.time_stamp
 - locale/en_GB/LC_MESSAGES/foreman_bootdisk.mo
+- locale/en_GB/foreman_bootdisk.edit.po
 - locale/en_GB/foreman_bootdisk.po
+- locale/en_GB/foreman_bootdisk.po.time_stamp
 - locale/es/LC_MESSAGES/foreman_bootdisk.mo
+- locale/es/foreman_bootdisk.edit.po
 - locale/es/foreman_bootdisk.po
+- locale/es/foreman_bootdisk.po.time_stamp
 - locale/foreman_bootdisk.pot
 - locale/fr/LC_MESSAGES/foreman_bootdisk.mo
+- locale/fr/foreman_bootdisk.edit.po
 - locale/fr/foreman_bootdisk.po
+- locale/fr/foreman_bootdisk.po.time_stamp
 - locale/gemspec.rb
 - locale/it/LC_MESSAGES/foreman_bootdisk.mo
+- locale/it/foreman_bootdisk.edit.po
 - locale/it/foreman_bootdisk.po
+- locale/it/foreman_bootdisk.po.time_stamp
 - locale/ja/LC_MESSAGES/foreman_bootdisk.mo
+- locale/ja/foreman_bootdisk.edit.po
 - locale/ja/foreman_bootdisk.po
+- locale/ja/foreman_bootdisk.po.time_stamp
 - locale/ko/LC_MESSAGES/foreman_bootdisk.mo
+- locale/ko/foreman_bootdisk.edit.po
 - locale/ko/foreman_bootdisk.po
+- locale/ko/foreman_bootdisk.po.time_stamp
 - locale/pt_BR/LC_MESSAGES/foreman_bootdisk.mo
+- locale/pt_BR/foreman_bootdisk.edit.po
 - locale/pt_BR/foreman_bootdisk.po
+- locale/pt_BR/foreman_bootdisk.po.time_stamp
 - locale/ru/LC_MESSAGES/foreman_bootdisk.mo
+- locale/ru/foreman_bootdisk.edit.po
 - locale/ru/foreman_bootdisk.po
+- locale/ru/foreman_bootdisk.po.time_stamp
 - locale/sv_SE/LC_MESSAGES/foreman_bootdisk.mo
+- locale/sv_SE/foreman_bootdisk.edit.po
 - locale/sv_SE/foreman_bootdisk.po
+- locale/sv_SE/foreman_bootdisk.po.time_stamp
 - locale/zanata.xml
 - locale/zh_CN/LC_MESSAGES/foreman_bootdisk.mo
+- locale/zh_CN/foreman_bootdisk.edit.po
 - locale/zh_CN/foreman_bootdisk.po
+- locale/zh_CN/foreman_bootdisk.po.time_stamp
 - locale/zh_TW/LC_MESSAGES/foreman_bootdisk.mo
+- locale/zh_TW/foreman_bootdisk.edit.po
 - locale/zh_TW/foreman_bootdisk.po
-- release-gem
-- test/functional/foreman_bootdisk/api/v2/disks_controller_test.rb
-- test/functional/foreman_bootdisk/api/v2/subnet_disks_controller_test.rb
-- test/functional/foreman_bootdisk/disks_controller_test.rb
-- test/functional/foreman_bootdisk/subnet_disks_controller_test.rb
-- test/models/host/managed_test.rb
-- test/test_plugin_helper.rb
-- test/unit/access_permissions_test.rb
-- test/unit/concerns/compute_resources/vmware_test.rb
-- test/unit/concerns/host_test.rb
-- test/unit/concerns/orchestration/compute_test.rb
-- test/unit/foreman_bootdisk/renderer_test.rb
-- test/unit/foreman_bootdisk/scope/bootdisk_test.rb
-- test/unit/foreman_bootdisk/scope/full_host_bootdisk_test.rb
-- test/unit/iso_generator_test.rb
-homepage: http://github.com/theforeman/foreman_bootdisk
+- locale/zh_TW/foreman_bootdisk.po.time_stamp
+- package.json
+- webpack/__mocks__/foremanReact/common/HOC.js
+- webpack/__mocks__/foremanReact/common/I18n.js
+- webpack/__mocks__/foremanReact/common/helpers.js
+- webpack/__mocks__/foremanReact/common/urlHelpers.js
+- webpack/__mocks__/foremanReact/constants.js
+- webpack/__mocks__/foremanReact/redux/API/APISelectors.js
+- webpack/__mocks__/foremanReact/redux/API/index.js
+- webpack/global_index.js
+- webpack/global_test_setup.js
+- webpack/index.js
+- webpack/src/extensions/constants.js
+- webpack/src/extensions/host/HostBootdiskButtons.js
+- webpack/src/extensions/host/HostBootdiskButtonsSelectors.js
+- webpack/src/extensions/host/__tests__/HostBootdiskButtonsSelectors.test.js
+- webpack/src/extensions/host/__tests__/__snapshots__/HostBootdiskButtonsSelectors.test.js.snap
+homepage: https://github.com/theforeman/foreman_bootdisk
 licenses:
 - GPL-3.0
 metadata: {}

data/.github/workflows/rubocop.yaml

@@ -1,19 +0,0 @@
-name: Rubocop
-
-on: [pull_request]
-
-jobs:
-  rubocop:
-    runs-on: ubuntu-latest
-    env:
-      BUNDLE_FOREMAN: '0'
-      BUNDLE_WITHOUT: development:test
-    steps:
-      - uses: actions/checkout@v2
-      - name: Setup Ruby
-        uses: ruby/setup-ruby@v1
-        with:
-          ruby-version: 2.5
-          bundler-cache: true
-      - name: Run rubocop
-        run: bundle exec rubocop -P --format github

data/.tx/config

@@ -1,8 +0,0 @@
-[main]
-host = https://www.transifex.com
-
-[foreman.foreman_bootdisk]
-file_filter = locale/<lang>/foreman_bootdisk.edit.po
-source_file = locale/foreman_bootdisk.pot
-source_lang = en
-type = PO

data/AUTHORS

@@ -1,34 +0,0 @@
-Adam Ruzicka
-Bernhard Suttner
-Bryan Kearney
-crito
-Damon Maria
-Dominic Cleal
-Dominik Matoulek
-Greg Sutcliffe
-Ido Kanner
-Jon Fautley
-Justin Sherrill
-Leos Stejskal
-Lukas Zapletal
-Lukáš Zapletal
-Marek Hulan
-Marek Hulán
-Markus Bucher
-Martin Bačovský
-Michael Hofer
-Michael Moll
-Ohad Levy
-Oleh Fedorenko
-Ondrej Prazak
-Ondřej Ezr
-Ondřej Pražák
-Pat Riehecky
-Robert Antoni Buj Gelonch
-Rohan21Lobo
-Ronald van Zantvoort
-Sam Kottler
-Sean O'Keeffe
-Stephen Benjamin
-Swapnil Abnave
-Timo Goebel

data/release-gem

@@ -1,84 +0,0 @@
-#!/bin/env ruby
-# frozen_string_literal: true
-# rubocop:disable all
-
-$plugin_name = "foreman_bootdisk"
-RUBY_VERSION_FILE = "lib/#{$plugin_name}/version.rb"
-
-def die(msg)
-  STDERR.puts msg; exit 1
-end
-
-def run(cmd)
-  puts `#{cmd}`
-  die("Command #{cmd} failed with #{$?}!") if $?.to_i != 0
-  true
-end
-
-def action(question)
-  puts "#{question} ? [Y/n]"
-  die("Aborted.") if STDIN.gets.chomp.match(/^(n|no|nope)$/i)
-  die("Check failed, exiting.") if block_given? && ! yield
-end
-
-action "Extracted strings with: rake plugin:gettext[#{$plugin_name}] (Foreman dir)"
-
-action "Pulled updates with: make -C locale tx-update"
-
-action "Merged the updates with: rake plugin:gettext[#{$plugin_name}] (Foreman dir)"
-
-action "Rebuilt the MO files with: make -C locale mo-files" do
-  Dir["locale/**/*.po"].each do |po|
-    mo = po.sub(/#{$plugin_name}\.po$/, "LC_MESSAGES/#{$plugin_name}.mo")
-    puts("Warning, file #{mo} does not exist!") unless File.exist?(mo)
-    puts("Warning, file #{mo} outdated!") if File.mtime(po) > File.mtime(mo)
-  end
-end
-
-action "Want me to update AUTHORS file" do
-  run "git log --pretty=format:'%an' | sort -u > AUTHORS"
-end
-
-action "Modified lib/#{$plugin_name}/engine.rb to match minimum Foreman core version"
-
-action "Modified #{RUBY_VERSION_FILE} to match desired version"
-
-action "Created 'Version X.Y' commit with engine.rb/version.rb change"
-
-action "Created git tag" do
-  $git_version = `git describe --abbrev=0 --tags`.chomp.sub("v", "")
-  $git_version == File.read(RUBY_VERSION_FILE).match(/VERSION\s*=\s'([0-9\.]+)'/)[1]
-end
-
-action "Is version #{$git_version} the one to be released"
-
-$gem_filename = "#{$plugin_name}-#{$git_version}.gem"
-action "Shall I build the #{$gem_filename} now" do
-  run "gem build #{$plugin_name}.gemspec"
-end
-
-action "Investigated the #{$gem_filename} via gem compare -k -b #{$plugin_name} OLD #{$git_version} -k" do
-  run "tar -xOf #{$gem_filename} data.tar.gz | tar -z -list | sort"
-end
-
-action "Created and built OS distribution packages"
-
-action "Updated dependencies in both RPM and DEB spec files"
-
-action "All commits and *tags* pushed to origin"
-
-action "Shell I push the #{$gem_filename} to rubygems.org now" do
-  run "gem push #{$gem_filename}"
-end
-
-action "Is https://rubygems.org/gems/#{$plugin_name} looking good"
-
-action "Created new stable branch and pushed to origin"
-
-action "Updated documentation at www.theforeman.org?"
-
-action "Updated jenkins_job_builder http://bit.ly/2iZKR3B branch"
-
-puts "Well done! Go ahead and create documentation and changelog:"
-puts "  git log --pretty=format:'%s' develop...X.Y.Z"
-puts "  git log --pretty=format:'%an' develop...X.Y.Z | sort -u"

data/test/functional/foreman_bootdisk/api/v2/disks_controller_test.rb

@@ -1,97 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-class ForemanBootdisk::Api::V2::DisksControllerTest < ActionController::TestCase
-  include ForemanBootdiskTestHelper
-  setup :setup_bootdisk
-  setup :setup_referer
-  setup :setup_org_loc
-
-  def perform_generic_generate
-    tmp = create_tempfile
-    ForemanBootdisk::ISOGenerator.expects(:generate).yields(create_tempfile.path)
-    get :generic
-    assert_response :success
-  ensure
-    tmp.unlink
-  end
-
-  def perform_host_generate
-    tmp = create_tempfile
-    ForemanBootdisk::ISOGenerator.expects(:generate).yields(create_tempfile.path)
-    get :host, params: { id: @host.name }
-    assert_response :success
-  ensure
-    tmp.unlink
-  end
-
-  def perform_full_host_generate
-    tmp = create_tempfile
-    ForemanBootdisk::ISOGenerator.expects(:generate_full_host).yields(create_tempfile.path)
-    get :host, params: { id: @host.name, full: true }
-    assert_response :success
-  ensure
-    tmp.unlink
-  end
-
-  describe '#generic with TFTP' do
-    setup :setup_subnet_with_tftp
-    setup :setup_host
-
-    test 'should generate generic image' do
-      perform_generic_generate
-    end
-
-    test 'should generate host image' do
-      perform_host_generate
-    end
-
-    test 'should generate full host image' do
-      perform_full_host_generate
-    end
-  end
-
-  describe '#subnet_host with TFTP and HTTPBOOT' do
-    setup :setup_subnet_with_tftp_httpboot_template
-    setup :setup_host
-
-    test 'should generate generic image' do
-      perform_generic_generate
-    end
-
-    test 'should generate host image' do
-      perform_host_generate
-    end
-
-    test 'should generate full host image' do
-      perform_full_host_generate
-    end
-  end
-
-  describe 'default API version 2' do
-    setup :setup_host_env
-
-    test 'path - /api/hosts/:host_id routes to #host' do
-      expected_path = if Rails::VERSION::MAJOR >= 5
-                        "/api/v2/hosts/#{@host.id}"
-                      else
-                        "/api/hosts/#{@host.id}"
-                      end
-      assert_routing expected_path,
-                     format: 'json',
-                     apiv: 'v2',
-                     controller: 'foreman_bootdisk/api/v2/disks',
-                     action: 'host',
-                     id: @host.id.to_s
-    end
-
-    test 'path - /api/generic routes to #generic' do
-      assert_routing '/api/generic',
-                     format: 'json',
-                     apiv: 'v2',
-                     controller: 'foreman_bootdisk/api/v2/disks',
-                     action: 'generic'
-    end
-  end
-end

data/test/functional/foreman_bootdisk/api/v2/subnet_disks_controller_test.rb

@@ -1,37 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-class ForemanBootdisk::Api::V2::SubnetDisksControllerTest < ActionController::TestCase
-  include ForemanBootdiskTestHelper
-  setup :setup_bootdisk
-  setup :setup_referer
-  setup :setup_org_loc
-
-  def perform_subnet_generate
-    tmp = create_tempfile
-    ForemanBootdisk::ISOGenerator.expects(:generate).yields(create_tempfile.path)
-    get :subnet, params: { id: @host.subnet.id }
-    assert_response :success
-  ensure
-    tmp.unlink
-  end
-
-  describe '#subnet_host with TFTP' do
-    setup :setup_subnet_with_tftp
-    setup :setup_host
-
-    test 'should generate subnet generic host image' do
-      perform_subnet_generate
-    end
-  end
-
-  describe '#subnet_host with TFTP and HTTPBOOT' do
-    setup :setup_subnet_with_tftp_httpboot_template
-    setup :setup_host
-
-    test 'should generate subnet generic host image' do
-      perform_subnet_generate
-    end
-  end
-end

data/test/functional/foreman_bootdisk/disks_controller_test.rb

@@ -1,97 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-class ForemanBootdisk::DisksControllerTest < ActionController::TestCase
-  include ForemanBootdiskTestHelper
-  setup :setup_bootdisk
-  setup :setup_referer
-  setup :setup_org_loc
-
-  def perform_generic_generate
-    tmp = create_tempfile
-    ForemanBootdisk::ISOGenerator.expects(:generate).yields(create_tempfile.path)
-    get :generic, session: set_session_user
-    assert_response :success
-  ensure
-    tmp.unlink
-  end
-
-  def perform_host_generate
-    tmp = create_tempfile
-    ForemanBootdisk::ISOGenerator.expects(:generate).yields(create_tempfile.path)
-    get :host, params: { id: @host.name }, session: set_session_user
-    assert_response :success
-  ensure
-    tmp.unlink
-  end
-
-  def perform_full_host_generate
-    tmp = create_tempfile
-    ForemanBootdisk::ISOGenerator.expects(:generate_full_host).yields(create_tempfile.path)
-    get :full_host, params: { id: @host.name }, session: set_session_user
-    assert_response :success
-  ensure
-    tmp.unlink
-  end
-
-  describe '#generic with TFTP' do
-    setup :setup_subnet_with_tftp
-    setup :setup_host
-
-    test 'should generate generic image' do
-      perform_generic_generate
-    end
-
-    test 'should generate host image' do
-      perform_host_generate
-    end
-
-    test 'should generate full host image' do
-      perform_full_host_generate
-    end
-  end
-
-  describe '#subnet_host with TFTP and HTTPBOOT' do
-    setup :setup_subnet_with_tftp_httpboot_template
-    setup :setup_host
-
-    test 'should generate generic image' do
-      perform_generic_generate
-    end
-
-    test 'should generate host image' do
-      perform_host_generate
-    end
-
-    test 'should generate full host image' do
-      perform_full_host_generate
-    end
-  end
-
-  describe '#host without tftp' do
-    setup :setup_referer
-    setup :setup_org_loc
-    setup :setup_subnet_no_tftp
-    setup :setup_host
-
-    test 'should prolong token for host image' do
-      Setting[:token_duration] = 60
-      @host.set_token
-      past = Time.now
-      assert @host.token.expires > past
-      travel_to(past + 2.hours)
-      assert @host.token_expired?
-
-      perform_full_host_generate
-
-      assert_equal @host.token_expired?, true
-      assert @host.token.expires > past
-    end
-  end
-
-  test 'should render help' do
-    get :help, session: set_session_user
-    assert_response :success
-  end
-end

data/test/functional/foreman_bootdisk/subnet_disks_controller_test.rb

@@ -1,51 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-class ForemanBootdisk::SubnetDisksControllerTest < ActionController::TestCase
-  include ForemanBootdiskTestHelper
-  setup :setup_bootdisk
-  setup :setup_referer
-  setup :setup_org_loc
-
-  def perform_subnet_generate
-    tmp = create_tempfile
-    ForemanBootdisk::ISOGenerator.expects(:generate).yields(create_tempfile.path)
-    get :subnet, params: { id: @subnet.id }, session: set_session_user
-    assert_empty flash[:error]
-    assert_response :success
-  ensure
-    tmp.unlink
-  end
-
-  describe '#generic with TFTP' do
-    setup :setup_subnet_with_tftp
-    setup :setup_host
-
-    test 'should generate subnet image' do
-      perform_subnet_generate
-    end
-  end
-
-  describe '#subnet_host with TFTP and HTTPBOOT' do
-    setup :setup_subnet_with_tftp_httpboot_template
-    setup :setup_host
-
-    test 'should generate subnet image' do
-      perform_subnet_generate
-    end
-  end
-
-  describe '#host without tftp' do
-    setup :setup_referer
-    setup :setup_org_loc
-    setup :setup_subnet_no_tftp
-    setup :setup_host
-
-    test 'should not generate subnet image' do
-      get :subnet, params: { id: @subnet.id }, session: set_session_user
-      assert_match(/Failed.*: TFTP feature not enabled/, flash[:error])
-      assert_response :redirect
-    end
-  end
-end