WIP - SAT-38755 - Convert RH Cloud Host Details snapshots to RTL

webpack/InsightsCloudSync/InsightsCloudSync.test.js

@@ -1,7 +1,12 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
+import React from 'react';
+import { render, screen } from '@testing-library/react';
+import '@testing-library/jest-dom';
 import { noop } from 'foremanReact/common/helpers';
 import InsightsCloudSync from './InsightsCloudSync';
 
+// Mock webpack share scopes for module federation
+global.__webpack_share_scopes__ = { default: {} };
+
 jest.mock('foremanReact/Root/Context/ForemanContext', () => ({
   useForemanContext: () => ({
     metadata: {
@@ -12,14 +17,27 @@ jest.mock('foremanReact/Root/Context/ForemanContext', () => ({
   }),
 }));
 
-const fixtures = {
-  render: {
-    status: 'RESOLVED',
-    syncInsights: noop,
-    fetchInsights: noop,
-    query: '',
-  },
-};
+jest.mock('foremanReact/components/PF4/TableIndexPage/Table/TableHooks', () => ({
+  useBulkSelect: () => ({
+    selectedCount: 0,
+    selectAll: () => {},
+    selectNone: () => {},
+    selectOne: () => {},
+    isSelected: () => false,
+    selectedResults: [],
+  }),
+}));
 
-describe('InsightsCloudSync', () =>
-  testComponentSnapshotsWithFixtures(InsightsCloudSync, fixtures));
+describe('InsightsCloudSync', () => {
+  it('should render with props', () => {
+    const { container } = render(
+      <InsightsCloudSync
+        status="RESOLVED"
+        syncInsights={noop}
+        fetchInsights={noop}
+        query=""
+      />
+    );
+    expect(container.querySelector('.insights-cloud-sync')).toBeInTheDocument();
+  });
+});

webpack/InsightsHostDetailsTab/__tests__/InsightsTab.test.js

@@ -1,13 +1,25 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
+import React from 'react';
+import { render, screen } from '@testing-library/react';
+import '@testing-library/jest-dom';
 
 import InsightsTab from '../InsightsTab';
 import { props } from './InsightsTab.fixtures';
 
-const fixtures = {
-  'render with props': props,
-};
-
 describe('InsightsTab', () => {
-  describe('rendering', () =>
-    testComponentSnapshotsWithFixtures(InsightsTab, fixtures));
+  describe('rendering', () => {
+    it('should render with props', () => {
+      render(<InsightsTab {...props} />);
+      expect(screen.getByText('Recommendations')).toBeInTheDocument();
+    });
+
+    it('should render empty state when hits array is empty', () => {
+      render(<InsightsTab hostID={1} hits={[]} />);
+      expect(screen.getByText('No recommendations were found for this host!')).toBeInTheDocument();
+    });
+
+    it('should use default props when hits is not provided', () => {
+      render(<InsightsTab hostID={1} />);
+      expect(screen.getByText('No recommendations were found for this host!')).toBeInTheDocument();
+    });
+  });
 });

webpack/InsightsHostDetailsTab/__tests__/InsightsTabActions.test.js

@@ -1,19 +1,52 @@
-import { testActionSnapshotWithFixtures } from '@theforeman/test';
 import { API } from 'foremanReact/redux/API';
 import { fetchHits } from '../InsightsTabActions';
 import { hostID, hits } from './InsightsTab.fixtures';
+import {
+  INSIGHTS_HITS_REQUEST,
+  INSIGHTS_HITS_SUCCESS,
+  INSIGHTS_HITS_FAILURE,
+} from '../InsightsTabConstants';
 
 jest.mock('foremanReact/redux/API');
-API.get.mockImplementation(async () => ({ data: { hits } }));
 
-const fixtures = {
-  'should fetchHits': () => fetchHits(hostID),
-  'should fetchHits with error': () => {
+describe('InsightsTab actions', () => {
+  it('should fetchHits', async () => {
+    API.get.mockImplementation(async () => ({ data: { hits } }));
+    const dispatch = jest.fn();
+
+    await fetchHits(hostID)(dispatch);
+
+    expect(dispatch).toHaveBeenCalledWith(
+      expect.objectContaining({
+        type: INSIGHTS_HITS_REQUEST,
+      })
+    );
+    expect(dispatch).toHaveBeenCalledWith(
+      expect.objectContaining({
+        type: INSIGHTS_HITS_SUCCESS,
+        payload: { hits },
+      })
+    );
+  });
+
+  it('should fetchHits with error', async () => {
     API.get.mockImplementationOnce(() =>
       Promise.reject(new Error('Network error!'))
     );
-    return fetchHits(hostID);
-  },
-};
+    const dispatch = jest.fn();
+
+    await fetchHits(hostID)(dispatch);
 
-describe('InsightsTab actions', () => testActionSnapshotWithFixtures(fixtures));
+    expect(dispatch).toHaveBeenCalledWith(
+      expect.objectContaining({
+        type: INSIGHTS_HITS_REQUEST,
+      })
+    );
+    // Error handling dispatches a toast notification, not INSIGHTS_HITS_FAILURE
+    expect(dispatch).toHaveBeenCalledWith(
+      expect.objectContaining({
+        type: 'TOASTS_ADD',
+      })
+    );
+  });
+});

webpack/InsightsHostDetailsTab/__tests__/InsightsTabReducer.test.js

@@ -1,26 +1,68 @@
-import { testReducerSnapshotWithFixtures } from '@theforeman/test';
+import Immutable from 'seamless-immutable';
 import reducer from '../InsightsTabReducer';
 import { hits } from './InsightsTab.fixtures';
 import {
   INSIGHTS_HITS_REQUEST,
   INSIGHTS_HITS_SUCCESS,
 } from '../InsightsTabConstants';
 
-const fixtures = {
-  'should return the initial state': {},
-  'should handle INSIGHTS_HITS_REQUEST': {
-    action: {
+describe('AccountList reducer', () => {
+  it('should return the initial state', () => {
+    const initialState = Immutable({ hits: [] });
+    expect(reducer(undefined, {})).toEqual(initialState);
+  });
+
+  it('should handle INSIGHTS_HITS_REQUEST', () => {
+    const initialState = Immutable({ hits: [] });
+    const action = {
       type: INSIGHTS_HITS_REQUEST,
       payload: {},
-    },
-  },
-  'should handle INSIGHTS_HITS_SUCCESS': {
-    action: {
+    };
+    const newState = reducer(initialState, action);
+    expect(newState).toHaveProperty('hits');
+    expect(newState.hits).toEqual([]);
+  });
+
+  it('should handle INSIGHTS_HITS_SUCCESS', () => {
+    const initialState = Immutable({ hits: [] });
+    const action = {
       type: INSIGHTS_HITS_SUCCESS,
       payload: { hits },
-    },
-  },
-};
+    };
+    const newState = reducer(initialState, action);
+    expect(newState.hits).toEqual(hits);
+  });
+
+  it('should handle INSIGHTS_HITS_SUCCESS with missing hits in payload', () => {
+    const initialState = Immutable({ hits: [] });
+    const action = {
+      type: INSIGHTS_HITS_SUCCESS,
+      payload: {},
+    };
+    const newState = reducer(initialState, action);
+    expect(newState).toHaveProperty('hits');
+    expect(newState.hits).toBeUndefined();
+  });
+
+  it('should handle INSIGHTS_HITS_SUCCESS with undefined hits', () => {
+    const initialState = Immutable({ hits: [] });
+    const action = {
+      type: INSIGHTS_HITS_SUCCESS,
+      payload: { hits: undefined },
+    };
+    const newState = reducer(initialState, action);
+    expect(newState).toHaveProperty('hits');
+    expect(newState.hits).toBeUndefined();
+  });
 
-describe('AccountList reducer', () =>
-  testReducerSnapshotWithFixtures(reducer, fixtures));
+  it('should handle INSIGHTS_HITS_SUCCESS with null hits', () => {
+    const initialState = Immutable({ hits: [] });
+    const action = {
+      type: INSIGHTS_HITS_SUCCESS,
+      payload: { hits: null },
+    };
+    const newState = reducer(initialState, action);
+    expect(newState).toHaveProperty('hits');
+    expect(newState.hits).toBeNull();
+  });
+});

webpack/InsightsHostDetailsTab/__tests__/InsightsTabSelectors.test.js

@@ -1,13 +1,11 @@
-import { testSelectorsSnapshotWithFixtures } from '@theforeman/test';
 import { hostInsightsStateWrapper } from '../../ForemanRhCloudTestHelpers';
 import { hits } from './InsightsTab.fixtures';
 import { selectHits } from '../InsightsTabSelectors';
 
 const state = hostInsightsStateWrapper({ hits });
 
-const fixtures = {
-  'should return hits': () => selectHits(state),
-};
-
-describe('InsightsTab selectors', () =>
-  testSelectorsSnapshotWithFixtures(fixtures));
+describe('InsightsTab selectors', () => {
+  it('should return hits', () => {
+    expect(selectHits(state)).toEqual(hits);
+  });
+});