foreman_rh_cloud 14.1.2 → 14.1.3

data/webpack/ForemanInventoryUpload/Components/AccountList/__tests__/AccountListReducer.test.js

@@ -1,5 +1,3 @@
-import { testReducerSnapshotWithFixtures } from '@theforeman/test';
-
 import {
   INVENTORY_ACCOUNT_STATUS_POLLING,
   INVENTORY_ACCOUNT_STATUS_POLLING_START,
@@ -14,45 +12,67 @@ import {
   accountID,
   processStatusName,
   pollingResponse,
+  accounts,
+  CloudConnectorStatus,
 } from '../AccountList.fixtures';
 
-const fixtures = {
-  'should return the initial state': {},
-  'should handle INVENTORY_ACCOUNT_STATUS_POLLING': {
-    action: {
+describe('AccountList reducer', () => {
+  it('returns the initial state', () => {
+    const state = reducer(undefined, { type: '@@INIT' });
+    expect(state.accounts).toEqual({});
+    expect(state.pollingProcessID).toBe(0);
+    expect(state.error).toBeNull();
+  });
+
+  it('handles INVENTORY_ACCOUNT_STATUS_POLLING', () => {
+    const state = reducer(undefined, {
       type: INVENTORY_ACCOUNT_STATUS_POLLING,
       payload: pollingResponse,
-    },
-  },
-  'should handle INVENTORY_ACCOUNT_STATUS_POLLING_ERROR': {
-    action: {
+    });
+    expect(state.accounts).toEqual(accounts);
+    expect(state.CloudConnectorStatus).toEqual(CloudConnectorStatus);
+    expect(state.error).toBeNull();
+  });
+
+  it('handles INVENTORY_ACCOUNT_STATUS_POLLING_ERROR', () => {
+    const state = reducer(undefined, {
       type: INVENTORY_ACCOUNT_STATUS_POLLING_ERROR,
       payload: { error },
-    },
-  },
-  'should handle INVENTORY_ACCOUNT_STATUS_POLLING_START': {
-    action: {
+    });
+    expect(state.accounts).toEqual({});
+    expect(state.error).toBe(error);
+  });
+
+  it('handles INVENTORY_ACCOUNT_STATUS_POLLING_START', () => {
+    const state = reducer(undefined, {
+      type: INVENTORY_ACCOUNT_STATUS_POLLING_START,
+      payload: { pollingProcessID },
+    });
+    expect(state.pollingProcessID).toBe(pollingProcessID);
+  });
+
+  it('handles INVENTORY_ACCOUNT_STATUS_POLLING_STOP (default case)', () => {
+    const prevState = reducer(undefined, {
       type: INVENTORY_ACCOUNT_STATUS_POLLING_START,
-      payload: {
-        pollingProcessID,
-      },
-    },
-  },
-  'should handle INVENTORY_ACCOUNT_STATUS_POLLING_STOP': {
-    action: {
+      payload: { pollingProcessID },
+    });
+    const state = reducer(prevState, {
       type: INVENTORY_ACCOUNT_STATUS_POLLING_STOP,
-    },
-  },
-  'should handle INVENTORY_PROCESS_RESTART': {
-    action: {
-      type: INVENTORY_PROCESS_RESTART,
-      payload: {
-        accountID,
-        processStatusName,
-      },
-    },
-  },
-};
+    });
+    expect(state.pollingProcessID).toBe(pollingProcessID);
+    expect(state.accounts).toEqual({});
+    expect(state.error).toBeNull();
+  });
 
-describe('AccountList reducer', () =>
-  testReducerSnapshotWithFixtures(reducer, fixtures));
+  it('handles INVENTORY_PROCESS_RESTART', () => {
+    const stateWithAccounts = reducer(undefined, {
+      type: INVENTORY_ACCOUNT_STATUS_POLLING,
+      payload: pollingResponse,
+    });
+    const state = reducer(stateWithAccounts, {
+      type: INVENTORY_PROCESS_RESTART,
+      payload: { accountID, processStatusName },
+    });
+    expect(state.accounts[accountID][processStatusName]).toBe('Restarting...');
+  });
+});

data/webpack/ForemanInventoryUpload/Components/FileDownload/__tests__/FileDownload.test.js

@@ -1,13 +1,17 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
-
+import React from 'react';
+import { render, screen, fireEvent } from '@testing-library/react';
 import FileDownload from '../FileDownload';
 
-const fixtures = {
-  'render without Props': {},
-  /** fixtures, props for the component */
-};
-
 describe('FileDownload', () => {
-  describe('rendering', () =>
-    testComponentSnapshotsWithFixtures(FileDownload, fixtures));
+  it('renders the download button', () => {
+    render(<FileDownload />);
+    expect(screen.getByRole('button', { name: /Download Report/ })).toBeTruthy();
+  });
+
+  it('calls onClick when button is clicked', () => {
+    const onClick = jest.fn();
+    render(<FileDownload onClick={onClick} />);
+    fireEvent.click(screen.getByRole('button', { name: /Download Report/ }));
+    expect(onClick).toHaveBeenCalledTimes(1);
+  });
 });

data/webpack/ForemanInventoryUpload/Components/InventoryFilter/__tests__/InventoryFilter.test.js

@@ -1,20 +1,17 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
-import { noop } from 'foremanReact/common/helpers';
-
+import React from 'react';
+import { render, screen } from '@testing-library/react';
 import InventoryFilter from '../InventoryFilter';
-import { filterTerm, organization } from '../InventoryFilter.fixtures';
 
-const fixtures = {
-  'render with props': {
-    handleFilterChange: noop,
-    handleFilterClear: noop,
-    filterTerm,
-    organization,
-  },
-  /** fixtures, props for the component */
-};
+jest.mock('foremanReact/Root/Context/ForemanContext');
 
 describe('InventoryFilter', () => {
-  describe('rendering', () =>
-    testComponentSnapshotsWithFixtures(InventoryFilter, fixtures));
+  it('renders the filter input', () => {
+    render(<InventoryFilter />);
+    expect(screen.getByPlaceholderText('Filter..')).toBeTruthy();
+  });
+
+  it('displays the current filter term', () => {
+    render(<InventoryFilter filterTerm="test-filter" />);
+    expect(screen.getByDisplayValue('test-filter')).toBeTruthy();
+  });
 });

data/webpack/ForemanInventoryUpload/Components/InventoryFilter/__tests__/integration.test.js

@@ -1,18 +1,38 @@
 import React from 'react';
-import { IntegrationTestHelper } from '@theforeman/test';
-import InventoryFilter from '../index';
-import reducers from '../../../../ForemanRhCloudReducers';
+import { render, screen, fireEvent } from '@testing-library/react';
+import { Provider } from 'react-redux';
+import configureMockStore from 'redux-mock-store';
+import thunk from 'redux-thunk';
+import ConnectedInventoryFilter from '../index';
+import { INVENTORY_FILTER_UPDATE } from '../InventoryFilterConstants';
+
+jest.mock('foremanReact/Root/Context/ForemanContext');
+
+const mockStore = configureMockStore([thunk]);
 
 describe('InventoryFilter integration test', () => {
-  it('should flow', async () => {
-    const integrationTestHelper = new IntegrationTestHelper(reducers);
-    const wrapper = integrationTestHelper.mount(<InventoryFilter />);
-    const input = wrapper.find('input[id="inventory_filter_input"]');
-    input.simulate('change', { target: { value: 'some_new_filter' } });
-    await IntegrationTestHelper.flushAllPromises();
-    wrapper.update();
-    integrationTestHelper.takeStoreAndLastActionSnapshot(
-      'filter have been updated'
+  it('dispatches filter update action on input change', () => {
+    const store = mockStore({
+      ForemanRhCloud: {
+        inventoryUpload: {
+          inventoryFilter: { filterTerm: '' },
+        },
+      },
+    });
+
+    render(
+      <Provider store={store}>
+        <ConnectedInventoryFilter />
+      </Provider>
+    );
+
+    const input = screen.getByPlaceholderText('Filter..');
+    fireEvent.change(input, { target: { value: 'some_new_filter' } });
+
+    const actions = store.getActions();
+    const filterAction = actions.find(
+      a => a.type === INVENTORY_FILTER_UPDATE && a.payload.filterTerm === 'some_new_filter'
     );
+    expect(filterAction).toBeTruthy();
   });
 });

data/webpack/ForemanInventoryUpload/Components/PageHeader/__tests__/PageTitle.test.js

@@ -1,12 +1,19 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
-
+import React from 'react';
+import { render, screen } from '@testing-library/react';
 import PageTitle from '../PageTitle';
 
-const fixtures = {
-  'render without Props': {},
-};
+jest.mock('../components/CloudPingModal', () => () => (
+  <div data-testid="cloud-ping-modal">CloudPingModal</div>
+));
 
 describe('PageTitle', () => {
-  describe('rendering', () =>
-    testComponentSnapshotsWithFixtures(PageTitle, fixtures));
+  it('renders the page title', () => {
+    render(<PageTitle />);
+    expect(screen.getByText('Red Hat Inventory')).toBeTruthy();
+  });
+
+  it('renders the kebab dropdown', () => {
+    const { container } = render(<PageTitle />);
+    expect(container.querySelector('.title-dropdown')).toBeTruthy();
+  });
 });

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/CloudConnectorButton/__tests__/CloudConnectorButton.test.js

@@ -1,22 +1,51 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
+import React from 'react';
+import { render, screen, fireEvent } from '@testing-library/react';
 import { CloudConnectorButton } from '../CloudConnectorButton';
 import { CONNECTOR_STATUS } from '../CloudConnectorConstants';
 
-const fixtures = {
-  'render no cloud connector': {
-    status: CONNECTOR_STATUS.NOT_RESOLVED,
-    onClick: jest.fn(),
-  },
-  'render resolved cloud connector': {
-    status: CONNECTOR_STATUS.RESOLVED,
-    onClick: jest.fn(),
-  },
-  'render pending connector': {
-    jobLink: '/job-link',
-    status: CONNECTOR_STATUS.PENDING,
-    onClick: jest.fn(),
-  },
-};
+describe('CloudConnectorButton', () => {
+  it('renders "Configure cloud connector" when not resolved', () => {
+    render(
+      <CloudConnectorButton
+        status={CONNECTOR_STATUS.NOT_RESOLVED}
+        onClick={jest.fn()}
+      />
+    );
+    expect(screen.getByRole('button', { name: /Configure cloud connector/ })).toBeTruthy();
+  });
 
-describe('CloudConnectorButton', () =>
-  testComponentSnapshotsWithFixtures(CloudConnectorButton, fixtures));
+  it('renders "Reconfigure cloud connector" when resolved', () => {
+    render(
+      <CloudConnectorButton
+        status={CONNECTOR_STATUS.RESOLVED}
+        onClick={jest.fn()}
+      />
+    );
+    expect(screen.getByRole('button', { name: /Reconfigure cloud connector/ })).toBeTruthy();
+  });
+
+  it('renders in-progress button when pending', () => {
+    render(
+      <CloudConnectorButton
+        status={CONNECTOR_STATUS.PENDING}
+        onClick={jest.fn()}
+        jobLink="/job-link"
+      />
+    );
+    expect(
+      screen.getByText('Cloud Connector is in progress')
+    ).toBeTruthy();
+  });
+
+  it('calls onClick when configure button is clicked', () => {
+    const onClick = jest.fn();
+    render(
+      <CloudConnectorButton
+        status={CONNECTOR_STATUS.NOT_RESOLVED}
+        onClick={onClick}
+      />
+    );
+    fireEvent.click(screen.getByRole('button', { name: /Configure cloud connector/ }));
+    expect(onClick).toHaveBeenCalledTimes(1);
+  });
+});

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SettingsWarning/SettingsWarning.test.js

@@ -1,18 +1,61 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
+import React from 'react';
+import { render, screen } from '@testing-library/react';
 import { SettingsWarning } from './SettingsWarning';
 
-const fixtures = {
-  'with 2 alerts': {
-    autoUpload: false,
-    hostObfuscation: true,
-    isCloudConnector: true,
-  },
-  'with isCloudConnector false': {
-    autoUpload: true,
-    hostObfuscation: true,
-    isCloudConnector: false,
-  },
-};
+describe('SettingsWarning', () => {
+  it('renders nothing when isCloudConnector is false', () => {
+    render(
+      <SettingsWarning autoUpload={false} isCloudConnector={false} />
+    );
+    expect(screen.queryByRole('alert')).toBeNull();
+  });
 
-describe('SettingsWarning', () =>
-  testComponentSnapshotsWithFixtures(SettingsWarning, fixtures));
+  it('renders nothing when autoUpload is on and obfuscation is off', () => {
+    render(
+      <SettingsWarning
+        autoUpload
+        hostObfuscation={false}
+        isCloudConnector
+      />
+    );
+    expect(screen.queryByRole('alert')).toBeNull();
+  });
+
+  it('renders upload warning when autoUpload is disabled', () => {
+    render(
+      <SettingsWarning
+        autoUpload={false}
+        hostObfuscation={false}
+        isCloudConnector
+      />
+    );
+    expect(
+      screen.getByText(/inventory auto-upload is disabled/)
+    ).toBeTruthy();
+  });
+
+  it('renders obfuscation warning when hostObfuscation is enabled', () => {
+    render(
+      <SettingsWarning autoUpload hostObfuscation isCloudConnector />
+    );
+    expect(
+      screen.getByText(/obfuscating host names setting is enabled/)
+    ).toBeTruthy();
+  });
+
+  it('renders both warnings when both conditions met', () => {
+    render(
+      <SettingsWarning
+        autoUpload={false}
+        hostObfuscation
+        isCloudConnector
+      />
+    );
+    expect(
+      screen.getByText(/inventory auto-upload is disabled/)
+    ).toBeTruthy();
+    expect(
+      screen.getByText(/obfuscating host names setting is enabled/)
+    ).toBeTruthy();
+  });
+});

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

@@ -1,13 +1,27 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
-import { noop } from 'foremanReact/common/helpers';
-
+import React from 'react';
+import { render, screen, fireEvent } from '@testing-library/react';
 import SyncButton from '../SyncButton';
 
-const fixtures = {
-  'render with Props': { handleSync: noop },
-};
-
 describe('SyncButton', () => {
-  describe('rendering', () =>
-    testComponentSnapshotsWithFixtures(SyncButton, fixtures));
+  it('renders the sync button text', () => {
+    render(<SyncButton handleSync={jest.fn()} />);
+    expect(screen.getByRole('button', { name: /Sync all inventory status/ })).toBeTruthy();
+  });
+
+  it('calls handleSync on click', () => {
+    const handleSync = jest.fn();
+    render(<SyncButton handleSync={handleSync} />);
+    fireEvent.click(screen.getByRole('button'));
+    expect(handleSync).toHaveBeenCalledTimes(1);
+  });
+
+  it('disables button when status is PENDING', () => {
+    render(<SyncButton handleSync={jest.fn()} status="PENDING" />);
+    expect(screen.getByRole('button').disabled).toBe(true);
+  });
+
+  it('enables button when status is not PENDING', () => {
+    render(<SyncButton handleSync={jest.fn()} status="RESOLVED" />);
+    expect(screen.getByRole('button').disabled).toBe(false);
+  });
 });

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

@@ -1,27 +1,29 @@
-import { testSelectorsSnapshotWithFixtures } from '@theforeman/test';
 import { selectTaskStatus } from '../SyncButtonSelectors';
 
-const state = {
+const buildState = result => ({
   API: {
     INVENTORY_SYNC_TASK_UPDATE: {
       response: {
-        endedAt: '2021-03-08T14:27:30.718+02:00',
-        output: {
-          host_statuses: {
-            sync: 0,
-            disconnect: 2,
-          },
-        },
-        result: 'pending',
+        result,
       },
-      status: 'RESOLVED',
     },
   },
-};
+});
 
-const fixtures = {
-  'should return InventorySync status': () => selectTaskStatus(state),
-};
+describe('SyncButton selectors', () => {
+  it('uppercases the result string', () => {
+    expect(selectTaskStatus(buildState('pending'))).toBe('PENDING');
+  });
 
-describe('SyncButton selectors', () =>
-  testSelectorsSnapshotWithFixtures(fixtures));
+  it('handles other result strings', () => {
+    expect(selectTaskStatus(buildState('success'))).toBe('SUCCESS');
+  });
+
+  it('returns null when result is not a string', () => {
+    expect(selectTaskStatus(buildState(null))).toBeNull();
+  });
+
+  it('returns null when result is undefined', () => {
+    expect(selectTaskStatus(buildState(undefined))).toBeNull();
+  });
+});

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

@@ -1,52 +1,40 @@
 import React from 'react';
-import { IntegrationTestHelper } from '@theforeman/test';
+import { render, screen, fireEvent } from '@testing-library/react';
+import { Provider } from 'react-redux';
+import configureMockStore from 'redux-mock-store';
+import thunk from 'redux-thunk';
 import * as API from 'foremanReact/redux/API';
-import { noop } from 'foremanReact/common/helpers';
-import SyncButton from '../index';
+import ConnectedSyncButton from '../index';
 import { successResponse } from './SyncButtonFixtures';
-import {
-  INVENTORY_SYNC,
-  INVENTORY_SYNC_TASK_UPDATE,
-} from '../SyncButtonConstants';
+import { INVENTORY_SYNC } from '../SyncButtonConstants';
 
 jest.spyOn(API, 'post');
-jest.spyOn(API, 'get');
+
+const mockStore = configureMockStore([thunk]);
 
 describe('SyncButton integration test', () => {
-  it('Successful task was triggered on the server resulting in an info toast and polling on the task', async () => {
-    API.post.mockImplementation(({ handleSuccess = noop, key, ...action }) => {
-      if (key === INVENTORY_SYNC) {
+  it('dispatches sync action when button is clicked', () => {
+    API.post.mockImplementation(({ handleSuccess, key, ...action }) => {
+      if (key === INVENTORY_SYNC && handleSuccess) {
         handleSuccess(successResponse);
       }
       return { type: 'API_POST', ...action };
     });
-    API.get.mockImplementation(({ handleSuccess = noop, key, ...action }) => {
-      if (key === INVENTORY_SYNC_TASK_UPDATE) {
-        handleSuccess(
-          {
-            data: {
-              endedAt: '2021-03-22T15:59:02.468+02:00',
-              output: {
-                host_statuses: {
-                  sync: 0,
-                  disconnect: 2,
-                  user_omitted: 1,
-                },
-              },
-              result: 'success',
-            },
-          },
-          jest.fn
-        );
-      }
-      return { type: 'API_GET', ...action };
+
+    const store = mockStore({
+      API: {},
     });
 
-    const integrationTestHelper = new IntegrationTestHelper();
-    const wrapper = integrationTestHelper.mount(<SyncButton />);
-    const instance = wrapper.find('SyncButton').instance();
-    instance.props.handleSync();
-    await IntegrationTestHelper.flushAllPromises();
-    integrationTestHelper.takeActionsSnapshot('handleSync was called');
+    render(
+      <Provider store={store}>
+        <ConnectedSyncButton />
+      </Provider>
+    );
+
+    fireEvent.click(screen.getByRole('button'));
+
+    const actions = store.getActions();
+    const syncAction = actions.find(a => a.type === 'API_POST');
+    expect(syncAction).toBeTruthy();
   });
 });

data/webpack/ForemanInventoryUpload/Components/ScheduledRun/__tests__/ScheduledRun.test.js

@@ -1,13 +1,33 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
-
+import React from 'react';
+import { render, screen } from '@testing-library/react';
+import { IntlProvider } from 'react-intl';
 import ScheduledRun from '../ScheduledRun';
-import { props } from '../ScheduledRun.fixtures';
 
-const fixtures = {
-  'render with Props': props,
-};
+const renderWithIntl = ui =>
+  render(<IntlProvider locale="en">{ui}</IntlProvider>);
 
 describe('ScheduledRun', () => {
-  describe('rendering', () =>
-    testComponentSnapshotsWithFixtures(ScheduledRun, fixtures));
+  it('renders next run info when autoUploadEnabled and date are set', () => {
+    renderWithIntl(
+      <ScheduledRun date="2019-08-21T16:14:16.520+03:00" autoUploadEnabled />
+    );
+    expect(screen.getByText(/Next run:/)).toBeTruthy();
+  });
+
+  it('renders nothing when autoUploadEnabled is false', () => {
+    renderWithIntl(
+      <ScheduledRun
+        date="2019-08-21T16:14:16.520+03:00"
+        autoUploadEnabled={false}
+      />
+    );
+    expect(screen.queryByText(/Next run:/)).toBeNull();
+  });
+
+  it('renders nothing when date is null', () => {
+    renderWithIntl(
+      <ScheduledRun date={null} autoUploadEnabled />
+    );
+    expect(screen.queryByText(/Next run:/)).toBeNull();
+  });
 });

data/webpack/ForemanInventoryUpload/Components/StatusChart/__tests__/StatusChart.test.js

@@ -1,13 +1,30 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
-
+import React from 'react';
+import { render } from '@testing-library/react';
 import StatusChart from '../StatusChart';
 
-const fixtures = {
-  'render without Props': {},
-  /** fixtures, props for the component */
-};
+jest.mock('patternfly-react', () => ({
+  Grid: { Col: ({ children }) => <div>{children}</div> },
+  DonutChart: ({ title }) => (
+    <div data-testid="donut-chart">
+      <span>{title?.primary}</span>
+      <span>{title?.secondary}</span>
+    </div>
+  ),
+}));
 
 describe('StatusChart', () => {
-  describe('rendering', () =>
-    testComponentSnapshotsWithFixtures(StatusChart, fixtures));
+  it('renders the status chart container', () => {
+    const { container } = render(<StatusChart />);
+    expect(container.querySelector('.status-chart')).toBeTruthy();
+  });
+
+  it('passes the completed percentage to the chart', () => {
+    const { getByText } = render(<StatusChart completed={75} />);
+    expect(getByText('75%')).toBeTruthy();
+  });
+
+  it('shows Completed label', () => {
+    const { getByText } = render(<StatusChart completed={50} />);
+    expect(getByText('Completed')).toBeTruthy();
+  });
 });

data/webpack/ForemanInventoryUpload/Components/TabContainer/__tests__/TabContainer.test.js

@@ -1,13 +1,15 @@
-import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
-
+import React from 'react';
+import { render, screen } from '@testing-library/react';
 import TabContainer from '../TabContainer';
 
-const fixtures = {
-  'render without Props': {},
-  /** fixtures, props for the component */
-};
-
 describe('TabContainer', () => {
-  describe('rendering', () =>
-    testComponentSnapshotsWithFixtures(TabContainer, fixtures));
+  it('renders children', () => {
+    render(<TabContainer><span>Tab content</span></TabContainer>);
+    expect(screen.getByText('Tab content')).toBeTruthy();
+  });
+
+  it('renders with custom className', () => {
+    const { container } = render(<TabContainer className="my-tab" />);
+    expect(container.querySelector('.my-tab')).toBeTruthy();
+  });
 });