foreman_rh_cloud 13.2.7 → 14.0.1

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/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();

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_rh_cloud
 version: !ruby/object:Gem::Version
-  version: 13.2.7
+  version: 14.0.1
 platform: ruby
 authors:
 - Foreman Red Hat Cloud team
+autorequire:
 bindir: bin
 cert_chain: []
-date: 1980-01-02 00:00:00.000000000 Z
+date: 2026-02-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: foreman_ansible
@@ -233,16 +234,26 @@ files:
 - locale/Makefile
 - locale/foreman_rh_cloud.pot
 - locale/fr/LC_MESSAGES/foreman_rh_cloud.mo
+- locale/fr/foreman_rh_cloud.edit.po
 - locale/fr/foreman_rh_cloud.po
+- locale/fr/foreman_rh_cloud.po.time_stamp
 - locale/gemspec.rb
 - locale/ja/LC_MESSAGES/foreman_rh_cloud.mo
+- locale/ja/foreman_rh_cloud.edit.po
 - locale/ja/foreman_rh_cloud.po
+- locale/ja/foreman_rh_cloud.po.time_stamp
 - locale/ka/LC_MESSAGES/foreman_rh_cloud.mo
+- locale/ka/foreman_rh_cloud.edit.po
 - locale/ka/foreman_rh_cloud.po
+- locale/ka/foreman_rh_cloud.po.time_stamp
 - locale/ko/LC_MESSAGES/foreman_rh_cloud.mo
+- locale/ko/foreman_rh_cloud.edit.po
 - locale/ko/foreman_rh_cloud.po
+- locale/ko/foreman_rh_cloud.po.time_stamp
 - locale/zh_CN/LC_MESSAGES/foreman_rh_cloud.mo
+- locale/zh_CN/foreman_rh_cloud.edit.po
 - locale/zh_CN/foreman_rh_cloud.po
+- locale/zh_CN/foreman_rh_cloud.po.time_stamp
 - package.json
 - test/controllers/accounts_controller_test.rb
 - test/controllers/insights_cloud/api/advisor_engine_controller_test.rb
@@ -586,7 +597,6 @@ files:
 - webpack/InsightsHostDetailsTab/__tests__/InsightsTabReducer.test.js
 - webpack/InsightsHostDetailsTab/__tests__/InsightsTabSelectors.test.js
 - webpack/InsightsHostDetailsTab/__tests__/InsightsTotalRiskChart.test.js
-- webpack/InsightsHostDetailsTab/__tests__/NewHostDetailsTab.test.js
 - webpack/InsightsHostDetailsTab/__tests__/__snapshots__/InsightsTab.test.js.snap
 - webpack/InsightsHostDetailsTab/__tests__/__snapshots__/InsightsTabActions.test.js.snap
 - webpack/InsightsHostDetailsTab/__tests__/__snapshots__/InsightsTabReducer.test.js.snap
@@ -652,6 +662,7 @@ homepage: https://github.com/theforeman/foreman_rh_cloud
 licenses:
 - GPL-3.0
 metadata: {}
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -669,7 +680,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.6
+rubygems_version: 3.2.33
+signing_key:
 specification_version: 4
 summary: Summary of ForemanRhCloud.
 test_files:

data/webpack/InsightsHostDetailsTab/__tests__/NewHostDetailsTab.test.js

@@ -1,154 +0,0 @@
-import React from 'react';
-import { render } from '@testing-library/react';
-import '@testing-library/jest-dom';
-import { Provider } from 'react-redux';
-import configureMockStore from 'redux-mock-store';
-import thunk from 'redux-thunk';
-import NewHostDetailsTab from '../NewHostDetailsTab';
-
-jest.mock('../../common/Hooks/ConfigHooks', () => ({
-  useIopConfig: jest.fn(() => false),
-}));
-
-jest.mock('foremanReact/common/I18n', () => ({
-  translate: jest.fn(str => str),
-}));
-
-const mockStore = configureMockStore([thunk]);
-
-describe('NewHostDetailsTab', () => {
-  let store;
-  let mockRouter;
-
-  beforeEach(() => {
-    mockRouter = {
-      push: jest.fn(),
-      replace: jest.fn(),
-      location: {
-        pathname: '/new/hosts/test-host.example.com',
-        search: '?page=1&per_page=20',
-        hash: '#/Insights',
-        query: { page: '1', per_page: '20' },
-      },
-    };
-
-    store = mockStore({
-      API: {},
-      ForemanRhCloud: {
-        InsightsCloudSync: {
-          table: {
-            selectedIds: {},
-            isAllSelected: false,
-            showSelectAllAlert: false,
-          },
-        },
-      },
-      insightsHostDetailsTab: {
-        query: '',
-        hits: [],
-        selectedIds: {},
-        error: null,
-      },
-      router: {
-        location: {
-          pathname: '/new/hosts/test-host.example.com',
-          search: '?page=1&per_page=20',
-          hash: '#/Insights',
-          query: { page: '1', per_page: '20' },
-        },
-      },
-    });
-  });
-
-  afterEach(() => {
-    jest.clearAllMocks();
-  });
-
-  describe('cleanup effect', () => {
-    it('should preserve hash when clearing search params on unmount', () => {
-      const { unmount } = render(
-        <Provider store={store}>
-          <NewHostDetailsTab
-            hostName="test-host.example.com"
-            router={mockRouter}
-          />
-        </Provider>
-      );
-
-      // Unmount the component to trigger cleanup
-      unmount();
-
-      // Verify router.replace was called with both search: null AND the existing hash
-      expect(mockRouter.replace).toHaveBeenCalledWith({
-        search: null,
-        hash: '#/Insights',
-      });
-    });
-
-    it('should only clear search params when no hash exists', () => {
-      mockRouter.location.hash = '';
-
-      const { unmount } = render(
-        <Provider store={store}>
-          <NewHostDetailsTab
-            hostName="test-host.example.com"
-            router={mockRouter}
-          />
-        </Provider>
-      );
-
-      unmount();
-
-      // When there's no hash, should only pass search: null
-      expect(mockRouter.replace).toHaveBeenCalledWith({
-        search: null,
-      });
-    });
-
-    it('should handle router.location being undefined gracefully', () => {
-      const routerWithoutLocation = {
-        push: jest.fn(),
-        replace: jest.fn(),
-        location: undefined,
-      };
-
-      const { unmount } = render(
-        <Provider store={store}>
-          <NewHostDetailsTab
-            hostName="test-host.example.com"
-            router={routerWithoutLocation}
-          />
-        </Provider>
-      );
-
-      unmount();
-
-      // Should still call replace with search: null even if location is undefined
-      expect(routerWithoutLocation.replace).toHaveBeenCalledWith({
-        search: null,
-      });
-    });
-
-    it('should use the latest hash value at unmount time, not a stale captured value', () => {
-      const { unmount } = render(
-        <Provider store={store}>
-          <NewHostDetailsTab
-            hostName="test-host.example.com"
-            router={mockRouter}
-          />
-        </Provider>
-      );
-
-      // Change the hash after mount, before unmount
-      mockRouter.location.hash = '#/Overview';
-
-      unmount();
-
-      // Verify router.replace was called with the UPDATED hash, not the initial '#/Insights'
-      expect(mockRouter.replace).toHaveBeenCalledWith({
-        search: null,
-        hash: '#/Overview',
-      });
-    });
-  });
-});