foreman_ovirt 2.0.2 → 2.0.4

data/test/models/ovirt_volume_test.rb

@@ -0,0 +1,104 @@
+# frozen_string_literal: true
+
+require 'test_plugin_helper'
+
+module ForemanOvirt
+  class OvirtVolumeTest < ActiveSupport::TestCase
+    setup do
+      User.current = users :admin
+    end
+
+    test '#new_volume should respect preallocate flag' do
+      ovirt = ForemanOvirt::Ovirt.new
+      volume = ovirt.new_volume(preallocate: '1')
+      assert_equal 'false',
+        volume.sparse
+      assert_equal 'raw',
+        volume.format
+
+      volume = ovirt.new_volume(preallocate: '0')
+      assert_equal 'true',
+        volume.sparse
+
+      volume = ovirt.new_volume
+      assert_equal 'true',
+        volume.sparse
+    end
+
+    test 'accepts multiple certificates' do
+      ovirt = ForemanOvirt::Ovirt.new
+
+      cert1 = "-----BEGIN CERTIFICATE-----\r\n" \
+              "MIIFdDCCA1ygAwIBAgIJAM5Uqykb3EAtMA0GCSqGSIb3DQEBCwUAME8xCzAJBgNV\r\n" \
+              "BAYTAklMMREwDwYDVQQIDAhUZWwgQXZpdjEUMBIGA1UECgwLVGhlIEZvcmVtYW4x\r\n" \
+              "FzAVBgNVBAMMDnRoZWZvcmVtYW4ub3JnMB4XDTE4MDMyNDEyMzYyOFoXDTI4MDMy\r\n" \
+              "MTEyMzYyOFowTzELMAkGA1UEBhMCSUwxETAPBgNVBAgMCFRlbCBBdml2MRQwEgYD\r\n" \
+              "VQQKDAtUaGUgRm9yZW1hbjEXMBUGA1UEAwwOdGhlZm9yZW1hbi5vcmcwggIiMA0G\r\n" \
+              "CSqGSIb3DQEBAQUAA4ICDwAwggIKAoICAQDF04/s4h+BgHPG1HDZ/sDlYq925pkc\r\n" \
+              "RTVAfnE2EXDAmZ6W4Q9ueDY65MHe3ZWO5Dg72kNSP2sK9kRI7Dk5CAFOgyw1rH8t\r\n" \
+              "Hd1+0xp/lv6e4SvSYghxIL68vFe0ftKkm1usqejBM5ZTgKr7JCI+XSIN36F65Kde\r\n" \
+              "c+vxwBnayuhP04r9/aaE/709SXML4eRVYW8I3qFy9FPtUOm+bY8U2PIv5fHayqbG\r\n" \
+              "cL/4t3+MCtMhHJsLzdBXya+1P5t+HcKjUNlmwoUF961YAktVuEFloGd0RMRlqF3/\r\n" \
+              "itU3QNlXgA5QBIciE5VPr/PiqgMC3zgd5avjF4OribZ+N9AATLiQMW78il5wSfcc\r\n" \
+              "kQjU9ChOLrzku455vQ8KE4bc0qvpCWGfUah6MvL9JB+TQkRl/8kxl0b9ZinIvJDH\r\n" \
+              "ynVMb4cB/TDEjrjOfzn9mWLH0ZJqjmc2bER/G12WQxOaYLxdVwRStD3Yh6PtiFWu\r\n" \
+              "sXOk19UOTVkeuvGFVtvzLfEwQ1lDEo7+VBQz8FG/HBu2Hpq3IwCFrHuicikwjQJk\r\n" \
+              "nfturgD0rBOKEc1qWNZRCvovYOLL6ihvv5Orujsx5ZCHOAtnVNxkvIlFt2RS45LF\r\n" \
+              "MtPJyhAc6SjitllfUEirxprsbmeSZqrIfzcGaEhgOSnyik1WMv6bYiqPfBg8Fzjh\r\n" \
+              "vOCbtiDNPmvgOwIDAQABo1MwUTAdBgNVHQ4EFgQUtkAgQopsTtG9zSG3MgW2IxHD\r\n" \
+              "MDwwHwYDVR0jBBgwFoAUtkAgQopsTtG9zSG3MgW2IxHDMDwwDwYDVR0TAQH/BAUw\r\n" \
+              "AwEB/zANBgkqhkiG9w0BAQsFAAOCAgEAJq7iN+ZroRBweNhvUobxs75bLIV6tNn1\r\n" \
+              "MdNHDRA+hezwf+gxHZhFyaAHfTpst2/9leK5Qe5Zd6gZLr3E5/8ppQuRod72H39B\r\n" \
+              "vxMlG5zxDss0WMo3vZeKZbTY6QhXi/lY2IZ6OGV4feSvCsYxn27GTjjrRUSLFeHH\r\n" \
+              "JVemCwCDMavaE3+OIY4v2P4FcG+MjUvfOB9ahI24TWL7YgrsNVmJjCILq+EeUj0t\r\n" \
+              "Gde1SXVyLkqt7PoxHRJAE0BCEMJSnjxaVB329acJgeehBUxjj4CCPqtDxtbz9HEH\r\n" \
+              "mOKfNdaKpFor+DUeEKUWVGnr9U9xOaC+Ws+oX7MIEUCDM7p2ob4JwcjnFs1jZgHh\r\n" \
+              "Hwig+i7doTlc701PvKWO96fuNHK3B3/jTb1fVvSZ49O/RvY1VWODdUdxWmXGHNh3\r\n" \
+              "LoR8tSPEb46lC2DXGaIQumqQt8PnBG+vL1qkQa1SGTV7dJ8TTbxbv0S+sS+igkk9\r\n" \
+              "zsIEK8Ea3Ep935cXximz0faAAKHSA+It+xHLAyDtqy2KaAEBgGsBuuWlUfK6TaP3\r\n" \
+              "Gwdjct3y4yYUO45lUsUfHqX8vk/4ttW5zYeDiW+HArJz+9VUXNbEdury4kGuHgBj\r\n" \
+              "xHD4Bsul65+hHZ9QywKU26F1A6TLkYpQ2rk/Dx9LGICM4m4IlHjWJPFsQdtkyOor\r\n" \
+              "osxMtcaZZ1E=\r\n" \
+              '-----END CERTIFICATE-----'
+
+      cert2 = "-----BEGIN CERTIFICATE-----\r\n" \
+              "MIIFdDCCA1ygAwIBAgIJAMJiz/f2HDayMA0GCSqGSIb3DQEBCwUAME8xCzAJBgNV\r\n" \
+              "BAYTAklMMREwDwYDVQQIDAhUZWwgQXZpdjEUMBIGA1UECgwLVGhlIEZvcmVtYW4x\r\n" \
+              "FzAVBgNVBAMMDnRoZWZvcmVtYW4ub3JnMB4XDTE4MDMyNDEyNDQxNFoXDTI4MDMy\r\n" \
+              "MTEyNDQxNFowTzELMAkGA1UEBhMCSUwxETAPBgNVBAgMCFRlbCBBdml2MRQwEgYD\r\n" \
+              "VQQKDAtUaGUgRm9yZW1hbjEXMBUGA1UEAwwOdGhlZm9yZW1hbi5vcmcwggIiMA0G\r\n" \
+              "CSqGSIb3DQEBAQUAA4ICDwAwggIKAoICAQClTtLvHCkP/DB2oYSb7lhh4yswr3we\r\n" \
+              "5AknEtM+MEgPRm8ou/iF4UT/u+IfCi0n2M/wUGceIpwuB5mKY29sAWP0IO+UBCmn\r\n" \
+              "SIdo8sK7jNmWhgcR4LG9MrX7pOQWFTUc583LSCWNF7V2Nhy2ZCqp5bzATEd207+L\r\n" \
+              "4LHCrn1kwdc22ltnR6CY1wWno2OrkVZOJFSGnLEwa8HhMxyBnGygZXt9JObDixp8\r\n" \
+              "r69heKpOLn8/QEIwq0FgAb4nxBsnT9igzd0L8a5UuvDQ7SmmOp7iYwJg5TNFtCm8\r\n" \
+              "u8fvXx6T+hLfyNFAHJuJ9xlRM5I7qQGSIK9a9xYqANXMpfcEz7p7JbT1AKQHDzJC\r\n" \
+              "ncoq/NNFXgJJIcMMxAFUsKrm75ElB0ph6fSaYMxUv2yCBxpZquLJCbavdH5avRDw\r\n" \
+              "JXWYzWjEAPG9YEFvF5LI7OhBrN5atrWxltsi3vfADRfiCMFTdgLMwPFI42EjIEK/\r\n" \
+              "DEMm4alWVJMRD8coWcsKJ0xxAku/84jWstV7P8+Kfip76XsAkUNbr46pIGIwjw++\r\n" \
+              "J+g77ERBugMyW30Te+CAuzVnVy01N+0dNsR2mV9hlRFk9HPohdTgWX9tr+/Px22n\r\n" \
+              "yRIXhPLls7Kl4q8J/05wNIbf6oDY2rYjroqJkI/xZdowanxBUWb6zV/1FfHHlAeO\r\n" \
+              "IT0l9LUfCdmgdQIDAQABo1MwUTAdBgNVHQ4EFgQUn3QR1z8ZICejC+7OLI6nBUxU\r\n" \
+              "0ngwHwYDVR0jBBgwFoAUn3QR1z8ZICejC+7OLI6nBUxU0ngwDwYDVR0TAQH/BAUw\r\n" \
+              "AwEB/zANBgkqhkiG9w0BAQsFAAOCAgEAUHqOREvTlPylNxHs+QG8hu+HbwsjY/g+\r\n" \
+              "VF4YBzB78tagaCw9McD1S9qvWWFNjeVRtFbKE+OyZdIkte2Yu6UWlQ5A5fHU4ADD\r\n" \
+              "CxCsThwiQaywJOZwVV8ZM6JOKtbf1Epa8kPegu1R/858aW1P/6VuPsTX3ZX8Qyon\r\n" \
+              "Fyh1GOmZ+uou/jGnaWSchHX1KHryK1UMA9Z3SbpdJQFILs04QyEXaWO/nPcbdd1h\r\n" \
+              "Ceshcd/88CDYoxhcJoi6wpjQRvyDOP4txEJw6SgsTXU8vPLDeISqXzr2znSPdL9k\r\n" \
+              "K8g7KJqr7KuhGuDk6dKQp0rL4FSFf8SNH3j7UvNa66/kd1N/ejtbwGmKti8uuQZ5\r\n" \
+              "z96knnC6SH0fIgQyXk9KVfHBTqPtHAxnBAOXvPRWlDObRNK8SobyEnAQTE8COGT/\r\n" \
+              "+9AuMVbmyEyTMRvY7znv1C7EhjEEOOlpr24on12RpqTJSbTFOppYeABu0rpKM9i7\r\n" \
+              "anZnga8BSVy4YbUsMpGdHZ5xhJq8ip9AFjcFvAN0qJFMozlGx6MkgHlmhcLGMPks\r\n" \
+              "wKFSycux0qh49d/9TQ0KOiooK/sjS2dbz5bj6Z//6MPmSvRlwPhDhRDaT7D8u7cU\r\n" \
+              "svKQIHrQJve7KJxXUuPTh3zawRXdDGmdm7D3CbDOqpAIAnR+/rKXnPzQYBMCvCDG\r\n" \
+              "2lSBFSVrAlo=\r\n" \
+              '-----END CERTIFICATE-----'
+
+      store = ovirt.send(
+        :ca_cert_store, "#{cert1}\r\n#{cert2}"
+      )
+      assert store.verify(OpenSSL::X509::Certificate.new(cert1))
+      assert store.verify(OpenSSL::X509::Certificate.new(cert2))
+    end
+  end
+end

data/test/test_plugin_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This calls the main test_helper in Foreman-core
 require 'test_helper'
 

data/test/unit/compute_resource_host_importer_test.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+require 'test_plugin_helper'
+
+module ForemanOvirt
+  class ComputeResourceHostImporterTest < ActiveSupport::TestCase
+    setup do
+      Fog.mock!
+      User.current = users(:admin)
+    end
+
+    teardown do
+      Fog.unmock!
+    end
+
+    let(:importer) do
+      ComputeResourceHostImporter.new(
+        compute_resource: compute_resource,
+        vm: vm
+      )
+    end
+
+    let(:host) { importer.host }
+    let(:vm) { compute_resource.find_vm_by_uuid(uuid) }
+
+    context 'on ovirt' do
+      let(:compute_resource) { FactoryBot.build(:ovirt_cr) }
+      let(:uuid) { '52b9406e-cf66-4867-8655-719a094e324c' }
+
+      test 'imports the VM with all parameters' do
+        assert_equal 'vm01', host.name
+        assert_equal uuid, host.uuid
+        assert_nil host.domain
+        assert_equal '00:1a:4a:23:1b:8f', host.mac
+        assert_equal(
+          {
+            'name' => 'nic1',
+            'network' => '00000000-0000-0000-0000-000000000009',
+            'interface' => 'virtio',
+            'vnic_profile' => '871f3a06-ef53-4ab1-922f-5aa2bea2e94e',
+          },
+          host.primary_interface.compute_attributes
+        )
+        assert_equal compute_resource, host.compute_resource
+      end
+    end
+  end
+end

data/test/unit/ovirt_connection_test.rb

@@ -0,0 +1,123 @@
+# frozen_string_literal: true
+
+require 'test_plugin_helper'
+
+module ForemanOvirt
+  class OvirtConnectionTest < ActiveSupport::TestCase
+    def setup
+      User.current = users(:admin)
+    end
+
+    def new_ovirt_cr(proto = 'http')
+      ComputeResource.new_provider(
+        provider: 'Ovirt',
+        name: :myovirt,
+        user: 'user',
+        password: 'password',
+        public_key: 'ovirtcacert',
+        url: "#{proto}://ovirt.example.com/"
+      )
+    end
+
+    test 'create a new oVirt compute resource' do
+      record = new_ovirt_cr
+      assert record.valid?
+    end
+
+    test 'fingerprint error is ignored on update of operating systems' do
+      record = new_ovirt_cr
+      record.stubs(:client).raises(Foreman::FingerprintException.new('fingerprint error'))
+      original_oses = record.attrs[:available_operating_systems] =
+        { foo: :bar }
+
+      assert_nothing_raised do
+        assert record.send(:update_available_operating_systems),
+          'after validation filter does not return true which would cancel the callback chain'
+      end
+
+      assert_equal original_oses,
+        record.attrs[:available_operating_systems]
+
+      record.url = ''
+      assert_not record.valid?
+
+      assert_nothing_raised do
+        assert_not record.send(:update_available_operating_systems),
+          'after validation filter does not return false which would not cancel the callback chain'
+      end
+
+      assert_equal original_oses,
+        record.attrs[:available_operating_systems]
+    end
+
+    test "#supports_operating_systems? defaults to false if there's SSL issue" do
+      record = new_ovirt_cr
+      record.stubs(:client).raises(Foreman::FingerprintException.new('fingerprint error'))
+
+      assert_nothing_raised do
+        assert_not record.supports_operating_systems?,
+          'ForemanOvirt::Ovirt#supports_operating_systems? returns true even if there is SSL issue'
+      end
+    end
+
+    test 'test_connection should fail if datacenters not found (404)' do
+      client = stub
+      client.stubs(:datacenters).raises(StandardError.new('404 error'))
+      record = new_ovirt_cr
+      record.stubs(:client).returns(client)
+      record.test_connection
+      assert_equal ['404 error'],
+        record.errors[:url]
+      assert_empty record.errors[:base]
+    end
+
+    test 'test_connection should fail if not authorized for datacenters (401)' do
+      client = stub
+      client.stubs(:datacenters).raises(StandardError.new('401 error'))
+      record = new_ovirt_cr
+      record.stubs(:client).returns(client)
+      record.test_connection
+      assert_equal ['401 error'],
+        record.errors[:user]
+      assert_empty record.errors[:base]
+    end
+
+    test 'test_connection should succeed with HTTP url' do
+      record = new_ovirt_cr 'http'
+      record.expects(:client).never
+      record.stubs(:datacenters).returns([
+                                           'example', 1
+                                         ])
+      # returned by oVirt when HTTP is valid and we POST to /api
+      RestClient.expects(:post).raises(StandardError.new('406 Not Acceptable'))
+      assert record.test_connection
+      assert_empty record.errors[:base]
+    end
+
+    test 'test_connection should succeed with HTTPS url' do
+      record = new_ovirt_cr 'https'
+      record.expects(:client).never
+      record.stubs(:datacenters).returns([
+                                           'example', 1
+                                         ])
+      RestClient.expects(:post).never
+      assert record.test_connection
+      assert_empty record.errors
+    end
+
+    test 'test_connection should detect 302 HTTPS redirect with HTTP url and fail' do
+      record = new_ovirt_cr 'http'
+      record.expects(:client).never
+      record.stubs(:datacenters).returns([
+                                           'example', 1
+                                         ])
+      # RestClient throws 302 as an error during POSTs
+      RestClient.expects(:post).raises(StandardError.new('302 Found'))
+      record.test_connection
+      assert_match(
+        /HTTPS/, record.errors[:url].first
+      )
+      assert_empty record.errors[:base]
+    end
+  end
+end

data/webpack/components/__tests__/ovirt.test.js

@@ -0,0 +1,17 @@
+import { datacenterSelected } from '../ovirt';
+
+jest.mock('jquery', () => {
+  const fn = jest.fn(() => fn);
+  fn.fn = fn;
+  return fn;
+});
+
+jest.mock('../ovirt', () => ({
+  datacenterSelected: jest.fn(),
+}));
+
+describe('ovirt', () => {
+  it('datacenterSelected is defined', () => {
+    expect(datacenterSelected).toBeDefined();
+  });
+});

data/webpack/components/extensions/HostDetails/DetailsTabCards/OvirtCard.js

@@ -6,25 +6,17 @@ import { useAPI } from 'foremanReact/common/hooks/API/APIHooks';
 import ErrorBoundary from 'foremanReact/components/common/ErrorBoundary';
 import CardTemplate from 'foremanReact/components/HostDetails/Templates/CardItem/CardTemplate';
 import {
-  Card,
-  CardBody,
-  CardHeader,
-  CardTitle,
   DescriptionList,
   DescriptionListGroup,
   DescriptionListTerm,
   DescriptionListDescription,
+  Spinner,
+  EmptyState,
 } from '@patternfly/react-core';
-import { Spinner } from '@patternfly/react-core';
 import { number_to_human_size as NumberToHumanSize } from 'number_helpers';
 
 const OvirtCard = ({ hostDetails }) => {
-  const {
-    id: hostId,
-    compute_resource_id: computeResourceId,
-    compute_resource_name: computeResourceName,
-    compute_resource_provider: provider,
-  } = hostDetails;
+  const { id: hostId, compute_resource_provider: provider } = hostDetails;
   const virtUrl = foremanUrl(`/api/hosts/${hostId}/vm_compute_attributes`);
   const { response: vm, status = {} } = useAPI('get', virtUrl) || {};
 
@@ -62,18 +54,24 @@ const OvirtCard = ({ hostDetails }) => {
           </DescriptionListGroup>
           <DescriptionListGroup>
             <DescriptionListTerm>{__('Sockets')}</DescriptionListTerm>
-            <DescriptionListDescription>{vm.sockets}</DescriptionListDescription>
+            <DescriptionListDescription>
+              {vm.sockets}
+            </DescriptionListDescription>
           </DescriptionListGroup>
           <DescriptionListGroup>
             <DescriptionListTerm>{__('Memory')}</DescriptionListTerm>
             <DescriptionListDescription>
-              {NumberToHumanSize(vm.memory, { strip_insignificant_zeros: true })}
+              {NumberToHumanSize(vm.memory, {
+                strip_insignificant_zeros: true,
+              })}
             </DescriptionListDescription>
           </DescriptionListGroup>
           {vm.display && (
             <DescriptionListGroup>
               <DescriptionListTerm>{__('Display')}</DescriptionListTerm>
-              <DescriptionListDescription>{vm.display.type}</DescriptionListDescription>
+              <DescriptionListDescription>
+                {vm.display.type}
+              </DescriptionListDescription>
             </DescriptionListGroup>
           )}
           {vm.display && vm.display.type === 'vnc' && (
@@ -84,32 +82,36 @@ const OvirtCard = ({ hostDetails }) => {
               </DescriptionListDescription>
             </DescriptionListGroup>
           )}
-          {vm.interfaces_attributes && Object.keys(vm.interfaces_attributes).length > 0 &&
+          {vm.interfaces_attributes &&
+            Object.keys(vm.interfaces_attributes).length > 0 &&
             Object.values(vm.interfaces_attributes).map((nic, index) => (
-            <DescriptionListGroup key={`nic-${index}`}>
-              <DescriptionListTerm>{__('NIC name')}</DescriptionListTerm>
-              <DescriptionListDescription>
-                {nic.compute_attributes.name}
-              </DescriptionListDescription>
-              <DescriptionListTerm>{__('Network')}</DescriptionListTerm>
-              <DescriptionListDescription>
-                {nic.compute_attributes.network}
-              </DescriptionListDescription>
-              <DescriptionListTerm>{__('MAC address')}</DescriptionListTerm>
-              <DescriptionListDescription>{nic.mac}</DescriptionListDescription>
-            </DescriptionListGroup>
-          ))}
-          {vm.volumes_attributes && Object.keys(vm.volumes_attributes).length > 0 &&
+              <DescriptionListGroup key={`nic-${index}`}>
+                <DescriptionListTerm>{__('NIC name')}</DescriptionListTerm>
+                <DescriptionListDescription>
+                  {nic.compute_attributes.name}
+                </DescriptionListDescription>
+                <DescriptionListTerm>{__('Network')}</DescriptionListTerm>
+                <DescriptionListDescription>
+                  {nic.compute_attributes.network}
+                </DescriptionListDescription>
+                <DescriptionListTerm>{__('MAC address')}</DescriptionListTerm>
+                <DescriptionListDescription>
+                  {nic.mac}
+                </DescriptionListDescription>
+              </DescriptionListGroup>
+            ))}
+          {vm.volumes_attributes &&
+            Object.keys(vm.volumes_attributes).length > 0 &&
             Object.values(vm.volumes_attributes).map((vol, index) => (
-            <DescriptionListGroup key={`volume-${index}`}>
-              <DescriptionListTerm>{__('Disk')}</DescriptionListTerm>
-              <DescriptionListDescription>
-                {NumberToHumanSize(vol.size_gb * 1024 ** 3, {
-                  strip_insignificant_zeros: true,
-                })}
-              </DescriptionListDescription>
-            </DescriptionListGroup>
-          ))}
+              <DescriptionListGroup key={`volume-${index}`}>
+                <DescriptionListTerm>{__('Disk')}</DescriptionListTerm>
+                <DescriptionListDescription>
+                  {NumberToHumanSize(vol.size_gb * 1024 ** 3, {
+                    strip_insignificant_zeros: true,
+                  })}
+                </DescriptionListDescription>
+              </DescriptionListGroup>
+            ))}
         </DescriptionList>
       </ErrorBoundary>
     </CardTemplate>

data/webpack/global_index.js

@@ -1,8 +1,6 @@
 import React from 'react';
 
-import { registerReducer } from 'foremanReact/common/MountingService';
 import { addGlobalFill } from 'foremanReact/components/common/Fill/GlobalFill';
-import { registerRoutes } from 'foremanReact/routes/RoutingService';
 
 import OvirtCard from './components/extensions/HostDetails/DetailsTabCards/OvirtCard';
 

data/webpack/global_test_setup.js

@@ -2,9 +2,9 @@
 // fail a test (i.e. default PropType missing). Check the error
 // output and traceback for actual error.
 global.console.error = (error, stack) => {
-    /* eslint-disable-next-line no-console */
-    if (stack) console.log(stack); // Prints out original stack trace
-    throw new Error(error);
+  /* eslint-disable-next-line no-console */
+  if (stack) console.log(stack); // Prints out original stack trace
+  throw new Error(error);
 };
 
 // Increase jest timeout as some tests using multiple http mocks can time out on CI systems.

data/webpack/index.js

@@ -1 +0,0 @@
-import componentRegistry from 'foremanReact/components/componentRegistry';

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_ovirt
 version: !ruby/object:Gem::Version
-  version: 2.0.2
+  version: 2.0.4
 platform: ruby
 authors:
 - markt.de
@@ -108,6 +108,7 @@ files:
 - app/assets/javascripts/foreman_ovirt/host_edit.js
 - app/assets/javascripts/foreman_ovirt/nic_info.js
 - app/assets/javascripts/foreman_ovirt/ovirt.js
+- app/controllers/concerns/foreman_ovirt/api_compute_resources_controller_extension.rb
 - app/controllers/concerns/foreman_ovirt/compute_resources_vms_controller.rb
 - app/controllers/concerns/foreman_ovirt/parameters_extension.rb
 - app/controllers/foreman_ovirt/concerns/compute_resources_controller_extensions.rb
@@ -137,9 +138,19 @@ files:
 - locale/foreman_ovirt.pot
 - locale/gemspec.rb
 - package.json
+- test/controllers/api/v2/compute_resources_controller_test.rb
+- test/controllers/hosts_controller_test.rb
 - test/factories/foreman_ovirt_factories.rb
+- test/fixtures/ovirt_operating_systems.xml
+- test/graphql/types/provider_enum_test.rb
+- test/integration/compute_profile_js_test.rb
+- test/models/compute_resources/compute_resource_test_helpers.rb
+- test/models/compute_resources/ovirt_test.rb
+- test/models/ovirt_volume_test.rb
 - test/test_plugin_helper.rb
-- test/unit/foreman_ovirt_test.rb
+- test/unit/compute_resource_host_importer_test.rb
+- test/unit/ovirt_connection_test.rb
+- webpack/components/__tests__/ovirt.test.js
 - webpack/components/extensions/HostDetails/DetailsTabCards/OvirtCard.js
 - webpack/components/ovirt.js
 - webpack/global_index.js
@@ -159,19 +170,26 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ">="
     - !ruby/object:Gem::Version
       version: '2.7'
-  - - "<"
-    - !ruby/object:Gem::Version
-      version: '4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 4.0.10
 specification_version: 4
-summary: oVirt as a compute resource for The Foreman
+summary: oVirt as a compute resource for Foreman
 test_files:
+- test/controllers/api/v2/compute_resources_controller_test.rb
+- test/controllers/hosts_controller_test.rb
 - test/factories/foreman_ovirt_factories.rb
+- test/fixtures/ovirt_operating_systems.xml
+- test/graphql/types/provider_enum_test.rb
+- test/integration/compute_profile_js_test.rb
+- test/models/compute_resources/compute_resource_test_helpers.rb
+- test/models/compute_resources/ovirt_test.rb
+- test/models/ovirt_volume_test.rb
 - test/test_plugin_helper.rb
-- test/unit/foreman_ovirt_test.rb
+- test/unit/compute_resource_host_importer_test.rb
+- test/unit/ovirt_connection_test.rb
+- webpack/components/__tests__/ovirt.test.js

data/test/unit/foreman_ovirt_test.rb

@@ -1,11 +0,0 @@
-require 'test_plugin_helper'
-
-class ForemanOvirtTest < ActiveSupport::TestCase
-  setup do
-    User.current = User.find_by_login 'admin'
-  end
-
-  test 'the truth' do
-    assert true
-  end
-end