foreman_rh_cloud 13.2.9 → 14.0.1

data/test/unit/rh_cloud_host_test.rb

@@ -188,218 +188,4 @@ class RhCloudHostTest < ActiveSupport::TestCase
       assert_equal local_uuid, @host.insights_uuid
     end
   end
-
-  context 'scoped search on insights_uuid' do
-    setup do
-      @org = FactoryBot.create(:organization)
-    end
-
-    teardown do
-      ForemanRhCloud.unstub(:with_iop_smart_proxy?)
-    end
-
-    test 'searches insights_facet.uuid in non-IoP mode with = operator' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      host1 = FactoryBot.create(:host, :managed, organization: @org)
-      host1.insights = FactoryBot.create(:insights_facet, host_id: host1.id, uuid: 'insights-uuid-123')
-      host2 = FactoryBot.create(:host, :managed, organization: @org)
-      host2.insights = FactoryBot.create(:insights_facet, host_id: host2.id, uuid: 'insights-uuid-456')
-
-      results = Host::Managed.search_for('insights_uuid = insights-uuid-123')
-
-      assert_includes results, host1
-      assert_not_includes results, host2
-    end
-
-    test 'searches subscription_facet.uuid in IoP mode with = operator' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      host1 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      host2 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-
-      # Even if insights_facet has different UUID, should use subscription_facet UUID
-      host1.insights = FactoryBot.create(:insights_facet, host_id: host1.id, uuid: 'stale-123')
-
-      results = Host::Managed.search_for("insights_uuid = #{host1.subscription_facet.uuid}")
-
-      assert_includes results, host1
-      assert_not_includes results, host2
-    end
-
-    test 'searches with ^ operator (IN) in non-IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      host1 = FactoryBot.create(:host, :managed, organization: @org)
-      host1.insights = FactoryBot.create(:insights_facet, host_id: host1.id, uuid: 'uuid-1')
-      host2 = FactoryBot.create(:host, :managed, organization: @org)
-      host2.insights = FactoryBot.create(:insights_facet, host_id: host2.id, uuid: 'uuid-2')
-      host3 = FactoryBot.create(:host, :managed, organization: @org)
-      host3.insights = FactoryBot.create(:insights_facet, host_id: host3.id, uuid: 'uuid-3')
-
-      results = Host::Managed.search_for('insights_uuid ^ (uuid-1,uuid-2)')
-
-      assert_includes results, host1
-      assert_includes results, host2
-      assert_not_includes results, host3
-    end
-
-    test 'searches with ^ operator (IN) in IoP mode - THE BUG FIX' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      host1 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      host2 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      host3 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-
-      # Create insights facets with stale UUIDs to verify we're using subscription_facet
-      host1.insights = FactoryBot.create(:insights_facet, host_id: host1.id, uuid: 'stale-1')
-      host2.insights = FactoryBot.create(:insights_facet, host_id: host2.id, uuid: 'stale-2')
-      host3.insights = FactoryBot.create(:insights_facet, host_id: host3.id, uuid: 'stale-3')
-
-      uuid1 = host1.subscription_facet.uuid
-      uuid2 = host2.subscription_facet.uuid
-
-      # This is the search query that remediation modal creates
-      results = Host::Managed.search_for("insights_uuid ^ (#{uuid1},#{uuid2})")
-
-      # Should find hosts by subscription_facet UUID, not insights_facet UUID
-      assert_includes results, host1
-      assert_includes results, host2
-      assert_not_includes results, host3
-    end
-
-    test 'searches with !^ operator (NOT IN) in non-IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      host1 = FactoryBot.create(:host, :managed, organization: @org)
-      host1.insights = FactoryBot.create(:insights_facet, host_id: host1.id, uuid: 'uuid-1')
-      host2 = FactoryBot.create(:host, :managed, organization: @org)
-      host2.insights = FactoryBot.create(:insights_facet, host_id: host2.id, uuid: 'uuid-2')
-      host3 = FactoryBot.create(:host, :managed, organization: @org)
-      host3.insights = FactoryBot.create(:insights_facet, host_id: host3.id, uuid: 'uuid-3')
-
-      results = Host::Managed.search_for('insights_uuid !^ (uuid-1,uuid-2)')
-
-      assert_not_includes results, host1
-      assert_not_includes results, host2
-      assert_includes results, host3
-    end
-
-    test 'searches with !^ operator (NOT IN) in IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      host1 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      host2 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      host3 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-
-      uuid1 = host1.subscription_facet.uuid
-      uuid2 = host2.subscription_facet.uuid
-
-      results = Host::Managed.search_for("insights_uuid !^ (#{uuid1},#{uuid2})")
-
-      assert_not_includes results, host1
-      assert_not_includes results, host2
-      assert_includes results, host3
-    end
-
-    test 'handles hosts without facets in non-IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      host_without_facet = FactoryBot.create(:host, :managed, organization: @org)
-      host_with_facet = FactoryBot.create(:host, :managed, organization: @org)
-      host_with_facet.insights = FactoryBot.create(:insights_facet, host_id: host_with_facet.id, uuid: 'uuid-1')
-
-      results = Host::Managed.search_for('insights_uuid = uuid-1')
-
-      assert_includes results, host_with_facet
-      assert_not_includes results, host_without_facet
-    end
-
-    test 'handles hosts without subscription_facet in IoP mode' do
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      host_without_sub = FactoryBot.create(:host, :managed, organization: @org)
-      host_with_sub = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-
-      uuid = host_with_sub.subscription_facet.uuid
-
-      results = Host::Managed.search_for("insights_uuid = #{uuid}")
-
-      assert_includes results, host_with_sub
-      assert_not_includes results, host_without_sub
-    end
-
-    test 'mode changes are reflected in searches' do
-      host1 = FactoryBot.create(:host, :managed, :with_subscription, organization: @org)
-      host1.insights = FactoryBot.create(:insights_facet, host_id: host1.id, uuid: 'insights-uuid-abc')
-      insights_uuid = 'insights-uuid-abc'
-      subscription_uuid = host1.subscription_facet.uuid
-
-      # Non-IoP mode: should find by insights_facet UUID
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
-      results = Host::Managed.search_for("insights_uuid = #{insights_uuid}")
-      assert_includes results, host1
-
-      # IoP mode: should find by subscription_facet UUID
-      ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-      results = Host::Managed.search_for("insights_uuid = #{subscription_uuid}")
-      assert_includes results, host1
-
-      # Should NOT find by old insights_facet UUID in IoP mode
-      results = Host::Managed.search_for("insights_uuid = #{insights_uuid}")
-      assert_not_includes results, host1
-    end
-  end
-
-  test 'scoped search for user_omitted inventory status works' do
-    host1 = FactoryBot.create(:host, :managed)
-    host2 = FactoryBot.create(:host, :managed)
-    host3 = FactoryBot.create(:host, :managed)
-
-    # Create different inventory statuses
-    InventorySync::InventoryStatus.create!(
-      host_id: host1.id,
-      status: InventorySync::InventoryStatus::SYNC,
-      reported_at: Time.zone.now
-    )
-
-    InventorySync::InventoryStatus.create!(
-      host_id: host2.id,
-      status: InventorySync::InventoryStatus::DISCONNECT,
-      reported_at: Time.zone.now
-    )
-
-    InventorySync::InventoryStatus.create!(
-      host_id: host3.id,
-      status: InventorySync::InventoryStatus::USER_OMITTED,
-      reported_at: Time.zone.now
-    )
-
-    # Search for user_omitted status
-    results = Host.search_for('insights_inventory_sync_status = user_omitted')
-    result_ids = results.pluck(:id)
-
-    assert_includes result_ids, host3.id, 'Host with USER_OMITTED status should be in search results'
-    assert_not_includes result_ids, host1.id, 'Host with SYNC status should not be in search results'
-    assert_not_includes result_ids, host2.id, 'Host with DISCONNECT status should not be in search results'
-  end
-
-  test 'scoped search for user_omitted insights client report status works' do
-    host1 = FactoryBot.create(:host, :managed)
-    host2 = FactoryBot.create(:host, :managed)
-    host3 = FactoryBot.create(:host, :managed)
-
-    # Create different insights client report statuses
-    status1 = host1.get_status(InsightsClientReportStatus)
-    status1.status = InsightsClientReportStatus::REPORTING
-    status1.save!
-
-    status2 = host2.get_status(InsightsClientReportStatus)
-    status2.status = InsightsClientReportStatus::NO_REPORT
-    status2.save!
-
-    status3 = host3.get_status(InsightsClientReportStatus)
-    status3.status = InsightsClientReportStatus::USER_OMITTED
-    status3.save!
-
-    # Search for user_omitted status
-    results = Host.search_for('insights_client_report_status = user_omitted')
-    result_ids = results.pluck(:id)
-
-    assert_includes result_ids, host3.id, 'Host with USER_OMITTED status should be in search results'
-    assert_not_includes result_ids, host1.id, 'Host with REPORTING status should not be in search results'
-    assert_not_includes result_ids, host2.id, 'Host with NO_REPORT status should not be in search results'
-  end
 end

data/test/unit/services/foreman_rh_cloud/branch_info_test.rb

@@ -19,7 +19,7 @@ class BranchInfoTest < ActiveSupport::TestCase
       organization: env.organization
     )
 
-    @host.subscription_facet.pools << FactoryBot.create(:katello_pool, account_number: '5678', cp_id: 1)
+    @host.organization.pools << FactoryBot.create(:katello_pool, account_number: '5678', cp_id: 1)
   end
 
   test 'should generate branch info for host' do

data/test/unit/services/foreman_rh_cloud/cloud_request_forwarder_test.rb

@@ -29,7 +29,7 @@ class CloudRequestForwarderTest < ActiveSupport::TestCase
       organization: env.organization
     )
 
-    @host.subscription_facet.pools << FactoryBot.create(:katello_pool, account_number: '5678', cp_id: 1)
+    @host.organization.pools << FactoryBot.create(:katello_pool, account_number: '5678', cp_id: 1)
   end
 
   test 'should prepare correct cloud url' do

data/test/unit/slice_generator_test.rb

@@ -641,6 +641,7 @@ class SliceGeneratorTest < ActiveSupport::TestCase
     assert_tag('1', actual_host, 'int_param', 'satellite_parameter')
 
     assert_equal false, satellite_facts['is_hostname_obfuscated']
+    assert_equal true, satellite_facts['is_simple_content_access']
 
     version = satellite_facts['satellite_version']
     if defined?(ForemanThemeSatellite)
@@ -702,22 +703,6 @@ class SliceGeneratorTest < ActiveSupport::TestCase
     assert_equal 'test_sla', fact_values['system_purpose_sla']
   end
 
-  test 'generates a report for a golden ticket' do
-    batch = Host.where(id: @host.id).in_batches.first
-    generator = create_generator(batch) do |generator|
-      generator.stubs(:golden_ticket?).returns(true)
-    end
-
-    json_str = generator.render
-    actual = JSON.parse(json_str.join("\n"))
-
-    assert_equal '00000000-0000-0000-0000-000000000000', actual['report_slice_id']
-    assert_not_nil(actual_host = actual['hosts'].first)
-    assert_equal @host.fqdn, actual_host['fqdn']
-    assert_equal '1234', actual_host['account']
-    assert_equal 1, generator.hosts_count
-  end
-
   test 'skips hosts without subscription' do
     a_host = FactoryBot.create(
       :host,
@@ -1131,11 +1116,7 @@ class SliceGeneratorTest < ActiveSupport::TestCase
 
   def create_generator(batch, name = '00000000-0000-0000-0000-000000000000')
     generator = ForemanInventoryUpload::Generators::Slice.new(batch, [], name)
-    if block_given?
-      yield(generator)
-    else
-      generator.stubs(:golden_ticket?).returns(false)
-    end
+    yield(generator) if block_given?
     generator
   end
 

data/webpack/CVEsHostDetailsTab/CVEsHostDetailsTab.js

@@ -10,12 +10,7 @@ const CVEsHostDetailsTab = ({ systemId }) => {
   const module = './SystemDetailTable';
   return (
     <div className="rh-cloud-insights-vulnerability-host-details-component vulnerability">
-      <ScalprumComponent
-        key={systemId}
-        scope={scope}
-        module={module}
-        systemId={systemId}
-      />
+      <ScalprumComponent scope={scope} module={module} systemId={systemId} />
     </div>
   );
 };

data/webpack/CVEsHostDetailsTab/__tests__/CVEsHostDetailsTab.test.js

@@ -11,25 +11,14 @@ jest.mock('foremanReact/Root/Context/ForemanContext', () => ({
   useForemanPermissions: () => new Set(['view_vulnerability']),
 }));
 
-const mockUnmountTracker = jest.fn();
-jest.mock('@scalprum/react-core', () => {
-  const ReactMock = require('react');
-  return {
-    ScalprumComponent: jest.fn(props => {
-      ReactMock.useEffect(() => mockUnmountTracker, []);
-      return (
-        <div data-testid="mock-scalprum-component">{JSON.stringify(props)}</div>
-      );
-    }),
-    ScalprumProvider: jest.fn(({ children }) => <div>{children}</div>),
-  };
-});
+jest.mock('@scalprum/react-core', () => ({
+  ScalprumComponent: jest.fn(props => (
+    <div data-testid="mock-scalprum-component">{JSON.stringify(props)}</div>
+  )),
+  ScalprumProvider: jest.fn(({ children }) => <div>{children}</div>),
+}));
 
 describe('CVEsHostDetailsTabWrapper', () => {
-  beforeEach(() => {
-    jest.clearAllMocks();
-  });
-
   it('renders without crashing', () => {
     const { container } = render(
       <CVEsHostDetailsTabWrapper
@@ -42,32 +31,4 @@ describe('CVEsHostDetailsTabWrapper', () => {
       )
     ).toBeTruthy();
   });
-
-  it('remounts ScalprumComponent when systemId changes', () => {
-    const { ScalprumComponent } = require('@scalprum/react-core');
-
-    const { rerender } = render(
-      <CVEsHostDetailsTabWrapper
-        response={{ subscription_facet_attributes: { uuid: 'uuid-host-A' } }}
-      />
-    );
-
-    expect(mockUnmountTracker).not.toHaveBeenCalled();
-    expect(ScalprumComponent).toHaveBeenLastCalledWith(
-      expect.objectContaining({ systemId: 'uuid-host-A' }),
-      expect.anything()
-    );
-
-    rerender(
-      <CVEsHostDetailsTabWrapper
-        response={{ subscription_facet_attributes: { uuid: 'uuid-host-B' } }}
-      />
-    );
-
-    expect(mockUnmountTracker).toHaveBeenCalledTimes(1);
-    expect(ScalprumComponent).toHaveBeenLastCalledWith(
-      expect.objectContaining({ systemId: 'uuid-host-B' }),
-      expect.anything()
-    );
-  });
 });

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/PageDescription/PageDescription.js

@@ -7,8 +7,6 @@ import { getDocsURL } from 'foremanReact/common/helpers';
 import { FormattedMessage } from 'react-intl';
 import { selectSubscriptionConnectionEnabled } from '../../../InventorySettings/InventorySettingsSelectors';
 
-import { getSubscriptionServiceDocsUrl } from '../../../../ForemanInventoryHelpers';
-
 export const PageDescription = () => {
   const subscriptionConnectionEnabled = useSelector(
     selectSubscriptionConnectionEnabled
@@ -82,7 +80,7 @@ export const PageDescription = () => {
         {__('For more information about the Subscriptions service, see:')}
          
         <a
-          href={getSubscriptionServiceDocsUrl()}
+          href="https://docs.redhat.com/en/documentation/subscription_central/1-latest/html/getting_started_with_the_subscriptions_service/index"
           target="_blank"
           rel="noopener noreferrer"
         >

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/SyncButtonActions.js

@@ -39,20 +39,14 @@ export const setupInventorySyncTaskPolling = (id, dispatch) =>
     key: INVENTORY_SYNC_TASK_UPDATE,
     onTaskSuccess: ({
       output: {
-        host_statuses: { sync, disconnect, user_omitted: userOmitted },
+        host_statuses: { sync, disconnect },
       },
     }) =>
       dispatch(
         addToast({
           sticky: true,
           type: 'success',
-          message: (
-            <Toast
-              syncHosts={sync}
-              disconnectHosts={disconnect}
-              userOmittedHosts={userOmitted}
-            />
-          ),
+          message: <Toast syncHosts={sync} disconnectHosts={disconnect} />,
         })
       ),
     dispatch,

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/__tests__/__snapshots__/integrations.test.js.snap

@@ -9,7 +9,6 @@ Array [
           "message": <Toast
             disconnectHosts={2}
             syncHosts={0}
-            userOmittedHosts={1}
           />,
           "sticky": true,
           "type": "success",

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/__tests__/integrations.test.js

@@ -30,7 +30,6 @@ describe('SyncButton integration test', () => {
                 host_statuses: {
                   sync: 0,
                   disconnect: 2,
-                  user_omitted: 1,
                 },
               },
               result: 'success',

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/components/Toast.js

@@ -1,55 +1,33 @@
 import React from 'react';
-import { Link } from 'react-router-dom';
 import PropTypes from 'prop-types';
 import { translate as __ } from 'foremanReact/common/I18n';
+import { foremanUrl } from '../../../../../../ForemanRhCloudHelpers';
 
-const statusSearchParams = statusName =>
-  `/new/hosts?search=insights_inventory_sync_status+%3D+${statusName}&page=1`;
-const DISCONNECT = 'disconnect';
-const SYNC = 'sync';
-const USER_OMITTED = 'user_omitted';
-const HostsWithStatusLink = ({ statusName, children }) => (
-  <Link to={statusSearchParams(statusName)}>{children}</Link>
-);
-HostsWithStatusLink.propTypes = {
-  statusName: PropTypes.string.isRequired,
-  children: PropTypes.node.isRequired,
-};
-
-const Toast = ({ syncHosts, disconnectHosts, userOmittedHosts }) => {
-  const totalHosts = syncHosts + disconnectHosts + userOmittedHosts;
+const Toast = ({ syncHosts, disconnectHosts }) => {
+  const totalHosts = syncHosts + disconnectHosts;
   return (
     <span>
       <p>
-        {__('Registered hosts in organization: ')}
-        <Link to="/new/hosts?search=set%3F+subscription_uuid&page=1">
-          {totalHosts}
-        </Link>
+        {__('Hosts with subscription in organization: ')}
+        <strong>{totalHosts}</strong>
       </p>
       <p>
-        {__('Uploaded and present on console.redhat.com Inventory service: ')}
-        <HostsWithStatusLink statusName={SYNC}>{syncHosts}</HostsWithStatusLink>
+        {__('Successfully synced hosts: ')}
+        <strong>{syncHosts}</strong>
       </p>
       <p>
-        {__('Not present on console.redhat.com Inventory service: ')}
-        <HostsWithStatusLink statusName={DISCONNECT}>
-          {disconnectHosts}
-        </HostsWithStatusLink>
+        {__('Disconnected hosts: ')}
+        <strong>{disconnectHosts}</strong>
       </p>
-      {!!userOmittedHosts && (
-        <p>
-          {__(
-            'Excluded from upload to console.redhat.com Inventory service because host_registration_insights_inventory parameter value is false: '
-          )}
-          <HostsWithStatusLink statusName={USER_OMITTED}>
-            {userOmittedHosts}
-          </HostsWithStatusLink>
-        </p>
-      )}
       <p>
-        {__(
-          'You can review this information later by looking at the Inventory status of each host.'
-        )}
+        {__('For more info, please visit the')}{' '}
+        <a
+          href={foremanUrl('/hosts')}
+          target="_blank"
+          rel="noopener noreferrer"
+        >
+          {__('hosts page')}
+        </a>
       </p>
     </span>
   );
@@ -58,10 +36,6 @@ const Toast = ({ syncHosts, disconnectHosts, userOmittedHosts }) => {
 Toast.propTypes = {
   syncHosts: PropTypes.number.isRequired,
   disconnectHosts: PropTypes.number.isRequired,
-  userOmittedHosts: PropTypes.number,
-};
-Toast.defaultProps = {
-  userOmittedHosts: 0,
 };
 
 export default Toast;

data/webpack/ForemanInventoryUpload/ForemanInventoryHelpers.js

@@ -11,13 +11,6 @@ export const getInventoryDocsUrl = () =>
     )}`
   );
 
-export const getSubscriptionServiceDocsUrl = () =>
-  foremanUrl(
-    `/links/manual/?root_url=${URI.encode(
-      'https://docs.redhat.com/en/documentation/subscription_central/1-latest/html-single/getting_started_with_the_subscriptions_service/index'
-    )}`
-  );
-
 export const getActionsHistoryUrl = () =>
   foremanUrl(
     '/foreman_tasks/tasks?search=label+%3D+ForemanInventoryUpload%3A%3AAsync%3A%3AHostInventoryReportJob+or+label+%3D+ForemanInventoryUpload%3A%3AAsync%3A%3AGenerateAllReportsJob&page=1'

data/webpack/InsightsHostDetailsTab/NewHostDetailsTab.js

@@ -33,28 +33,13 @@ const NewHostDetailsTab = ({ hostName, router }) => {
   const hits = useSelector(selectHits);
   const isIop = useIopConfig();
 
-  useEffect(
-    () => () => {
-      // Preserve hash when clearing search params to prevent tab navigation bugs
-      if (router && typeof router.replace === 'function') {
-        const replaceOptions = { search: null };
-        if (router.location && router.location.hash) {
-          replaceOptions.hash = router.location.hash;
-        }
-        router.replace(replaceOptions);
-      }
-    },
-    [router]
-  );
+  useEffect(() => () => router.replace({ search: null }), [router]);
 
   const onSearch = q => dispatch(fetchInsights({ query: q, page: 1 }));
 
   const [isDropdownOpen, setIsDropdownOpen] = useState(false);
-  const onSatInsightsClick = () => {
-    if (router && typeof router.push === 'function') {
-      router.push({ pathname: '/foreman_rh_cloud/insights_cloud' });
-    }
-  };
+  const onSatInsightsClick = () =>
+    router.push({ pathname: '/foreman_rh_cloud/insights_cloud' });
 
   const dropdownItems = [
     <DropdownItem key="insights-link" ouiaId="insights-link">
@@ -128,32 +113,17 @@ NewHostDetailsTab.defaultProps = {
 const scope = 'advisor';
 const module = './SystemDetailWrapped';
 
-const IopInsightsTab = props => {
-  // eslint-disable-next-line camelcase
-  const systemId = props.response?.subscription_facet_attributes?.uuid;
-  return (
-    <div className="advisor">
-      <ScalprumComponent
-        key={systemId || props.hostName}
-        scope={scope}
-        module={module}
-        IopRemediationModal={RemediationModal}
-        generateRuleUrl={generateRuleUrl}
-        {...props}
-      />
-    </div>
-  );
-};
-
-IopInsightsTab.propTypes = {
-  hostName: PropTypes.string,
-  response: PropTypes.object,
-};
-
-IopInsightsTab.defaultProps = {
-  hostName: '',
-  response: {},
-};
+const IopInsightsTab = props => (
+  <div className="advisor">
+    <ScalprumComponent
+      scope={scope}
+      module={module}
+      IopRemediationModal={RemediationModal}
+      generateRuleUrl={generateRuleUrl}
+      {...props}
+    />
+  </div>
+);
 
 const IopInsightsTabWrapped = props => {
   const permissions = useInsightsPermissions();