onApply(newColumns)}
+ isOpen={isModalOpen}
+ setOpen={setModalOpen}
+ onClose={() => setModalOpen(false)}
+ key='column-mgmt-modal'
+ />,
+ setModalOpen,
+ ];
};
diff --git a/src/Utilities/hooks/Hooks.test.js b/src/Utilities/hooks/Hooks.test.js
index 00aec7285..41321ccdc 100644
--- a/src/Utilities/hooks/Hooks.test.js
+++ b/src/Utilities/hooks/Hooks.test.js
@@ -1,139 +1,150 @@
-/* eslint-disable */
import { SortByDirection } from '@patternfly/react-table';
-import { useEntitlements, useHandleRefresh, usePagePerPage,
- usePerPageSelect, useRemoveFilter, useSetPage, useSortColumn } from './Hooks';
+import {
+ useEntitlements,
+ useHandleRefresh,
+ usePagePerPage,
+ usePerPageSelect,
+ useRemoveFilter,
+ useSetPage,
+ useSortColumn,
+} from './Hooks';
import { packagesListDefaultFilters } from '../constants';
import { renderHook } from '@testing-library/react';
jest.mock('@redhat-cloud-services/frontend-components/useChrome', () => ({
- useChrome: jest.fn(() => ({
- auth: {
- getUser: () => new Promise(
- (resolve) => resolve({ entitlements: { 'test-entitelement': true } })
- )
- }
- }))
+ useChrome: jest.fn(() => ({
+ auth: {
+ getUser: () =>
+ new Promise((resolve) => resolve({ entitlements: { 'test-entitelement': true } })),
+ },
+ })),
}));
-
describe('Custom hooks tests', () => {
- it('useSetPage, should return correct offset for pagination', () => {
- let limit = 10;
- let page = 5;
- let finalResult = 40;
- let callback = jest.fn();
- const { result } = renderHook(() => useSetPage(limit, callback));
- result.current({}, page);
+ it('useSetPage, should return correct offset for pagination', () => {
+ let limit = 10;
+ let page = 5;
+ let finalResult = 40;
+ let callback = jest.fn();
+ const { result } = renderHook(() => useSetPage(limit, callback));
+ result.current({}, page);
- expect(callback).toHaveBeenCalledWith({ offset: finalResult });
- });
- it('usePerPageSelect, should return correct limit and offset', () => {
- let perPage = 15;
- let finalResult = { offset: 0, limit: perPage };
- let callback = jest.fn();
- const { result } = renderHook(() => usePerPageSelect(callback));
- result.current({}, perPage);
+ expect(callback).toHaveBeenCalledWith({ offset: finalResult });
+ });
+ it('usePerPageSelect, should return correct limit and offset', () => {
+ let perPage = 15;
+ let finalResult = { offset: 0, limit: perPage };
+ let callback = jest.fn();
+ const { result } = renderHook(() => usePerPageSelect(callback));
+ result.current({}, perPage);
- expect(callback).toHaveBeenCalledWith(finalResult);
- });
- it('useSortColumn, should return desc sort with offset ', () => {
- let columns = [
- {
- key: 'a'
- },
- {
- key: 'b'
- },
- {
- key: 'c'
- },
- {
- key: 'd'
- }
- ];
- let offset = 2;
- let index = 2;
- let direction = SortByDirection.desc;
- let finalResult = { sort: '-a' };
- let callback = jest.fn();
+ expect(callback).toHaveBeenCalledWith(finalResult);
+ });
+ it('useSortColumn, should return desc sort with offset ', () => {
+ let columns = [
+ {
+ key: 'a',
+ },
+ {
+ key: 'b',
+ },
+ {
+ key: 'c',
+ },
+ {
+ key: 'd',
+ },
+ ];
+ let offset = 2;
+ let index = 2;
+ let direction = SortByDirection.desc;
+ let finalResult = { sort: '-a' };
+ let callback = jest.fn();
- const { result } = renderHook(() => useSortColumn(columns, callback, offset));
- result.current({}, index, direction);
+ const { result } = renderHook(() => useSortColumn(columns, callback, offset));
+ result.current({}, index, direction);
- expect(callback).toHaveBeenCalledWith(finalResult);
- });
- it('useSortColumn, should return asc sort without offset', () => {
- let columns = [
- {
- key: 'a'
- },
- {
- key: 'b'
- },
- {
- key: 'c'
- },
- {
- key: 'd'
- }
- ];
- let index = 0;
- let direction = SortByDirection.asc;
- let finalResult = { sort: 'a' };
- let callback = jest.fn();
- const { result } = renderHook(() => useSortColumn(columns, callback));
- result.current({}, index, direction);
+ expect(callback).toHaveBeenCalledWith(finalResult);
+ });
+ it('useSortColumn, should return asc sort without offset', () => {
+ let columns = [
+ {
+ key: 'a',
+ },
+ {
+ key: 'b',
+ },
+ {
+ key: 'c',
+ },
+ {
+ key: 'd',
+ },
+ ];
+ let index = 0;
+ let direction = SortByDirection.asc;
+ let finalResult = { sort: 'a' };
+ let callback = jest.fn();
+ const { result } = renderHook(() => useSortColumn(columns, callback));
+ result.current({}, index, direction);
- expect(callback).toHaveBeenCalledWith(finalResult);
- });
- it('usePagePerPage, should return correct page and perPage', () => {
- let limit = 50;
- let offset = 1100;
- let expected = [23, 50];
- const { result } = renderHook(() => usePagePerPage(limit, offset));
- expect(result.current).toEqual(expected);
- });
+ expect(callback).toHaveBeenCalledWith(finalResult);
+ });
+ it('usePagePerPage, should return correct page and perPage', () => {
+ let limit = 50;
+ let offset = 1100;
+ let expected = [23, 50];
+ const { result } = renderHook(() => usePagePerPage(limit, offset));
+ expect(result.current).toEqual(expected);
+ });
- it.each`
- filter | apply | selected |finalResult
- ${{ advisory_type: 2 }} | ${jest.fn()} | ${[{ id: "advisory_type", chips: [{ id: 1 }] }]} | ${{ filter: { advisory_type: undefined }}}
- ${{advisory_type: [2,1]}} | ${jest.fn()} | ${[{ id: "advisory_type", chips: [{id: 1}] }]} | ${{filter: {advisory_type: [2]}}}
- ${{search: "asd"}} | ${jest.fn()} | ${[{ id: "search"}]} | ${{filter: {}, search: ''}}
- `('useRemoveFilter: should return correct filter for $filter',({filter, apply, finalResult, selected}) => {
- const { result } = renderHook(() => useRemoveFilter(filter, apply));
- result.current[0]({}, selected)
- expect(apply).toHaveBeenCalledWith(finalResult);
- });
+ it.each`
+ filter | apply | selected | finalResult
+ ${{ advisory_type: 2 }} | ${jest.fn()} | ${[{ id: 'advisory_type', chips: [{ id: 1 }] }]} | ${{ filter: { advisory_type: undefined } }}
+ ${{ advisory_type: [2, 1] }} | ${jest.fn()} | ${[{ id: 'advisory_type', chips: [{ id: 1 }] }]} | ${{ filter: { advisory_type: [2] } }}
+ ${{ search: 'asd' }} | ${jest.fn()} | ${[{ id: 'search' }]} | ${{ filter: {}, search: '' }}
+ `(
+ 'useRemoveFilter: should return correct filter for $filter',
+ ({ filter, apply, finalResult, selected }) => {
+ const { result } = renderHook(() => useRemoveFilter(filter, apply));
+ result.current[0]({}, selected);
+ expect(apply).toHaveBeenCalledWith(finalResult);
+ },
+ );
- it.each`
- filter | apply | selected |finalResult
- ${{ advisory_type: 2 }} | ${jest.fn()} | ${[{ id: "advisory_type", chips: [{ id: 1 }] }]} | ${{ filter: { advisory_type: undefined, systems_applicable: ["gt:0"] } }}
- ${{ search: "asd" }} | ${jest.fn()} | ${[{ id: "search" }]} | ${{ filter: { systems_applicable: ["gt:0"] }, search: '' }}
- `('useRemoveFilter: should reset to default filters for $filter while ', ({ filter, apply, finalResult, selected }) => {
- const { result } = renderHook(() => useRemoveFilter(filter, apply, packagesListDefaultFilters));
- result.current[0]({}, selected, true)
- expect(apply).toHaveBeenCalledWith(finalResult);
- });
+ it.each`
+ filter | apply | selected | finalResult
+ ${{ advisory_type: 2 }} | ${jest.fn()} | ${[{ id: 'advisory_type', chips: [{ id: 1 }] }]} | ${{ filter: { advisory_type: undefined, systems_applicable: ['gt:0'] } }}
+ ${{ search: 'asd' }} | ${jest.fn()} | ${[{ id: 'search' }]} | ${{ filter: { systems_applicable: ['gt:0'] }, search: '' }}
+ `(
+ 'useRemoveFilter: should reset to default filters for $filter while ',
+ ({ filter, apply, finalResult, selected }) => {
+ const { result } = renderHook(() =>
+ useRemoveFilter(filter, apply, packagesListDefaultFilters),
+ );
+ result.current[0]({}, selected, true);
+ expect(apply).toHaveBeenCalledWith(finalResult);
+ },
+ );
- it.each`
- metadata | apply | input | finalResult
- ${{limit: 10, offset: 0}} | ${jest.fn()} | ${{page: 2, per_page: 10}} | ${{offset:10}}
- ${{limit: 10, offset: 10}} | ${jest.fn()} | ${{page: 2, per_page: 20}} | ${{offset:20, limit: 20}}
- ${{limit: 10, offset: 10}} | ${jest.fn()} | ${{page: 2, per_page: 10}} | ${undefined}
- `('useHandleRefresh: should return correct filter',({metadata, apply, finalResult, input}) => {
- const { result } = renderHook(() =>useHandleRefresh(metadata, apply));
- result.current(input)
- if (finalResult) {
- expect(apply).toHaveBeenCalledWith(finalResult);
- }
- else {
- expect(apply).not.toHaveBeenCalled();
- }
- });
+ it.each`
+ metadata | apply | input | finalResult
+ ${{ limit: 10, offset: 0 }} | ${jest.fn()} | ${{ page: 2, per_page: 10 }} | ${{ offset: 10 }}
+ ${{ limit: 10, offset: 10 }} | ${jest.fn()} | ${{ page: 2, per_page: 20 }} | ${{ offset: 20, limit: 20 }}
+ ${{ limit: 10, offset: 10 }} | ${jest.fn()} | ${{ page: 2, per_page: 10 }} | ${undefined}
+ `('useHandleRefresh: should return correct filter', ({ metadata, apply, finalResult, input }) => {
+ const { result } = renderHook(() => useHandleRefresh(metadata, apply));
+ result.current(input);
+ if (finalResult) {
+ expect(apply).toHaveBeenCalledWith(finalResult);
+ } else {
+ expect(apply).not.toHaveBeenCalled();
+ }
+ });
- it('useEntitlements, should return correct entitlements', async () => {
- const { result } = renderHook(() => useEntitlements());
- const finalResult = await result.current();
- expect(finalResult).toEqual({ 'test-entitelement': true });
- });
+ it('useEntitlements, should return correct entitlements', async () => {
+ const { result } = renderHook(() => useEntitlements());
+ const finalResult = await result.current();
+ expect(finalResult).toEqual({ 'test-entitelement': true });
+ });
});
diff --git a/src/Utilities/hooks/useFeatureFlag.js b/src/Utilities/hooks/useFeatureFlag.js
index 012e7c7b9..7977b369d 100644
--- a/src/Utilities/hooks/useFeatureFlag.js
+++ b/src/Utilities/hooks/useFeatureFlag.js
@@ -1,7 +1,7 @@
import { useFlag, useFlagsStatus } from '@unleash/proxy-client-react';
export default (flag) => {
- const { flagsReady } = useFlagsStatus();
- const isFlagEnabled = useFlag(flag);
- return flagsReady ? isFlagEnabled : false;
+ const { flagsReady } = useFlagsStatus();
+ const isFlagEnabled = useFlag(flag);
+ return flagsReady ? isFlagEnabled : false;
};
diff --git a/src/Utilities/hooks/useFetchBatched.js b/src/Utilities/hooks/useFetchBatched.js
index d4ba16778..df15dfa4b 100644
--- a/src/Utilities/hooks/useFetchBatched.js
+++ b/src/Utilities/hooks/useFetchBatched.js
@@ -1,48 +1,43 @@
import usePromiseQueue from './usePromiseQueue';
export const useFetchBatched = () => {
- const { isResolving: isLoading, resolve } = usePromiseQueue();
-
- return {
- isLoading,
- fetchBatched: async (fetchFunction, filter, total, batchSize = 50) => {
- if (!total) {
- total = await fetchFunction({ ...filter, limit: 1 }).then(
- response => response?.meta?.total_items || 0
- );
- }
-
- const pages = Math.ceil(total / batchSize) || 1;
-
- const results = resolve(
- [...new Array(pages)].map(
- // eslint-disable-next-line camelcase
- (_, pageIdx) => () => {
- return fetchFunction({
- ...filter,
- offset: pageIdx * batchSize,
- limit: batchSize
- });
- }
- )
- );
-
- return results;
- },
- fetchBatchedInline: (fetchFunction, list, batchSize = 20) => {
- const pages = Math.ceil(list.length / batchSize) || 1;
-
- const results = resolve(
- [...new Array(pages)].map(
- (_, pageIdx) => () =>
- fetchFunction(
- list.slice(batchSize * pageIdx, batchSize * (pageIdx + 1))
- )
- )
- );
-
- return results;
- }
- };
+ const { isResolving: isLoading, resolve } = usePromiseQueue();
+
+ return {
+ isLoading,
+ fetchBatched: async (fetchFunction, filter, total, batchSize = 50) => {
+ if (!total) {
+ total = await fetchFunction({ ...filter, limit: 1 }).then(
+ (response) => response?.meta?.total_items || 0,
+ );
+ }
+
+ const pages = Math.ceil(total / batchSize) || 1;
+
+ const results = resolve(
+ [...new Array(pages)].map(
+ (_, pageIdx) => () =>
+ fetchFunction({
+ ...filter,
+ offset: pageIdx * batchSize,
+ limit: batchSize,
+ }),
+ ),
+ );
+
+ return results;
+ },
+ fetchBatchedInline: (fetchFunction, list, batchSize = 20) => {
+ const pages = Math.ceil(list.length / batchSize) || 1;
+
+ const results = resolve(
+ [...new Array(pages)].map(
+ (_, pageIdx) => () =>
+ fetchFunction(list.slice(batchSize * pageIdx, batchSize * (pageIdx + 1))),
+ ),
+ );
+
+ return results;
+ },
+ };
};
-
diff --git a/src/Utilities/hooks/useOnSelect.js b/src/Utilities/hooks/useOnSelect.js
index d2c1533af..cd5d781d5 100644
--- a/src/Utilities/hooks/useOnSelect.js
+++ b/src/Utilities/hooks/useOnSelect.js
@@ -7,176 +7,168 @@ import { useFetchBatched } from './useFetchBatched';
import isArray from 'lodash/isArray';
export const ID_API_ENDPOINTS = {
- advisories: '/ids/advisories',
- systems: '/ids/systems',
- templates: '/baselines',
- advisorySystems: (advisoryID) => `/ids/advisories/${advisoryID}/systems`,
- systemAdvisories: (systemID) => `/ids/systems/${systemID}/advisories`,
- packageSystems: (packageName) => `/packages/${packageName}/systems`,
- systemPackages: (systemID) => `/systems/${systemID}/packages`,
- templateSystems: (templateId) => `/ids/baselines/${templateId}/systems`
-};
-const isArrayWithData = (dataStructure) => {
- return isArray(dataStructure) && dataStructure.length;
+ advisories: '/ids/advisories',
+ systems: '/ids/systems',
+ templates: '/baselines',
+ advisorySystems: (advisoryID) => `/ids/advisories/${advisoryID}/systems`,
+ systemAdvisories: (systemID) => `/ids/systems/${systemID}/advisories`,
+ packageSystems: (packageName) => `/packages/${packageName}/systems`,
+ systemPackages: (systemID) => `/systems/${systemID}/packages`,
+ templateSystems: (templateId) => `/ids/baselines/${templateId}/systems`,
};
+const isArrayWithData = (dataStructure) => isArray(dataStructure) && dataStructure.length;
+
+const useFetchAllIDs = (endpoint, apiResponseTransformer, totalItems) => {
+ const { fetchBatched } = useFetchBatched();
+ return useCallback(
+ async (queryParams) => {
+ const response = await fetchBatched(
+ (filter) => fetchIDs(endpoint, filter),
+ queryParams,
+ totalItems,
+ 100,
+ );
+
+ const aggregatedResponse = response.reduce(
+ (accumulator = {}, currentValue) => {
+ Object.keys(accumulator).forEach((key) => {
+ if (isArrayWithData(currentValue[key])) {
+ accumulator[key] = accumulator[key].concat(currentValue[key]);
+ }
+ });
-const useFetchAllIDs = (
- endpoint,
- apiResponseTransformer,
- totalItems
-) => {
- const { fetchBatched } = useFetchBatched();
- return useCallback(async (queryParams) => {
- const response = await fetchBatched(
- (filter) => fetchIDs(endpoint, filter),
- queryParams,
- totalItems,
- 100
- );
-
- const aggregatedResponse = response.reduce((accumulator = {}, currentValue) => {
- Object.keys(accumulator).forEach(key => {
- if (isArrayWithData(currentValue[key])) {
- accumulator[key] = accumulator[key].concat(currentValue[key]);
- }
- });
-
- return accumulator;
- }, { data: [], ids: [] });
-
- return apiResponseTransformer ? apiResponseTransformer(aggregatedResponse) : aggregatedResponse;
+ return accumulator;
+ },
+ { data: [], ids: [] },
+ );
+
+ return apiResponseTransformer
+ ? apiResponseTransformer(aggregatedResponse)
+ : aggregatedResponse;
},
- [totalItems, endpoint, fetchBatched]);
+ [totalItems, endpoint, fetchBatched],
+ );
};
const useCreateSelectedRow = (transformKey, constructFilename) =>
- useCallback((rows, toSelect = []) => {
- const { ids, data } = rows;
- const shouldUseOnlyIDs = !isArrayWithData(data);
- const items = shouldUseOnlyIDs ? ids : data;
-
- items.forEach((item) => {
- const id = shouldUseOnlyIDs ? item : item.id;
-
- //expanded rows does not have ID and should be disabled for selection
- if (!(isObject(item) && item.isExpandedRow)) {
- toSelect.push(
- {
- id: transformKey ? transformKey(item) : id,
- selected: constructFilename ? constructFilename(item) : id
- }
- );
- }
+ useCallback((rows, toSelect = []) => {
+ const { ids, data } = rows;
+ const shouldUseOnlyIDs = !isArrayWithData(data);
+ const items = shouldUseOnlyIDs ? ids : data;
+
+ items.forEach((item) => {
+ const id = shouldUseOnlyIDs ? item : item.id;
+
+ // expanded rows does not have ID and should be disabled for selection
+ if (!(isObject(item) && item.isExpandedRow)) {
+ toSelect.push({
+ id: transformKey ? transformKey(item) : id,
+ selected: constructFilename ? constructFilename(item) : id,
});
-
- return toSelect;
+ }
});
-const createSelectors = (
- createSelectedRow,
- dispatchSelection,
- toggleAllSystemsSelected
-) => {
- const selectNone = (rows) => {
- const toSelect = [];
- Object.keys(rows).forEach(id => {
- toSelect.push(
- {
- id,
- selected: false
- }
- );
- });
+ return toSelect;
+ });
+
+const createSelectors = (createSelectedRow, dispatchSelection, toggleAllSystemsSelected) => {
+ const selectNone = (rows) => {
+ const toSelect = [];
+ Object.keys(rows).forEach((id) => {
+ toSelect.push({
+ id,
+ selected: false,
+ });
+ });
- dispatchSelection(toSelect);
- toggleAllSystemsSelected(false);
- };
-
- const selectPage = (pageRows) => {
- if (Array.isArray(pageRows)) {
- pageRows = pageRows.filter(row => !row.disableSelection);
- }
-
- dispatchSelection(createSelectedRow({ data: pageRows }));
- };
-
- const selectAll = (fetchIDs, queryParams) => {
- return fetchIDs(queryParams).then(response => {
- if (isArrayWithData(response.data)) {
- let rowsToSelect = response.data.filter(row => row.status !== 'Applicable');
- dispatchSelection(createSelectedRow({ data: rowsToSelect }));
- } else {
- dispatchSelection(createSelectedRow(response));
- toggleAllSystemsSelected(true);
- }
- });
- };
+ dispatchSelection(toSelect);
+ toggleAllSystemsSelected(false);
+ };
+
+ const selectPage = (pageRows) => {
+ if (Array.isArray(pageRows)) {
+ pageRows = pageRows.filter((row) => !row.disableSelection);
+ }
+
+ dispatchSelection(createSelectedRow({ data: pageRows }));
+ };
+
+ const selectAll = (fetchIDs, queryParams) =>
+ fetchIDs(queryParams).then((response) => {
+ if (isArrayWithData(response.data)) {
+ let rowsToSelect = response.data.filter((row) => row.status !== 'Applicable');
+ dispatchSelection(createSelectedRow({ data: rowsToSelect }));
+ } else {
+ dispatchSelection(createSelectedRow(response));
+ toggleAllSystemsSelected(true);
+ }
+ });
- return { selectNone, selectPage, selectAll };
+ return { selectNone, selectPage, selectAll };
};
export const useOnSelect = (rawData, selectedRows, config) => {
- const {
- endpoint,
- queryParams,
- selectionDispatcher,
- constructFilename,
- transformKey,
- apiResponseTransformer,
- //TODO: get rid of this custom selector
- customSelector,
- totalItems
- } = config;
-
- const dispatch = useDispatch();
- const fetchIDs = useFetchAllIDs(endpoint, apiResponseTransformer, totalItems);
- const createSelectedRow = useCreateSelectedRow(transformKey, constructFilename);
-
- const toggleAllSystemsSelected = (flagState) => {
- dispatch(toggleAllSelectedAction(flagState));
- };
-
- const dispatchSelection = (toSelect) => {
- if (customSelector) {
- customSelector(toSelect);
- } else {
- dispatch(selectionDispatcher(toSelect));
- }
- };
-
- const { selectNone, selectPage, selectAll } = createSelectors(
- createSelectedRow,
- dispatchSelection,
- toggleAllSystemsSelected
- );
-
- const onSelect = useCallback(
- async (event, selected, rowId, setBulkLoading = () => {}) => {
- switch (event) {
- case 'none': {
- selectNone(selectedRows);
- break;
- }
-
- case 'page': {
- selectPage(rawData);
- break;
- }
-
- case 'all': {
- selectAll(fetchIDs, queryParams)
- .then(() => setBulkLoading(false));
- break;
- }
-
- default: {
- dispatchSelection([{
- id: transformKey ? transformKey(rawData[rowId]) : rawData[rowId].id,
- selected: selected && (constructFilename ? constructFilename(rawData[rowId]) : true)
- }]);
- }
- }
- });
-
- return onSelect;
+ const {
+ endpoint,
+ queryParams,
+ selectionDispatcher,
+ constructFilename,
+ transformKey,
+ apiResponseTransformer,
+ // TODO: get rid of this custom selector
+ customSelector,
+ totalItems,
+ } = config;
+
+ const dispatch = useDispatch();
+ const fetchIDs = useFetchAllIDs(endpoint, apiResponseTransformer, totalItems);
+ const createSelectedRow = useCreateSelectedRow(transformKey, constructFilename);
+
+ const toggleAllSystemsSelected = (flagState) => {
+ dispatch(toggleAllSelectedAction(flagState));
+ };
+
+ const dispatchSelection = (toSelect) => {
+ if (customSelector) {
+ customSelector(toSelect);
+ } else {
+ dispatch(selectionDispatcher(toSelect));
+ }
+ };
+
+ const { selectNone, selectPage, selectAll } = createSelectors(
+ createSelectedRow,
+ dispatchSelection,
+ toggleAllSystemsSelected,
+ );
+
+ const onSelect = useCallback(async (event, selected, rowId, setBulkLoading = () => {}) => {
+ switch (event) {
+ case 'none': {
+ selectNone(selectedRows);
+ break;
+ }
+
+ case 'page': {
+ selectPage(rawData);
+ break;
+ }
+
+ case 'all': {
+ selectAll(fetchIDs, queryParams).then(() => setBulkLoading(false));
+ break;
+ }
+
+ default: {
+ dispatchSelection([
+ {
+ id: transformKey ? transformKey(rawData[rowId]) : rawData[rowId].id,
+ selected: selected && (constructFilename ? constructFilename(rawData[rowId]) : true),
+ },
+ ]);
+ }
+ }
+ });
+
+ return onSelect;
};
diff --git a/src/Utilities/hooks/useOnSelect.test.js b/src/Utilities/hooks/useOnSelect.test.js
index fe7445b01..0465e0738 100644
--- a/src/Utilities/hooks/useOnSelect.test.js
+++ b/src/Utilities/hooks/useOnSelect.test.js
@@ -3,177 +3,166 @@ import { fetchIDs } from '../api';
import { useOnSelect } from './useOnSelect';
jest.mock('react-redux', () => ({
- ...jest.requireActual('react-redux'),
- useDispatch: jest.fn(() => () => {})
+ ...jest.requireActual('react-redux'),
+ useDispatch: jest.fn(() => () => {}),
}));
jest.mock('../api', () => ({
- ...jest.requireActual('../api'),
- fetchIDs: jest.fn(() => {
- return Promise.resolve({
- data: [{ id: 'db-item' }]
- });
- })
+ ...jest.requireActual('../api'),
+ fetchIDs: jest.fn(() =>
+ Promise.resolve({
+ data: [{ id: 'db-item' }],
+ }),
+ ),
}));
const rows = [
- {
- id: 'row-1',
- selected: true
- },
- {
- id: 'row-2',
- selected: true
- }
+ {
+ id: 'row-1',
+ selected: true,
+ },
+ {
+ id: 'row-2',
+ selected: true,
+ },
];
let selectedRows = {};
let config = {
- endpoint: '/some/api/endpoint',
- queryParams: { search: 'test-search' },
- selectionDispatcher: jest.fn((...args) => args)
+ endpoint: '/some/api/endpoint',
+ queryParams: { search: 'test-search' },
+ selectionDispatcher: jest.fn((...args) => args),
};
describe('useOnSelect', () => {
- it('Should select single item', () => {
- const { result } = renderHook(() =>
- useOnSelect(rows, selectedRows, config)
- );
-
- act(() => {
- result.current('default-single', { 'row-2': true }, 0);
- });
-
- expect(config.selectionDispatcher).toHaveBeenCalledWith([
- { id: 'row-1', selected: true }
- ]);
+ it('Should select single item', () => {
+ const { result } = renderHook(() => useOnSelect(rows, selectedRows, config));
+
+ act(() => {
+ result.current('default-single', { 'row-2': true }, 0);
});
- it('Should select single item while constructing new selection name', () => {
- config.constructFilename = jest.fn(() => 'constructed-name');
+ expect(config.selectionDispatcher).toHaveBeenCalledWith([{ id: 'row-1', selected: true }]);
+ });
- const { result } = renderHook(() =>
- useOnSelect(rows, selectedRows, config)
- );
+ it('Should select single item while constructing new selection name', () => {
+ config.constructFilename = jest.fn(() => 'constructed-name');
- act(() => {
- result.current('default-single', { 'row-2': true }, 0);
- });
+ const { result } = renderHook(() => useOnSelect(rows, selectedRows, config));
- expect(config.selectionDispatcher).toHaveBeenCalledWith([
- { id: 'row-1', selected: 'constructed-name' }
- ]);
+ act(() => {
+ result.current('default-single', { 'row-2': true }, 0);
});
- it('Should select single item while transforming the selection id', () => {
- config.transformKey = jest.fn(() => 'transformed-name');
+ expect(config.selectionDispatcher).toHaveBeenCalledWith([
+ { id: 'row-1', selected: 'constructed-name' },
+ ]);
+ });
- const { result } = renderHook(() =>
- useOnSelect(rows, selectedRows, config)
- );
+ it('Should select single item while transforming the selection id', () => {
+ config.transformKey = jest.fn(() => 'transformed-name');
- act(() => {
- result.current('default-single', {}, 0);
- });
+ const { result } = renderHook(() => useOnSelect(rows, selectedRows, config));
- expect(config.selectionDispatcher).toHaveBeenCalledWith([
- { id: 'transformed-name', selected: 'constructed-name' }
- ]);
+ act(() => {
+ result.current('default-single', {}, 0);
});
- it('Should should use custom selector if provided', () => {
- config.customSelector = jest.fn(() => {});
+ expect(config.selectionDispatcher).toHaveBeenCalledWith([
+ { id: 'transformed-name', selected: 'constructed-name' },
+ ]);
+ });
- const { result } = renderHook(() =>
- useOnSelect(rows, selectedRows, config)
- );
+ it('Should should use custom selector if provided', () => {
+ config.customSelector = jest.fn(() => {});
- act(() => {
- result.current('default-single', {}, 0);
- });
+ const { result } = renderHook(() => useOnSelect(rows, selectedRows, config));
- expect(config.customSelector).toHaveBeenCalledWith([
- { id: 'transformed-name', selected: 'constructed-name' }
- ]);
+ act(() => {
+ result.current('default-single', {}, 0);
});
- it('Should deselect all', () => {
- const { result } = renderHook(() =>
- useOnSelect(rows, { 'row-1': true, 'row-2': true }, config)
- );
+ expect(config.customSelector).toHaveBeenCalledWith([
+ { id: 'transformed-name', selected: 'constructed-name' },
+ ]);
+ });
- act(() => {
- result.current('none', {}, 0);
- });
+ it('Should deselect all', () => {
+ const { result } = renderHook(() =>
+ useOnSelect(rows, { 'row-1': true, 'row-2': true }, config),
+ );
- expect(config.customSelector).toHaveBeenCalledWith([
- { id: 'row-1', selected: false },
- { id: 'row-2', selected: false }
- ]);
+ act(() => {
+ result.current('none', {}, 0);
});
- it('Should select page', () => {
- config.transformKey = undefined;
- config.constructFilename = undefined;
+ expect(config.customSelector).toHaveBeenCalledWith([
+ { id: 'row-1', selected: false },
+ { id: 'row-2', selected: false },
+ ]);
+ });
- const { result } = renderHook(() =>
- useOnSelect(rows, {}, config)
- );
+ it('Should select page', () => {
+ config.transformKey = undefined;
+ config.constructFilename = undefined;
- act(() => {
- result.current('page', {});
- });
+ const { result } = renderHook(() => useOnSelect(rows, {}, config));
- expect(config.customSelector).toHaveBeenCalledWith([
- { id: 'row-1', selected: 'row-1' },
- { id: 'row-2', selected: 'row-2' }
- ]);
+ act(() => {
+ result.current('page', {});
});
- it('Should select all items from db', async () => {
- config.totalItems = 102;
- const { result } = renderHook(() =>
- useOnSelect(rows, {}, config)
- );
-
- result.current('all', {});
-
- await waitFor(
- () => {
- expect(fetchIDs).toHaveBeenCalledTimes(2);
- expect(fetchIDs).toHaveBeenCalledWith('/some/api/endpoint', { limit: 100, offset: 0, search: 'test-search' });
- expect(fetchIDs).toHaveBeenCalledWith('/some/api/endpoint', { limit: 100, offset: 100, search: 'test-search' });
- }
- );
+ expect(config.customSelector).toHaveBeenCalledWith([
+ { id: 'row-1', selected: 'row-1' },
+ { id: 'row-2', selected: 'row-2' },
+ ]);
+ });
+
+ it('Should select all items from db', async () => {
+ config.totalItems = 102;
+ const { result } = renderHook(() => useOnSelect(rows, {}, config));
+
+ result.current('all', {});
+
+ await waitFor(() => {
+ expect(fetchIDs).toHaveBeenCalledTimes(2);
+ expect(fetchIDs).toHaveBeenCalledWith('/some/api/endpoint', {
+ limit: 100,
+ offset: 0,
+ search: 'test-search',
+ });
+ expect(fetchIDs).toHaveBeenCalledWith('/some/api/endpoint', {
+ limit: 100,
+ offset: 100,
+ search: 'test-search',
+ });
});
-
- it('Should skip invalid rows while selection', () => {
- const rows = [
- {
- id: 'valid-1',
- selected: true
- },
- {
- id: 'is-expanded-row',
- isExpandedRow: true
-
- },
- {
- id: 'valid-2',
- selected: true
- }
- ];
-
- const { result } = renderHook(() =>
- useOnSelect(rows, {}, config)
- );
-
- act(() => {
- result.current('page', {});
- });
-
- expect(config.customSelector).toHaveBeenCalledWith([
- { id: 'valid-1', selected: 'valid-1' },
- { id: 'valid-2', selected: 'valid-2' }
- ]);
+ });
+
+ it('Should skip invalid rows while selection', () => {
+ const rows = [
+ {
+ id: 'valid-1',
+ selected: true,
+ },
+ {
+ id: 'is-expanded-row',
+ isExpandedRow: true,
+ },
+ {
+ id: 'valid-2',
+ selected: true,
+ },
+ ];
+
+ const { result } = renderHook(() => useOnSelect(rows, {}, config));
+
+ act(() => {
+ result.current('page', {});
});
+
+ expect(config.customSelector).toHaveBeenCalledWith([
+ { id: 'valid-1', selected: 'valid-1' },
+ { id: 'valid-2', selected: 'valid-2' },
+ ]);
+ });
});
diff --git a/src/Utilities/hooks/usePatchSetState.js b/src/Utilities/hooks/usePatchSetState.js
index b730c0fbc..295f7ebb4 100644
--- a/src/Utilities/hooks/usePatchSetState.js
+++ b/src/Utilities/hooks/usePatchSetState.js
@@ -1,59 +1,57 @@
import { useState } from 'react';
-import {
- filterSelectedActiveSystemIDs
-} from '../Helpers';
+import { filterSelectedActiveSystemIDs } from '../Helpers';
/**
-* Manages unified state for patch set wizard and unassign modal.
-* @param {Object} [selectedRows] selected rows object to be removed.
-* @returns {patchSetState, setPatchSetState, openPatchSetAssignWizard, openUnassignSystemsModal, openPatchSetEditModal}
-*/
+ * Manages unified state for patch set wizard and unassign modal.
+ * @param {Object} [selectedRows] selected rows object to be removed.
+ * @returns {patchSetState, setPatchSetState, openPatchSetAssignWizard, openUnassignSystemsModal, openPatchSetEditModal}
+ */
export const usePatchSetState = (selectedRows) => {
- const [patchSetState, setPatchSetState] = useState({
- isPatchSetWizardOpen: false,
- isUnassignSystemsModalOpen: false,
- isAssignSystemsModalOpen: false,
- shouldRefresh: false,
- systemsIDs: {}
- });
+ const [patchSetState, setPatchSetState] = useState({
+ isPatchSetWizardOpen: false,
+ isUnassignSystemsModalOpen: false,
+ isAssignSystemsModalOpen: false,
+ shouldRefresh: false,
+ systemsIDs: {},
+ });
- const openPatchSetAssignWizard = (systemID) => {
- setPatchSetState({
- isPatchSetWizardOpen: true,
- systemsIDs: typeof systemID === 'string' && systemID !== ''
- ? [systemID]
- : filterSelectedActiveSystemIDs(selectedRows),
- shouldRefresh: false
- }
- );
- };
+ const openPatchSetAssignWizard = (systemID) => {
+ setPatchSetState({
+ isPatchSetWizardOpen: true,
+ systemsIDs:
+ typeof systemID === 'string' && systemID !== ''
+ ? [systemID]
+ : filterSelectedActiveSystemIDs(selectedRows),
+ shouldRefresh: false,
+ });
+ };
- const openUnassignSystemsModal = (systemsIDs) => {
- setPatchSetState({
- isUnassignSystemsModalOpen: true,
- systemsIDs,
- shouldRefresh: false
- });
- };
+ const openUnassignSystemsModal = (systemsIDs) => {
+ setPatchSetState({
+ isUnassignSystemsModalOpen: true,
+ systemsIDs,
+ shouldRefresh: false,
+ });
+ };
- const openAssignSystemsModal = (systemsIDs) => {
- setPatchSetState({
- isAssignSystemsModalOpen: true,
- systemsIDs,
- shouldRefresh: false
- });
- };
+ const openAssignSystemsModal = (systemsIDs) => {
+ setPatchSetState({
+ isAssignSystemsModalOpen: true,
+ systemsIDs,
+ shouldRefresh: false,
+ });
+ };
- const openPatchSetEditModal = (patchSetID) => {
- setPatchSetState({ isPatchSetWizardOpen: true, patchSetID });
- };
+ const openPatchSetEditModal = (patchSetID) => {
+ setPatchSetState({ isPatchSetWizardOpen: true, patchSetID });
+ };
- return {
- patchSetState,
- setPatchSetState,
- openPatchSetAssignWizard,
- openUnassignSystemsModal,
- openAssignSystemsModal,
- openPatchSetEditModal
- };
+ return {
+ patchSetState,
+ setPatchSetState,
+ openPatchSetAssignWizard,
+ openUnassignSystemsModal,
+ openAssignSystemsModal,
+ openPatchSetEditModal,
+ };
};
diff --git a/src/Utilities/hooks/usePromiseQueue.js b/src/Utilities/hooks/usePromiseQueue.js
index 409ee6011..2dfb78890 100644
--- a/src/Utilities/hooks/usePromiseQueue.js
+++ b/src/Utilities/hooks/usePromiseQueue.js
@@ -4,35 +4,35 @@ import pAll from 'p-all';
const DEFAULT_CONCURRENT_PROMISES = 2;
const usePromiseQueue = (limit = DEFAULT_CONCURRENT_PROMISES) => {
- const [results, setResults] = useState({
- isResolving: false,
- promiseResults: undefined
- });
+ const [results, setResults] = useState({
+ isResolving: false,
+ promiseResults: undefined,
+ });
- const resolve = useCallback(
- async (fns) => {
- setResults((state) => ({
- ...state,
- isResolving: true
- }));
- const results = await pAll(fns, {
- concurrency: limit
- });
- setResults({
- isResolving: false,
- promiseResults: results
- });
+ const resolve = useCallback(
+ async (fns) => {
+ setResults((state) => ({
+ ...state,
+ isResolving: true,
+ }));
+ const results = await pAll(fns, {
+ concurrency: limit,
+ });
+ setResults({
+ isResolving: false,
+ promiseResults: results,
+ });
- return results;
- },
- [limit]
- );
+ return results;
+ },
+ [limit],
+ );
- return {
- isResolving: results.isResolving,
- results: results.promiseResults,
- resolve
- };
+ return {
+ isResolving: results.isResolving,
+ results: results.promiseResults,
+ resolve,
+ };
};
export default usePromiseQueue;
diff --git a/src/Utilities/hooks/useRemediationDataProvider.js b/src/Utilities/hooks/useRemediationDataProvider.js
index 15c3e43cb..34af9ced7 100644
--- a/src/Utilities/hooks/useRemediationDataProvider.js
+++ b/src/Utilities/hooks/useRemediationDataProvider.js
@@ -1,80 +1,82 @@
import { useDispatch } from 'react-redux';
import { addNotification } from '@redhat-cloud-services/frontend-components-notifications/redux';
import { useChrome } from '@redhat-cloud-services/frontend-components/useChrome';
-import {
- removeUndefinedObjectKeys
-} from '../Helpers';
+import { removeUndefinedObjectKeys } from '../Helpers';
const initializeWorker = () => {
- const worker = new Worker(new URL('../RemediationPairs.js', import.meta.url));
- return [worker, () => worker.terminate()];
+ const worker = new Worker(new URL('../RemediationPairs.js', import.meta.url));
+ return [worker, () => worker.terminate()];
};
const deligateWorkerTask = (worker, task) => {
- worker.postMessage(task);
+ worker.postMessage(task);
- //waits web worker response
- return new Promise((resolve, reject) => {
- worker.onmessage = ({ data: { status, error, result } } = {}) => {
- if (status === 'resolved') {
- resolve(result);
- }
+ // waits web worker response
+ return new Promise((resolve, reject) => {
+ worker.onmessage = ({ data: { status, error, result } } = {}) => {
+ if (status === 'resolved') {
+ resolve(result);
+ }
- reject(error);
- };
- });
+ reject(error);
+ };
+ });
};
export const prepareRemediationPairs = async (task, dispatch) => {
- const [worker, terminateWorker] = initializeWorker();
- const deligatedTask = deligateWorkerTask(worker, task);
+ const [worker, terminateWorker] = initializeWorker();
+ const deligatedTask = deligateWorkerTask(worker, task);
- const response = await deligatedTask.catch(err =>
- dispatch(
- addNotification(
- {
- title: `There was an error while processing.`,
- description: err,
- variant: 'danger'
- }
- )
- ));
+ const response = await deligatedTask.catch((err) =>
+ dispatch(
+ addNotification({
+ title: `There was an error while processing.`,
+ description: err,
+ variant: 'danger',
+ }),
+ ),
+ );
- terminateWorker();
+ terminateWorker();
- //displays NoDataModal when there is no patch updates available
- return response?.issues?.length ? response : false;
+ // displays NoDataModal when there is no patch updates available
+ return response?.issues?.length ? response : false;
};
/**
-* Provides remediation data, systems with all of their corresponding issues.
-* @param {Function} [setRemediationLoading] function to toggle remediation loading state
-* @param {Array} [selectedRows] array of systems to calculate
-* @returns {handleSystemsRemoval}
-*/
-export const useRemediationDataProvider = (selectedRows, setRemediationLoading, remediationType, areAllSelected) => {
- const dispatch = useDispatch();
- const chrome = useChrome();
- const remediationDataProvider = async () => {
- setRemediationLoading(true);
+ * Provides remediation data, systems with all of their corresponding issues.
+ * @param {Function} [setRemediationLoading] function to toggle remediation loading state
+ * @param {Array} [selectedRows] array of systems to calculate
+ * @returns {handleSystemsRemoval}
+ */
+export const useRemediationDataProvider = (
+ selectedRows,
+ setRemediationLoading,
+ remediationType,
+ areAllSelected,
+) => {
+ const dispatch = useDispatch();
+ const chrome = useChrome();
+ const remediationDataProvider = async () => {
+ setRemediationLoading(true);
- //Auth token must be added to webworker request as webworker does not have access
- //to default token by platform
- const authToken = await chrome.auth.getToken();
- const remediationPairs = await prepareRemediationPairs(
- {
- payload: removeUndefinedObjectKeys(selectedRows),
- remediationType,
- areAllSelected,
- authToken
- },
- dispatch
- );
+ // Auth token must be added to webworker request as webworker does not have access
+ // to default token by platform
+ const authToken = await chrome.auth.getToken();
+ const remediationPairs = await prepareRemediationPairs(
+ {
+ payload: removeUndefinedObjectKeys(selectedRows),
+ remediationType,
+ areAllSelected,
+ authToken,
+ },
+ dispatch,
+ );
- setRemediationLoading(false);
+ setRemediationLoading(false);
- return remediationPairs;
- };
+ return remediationPairs;
+ };
- return remediationDataProvider;
+ return remediationDataProvider;
};
diff --git a/src/Utilities/hooks/useRemediationDataProvider.test.js b/src/Utilities/hooks/useRemediationDataProvider.test.js
index fed014c19..62764036f 100644
--- a/src/Utilities/hooks/useRemediationDataProvider.test.js
+++ b/src/Utilities/hooks/useRemediationDataProvider.test.js
@@ -1,6 +1,7 @@
-describe('useRemediationDataProvider', () => it('temproray test', () => {
+describe('useRemediationDataProvider', () =>
+ it('temproray test', () => {
expect(true).toBeTruthy();
-}));
+ }));
// /**
// * @jest-environment jsdom
diff --git a/src/Utilities/unitTestingUtilities.js b/src/Utilities/unitTestingUtilities.js
index 55262c059..d7880589d 100644
--- a/src/Utilities/unitTestingUtilities.js
+++ b/src/Utilities/unitTestingUtilities.js
@@ -2,73 +2,71 @@
import { useSelector } from 'react-redux';
import { IntlProvider } from '@redhat-cloud-services/frontend-components-translations';
export const initMocks = () => {
- window.insights = {
- chrome: {
- appNavClick: () => {},
- init: jest.fn(),
- identifyApp: () => {},
- navigation: () => {},
- on: () => {
- return () => {};
- },
- isBeta: () => true,
- auth: {
- getUser: () =>
- new Promise((resolve) =>
- resolve({
- identity: {
- user: {}
- }
- })
- )
- },
- getUserPermissions: () => Promise.resolve([])
- },
- loadInventory: () => new Promise((resolve) => {
- resolve(({
- inventoryConnector: () => {
- const InventoryTable = ({ children }) => A mock passed! {children}
;
- const InventoryDetailHead = ({ children }) => A mock passed! {children}
;
- const AppInfo = ({ children }) => A mock passed! {children}
;
- const DetailWrapper = ({ children }) => A mock passed! {children}
;
-
- return ({ InventoryTable, InventoryDetailHead, AppInfo, DetailWrapper });
- },
- mergeWithEntities: () => {},
- mergeWithDetail: () => {}
- }));
- }),
- loadRemediations: () => new Promise((resolve) => {
+ window.insights = {
+ chrome: {
+ appNavClick: () => {},
+ init: jest.fn(),
+ identifyApp: () => {},
+ navigation: () => {},
+ on: () => () => {},
+ isBeta: () => true,
+ auth: {
+ getUser: () =>
+ new Promise((resolve) =>
resolve({
- openWizard: () => new Promise((resolve) => {
- resolve(true);
- })
- });
- })
- };
+ identity: {
+ user: {},
+ },
+ }),
+ ),
+ },
+ getUserPermissions: () => Promise.resolve([]),
+ },
+ loadInventory: () =>
+ new Promise((resolve) => {
+ resolve({
+ inventoryConnector: () => {
+ const InventoryTable = ({ children }) => A mock passed! {children}
;
+ const InventoryDetailHead = ({ children }) => A mock passed! {children}
;
+ const AppInfo = ({ children }) => A mock passed! {children}
;
+ const DetailWrapper = ({ children }) => A mock passed! {children}
;
+ return { InventoryTable, InventoryDetailHead, AppInfo, DetailWrapper };
+ },
+ mergeWithEntities: () => {},
+ mergeWithDetail: () => {},
+ });
+ }),
+ loadRemediations: () =>
+ new Promise((resolve) => {
+ resolve({
+ openWizard: () =>
+ new Promise((resolve) => {
+ resolve(true);
+ }),
+ });
+ }),
+ };
};
export const mockStore = (initialState, mutatedState) => {
- const customMiddleWare = () => next => action => {
- useSelector.mockImplementation(callback => {
- return callback(mutatedState);
- });
- next(action);
- };
+ const customMiddleWare = () => (next) => (action) => {
+ useSelector.mockImplementation((callback) => callback(mutatedState));
+ next(action);
+ };
- // eslint-disable-next-line no-undef
- const configuredMockStore = configureStore([customMiddleWare]);
+ // eslint-disable-next-line no-undef
+ const configuredMockStore = configureStore([customMiddleWare]);
- return configuredMockStore(initialState);
+ return configuredMockStore(initialState);
};
export const mountWithIntl = (Component) => {
- const wrapper = mount(Component, {
- wrappingComponent: IntlProvider
- });
- const provider = wrapper.getWrappingComponent();
- provider.setProps({ locale: 'en' });
+ const wrapper = mount(Component, {
+ wrappingComponent: IntlProvider,
+ });
+ const provider = wrapper.getWrappingComponent();
+ provider.setProps({ locale: 'en' });
- return wrapper;
+ return wrapper;
};
diff --git a/src/index.js b/src/index.js
index b4e6518e8..e5b2f4a61 100644
--- a/src/index.js
+++ b/src/index.js
@@ -7,25 +7,27 @@ import { Provider } from 'react-redux';
import PropTypes from 'prop-types';
const WrappedSystemDetail = ({ getRegistry, ...props }) => {
- const [Wrapper, setWrapper] = useState();
- useEffect(() => {
- if (getRegistry) {
- getRegistry()?.register?.({ SystemAdvisoryListStore, SystemPackageListStore });
- }
+ const [Wrapper, setWrapper] = useState();
+ useEffect(() => {
+ if (getRegistry) {
+ getRegistry()?.register?.({ SystemAdvisoryListStore, SystemPackageListStore });
+ }
- setWrapper(() => getRegistry ? Provider : Fragment);
- }, []);
- return (
- Wrapper ?
-
- :
-
-
- );
+ setWrapper(() => (getRegistry ? Provider : Fragment));
+ }, []);
+ return Wrapper ? (
+
+
+
+ ) : (
+
+
+
+ );
};
WrappedSystemDetail.propTypes = {
- getRegistry: PropTypes.func
+ getRegistry: PropTypes.func,
};
export { SystemPackageListStore, SystemAdvisoryListStore };
diff --git a/src/store/ActionTypes.js b/src/store/ActionTypes.js
index d109e1397..cac9efe15 100644
--- a/src/store/ActionTypes.js
+++ b/src/store/ActionTypes.js
@@ -2,19 +2,15 @@ export const FETCH_APPLICABLE_ADVISORIES = 'FETCH_APPLICABLE_ADVISORIES';
export const EXPAND_ADVISORY_ROW = 'EXPAND_ADVISORY_ROW';
export const SELECT_ADVISORY_ROW = 'SELECT_ADVISORY_ROW';
export const CHANGE_ADVISORY_LIST_PARAMS = 'CHANGE_ADVISORY_LIST_PARAMS';
-export const CHANGE_SYSTEM_ADVISORY_LIST_PARAMS =
- 'CHANGE_SYSTEM_SYSTEMS_LIST_PARAMS';
+export const CHANGE_SYSTEM_ADVISORY_LIST_PARAMS = 'CHANGE_SYSTEM_SYSTEMS_LIST_PARAMS';
export const EXPAND_SYSTEM_ADVISORY_ROW = 'EXPAND_SYSTEM_ADVISORY_ROW';
-export const FETCH_APPLICABLE_SYSTEM_ADVISORIES =
- 'FETCH_APPLICABLE_SYSTEM_ADVISORIES';
+export const FETCH_APPLICABLE_SYSTEM_ADVISORIES = 'FETCH_APPLICABLE_SYSTEM_ADVISORIES';
export const SELECT_SYSTEM_ADVISORY_ROW = 'SELECT_SYSTEM_ADVISORY_ROW';
export const CLEAR_SYSTEM_ADVISORIES = 'CLEAR_SYSTEM_ADVISORIES';
export const FETCH_ADVISORY_DETAILS = 'FETCH_ADVISORY_DETAILS';
export const CLEAR_ADVISORY_DETAILS = 'CLEAR_ADVISORY_DETAILS';
-export const CHANGE_SYSTEM_PACKAGES_LIST_PARAMS =
- 'CHANGE_SYSTEM_PACKAGES_LIST_PARAMS';
-export const FETCH_APPLICABLE_SYSTEM_PACKAGES =
- 'FETCH_APPLICABLE_SYSTEM_PACKAGES';
+export const CHANGE_SYSTEM_PACKAGES_LIST_PARAMS = 'CHANGE_SYSTEM_PACKAGES_LIST_PARAMS';
+export const FETCH_APPLICABLE_SYSTEM_PACKAGES = 'FETCH_APPLICABLE_SYSTEM_PACKAGES';
export const SELECT_SYSTEM_PACKAGES_ROW = 'SELECT_SYSTEM_PACKAGES_ROW';
export const CLEAR_SYSTEM_PACKAGES = 'CLEAR_SYSTEM_PACKAGES';
export const TRIGGER_GLOBAL_FILTER = 'TRIGGER_GLOBAL_FILTER';
diff --git a/src/store/Actions/Actions.js b/src/store/Actions/Actions.js
index 23ad9b79c..38c99cd05 100644
--- a/src/store/Actions/Actions.js
+++ b/src/store/Actions/Actions.js
@@ -1,297 +1,305 @@
import {
- fetchAdvisoryDetailsApi, fetchApplicableAdvisoriesApi,
- fetchApplicablePackagesApi, fetchApplicableSystemAdvisoriesApi, fetchPackageDetailsApi,
- fetchPackagesList, fetchPackageSystems, fetchCvesInfo, fetchSystemDetails, fetchPatchSets, fetchPatchSet,
- fetchPatchSetSystems
+ fetchAdvisoryDetailsApi,
+ fetchApplicableAdvisoriesApi,
+ fetchApplicablePackagesApi,
+ fetchApplicableSystemAdvisoriesApi,
+ fetchPackageDetailsApi,
+ fetchPackagesList,
+ fetchPackageSystems,
+ fetchCvesInfo,
+ fetchSystemDetails,
+ fetchPatchSets,
+ fetchPatchSet,
+ fetchPatchSetSystems,
} from '../../Utilities/api';
import * as ActionTypes from '../ActionTypes';
-export const fetchApplicableAdvisories = params => ({
- type: ActionTypes.FETCH_APPLICABLE_ADVISORIES,
- payload: new Promise(resolve => {
- resolve(fetchApplicableAdvisoriesApi(params));
- }).then(result => result)
+export const fetchApplicableAdvisories = (params) => ({
+ type: ActionTypes.FETCH_APPLICABLE_ADVISORIES,
+ payload: new Promise((resolve) => {
+ resolve(fetchApplicableAdvisoriesApi(params));
+ }).then((result) => result),
});
-export const fetchApplicableSystemAdvisories = params => ({
- type: ActionTypes.FETCH_APPLICABLE_SYSTEM_ADVISORIES,
- payload: new Promise(resolve => {
- resolve(fetchApplicableSystemAdvisoriesApi(params));
- }).then(result => result)
+export const fetchApplicableSystemAdvisories = (params) => ({
+ type: ActionTypes.FETCH_APPLICABLE_SYSTEM_ADVISORIES,
+ payload: new Promise((resolve) => {
+ resolve(fetchApplicableSystemAdvisoriesApi(params));
+ }).then((result) => result),
});
-export const fetchAvisoryDetails = params => ({
- type: ActionTypes.FETCH_ADVISORY_DETAILS,
- payload: new Promise(resolve => {
- resolve(fetchAdvisoryDetailsApi(params));
- }).then(result => result),
- noError: true
+export const fetchAvisoryDetails = (params) => ({
+ type: ActionTypes.FETCH_ADVISORY_DETAILS,
+ payload: new Promise((resolve) => {
+ resolve(fetchAdvisoryDetailsApi(params));
+ }).then((result) => result),
+ noError: true,
});
-export const fetchPackageDetails = params => ({
- type: ActionTypes.FETCH_PACKAGE_DETAILS,
- payload: new Promise(resolve => {
- resolve(fetchPackageDetailsApi(params));
- }).then(result => result)
+export const fetchPackageDetails = (params) => ({
+ type: ActionTypes.FETCH_PACKAGE_DETAILS,
+ payload: new Promise((resolve) => {
+ resolve(fetchPackageDetailsApi(params));
+ }).then((result) => result),
});
-export const changeAdvisoryListParams = params => ({
- type: ActionTypes.CHANGE_ADVISORY_LIST_PARAMS,
- payload: params
+export const changeAdvisoryListParams = (params) => ({
+ type: ActionTypes.CHANGE_ADVISORY_LIST_PARAMS,
+ payload: params,
});
-export const changeSystemAdvisoryListParams = params => ({
- type: ActionTypes.CHANGE_SYSTEM_ADVISORY_LIST_PARAMS,
- payload: params
+export const changeSystemAdvisoryListParams = (params) => ({
+ type: ActionTypes.CHANGE_SYSTEM_ADVISORY_LIST_PARAMS,
+ payload: params,
});
-export const changeAdvisorySystemsParams = params => ({
- type: ActionTypes.CHANGE_AFFECTED_SYSTEMS_PARAMS,
- payload: params
+export const changeAdvisorySystemsParams = (params) => ({
+ type: ActionTypes.CHANGE_AFFECTED_SYSTEMS_PARAMS,
+ payload: params,
});
-export const expandAdvisoryRow = rowState => ({
- type: ActionTypes.EXPAND_ADVISORY_ROW,
- payload: rowState
+export const expandAdvisoryRow = (rowState) => ({
+ type: ActionTypes.EXPAND_ADVISORY_ROW,
+ payload: rowState,
});
-export const expandSystemAdvisoryRow = rowState => ({
- type: ActionTypes.EXPAND_SYSTEM_ADVISORY_ROW,
- payload: rowState
+export const expandSystemAdvisoryRow = (rowState) => ({
+ type: ActionTypes.EXPAND_SYSTEM_ADVISORY_ROW,
+ payload: rowState,
});
-export const selectAdvisoryRow = rowState => ({
- type: ActionTypes.SELECT_ADVISORY_ROW,
- payload: rowState
+export const selectAdvisoryRow = (rowState) => ({
+ type: ActionTypes.SELECT_ADVISORY_ROW,
+ payload: rowState,
});
-export const selectSystemAdvisoryRow = rowState => ({
- type: ActionTypes.SELECT_SYSTEM_ADVISORY_ROW,
- payload: rowState
+export const selectSystemAdvisoryRow = (rowState) => ({
+ type: ActionTypes.SELECT_SYSTEM_ADVISORY_ROW,
+ payload: rowState,
});
export const clearSystemAdvisoriesStore = () => ({
- type: ActionTypes.CLEAR_SYSTEM_ADVISORIES,
- payload: []
+ type: ActionTypes.CLEAR_SYSTEM_ADVISORIES,
+ payload: [],
});
export const clearSystemPackagesStore = () => ({
- type: ActionTypes.CLEAR_SYSTEM_PACKAGES,
- payload: []
+ type: ActionTypes.CLEAR_SYSTEM_PACKAGES,
+ payload: [],
});
export const clearAdvisoryDetailStore = () => ({
- type: ActionTypes.CLEAR_ADVISORY_DETAILS,
- payload: []
+ type: ActionTypes.CLEAR_ADVISORY_DETAILS,
+ payload: [],
});
export const clearPackageDetailStore = () => ({
- type: ActionTypes.CLEAR_PACKAGE_DETAILS,
- payload: []
+ type: ActionTypes.CLEAR_PACKAGE_DETAILS,
+ payload: [],
});
-export const fetchPackageSystemsAction = params => ({
- type: ActionTypes.FETCH_PACKAGE_SYSTEMS,
- payload: new Promise(resolve => {
- resolve(fetchPackageSystems(params));
- }).then(result => result)
+export const fetchPackageSystemsAction = (params) => ({
+ type: ActionTypes.FETCH_PACKAGE_SYSTEMS,
+ payload: new Promise((resolve) => {
+ resolve(fetchPackageSystems(params));
+ }).then((result) => result),
});
-export const fetchApplicableSystemPackages = params => ({
- type: ActionTypes.FETCH_APPLICABLE_SYSTEM_PACKAGES,
- payload: new Promise(resolve => {
- resolve(fetchApplicablePackagesApi(params));
- }).then(result => result)
+export const fetchApplicableSystemPackages = (params) => ({
+ type: ActionTypes.FETCH_APPLICABLE_SYSTEM_PACKAGES,
+ payload: new Promise((resolve) => {
+ resolve(fetchApplicablePackagesApi(params));
+ }).then((result) => result),
});
-export const selectSystemPackagesRow = rowState => ({
- type: ActionTypes.SELECT_SYSTEM_PACKAGES_ROW,
- payload: rowState
+export const selectSystemPackagesRow = (rowState) => ({
+ type: ActionTypes.SELECT_SYSTEM_PACKAGES_ROW,
+ payload: rowState,
});
-export const changeSystemPackagesParams = params => ({
- type: ActionTypes.CHANGE_SYSTEM_PACKAGES_LIST_PARAMS,
- payload: params
+export const changeSystemPackagesParams = (params) => ({
+ type: ActionTypes.CHANGE_SYSTEM_PACKAGES_LIST_PARAMS,
+ payload: params,
});
-export const globalFilter = params => ({
- type: ActionTypes.TRIGGER_GLOBAL_FILTER,
- payload: params
+export const globalFilter = (params) => ({
+ type: ActionTypes.TRIGGER_GLOBAL_FILTER,
+ payload: params,
});
-export const changeGlobalTags = params => ({
- type: ActionTypes.CHANGE_GLOBAL_TAGS,
- payload: params
+export const changeGlobalTags = (params) => ({
+ type: ActionTypes.CHANGE_GLOBAL_TAGS,
+ payload: params,
});
-export const changeTags = params => ({
- type: ActionTypes.CHANGE_TAGS,
- payload: params
+export const changeTags = (params) => ({
+ type: ActionTypes.CHANGE_TAGS,
+ payload: params,
});
-export const changeProfile = params => ({
- type: ActionTypes.CHANGE_PROFILE,
- payload: params
+export const changeProfile = (params) => ({
+ type: ActionTypes.CHANGE_PROFILE,
+ payload: params,
});
-export const fetchPackagesAction = params => ({
- type: ActionTypes.FETCH_PACKAGES_LIST,
- payload: new Promise(resolve => {
- resolve(fetchPackagesList(params));
- }).then(result => result)
+export const fetchPackagesAction = (params) => ({
+ type: ActionTypes.FETCH_PACKAGES_LIST,
+ payload: new Promise((resolve) => {
+ resolve(fetchPackagesList(params));
+ }).then((result) => result),
});
-export const changePackagesListParams = params => ({
- type: ActionTypes.CHANGE_PACKAGES_LIST_PARAMS,
- payload: params
+export const changePackagesListParams = (params) => ({
+ type: ActionTypes.CHANGE_PACKAGES_LIST_PARAMS,
+ payload: params,
});
-export const changePackageSystemsParams = params => ({
- type: ActionTypes.CHANGE_PACKAGE_SYSTEMS_PARAMS,
- payload: params
+export const changePackageSystemsParams = (params) => ({
+ type: ActionTypes.CHANGE_PACKAGE_SYSTEMS_PARAMS,
+ payload: params,
});
export const fetchCves = (params) => ({
- type: ActionTypes.FETCH_CVES_INFO,
- payload: new Promise(resolve => {
- resolve(fetchCvesInfo(params));
- }).then(result => result)
+ type: ActionTypes.FETCH_CVES_INFO,
+ payload: new Promise((resolve) => {
+ resolve(fetchCvesInfo(params));
+ }).then((result) => result),
});
-export const changeCvesListParams = params => ({
- type: ActionTypes.CHANGE_CVES_STORE_PARAMS,
- payload: params
+export const changeCvesListParams = (params) => ({
+ type: ActionTypes.CHANGE_CVES_STORE_PARAMS,
+ payload: params,
});
-export const fetchSystemDetailsAction = params => ({
- type: ActionTypes.FETCH_SYSTEM_DETAIL,
- payload: new Promise(resolve => {
- resolve(fetchSystemDetails(params));
- }).then(result => result),
- meta: { noError: true }
+export const fetchSystemDetailsAction = (params) => ({
+ type: ActionTypes.FETCH_SYSTEM_DETAIL,
+ payload: new Promise((resolve) => {
+ resolve(fetchSystemDetails(params));
+ }).then((result) => result),
+ meta: { noError: true },
});
export const clearEntitiesStore = () => ({
- type: ActionTypes.CLEAR_ENTITIES,
- payload: []
+ type: ActionTypes.CLEAR_ENTITIES,
+ payload: [],
});
export const clearInventoryReducer = () => ({
- type: ActionTypes.CLEAR_INVENTORY_REDUCER,
- payload: []
+ type: ActionTypes.CLEAR_INVENTORY_REDUCER,
+ payload: [],
});
-export const changeEntitiesParams = params => ({
- type: ActionTypes.CHANGE_ENTITIES_PARAMS,
- payload: params
+export const changeEntitiesParams = (params) => ({
+ type: ActionTypes.CHANGE_ENTITIES_PARAMS,
+ payload: params,
});
-export const changeSystemsParams = params => ({
- type: ActionTypes.CHANGE_SYSTEMS_PARAMS,
- payload: params
+export const changeSystemsParams = (params) => ({
+ type: ActionTypes.CHANGE_SYSTEMS_PARAMS,
+ payload: params,
});
-export const changeAffectedSystemsParams = params => ({
- type: ActionTypes.CHANGE_AFFECTED_SYSTEMS_PARAMS,
- payload: params
+export const changeAffectedSystemsParams = (params) => ({
+ type: ActionTypes.CHANGE_AFFECTED_SYSTEMS_PARAMS,
+ payload: params,
});
export const clearPackageSystemsReducer = () => ({
- type: ActionTypes.CLEAR_PACKAGE_SYSTEMS_REDUCER,
- payload: []
+ type: ActionTypes.CLEAR_PACKAGE_SYSTEMS_REDUCER,
+ payload: [],
});
export const clearAdvisorySystemsReducer = () => ({
- type: ActionTypes.CLEAR_ADVISORY_SYSTEMS_REDUCER,
- payload: []
+ type: ActionTypes.CLEAR_ADVISORY_SYSTEMS_REDUCER,
+ payload: [],
});
export const changeSystemsMetadata = (params) => ({
- type: ActionTypes.CHANGE_SYSTEMS_METADATA,
- payload: params
+ type: ActionTypes.CHANGE_SYSTEMS_METADATA,
+ payload: params,
});
-export const fetchPatchSetsAction = params => ({
- type: ActionTypes.FETCH_ALL_PATCH_SETS,
- payload: new Promise(resolve => {
- resolve(fetchPatchSets(params));
- }).then(result => result)
+export const fetchPatchSetsAction = (params) => ({
+ type: ActionTypes.FETCH_ALL_PATCH_SETS,
+ payload: new Promise((resolve) => {
+ resolve(fetchPatchSets(params));
+ }).then((result) => result),
});
export const changePatchSetsParams = (params) => ({
- type: ActionTypes.CHANGE_PATCH_SET_PARAMS,
- payload: params
+ type: ActionTypes.CHANGE_PATCH_SET_PARAMS,
+ payload: params,
});
-export const selectPatchSetRow = rowState => ({
- type: ActionTypes.SELECT_PATCH_SET_ROW,
- payload: rowState
+export const selectPatchSetRow = (rowState) => ({
+ type: ActionTypes.SELECT_PATCH_SET_ROW,
+ payload: rowState,
});
export const fetchPatchSetAction = (id) => ({
- type: ActionTypes.FETCH_PATCH_SET,
- payload: new Promise(resolve => {
- resolve(fetchPatchSet(id));
- }).then(result => result)
+ type: ActionTypes.FETCH_PATCH_SET,
+ payload: new Promise((resolve) => {
+ resolve(fetchPatchSet(id));
+ }).then((result) => result),
});
export const fetchTemplateDetail = (id) => ({
- type: ActionTypes.FETCH_TEMPLATE_DETAIL,
- payload: new Promise(resolve => {
- resolve(fetchPatchSet(id));
- }).then(result => result)
+ type: ActionTypes.FETCH_TEMPLATE_DETAIL,
+ payload: new Promise((resolve) => {
+ resolve(fetchPatchSet(id));
+ }).then((result) => result),
});
export const clearTemplateDetail = () => ({
- type: ActionTypes.CLEAR_TEMPLATE_DETAIL,
- payload: []
+ type: ActionTypes.CLEAR_TEMPLATE_DETAIL,
+ payload: [],
});
export const fetchPatchSetDetailSystemsAction = (id, params) => ({
- type: ActionTypes.FETCH_PATCH_SET_DETAIL_SYSTEMS,
- payload: new Promise(resolve => {
- resolve(fetchPatchSetSystems(id, params));
- }).then(result => result)
+ type: ActionTypes.FETCH_PATCH_SET_DETAIL_SYSTEMS,
+ payload: new Promise((resolve) => {
+ resolve(fetchPatchSetSystems(id, params));
+ }).then((result) => result),
});
export const changePatchSetDetailsSystemsParams = (params) => ({
- type: ActionTypes.CHANGE_PATCH_SET_DETAIL_SYSTEMS_PARAMS,
- payload: params
+ type: ActionTypes.CHANGE_PATCH_SET_DETAIL_SYSTEMS_PARAMS,
+ payload: params,
});
export const changePatchSetDetailsSystemsMetadata = (params) => ({
- type: ActionTypes.CHANGE_PATCH_SET_DETAIL_SYSTEMS_METADATA,
- payload: params
+ type: ActionTypes.CHANGE_PATCH_SET_DETAIL_SYSTEMS_METADATA,
+ payload: params,
});
export const clearPatchSetDetailsSystems = () => ({
- type: ActionTypes.CLEAR_PATCH_SET_DETAIL_SYSTEMS,
- payload: []
+ type: ActionTypes.CLEAR_PATCH_SET_DETAIL_SYSTEMS,
+ payload: [],
});
export const clearPatchSetAction = () => ({
- type: ActionTypes.CLEAR_PATCH_SET,
- payload: []
+ type: ActionTypes.CLEAR_PATCH_SET,
+ payload: [],
});
export const fetchPatchSetSystemsNoFiltersAction = (params) => ({
- type: ActionTypes.FETCH_PATCH_SET_SYSTEMS_NO_FILTERS,
- payload: new Promise(resolve => {
- resolve(fetchPatchSetSystems(params));
- }).then(result => result)
+ type: ActionTypes.FETCH_PATCH_SET_SYSTEMS_NO_FILTERS,
+ payload: new Promise((resolve) => {
+ resolve(fetchPatchSetSystems(params));
+ }).then((result) => result),
});
export const clearPatchSetsAction = () => ({
- type: ActionTypes.CLEAR_PATCH_SETS,
- payload: []
+ type: ActionTypes.CLEAR_PATCH_SETS,
+ payload: [],
});
export const toggleAllSelectedAction = (state) => ({
- type: ActionTypes.TOGGLE_ALL_SELECTED,
- payload: state
+ type: ActionTypes.TOGGLE_ALL_SELECTED,
+ payload: state,
});
export const systemSelectAction = (toSelect) => ({
- type: 'SELECT_ENTITY',
- payload: toSelect
+ type: 'SELECT_ENTITY',
+ payload: toSelect,
});
diff --git a/src/store/Reducers/AdvisoryDetailStore.js b/src/store/Reducers/AdvisoryDetailStore.js
index f8ead15e2..280ee0308 100644
--- a/src/store/Reducers/AdvisoryDetailStore.js
+++ b/src/store/Reducers/AdvisoryDetailStore.js
@@ -1,37 +1,33 @@
import { CLEAR_ADVISORY_DETAILS, FETCH_ADVISORY_DETAILS } from '../ActionTypes';
-import {
-
- fetchPending,
- fetchRejected
-} from './HelperReducers';
+import { fetchPending, fetchRejected } from './HelperReducers';
export let initialState = {
- data: { attributes: {} },
- status: { isLoading: true }
+ data: { attributes: {} },
+ status: { isLoading: true },
};
// Reducer
export const AdvisoryDetailStore = (state = initialState, action) => {
- let newState = { ...state };
- switch (action.type) {
- case FETCH_ADVISORY_DETAILS + '_FULFILLED':
- return {
- ...state,
- status: { isLoading: false },
- data: action.payload.data,
- error: {}
- };
+ let newState = { ...state };
+ switch (action.type) {
+ case FETCH_ADVISORY_DETAILS + '_FULFILLED':
+ return {
+ ...state,
+ status: { isLoading: false },
+ data: action.payload.data,
+ error: {},
+ };
- case FETCH_ADVISORY_DETAILS + '_PENDING':
- return fetchPending(newState);
+ case FETCH_ADVISORY_DETAILS + '_PENDING':
+ return fetchPending(newState);
- case FETCH_ADVISORY_DETAILS + '_REJECTED':
- return fetchRejected(newState, action);
+ case FETCH_ADVISORY_DETAILS + '_REJECTED':
+ return fetchRejected(newState, action);
- case CLEAR_ADVISORY_DETAILS:
- return initialState;
+ case CLEAR_ADVISORY_DETAILS:
+ return initialState;
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
diff --git a/src/store/Reducers/AdvisoryDetailStore.test.js b/src/store/Reducers/AdvisoryDetailStore.test.js
index 06b47ffd0..0aca8e295 100644
--- a/src/store/Reducers/AdvisoryDetailStore.test.js
+++ b/src/store/Reducers/AdvisoryDetailStore.test.js
@@ -8,22 +8,22 @@ const action_rejected = FETCH_ADVISORY_DETAILS + '_REJECTED';
const action_pending = FETCH_ADVISORY_DETAILS + '_PENDING';
const fulfilled_payload = {
- data: []
-}
+ data: [],
+};
-const error = "Error";
+const error = 'Error';
describe('AdvisoryDetailStore tests', () => {
- it.each`
- state | action | result
- ${initialState} | ${{type: action_fulfilled, payload: fulfilled_payload}} | ${{...fulfilled_payload, status: { isLoading: false }, error: {}}}
- ${initialState} | ${{ type: action_pending, payload: {} }} | ${{ ...initialState, status: { code: undefined, isLoading: true, hasError: false }, error: {}}}
- ${initialState} | ${{type: CLEAR_ADVISORY_DETAILS, payload: {}}} | ${initialState}
- ${initialState} | ${{type: "NONSENSE", payload: {}}} | ${initialState}
- `('$action',({state, action: {type, payload}, result}) => {
- const res = AdvisoryDetailStore(state, {type, payload})
- expect(res).toEqual(result);
- });
+ it.each`
+ state | action | result
+ ${initialState} | ${{ type: action_fulfilled, payload: fulfilled_payload }} | ${{ ...fulfilled_payload, status: { isLoading: false }, error: {} }}
+ ${initialState} | ${{ type: action_pending, payload: {} }} | ${{ ...initialState, status: { code: undefined, isLoading: true, hasError: false }, error: {} }}
+ ${initialState} | ${{ type: CLEAR_ADVISORY_DETAILS, payload: {} }} | ${initialState}
+ ${initialState} | ${{ type: 'NONSENSE', payload: {} }} | ${initialState}
+ `('$action', ({ state, action: { type, payload }, result }) => {
+ const res = AdvisoryDetailStore(state, { type, payload });
+ expect(res).toEqual(result);
+ });
});
// ${initialState} | ${{type: action_rejected, payload: error}} | ${{...initialState, status: STATUS_REJECTED, error: error}}
diff --git a/src/store/Reducers/AdvisoryListStore.js b/src/store/Reducers/AdvisoryListStore.js
index ce426e85e..6c5fb18f0 100644
--- a/src/store/Reducers/AdvisoryListStore.js
+++ b/src/store/Reducers/AdvisoryListStore.js
@@ -1,49 +1,48 @@
import { storeListDefaults } from '../../Utilities/constants';
import * as ActionTypes from '../ActionTypes';
import {
- changeFilters,
- selectRows,
- expandRows,
- fetchPending,
- fetchRejected,
- fetchFulfilled
+ changeFilters,
+ selectRows,
+ expandRows,
+ fetchPending,
+ fetchRejected,
+ fetchFulfilled,
} from './HelperReducers';
-// eslint-disable-next-line
const storeListDefaultsModified = {
- ...storeListDefaults,
- queryParams: {}
+ ...storeListDefaults,
+ queryParams: {},
};
export const AdvisoryListStore = (state = storeListDefaultsModified, action) => {
- let newState = { ...state };
- switch (action.type) {
- case ActionTypes.FETCH_APPLICABLE_ADVISORIES + '_FULFILLED':
- return fetchFulfilled(newState, action);
+ let newState = { ...state };
+ switch (action.type) {
+ case ActionTypes.FETCH_APPLICABLE_ADVISORIES + '_FULFILLED':
+ return fetchFulfilled(newState, action);
- case ActionTypes.FETCH_APPLICABLE_ADVISORIES + '_PENDING':
- return fetchPending(newState);
+ case ActionTypes.FETCH_APPLICABLE_ADVISORIES + '_PENDING':
+ return fetchPending(newState);
- case ActionTypes.FETCH_APPLICABLE_ADVISORIES + '_REJECTED':
- return fetchRejected(newState, action);
+ case ActionTypes.FETCH_APPLICABLE_ADVISORIES + '_REJECTED':
+ return fetchRejected(newState, action);
- case ActionTypes.CHANGE_ADVISORY_LIST_PARAMS:
- return changeFilters(newState, action);
+ case ActionTypes.CHANGE_ADVISORY_LIST_PARAMS:
+ return changeFilters(newState, action);
- case ActionTypes.EXPAND_ADVISORY_ROW:
- return expandRows(newState, action);
+ case ActionTypes.EXPAND_ADVISORY_ROW:
+ return expandRows(newState, action);
- case ActionTypes.SELECT_ADVISORY_ROW:
- return selectRows(newState, action);
+ case ActionTypes.SELECT_ADVISORY_ROW:
+ return selectRows(newState, action);
- case ActionTypes.TRIGGER_GLOBAL_FILTER:
- return changeFilters(newState, action);
+ case ActionTypes.TRIGGER_GLOBAL_FILTER:
+ return changeFilters(newState, action);
- case ActionTypes.TOGGLE_ALL_SELECTED:
- newState.areAllSelected = action.payload;
- return newState;
+ case ActionTypes.TOGGLE_ALL_SELECTED:
+ newState.areAllSelected = action.payload;
+ return newState;
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
diff --git a/src/store/Reducers/AdvisoryListStore.test.js b/src/store/Reducers/AdvisoryListStore.test.js
index f02504665..114b8cf33 100644
--- a/src/store/Reducers/AdvisoryListStore.test.js
+++ b/src/store/Reducers/AdvisoryListStore.test.js
@@ -1,43 +1,51 @@
-
/* eslint-disable */
-import { STATUS_LOADING, STATUS_REJECTED, STATUS_RESOLVED, storeListDefaults } from '../../Utilities/constants';
-import { CHANGE_ADVISORY_LIST_PARAMS, EXPAND_ADVISORY_ROW, FETCH_APPLICABLE_ADVISORIES, SELECT_ADVISORY_ROW } from '../ActionTypes';
+import {
+ STATUS_LOADING,
+ STATUS_REJECTED,
+ STATUS_RESOLVED,
+ storeListDefaults,
+} from '../../Utilities/constants';
+import {
+ CHANGE_ADVISORY_LIST_PARAMS,
+ EXPAND_ADVISORY_ROW,
+ FETCH_APPLICABLE_ADVISORIES,
+ SELECT_ADVISORY_ROW,
+} from '../ActionTypes';
import { AdvisoryListStore } from './AdvisoryListStore';
const action_fulfilled = FETCH_APPLICABLE_ADVISORIES + '_FULFILLED';
const action_rejected = FETCH_APPLICABLE_ADVISORIES + '_REJECTED';
const action_pending = FETCH_APPLICABLE_ADVISORIES + '_PENDING';
const fulfilled_payload = {
- data: [],
- meta: {
- limit: 50,
- offset: 25,
- total_items: 50
- }
+ data: [],
+ meta: {
+ limit: 50,
+ offset: 25,
+ total_items: 50,
+ },
};
-// eslint-disable-next-line
const advisoryListStoreDefaults = {
- ...storeListDefaults,
- queryParams: {}
+ ...storeListDefaults,
+ queryParams: {},
};
const error = 'Error';
describe('AdvisoryListStore tests', () => {
- it.each`
- state | action | result
+ it.each`
+ state | action | result
${advisoryListStoreDefaults} | ${{ type: action_fulfilled, payload: fulfilled_payload }} | ${{ ...advisoryListStoreDefaults, metadata: fulfilled_payload.meta, rows: fulfilled_payload.data, status: { code: undefined, isLoading: false, hasError: false }, error: {} }}
${advisoryListStoreDefaults} | ${{ type: action_pending, payload: {} }} | ${{ ...advisoryListStoreDefaults, status: { code: undefined, isLoading: true, hasError: false }, error: {} }}
${advisoryListStoreDefaults} | ${{ type: EXPAND_ADVISORY_ROW, payload: { rowId: 1, value: true } }} | ${{ ...advisoryListStoreDefaults, expandedRows: { 1: true } }}
${advisoryListStoreDefaults} | ${{ type: CHANGE_ADVISORY_LIST_PARAMS, payload: { limit: 10 } }} | ${{ ...advisoryListStoreDefaults, queryParams: { limit: 10, offset: 0 } }}
${advisoryListStoreDefaults} | ${{ type: SELECT_ADVISORY_ROW, payload: { id: 1, selected: true } }} | ${{ ...advisoryListStoreDefaults, selectedRows: { 1: true } }}
${advisoryListStoreDefaults} | ${{ type: 'NONSENSE', payload: {} }} | ${advisoryListStoreDefaults}
- ${undefined} | ${{ type: 'NONSENSE', payload: {} }} | ${advisoryListStoreDefaults}
- `('$action', ({ state, action: { type, payload }, result }) => {
+ ${undefined} | ${{ type: 'NONSENSE', payload: {} }} | ${advisoryListStoreDefaults}
+ `('$action', ({ state, action: { type, payload }, result }) => {
const res = AdvisoryListStore(state, { type, payload });
expect(res).toEqual(result);
-});
+ });
});
// ${ storeListDefaults } | ${ { type: action_rejected, payload: error } } | ${ { ...storeListDefaults, status: STATUS_REJECTED, error } }
diff --git a/src/store/Reducers/AdvisorySystemsStore.js b/src/store/Reducers/AdvisorySystemsStore.js
index ce1749957..c41c5ed63 100644
--- a/src/store/Reducers/AdvisorySystemsStore.js
+++ b/src/store/Reducers/AdvisorySystemsStore.js
@@ -4,30 +4,28 @@ import { changeListParams } from '../../Utilities/Helpers';
// Initial State. It should not include page and perPage to persist them dynamically
export const initialState = {
- selectedRows: {},
- queryParams: {
- page: 1,
- perPage: 20
- }
+ selectedRows: {},
+ queryParams: {
+ page: 1,
+ perPage: 20,
+ },
};
// Reducer
export const AdvisorySystemsStore = (state = initialState, action) => {
- let newState = { ...state };
- switch (action.type) {
+ let newState = { ...state };
+ switch (action.type) {
+ case ActionTypes.CHANGE_AFFECTED_SYSTEMS_PARAMS:
+ newState.queryParams = changeListParams(newState.queryParams, action.payload);
+ return newState;
- case ActionTypes.CHANGE_AFFECTED_SYSTEMS_PARAMS:
- newState.queryParams = changeListParams(newState.queryParams, action.payload);
- return newState;
+ case ActionTypes.TRIGGER_GLOBAL_FILTER:
+ return changeFilters(newState, action);
- case ActionTypes.TRIGGER_GLOBAL_FILTER:
- return changeFilters(newState, action);
+ case ActionTypes.CLEAR_ADVISORY_SYSTEMS_REDUCER:
+ return initialState;
- case ActionTypes.CLEAR_ADVISORY_SYSTEMS_REDUCER:
- return initialState;
-
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
-
diff --git a/src/store/Reducers/CvesListStore.js b/src/store/Reducers/CvesListStore.js
index cf69ae84c..6d1ed4dde 100644
--- a/src/store/Reducers/CvesListStore.js
+++ b/src/store/Reducers/CvesListStore.js
@@ -1,28 +1,23 @@
import { storeListDefaults } from '../../Utilities/constants';
import * as ActionTypes from '../ActionTypes';
-import {
- changeFilters,
- fetchPending,
- fetchRejected,
- fetchFulfilled
-} from './HelperReducers';
+import { changeFilters, fetchPending, fetchRejected, fetchFulfilled } from './HelperReducers';
export const CvesListStore = (state = storeListDefaults, action) => {
- let newState = { ...state };
- switch (action.type) {
- case ActionTypes.FETCH_CVES_INFO + '_FULFILLED':
- return fetchFulfilled(newState, action);
+ let newState = { ...state };
+ switch (action.type) {
+ case ActionTypes.FETCH_CVES_INFO + '_FULFILLED':
+ return fetchFulfilled(newState, action);
- case ActionTypes.FETCH_CVES_INFO + '_PENDING':
- return fetchPending(newState);
+ case ActionTypes.FETCH_CVES_INFO + '_PENDING':
+ return fetchPending(newState);
- case ActionTypes.FETCH_CVES_INFO + '_REJECTED':
- return fetchRejected(newState, action);
+ case ActionTypes.FETCH_CVES_INFO + '_REJECTED':
+ return fetchRejected(newState, action);
- case ActionTypes.CHANGE_CVES_STORE_PARAMS:
- return changeFilters(newState, action);
+ case ActionTypes.CHANGE_CVES_STORE_PARAMS:
+ return changeFilters(newState, action);
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
diff --git a/src/store/Reducers/CvesListStore.test.js b/src/store/Reducers/CvesListStore.test.js
index fd30e01b0..417991d7c 100644
--- a/src/store/Reducers/CvesListStore.test.js
+++ b/src/store/Reducers/CvesListStore.test.js
@@ -1,15 +1,13 @@
-/* eslint-disable new-cap */
import { changeFilters, fetchFulfilled, fetchPending, fetchRejected } from './HelperReducers';
import { CvesListStore } from './CvesListStore';
import { FETCH_CVES_INFO } from '../ActionTypes';
jest.mock('./HelperReducers', () => ({
- ...jest.requireActual('./HelperReducers'),
- changeFilters: jest.fn(),
- fetchFulfilled: jest.fn(),
- fetchPending: jest.fn(),
- fetchRejected: jest.fn()
-})
-);
+ ...jest.requireActual('./HelperReducers'),
+ changeFilters: jest.fn(),
+ fetchFulfilled: jest.fn(),
+ fetchPending: jest.fn(),
+ fetchRejected: jest.fn(),
+}));
const state = { testObj: 'testVal' };
const actionFulfilled = FETCH_CVES_INFO + '_FULFILLED';
@@ -17,23 +15,30 @@ const actionRejected = FETCH_CVES_INFO + '_REJECTED';
const actionPending = FETCH_CVES_INFO + '_PENDING';
describe('PackageListStore', () => {
- it('should start fetching package list', () => {
- CvesListStore(state, { type: actionFulfilled, payload: { search: 'testSearch' } });
- expect(fetchFulfilled).toHaveBeenCalledWith(state, { type: actionFulfilled, payload: { search: 'testSearch' } });
+ it('should start fetching package list', () => {
+ CvesListStore(state, { type: actionFulfilled, payload: { search: 'testSearch' } });
+ expect(fetchFulfilled).toHaveBeenCalledWith(state, {
+ type: actionFulfilled,
+ payload: { search: 'testSearch' },
});
- it('should fetch package list', () => {
- CvesListStore(state, { type: actionPending, payload: { search: 'testSearch' } });
- expect(fetchPending).toHaveBeenCalledWith(state);
+ });
+ it('should fetch package list', () => {
+ CvesListStore(state, { type: actionPending, payload: { search: 'testSearch' } });
+ expect(fetchPending).toHaveBeenCalledWith(state);
+ });
+ it('should handle rejected call', () => {
+ CvesListStore(state, { type: actionRejected, payload: { search: 'testSearch' } });
+ expect(fetchRejected).toHaveBeenCalledWith(state, {
+ type: actionRejected,
+ payload: { search: 'testSearch' },
});
- it('should handle rejected call', () => {
- CvesListStore(state, { type: actionRejected, payload: { search: 'testSearch' } });
- expect(fetchRejected).toHaveBeenCalledWith(state, { type: actionRejected, payload: { search: 'testSearch' } });
+ });
+ it('should change params', () => {
+ CvesListStore(state, { type: 'CHANGE_CVES_STORE_PARAMS', payload: { search: 'testSearch' } });
+ expect(changeFilters).toHaveBeenCalledWith(state, {
+ type: 'CHANGE_CVES_STORE_PARAMS',
+ payload: { search: 'testSearch' },
});
-
- it('should change params', () => {
- CvesListStore(state, { type: 'CHANGE_CVES_STORE_PARAMS', payload: { search: 'testSearch' } });
- expect(changeFilters).toHaveBeenCalledWith(state,
- { type: 'CHANGE_CVES_STORE_PARAMS', payload: { search: 'testSearch' } });
- });
+ });
});
diff --git a/src/store/Reducers/GlobalFilterStore.js b/src/store/Reducers/GlobalFilterStore.js
index 593924ccc..795e6c6d8 100644
--- a/src/store/Reducers/GlobalFilterStore.js
+++ b/src/store/Reducers/GlobalFilterStore.js
@@ -1,32 +1,32 @@
import * as ActionTypes from '../ActionTypes';
const initialState = {
- selectedGlobalTags: [],
- selectedTags: [],
- systemProfile: undefined
+ selectedGlobalTags: [],
+ selectedTags: [],
+ systemProfile: undefined,
};
export const GlobalFilterStore = (state = initialState, action) => {
- switch (action.type) {
- case ActionTypes.CHANGE_TAGS:
- return {
- ...state,
- selectedTags: action.payload
- };
+ switch (action.type) {
+ case ActionTypes.CHANGE_TAGS:
+ return {
+ ...state,
+ selectedTags: action.payload,
+ };
- case ActionTypes.CHANGE_GLOBAL_TAGS:
- return {
- ...state,
- selectedGlobalTags: action.payload
- };
+ case ActionTypes.CHANGE_GLOBAL_TAGS:
+ return {
+ ...state,
+ selectedGlobalTags: action.payload,
+ };
- case ActionTypes.CHANGE_PROFILE:
- return {
- ...state,
- systemProfile: action.payload
- };
+ case ActionTypes.CHANGE_PROFILE:
+ return {
+ ...state,
+ systemProfile: action.payload,
+ };
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
diff --git a/src/store/Reducers/HelperReducers.js b/src/store/Reducers/HelperReducers.js
index c5f3970f4..31878150c 100644
--- a/src/store/Reducers/HelperReducers.js
+++ b/src/store/Reducers/HelperReducers.js
@@ -1,46 +1,44 @@
-import { addOrRemoveItemFromSet, changeListParams, getNewSelectedItems } from '../../Utilities/Helpers';
+import {
+ addOrRemoveItemFromSet,
+ changeListParams,
+ getNewSelectedItems,
+} from '../../Utilities/Helpers';
export const changeFilters = (state, action) => {
- state.queryParams = changeListParams(
- state.queryParams,
- action.payload
- );
+ state.queryParams = changeListParams(state.queryParams, action.payload);
- return state;
+ return state;
};
export const selectRows = (state, action) => {
- const selectedUpdated = getNewSelectedItems(action.payload, state.selectedRows);
- state = { ...state, selectedRows: selectedUpdated };
- return state;
+ const selectedUpdated = getNewSelectedItems(action.payload, state.selectedRows);
+ state = { ...state, selectedRows: selectedUpdated };
+ return state;
};
export const expandRows = (state, action) => {
- const expandedUpdated = addOrRemoveItemFromSet(
- state.expandedRows,
- [].concat(action.payload)
- );
- state = { ...state, expandedRows: expandedUpdated };
- return state;
+ const expandedUpdated = addOrRemoveItemFromSet(state.expandedRows, [].concat(action.payload));
+ state = { ...state, expandedRows: expandedUpdated };
+ return state;
};
export const fetchPending = (state) => {
- state.error = {};
- state.status = { isLoading: true, hasError: false, code: undefined };
- return state;
+ state.error = {};
+ state.status = { isLoading: true, hasError: false, code: undefined };
+ return state;
};
export const fetchRejected = (state, action) => {
- state.metadata = action.payload.meta || {};
- state.error = action.payload;
- state.status = { code: action.payload.status, isLoading: false, hasError: true };
- return state;
+ state.metadata = action.payload.meta || {};
+ state.error = action.payload;
+ state.status = { code: action.payload.status, isLoading: false, hasError: true };
+ return state;
};
export const fetchFulfilled = (state, action) => {
- state.rows = action.payload.data;
- state.metadata = action.payload.meta || {};
- state.error = {};
- state.status = { code: action.payload.status, isLoading: false, hasError: false };
- return state;
+ state.rows = action.payload.data;
+ state.metadata = action.payload.meta || {};
+ state.error = {};
+ state.status = { code: action.payload.status, isLoading: false, hasError: false };
+ return state;
};
diff --git a/src/store/Reducers/InventoryEntitiesReducer.js b/src/store/Reducers/InventoryEntitiesReducer.js
index 5e3359f2b..d2bb0ba34 100644
--- a/src/store/Reducers/InventoryEntitiesReducer.js
+++ b/src/store/Reducers/InventoryEntitiesReducer.js
@@ -1,120 +1,128 @@
-import { createAdvisorySystemsRows, createSystemsRows, createPatchSetDetailRows } from '../../Utilities/DataMappers';
-import { createPackageSystemsRows } from '../../Utilities/DataMappers';
+import {
+ createAdvisorySystemsRows,
+ createSystemsRows,
+ createPatchSetDetailRows,
+ createPackageSystemsRows,
+} from '../../Utilities/DataMappers';
import { selectRows, fetchRejected } from './HelperReducers';
import * as ActionTypes from '../ActionTypes';
// Initial State. It should not include page and perPage to persist them dynamically
const initialState = {
- rows: [],
- entities: [],
- selectedRows: {},
- status: {},
- page: 1,
- perPage: 20,
- metadata: {
- limit: 20,
- offset: 0,
- total_items: 0
- }
+ rows: [],
+ entities: [],
+ selectedRows: {},
+ status: {},
+ page: 1,
+ perPage: 20,
+ metadata: {
+ limit: 20,
+ offset: 0,
+ total_items: 0,
+ },
};
// Reducer
export const modifyInventory = (columns, state) => {
- if (state.loaded) {
- return {
- ...state,
- status: { isLoading: false, hasError: false },
- rows: createSystemsRows(state.rows, state.selectedRows)
- };
- }
-
- return state;
+ if (state.loaded) {
+ return {
+ ...state,
+ status: { isLoading: false, hasError: false },
+ rows: createSystemsRows(state.rows, state.selectedRows),
+ };
+ }
+
+ return state;
};
export const modifyPackageSystems = (columns, state, lastAction) => {
- // this is pretty scuffed, but trust me it cannot be solved more elegantly because
- // single item select actions are triggered by Inventory and contain only system id
- // but we also need to match available_evra for each selected system for remediations
- // this does not happen with bulk select action because they are implemented by Patchman
- // the id-evra mapping is cached to minimize O(n^2) time complexity of id-evra matching
- if (lastAction === 'SELECT_ENTITY') {
- let idToEvraMapping = state.packageEvraCache;
-
- if (idToEvraMapping === undefined) {
- idToEvraMapping = {};
- state.rows.forEach(row => idToEvraMapping[row.id] = row.available_evra);
- state.packageEvraCache = idToEvraMapping;
- }
-
- state.selectedRows = state.selectedRows ? Object.fromEntries(
- Object.entries(state.selectedRows).map(([id, value]) =>
- value === true ? [id, idToEvraMapping[id]] : [id, value]
- )
- ) : state.selectedRows;
- }
-
- if (lastAction === 'LOAD_ENTITIES_FULFILLED') {
- state.packageEvraCache = undefined;
- }
-
- if (state.loaded) {
- return {
- ...state,
- columns,
- rows: createPackageSystemsRows(state.rows, state.selectedRows)
- };
+ // this is pretty scuffed, but trust me it cannot be solved more elegantly because
+ // single item select actions are triggered by Inventory and contain only system id
+ // but we also need to match available_evra for each selected system for remediations
+ // this does not happen with bulk select action because they are implemented by Patchman
+ // the id-evra mapping is cached to minimize O(n^2) time complexity of id-evra matching
+ if (lastAction === 'SELECT_ENTITY') {
+ let idToEvraMapping = state.packageEvraCache;
+
+ if (idToEvraMapping === undefined) {
+ idToEvraMapping = {};
+ state.rows.forEach((row) => (idToEvraMapping[row.id] = row.available_evra));
+ state.packageEvraCache = idToEvraMapping;
}
- return state;
+ state.selectedRows = state.selectedRows
+ ? Object.fromEntries(
+ Object.entries(state.selectedRows).map(([id, value]) =>
+ value === true ? [id, idToEvraMapping[id]] : [id, value],
+ ),
+ )
+ : state.selectedRows;
+ }
+
+ if (lastAction === 'LOAD_ENTITIES_FULFILLED') {
+ state.packageEvraCache = undefined;
+ }
+
+ if (state.loaded) {
+ return {
+ ...state,
+ columns,
+ rows: createPackageSystemsRows(state.rows, state.selectedRows),
+ };
+ }
+
+ return state;
};
export const modifyTemplateDetailSystems = (columns, state) => {
- if (state.loaded) {
- return {
- ...state,
- status: { isLoading: false, hasError: false },
- rows: createPatchSetDetailRows(state.rows, state.selectedRows)
- };
- }
-
- return state;
+ if (state.loaded) {
+ return {
+ ...state,
+ status: { isLoading: false, hasError: false },
+ rows: createPatchSetDetailRows(state.rows, state.selectedRows),
+ };
+ }
+
+ return state;
};
export const modifyAdvisorySystems = (columns, state) => {
- if (state.loaded) {
- return {
- ...state,
- status: { isLoading: false, hasError: false },
- rows: createAdvisorySystemsRows(state.rows, state.selectedRows)
- };
- }
-
- return state;
+ if (state.loaded) {
+ return {
+ ...state,
+ status: { isLoading: false, hasError: false },
+ rows: createAdvisorySystemsRows(state.rows, state.selectedRows),
+ };
+ }
+
+ return state;
};
-export const inventoryEntitiesReducer = (columns, inventoryModifier) => (state = initialState, action) => {
+export const inventoryEntitiesReducer =
+ (columns, inventoryModifier) =>
+ (state = initialState, action) => {
let newState = { ...state };
switch (action.type) {
- case 'LOAD_ENTITIES_FULFILLED':
- return inventoryModifier(columns, newState, action.type);
+ case 'LOAD_ENTITIES_FULFILLED':
+ return inventoryModifier(columns, newState, action.type);
- case 'LOAD_ENTITIES_PENDING':
- newState.status = { isLoading: true, hasError: false };
- return newState;
+ case 'LOAD_ENTITIES_PENDING':
+ newState.status = { isLoading: true, hasError: false };
+ return newState;
- case 'LOAD_ENTITIES_REJECTED':
- return fetchRejected(newState, action);
+ case 'LOAD_ENTITIES_REJECTED':
+ return fetchRejected(newState, action);
- case 'SELECT_ENTITY': {
- const stateAfterSelection = selectRows(newState, action);
- return inventoryModifier(columns, stateAfterSelection, action.type);
- }
+ case 'SELECT_ENTITY': {
+ const stateAfterSelection = selectRows(newState, action);
+ return inventoryModifier(columns, stateAfterSelection, action.type);
+ }
- case ActionTypes.CLEAR_INVENTORY_REDUCER:
- return initialState;
+ case ActionTypes.CLEAR_INVENTORY_REDUCER:
+ return initialState;
- default:
- return state;
+ default:
+ return state;
}
-};
+ };
diff --git a/src/store/Reducers/InventoryEntitiesReducer.test.js b/src/store/Reducers/InventoryEntitiesReducer.test.js
index 28d91697e..0a91cff22 100644
--- a/src/store/Reducers/InventoryEntitiesReducer.test.js
+++ b/src/store/Reducers/InventoryEntitiesReducer.test.js
@@ -1,76 +1,84 @@
-import { inventoryEntitiesReducer, initialState } from './InventoryEntitiesReducer';
-import { modifyInventory, modifyPackageSystems } from './InventoryEntitiesReducer';
+import {
+ inventoryEntitiesReducer,
+ initialState,
+ modifyInventory,
+ modifyPackageSystems,
+} from './InventoryEntitiesReducer';
import { systemRows } from '../../Utilities/RawDataForTesting';
-/* eslint-disable */
describe('InventoryEntitiesReducer tests', () => {
- it.each`
- columns | state | action | result
- ${[]} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }} | ${{ type: 'LOAD_ENTITIES_FULFILLED', payload: {} }} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }}
- ${[]} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }} | ${{ type: 'LOAD_ENTITIES_PENDING', payload: {} }} | ${{ ...initialState, loaded: false, status: { isLoading: true, hasError: false, }, columns: [{ key: 'testCol' }] }}
- ${[]} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }} | ${{ type: 'LOAD_ENTITIES_REJECTED', payload: { status: 400 } }} | ${{ ...initialState, loaded: false, status: { isLoading: false, code: 400, hasError: true }, error: { status: 400 }, metadata: {}, columns: [{ key: 'testCol' }] }}
- ${[]} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }} | ${{ type: 'SELECT_ENTITY', payload: [{ id: '83e97cde-74b4-4752-819d-704687bbc286' }] }}} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }], selectedRows: { "83e97cde-74b4-4752-819d-704687bbc286": undefined } }}
+ it.each`
+ columns | state | action | result
+ ${[]} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }} | ${{ type: 'LOAD_ENTITIES_FULFILLED', payload: {} }} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }}
+ ${[]} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }} | ${{ type: 'LOAD_ENTITIES_PENDING', payload: {} }} | ${{ ...initialState, loaded: false, status: { isLoading: true, hasError: false }, columns: [{ key: 'testCol' }] }}
+ ${[]} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }} | ${{ type: 'LOAD_ENTITIES_REJECTED', payload: { status: 400 } }} | ${{ ...initialState, loaded: false, status: { isLoading: false, code: 400, hasError: true }, error: { status: 400 }, metadata: {}, columns: [{ key: 'testCol' }] }}
+ ${[]} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }} | ${{ type: 'SELECT_ENTITY', payload: [{ id: '83e97cde-74b4-4752-819d-704687bbc286' }] }} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }], selectedRows: { '83e97cde-74b4-4752-819d-704687bbc286': undefined } }}
+ `('$action', ({ columns, state, action: { type, payload }, result }) => {
+ const wrapperReducer = inventoryEntitiesReducer(columns, modifyInventory);
+ const res = wrapperReducer(state, { type, payload });
+ expect(res).toEqual(result);
+ });
- `('$action', ({ columns, state, action: { type, payload }, result }) => {
- const wrapperReducer = inventoryEntitiesReducer(columns, modifyInventory);
- const res = wrapperReducer(state, { type, payload })
- expect(res).toEqual(result);
- });
-
- it.each`
- columns | state | action | result
- ${[]} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }} | ${{ type: 'LOAD_ENTITIES_FULFILLED', payload: {} }} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }}
-
- `('$action', ({ columns, state, action: { type, payload }, result }) => {
- const wrapperReducer = inventoryEntitiesReducer(columns, modifyPackageSystems);
- const res = wrapperReducer(state, { type, payload })
- expect(res).toEqual(result);
- });
+ it.each`
+ columns | state | action | result
+ ${[]} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }} | ${{ type: 'LOAD_ENTITIES_FULFILLED', payload: {} }} | ${{ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }}
+ `('$action', ({ columns, state, action: { type, payload }, result }) => {
+ const wrapperReducer = inventoryEntitiesReducer(columns, modifyPackageSystems);
+ const res = wrapperReducer(state, { type, payload });
+ expect(res).toEqual(result);
+ });
- it('should modifyInventory nicely', () => {
- const wrapperReducer = inventoryEntitiesReducer([{ key: 'updated' }], modifyInventory);
- const res = wrapperReducer({ ...initialState, loaded: true, columns: [{ key: 'testCol' }], rows: systemRows }, { type: 'LOAD_ENTITIES_FULFILLED', payload: {} });
- expect(res).toEqual({
- ...initialState,
- loaded: true,
- columns: [{ key: 'testCol' }],
- rows: expect.any(Object),
- status: {
- hasError: false,
- isLoading: false,
- }
- });
+ it('should modifyInventory nicely', () => {
+ const wrapperReducer = inventoryEntitiesReducer([{ key: 'updated' }], modifyInventory);
+ const res = wrapperReducer(
+ { ...initialState, loaded: true, columns: [{ key: 'testCol' }], rows: systemRows },
+ { type: 'LOAD_ENTITIES_FULFILLED', payload: {} },
+ );
+ expect(res).toEqual({
+ ...initialState,
+ loaded: true,
+ columns: [{ key: 'testCol' }],
+ rows: expect.any(Object),
+ status: {
+ hasError: false,
+ isLoading: false,
+ },
});
+ });
- it('should use modifyPackageSystems nicely', () => {
- const wrapperReducer = inventoryEntitiesReducer([], modifyPackageSystems);
- const res = wrapperReducer({ ...initialState, loaded: true, rows: systemRows }, { type: 'LOAD_ENTITIES_FULFILLED', payload: {} });
- expect(res).toEqual({
- ...initialState,
- loaded: true,
- columns: [],
- rows: expect.any(Object)
- });
+ it('should use modifyPackageSystems nicely', () => {
+ const wrapperReducer = inventoryEntitiesReducer([], modifyPackageSystems);
+ const res = wrapperReducer(
+ { ...initialState, loaded: true, rows: systemRows },
+ { type: 'LOAD_ENTITIES_FULFILLED', payload: {} },
+ );
+ expect(res).toEqual({
+ ...initialState,
+ loaded: true,
+ columns: [],
+ rows: expect.any(Object),
});
+ });
+ it('should return initial state on clear', () => {
+ const wrapperReducer = inventoryEntitiesReducer([], modifyPackageSystems);
- it('should return initial state on clear', () => {
- const wrapperReducer = inventoryEntitiesReducer([], modifyPackageSystems);
-
- const res = wrapperReducer({ ...initialState, loaded: false, columns: [{ key: 'testCol' }] }, { type: 'CLEAR_INVENTORY_REDUCER', payload: {} });
- expect(res).toEqual({
- entities:[],
- metadata: {
- limit: 20,
- offset: 0,
- total_items: 0,
- },
- page: 1,
- perPage: 20,
- rows: [],
- selectedRows: {},
- status: {},
- });
- });
+ const res = wrapperReducer(
+ { ...initialState, loaded: false, columns: [{ key: 'testCol' }] },
+ { type: 'CLEAR_INVENTORY_REDUCER', payload: {} },
+ );
+ expect(res).toEqual({
+ entities: [],
+ metadata: {
+ limit: 20,
+ offset: 0,
+ total_items: 0,
+ },
+ page: 1,
+ perPage: 20,
+ rows: [],
+ selectedRows: {},
+ status: {},
+ });
+ });
});
-/* eslint-enable */
diff --git a/src/store/Reducers/InventoryReducers.test.js b/src/store/Reducers/InventoryReducers.test.js
index e365a50f7..13b7837a9 100644
--- a/src/store/Reducers/InventoryReducers.test.js
+++ b/src/store/Reducers/InventoryReducers.test.js
@@ -1,4 +1,3 @@
-/* eslint-disable no-unused-vars */
import { AdvisorySystemsStore, initialState } from './AdvisorySystemsStore';
import { SystemsStore } from './SystemsStore';
import { PackageSystemsStore } from './PackageSystemsStore';
@@ -6,54 +5,73 @@ import { systemRows, systemsStoreState } from '../../Utilities/RawDataForTesting
const state = { ...initialState, loaded: true, columns: [{ key: 'testCol' }], rows: systemRows };
-/* eslint-disable */
describe('InventoryEntitiesReducer tests', () => {
-
- it('should change AdvisorySystemsStore nicely', () => {
- const res = AdvisorySystemsStore(state, { type: 'CHANGE_AFFECTED_SYSTEMS_PARAMS', payload: { search: 'testSearch' } });
- expect(res).toEqual(systemsStoreState);
- });
-
- it('should change SystemsStore nicely', () => {
- const res = SystemsStore(state, { type: 'CHANGE_SYSTEMS_PARAMS', payload: { search: 'testSearch' } });
- expect(res).toEqual(systemsStoreState);
+ it('should change AdvisorySystemsStore nicely', () => {
+ const res = AdvisorySystemsStore(state, {
+ type: 'CHANGE_AFFECTED_SYSTEMS_PARAMS',
+ payload: { search: 'testSearch' },
});
+ expect(res).toEqual(systemsStoreState);
+ });
- it('should change PackageSystemsStore nicely', () => {
- const res = PackageSystemsStore(state, { type: 'CHANGE_PACKAGE_SYSTEMS_PARAMS', payload: { search: 'testSearch' } });
- expect(res).toEqual(systemsStoreState);
+ it('should change SystemsStore nicely', () => {
+ const res = SystemsStore(state, {
+ type: 'CHANGE_SYSTEMS_PARAMS',
+ payload: { search: 'testSearch' },
});
+ expect(res).toEqual(systemsStoreState);
+ });
- it('should change AdvisorySystemsStore on global filter', () => {
- const res = AdvisorySystemsStore(state, { type: 'TRIGGER_GLOBAL_FILTER', payload: { search: 'testSearch' } });
- expect(res).toEqual(systemsStoreState);
+ it('should change PackageSystemsStore nicely', () => {
+ const res = PackageSystemsStore(state, {
+ type: 'CHANGE_PACKAGE_SYSTEMS_PARAMS',
+ payload: { search: 'testSearch' },
});
+ expect(res).toEqual(systemsStoreState);
+ });
- it('should change SystemsStore on global filter', () => {
- const res = SystemsStore(state, { type: 'TRIGGER_GLOBAL_FILTER', payload: { search: 'testSearch' } });
- expect(res).toEqual(systemsStoreState);
+ it('should change AdvisorySystemsStore on global filter', () => {
+ const res = AdvisorySystemsStore(state, {
+ type: 'TRIGGER_GLOBAL_FILTER',
+ payload: { search: 'testSearch' },
});
+ expect(res).toEqual(systemsStoreState);
+ });
- it('should change PackageSystemsStore on global filter', () => {
- const res = PackageSystemsStore(state, { type: 'TRIGGER_GLOBAL_FILTER', payload: { search: 'testSearch' } });
- expect(res).toEqual(systemsStoreState);
+ it('should change SystemsStore on global filter', () => {
+ const res = SystemsStore(state, {
+ type: 'TRIGGER_GLOBAL_FILTER',
+ payload: { search: 'testSearch' },
});
+ expect(res).toEqual(systemsStoreState);
+ });
-
- it('should clear AdvisorySystemsStore nicely', () => {
- const res = AdvisorySystemsStore(state, { type: 'CLEAR_ADVISORY_SYSTEMS_REDUCER', payload: {} });
- expect(res).toEqual(initialState);
+ it('should change PackageSystemsStore on global filter', () => {
+ const res = PackageSystemsStore(state, {
+ type: 'TRIGGER_GLOBAL_FILTER',
+ payload: { search: 'testSearch' },
});
+ expect(res).toEqual(systemsStoreState);
+ });
- it('should clear PackageSystemsStore nicely', () => {
- const res = PackageSystemsStore(state, { type: 'CLEAR_PACKAGE_SYSTEMS_REDUCER', payload: {} });
- expect(res).toEqual(initialState);
+ it('should clear AdvisorySystemsStore nicely', () => {
+ const res = AdvisorySystemsStore(state, {
+ type: 'CLEAR_ADVISORY_SYSTEMS_REDUCER',
+ payload: {},
});
+ expect(res).toEqual(initialState);
+ });
+ it('should clear PackageSystemsStore nicely', () => {
+ const res = PackageSystemsStore(state, { type: 'CLEAR_PACKAGE_SYSTEMS_REDUCER', payload: {} });
+ expect(res).toEqual(initialState);
+ });
- it('should change SystemsStore metadata on CHANGE_SYSTEMS_METADATA', () => {
- const res = SystemsStore(state, { type: 'CHANGE_SYSTEMS_METADATA', payload: { metadata: 'testMetadata' } });
- expect(res).toEqual({ ...state, metadata: { metadata : 'testMetadata' } }) ;
+ it('should change SystemsStore metadata on CHANGE_SYSTEMS_METADATA', () => {
+ const res = SystemsStore(state, {
+ type: 'CHANGE_SYSTEMS_METADATA',
+ payload: { metadata: 'testMetadata' },
});
+ expect(res).toEqual({ ...state, metadata: { metadata: 'testMetadata' } });
+ });
});
-/* eslint-enable */
diff --git a/src/store/Reducers/PackageDetailStore.js b/src/store/Reducers/PackageDetailStore.js
index a3d812eec..015456303 100644
--- a/src/store/Reducers/PackageDetailStore.js
+++ b/src/store/Reducers/PackageDetailStore.js
@@ -2,32 +2,32 @@ import { CLEAR_PACKAGE_DETAILS, FETCH_PACKAGE_DETAILS } from '../ActionTypes';
import { fetchPending, fetchRejected } from './HelperReducers';
export let initialState = {
- data: { attributes: {} },
- status: { isLoading: true }
+ data: { attributes: {} },
+ status: { isLoading: true },
};
// Reducer
export const PackageDetailStore = (state = initialState, action) => {
- let newState = { ...state };
- switch (action.type) {
- case FETCH_PACKAGE_DETAILS + '_FULFILLED':
- return {
- ...state,
- status: { isLoading: false },
- data: action.payload.data,
- error: {}
- };
+ let newState = { ...state };
+ switch (action.type) {
+ case FETCH_PACKAGE_DETAILS + '_FULFILLED':
+ return {
+ ...state,
+ status: { isLoading: false },
+ data: action.payload.data,
+ error: {},
+ };
- case FETCH_PACKAGE_DETAILS + '_PENDING':
- return fetchPending(newState);
+ case FETCH_PACKAGE_DETAILS + '_PENDING':
+ return fetchPending(newState);
- case FETCH_PACKAGE_DETAILS + '_REJECTED':
- return fetchRejected(newState, action);
+ case FETCH_PACKAGE_DETAILS + '_REJECTED':
+ return fetchRejected(newState, action);
- case CLEAR_PACKAGE_DETAILS:
- return initialState;
+ case CLEAR_PACKAGE_DETAILS:
+ return initialState;
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
diff --git a/src/store/Reducers/PackageDetailStore.test.js b/src/store/Reducers/PackageDetailStore.test.js
index b2e80cf71..712ccd6f9 100644
--- a/src/store/Reducers/PackageDetailStore.test.js
+++ b/src/store/Reducers/PackageDetailStore.test.js
@@ -1,14 +1,12 @@
-/* eslint-disable new-cap */
import { fetchPending, fetchRejected } from './HelperReducers';
import { PackageDetailStore } from './PackageDetailStore';
import { FETCH_PACKAGE_DETAILS } from '../ActionTypes';
jest.mock('./HelperReducers', () => ({
- ...jest.requireActual('./HelperReducers'),
- fetchFulfilled: jest.fn(),
- fetchPending: jest.fn(),
- fetchRejected: jest.fn()
-})
-);
+ ...jest.requireActual('./HelperReducers'),
+ fetchFulfilled: jest.fn(),
+ fetchPending: jest.fn(),
+ fetchRejected: jest.fn(),
+}));
const state = { testObj: 'testVal' };
const actionFulfilled = FETCH_PACKAGE_DETAILS + '_FULFILLED';
@@ -16,45 +14,51 @@ const actionRejected = FETCH_PACKAGE_DETAILS + '_REJECTED';
const actionPending = FETCH_PACKAGE_DETAILS + '_PENDING';
describe('PackageListStore', () => {
- it('should start fetching package list', () => {
- const res = PackageDetailStore(state, { type: actionFulfilled, payload: { data: 'testData' } });
- expect(res).toEqual({
- ...state, status: { isLoading: false },
- data: 'testData',
- error: {}
- });
+ it('should start fetching package list', () => {
+ const res = PackageDetailStore(state, { type: actionFulfilled, payload: { data: 'testData' } });
+ expect(res).toEqual({
+ ...state,
+ status: { isLoading: false },
+ data: 'testData',
+ error: {},
});
+ });
- it('should fetch package detail', () => {
- PackageDetailStore(state, { type: actionPending, payload: { search: 'testSearch' } });
- expect(fetchPending).toHaveBeenCalledWith(state);
+ it('should fetch package detail', () => {
+ PackageDetailStore(state, { type: actionPending, payload: { search: 'testSearch' } });
+ expect(fetchPending).toHaveBeenCalledWith(state);
+ });
+ it('should handle rejected call', () => {
+ PackageDetailStore(state, { type: actionRejected, payload: { search: 'testSearch' } });
+ expect(fetchRejected).toHaveBeenCalledWith(state, {
+ type: actionRejected,
+ payload: { search: 'testSearch' },
});
- it('should handle rejected call', () => {
- PackageDetailStore(state, { type: actionRejected, payload: { search: 'testSearch' } });
- expect(fetchRejected).toHaveBeenCalledWith(state, { type: actionRejected, payload: { search: 'testSearch' } });
-
+ });
+ it('should handle clear call', () => {
+ const res = PackageDetailStore(state, {
+ type: 'CLEAR_PACKAGE_DETAILS',
+ payload: { search: 'testSearch' },
});
- it('should handle clear call', () => {
- const res = PackageDetailStore(state, { type: 'CLEAR_PACKAGE_DETAILS', payload: { search: 'testSearch' } });
- expect(res).toEqual({
- data: {
- attributes: {}
- },
- status: {
- isLoading: true
- }
- });
+ expect(res).toEqual({
+ data: {
+ attributes: {},
+ },
+ status: {
+ isLoading: true,
+ },
});
+ });
- // it('should change params', () => {
- // PackageDetailStore(state, { type: 'CHANGE_PACKAGES_LIST_PARAMS', payload: { search: 'testSearch' } });
- // expect(changeFilters).toHaveBeenCalledWith(state,
- // { type: 'CHANGE_PACKAGES_LIST_PARAMS', payload: { search: 'testSearch' } });
- // });
+ // it('should change params', () => {
+ // PackageDetailStore(state, { type: 'CHANGE_PACKAGES_LIST_PARAMS', payload: { search: 'testSearch' } });
+ // expect(changeFilters).toHaveBeenCalledWith(state,
+ // { type: 'CHANGE_PACKAGES_LIST_PARAMS', payload: { search: 'testSearch' } });
+ // });
- // it('should apply global filter', () => {
- // PackagesListStore(state, { type: 'TRIGGER_GLOBAL_FILTER', payload: { search: 'testSearch' } });
- // expect(changeFilters).toHaveBeenCalledWith(state,
- // { type: 'TRIGGER_GLOBAL_FILTER', payload: { search: 'testSearch' } });
- // });
+ // it('should apply global filter', () => {
+ // PackagesListStore(state, { type: 'TRIGGER_GLOBAL_FILTER', payload: { search: 'testSearch' } });
+ // expect(changeFilters).toHaveBeenCalledWith(state,
+ // { type: 'TRIGGER_GLOBAL_FILTER', payload: { search: 'testSearch' } });
+ // });
});
diff --git a/src/store/Reducers/PackageListStore.test.js b/src/store/Reducers/PackageListStore.test.js
index a8c7d4643..a73ce9c62 100644
--- a/src/store/Reducers/PackageListStore.test.js
+++ b/src/store/Reducers/PackageListStore.test.js
@@ -1,15 +1,13 @@
-/* eslint-disable new-cap */
import { changeFilters, fetchFulfilled, fetchPending, fetchRejected } from './HelperReducers';
import { PackagesListStore } from './PackagesListStore';
import { FETCH_PACKAGES_LIST } from '../ActionTypes';
jest.mock('./HelperReducers', () => ({
- ...jest.requireActual('./HelperReducers'),
- changeFilters: jest.fn(),
- fetchFulfilled: jest.fn(),
- fetchPending: jest.fn(),
- fetchRejected: jest.fn()
-})
-);
+ ...jest.requireActual('./HelperReducers'),
+ changeFilters: jest.fn(),
+ fetchFulfilled: jest.fn(),
+ fetchPending: jest.fn(),
+ fetchRejected: jest.fn(),
+}));
const state = { testObj: 'testVal' };
const actionFulfilled = FETCH_PACKAGES_LIST + '_FULFILLED';
@@ -17,29 +15,41 @@ const actionRejected = FETCH_PACKAGES_LIST + '_REJECTED';
const actionPending = FETCH_PACKAGES_LIST + '_PENDING';
describe('PackageListStore', () => {
- it('should start fetching package list', () => {
- PackagesListStore(state, { type: actionFulfilled, payload: { search: 'testSearch' } });
- expect(fetchFulfilled).toHaveBeenCalledWith(state, { type: actionFulfilled, payload: { search: 'testSearch' } });
+ it('should start fetching package list', () => {
+ PackagesListStore(state, { type: actionFulfilled, payload: { search: 'testSearch' } });
+ expect(fetchFulfilled).toHaveBeenCalledWith(state, {
+ type: actionFulfilled,
+ payload: { search: 'testSearch' },
});
- it('should fetch package list', () => {
- PackagesListStore(state, { type: actionPending, payload: { search: 'testSearch' } });
- expect(fetchPending).toHaveBeenCalledWith(state);
+ });
+ it('should fetch package list', () => {
+ PackagesListStore(state, { type: actionPending, payload: { search: 'testSearch' } });
+ expect(fetchPending).toHaveBeenCalledWith(state);
+ });
+ it('should handle rejected call', () => {
+ PackagesListStore(state, { type: actionRejected, payload: { search: 'testSearch' } });
+ expect(fetchRejected).toHaveBeenCalledWith(state, {
+ type: actionRejected,
+ payload: { search: 'testSearch' },
});
- it('should handle rejected call', () => {
- PackagesListStore(state, { type: actionRejected, payload: { search: 'testSearch' } });
- expect(fetchRejected).toHaveBeenCalledWith(state, { type: actionRejected, payload: { search: 'testSearch' } });
+ });
+ it('should change params', () => {
+ PackagesListStore(state, {
+ type: 'CHANGE_PACKAGES_LIST_PARAMS',
+ payload: { search: 'testSearch' },
});
-
- it('should change params', () => {
- PackagesListStore(state, { type: 'CHANGE_PACKAGES_LIST_PARAMS', payload: { search: 'testSearch' } });
- expect(changeFilters).toHaveBeenCalledWith(state,
- { type: 'CHANGE_PACKAGES_LIST_PARAMS', payload: { search: 'testSearch' } });
+ expect(changeFilters).toHaveBeenCalledWith(state, {
+ type: 'CHANGE_PACKAGES_LIST_PARAMS',
+ payload: { search: 'testSearch' },
});
+ });
- it('should apply global filter', () => {
- PackagesListStore(state, { type: 'TRIGGER_GLOBAL_FILTER', payload: { search: 'testSearch' } });
- expect(changeFilters).toHaveBeenCalledWith(state,
- { type: 'TRIGGER_GLOBAL_FILTER', payload: { search: 'testSearch' } });
+ it('should apply global filter', () => {
+ PackagesListStore(state, { type: 'TRIGGER_GLOBAL_FILTER', payload: { search: 'testSearch' } });
+ expect(changeFilters).toHaveBeenCalledWith(state, {
+ type: 'TRIGGER_GLOBAL_FILTER',
+ payload: { search: 'testSearch' },
});
+ });
});
diff --git a/src/store/Reducers/PackageSystemsStore.js b/src/store/Reducers/PackageSystemsStore.js
index 368b0e342..7a1453f4b 100644
--- a/src/store/Reducers/PackageSystemsStore.js
+++ b/src/store/Reducers/PackageSystemsStore.js
@@ -4,28 +4,27 @@ import { changeListParams } from '../../Utilities/Helpers';
// Initial State. It should not include page and perPage to persist them dynamically
export const initialState = {
- selectedRows: {},
- queryParams: {
- page: 1,
- perPage: 20
- }
+ selectedRows: {},
+ queryParams: {
+ page: 1,
+ perPage: 20,
+ },
};
export const PackageSystemsStore = (state = initialState, action) => {
- let newState = { ...state };
- switch (action.type) {
+ let newState = { ...state };
+ switch (action.type) {
+ case ActionTypes.CHANGE_PACKAGE_SYSTEMS_PARAMS:
+ newState.queryParams = changeListParams(newState.queryParams, action.payload);
+ return newState;
- case ActionTypes.CHANGE_PACKAGE_SYSTEMS_PARAMS:
- newState.queryParams = changeListParams(newState.queryParams, action.payload);
- return newState;
+ case ActionTypes.TRIGGER_GLOBAL_FILTER:
+ return changeFilters(newState, action);
- case ActionTypes.TRIGGER_GLOBAL_FILTER:
- return changeFilters(newState, action);
+ case ActionTypes.CLEAR_PACKAGE_SYSTEMS_REDUCER:
+ return initialState;
- case ActionTypes.CLEAR_PACKAGE_SYSTEMS_REDUCER:
- return initialState;
-
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
diff --git a/src/store/Reducers/PackagesListStore.js b/src/store/Reducers/PackagesListStore.js
index 417d05a95..3069a8386 100644
--- a/src/store/Reducers/PackagesListStore.js
+++ b/src/store/Reducers/PackagesListStore.js
@@ -2,27 +2,27 @@ import { storeListDefaults, packagesListDefaultFilters } from '../../Utilities/c
import * as ActionTypes from '../ActionTypes';
import { changeFilters, fetchFulfilled, fetchPending, fetchRejected } from './HelperReducers';
-const initialStore = { ...storeListDefaults, queryParams: packagesListDefaultFilters };
+const initialStore = { ...storeListDefaults, queryParams: packagesListDefaultFilters };
export const PackagesListStore = (state = initialStore, action) => {
- let newState = { ...state };
- switch (action.type) {
- case ActionTypes.FETCH_PACKAGES_LIST + '_PENDING':
- return fetchPending(newState);
+ let newState = { ...state };
+ switch (action.type) {
+ case ActionTypes.FETCH_PACKAGES_LIST + '_PENDING':
+ return fetchPending(newState);
- case ActionTypes.FETCH_PACKAGES_LIST + '_FULFILLED':
- return fetchFulfilled(newState, action);
+ case ActionTypes.FETCH_PACKAGES_LIST + '_FULFILLED':
+ return fetchFulfilled(newState, action);
- case ActionTypes.FETCH_PACKAGES_LIST + '_REJECTED':
- return fetchRejected(newState, action);
+ case ActionTypes.FETCH_PACKAGES_LIST + '_REJECTED':
+ return fetchRejected(newState, action);
- case ActionTypes.CHANGE_PACKAGES_LIST_PARAMS:
- return changeFilters(newState, action);
+ case ActionTypes.CHANGE_PACKAGES_LIST_PARAMS:
+ return changeFilters(newState, action);
- case ActionTypes.TRIGGER_GLOBAL_FILTER:
- return changeFilters(newState, action);
+ case ActionTypes.TRIGGER_GLOBAL_FILTER:
+ return changeFilters(newState, action);
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
diff --git a/src/store/Reducers/PatchSetDetailStore.js b/src/store/Reducers/PatchSetDetailStore.js
index 6bcebcb5d..c1e6809b3 100644
--- a/src/store/Reducers/PatchSetDetailStore.js
+++ b/src/store/Reducers/PatchSetDetailStore.js
@@ -2,32 +2,32 @@ import { FETCH_TEMPLATE_DETAIL, CLEAR_TEMPLATE_DETAIL } from '../ActionTypes';
import { fetchPending, fetchRejected } from './HelperReducers';
export let initialState = {
- data: { attributes: {} },
- status: { isLoading: true }
+ data: { attributes: {} },
+ status: { isLoading: true },
};
export const PatchSetDetailStore = (state = initialState, action) => {
- let newState = { ...state };
+ let newState = { ...state };
- switch (action.type) {
- case FETCH_TEMPLATE_DETAIL + '_FULFILLED':
- return {
- ...state,
- status: { isLoading: false },
- data: action.payload.data,
- error: {}
- };
+ switch (action.type) {
+ case FETCH_TEMPLATE_DETAIL + '_FULFILLED':
+ return {
+ ...state,
+ status: { isLoading: false },
+ data: action.payload.data,
+ error: {},
+ };
- case FETCH_TEMPLATE_DETAIL + '_PENDING':
- return fetchPending(newState);
+ case FETCH_TEMPLATE_DETAIL + '_PENDING':
+ return fetchPending(newState);
- case FETCH_TEMPLATE_DETAIL + '_REJECTED':
- return fetchRejected(newState, action);
+ case FETCH_TEMPLATE_DETAIL + '_REJECTED':
+ return fetchRejected(newState, action);
- case CLEAR_TEMPLATE_DETAIL:
- return initialState;
+ case CLEAR_TEMPLATE_DETAIL:
+ return initialState;
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
diff --git a/src/store/Reducers/PatchSetDetailSystemsStore.js b/src/store/Reducers/PatchSetDetailSystemsStore.js
index be0fc14b9..568b6b97c 100644
--- a/src/store/Reducers/PatchSetDetailSystemsStore.js
+++ b/src/store/Reducers/PatchSetDetailSystemsStore.js
@@ -1,61 +1,60 @@
import { changeListParams } from '../../Utilities/Helpers';
import {
- CHANGE_PATCH_SET_DETAIL_SYSTEMS_PARAMS,
- TRIGGER_GLOBAL_FILTER,
- CHANGE_PATCH_SET_DETAIL_SYSTEMS_METADATA,
- FETCH_PATCH_SET_SYSTEMS_NO_FILTERS
+ CHANGE_PATCH_SET_DETAIL_SYSTEMS_PARAMS,
+ TRIGGER_GLOBAL_FILTER,
+ CHANGE_PATCH_SET_DETAIL_SYSTEMS_METADATA,
+ FETCH_PATCH_SET_SYSTEMS_NO_FILTERS,
} from '../ActionTypes';
import { changeFilters } from './HelperReducers';
const initialState = {
- selectedRows: {},
- queryParams: {
- page: 1,
- perPage: 20
- },
- templateHasSystemsLoading: true,
- templateHasSystems: null
+ selectedRows: {},
+ queryParams: {
+ page: 1,
+ perPage: 20,
+ },
+ templateHasSystemsLoading: true,
+ templateHasSystems: null,
};
export const PatchSetDetailSystemsStore = (state = initialState, action) => {
- let newState = { ...state };
-
- switch (action.type) {
- case CHANGE_PATCH_SET_DETAIL_SYSTEMS_PARAMS:
- newState.queryParams = changeListParams(newState.queryParams, action.payload);
- return newState;
-
- case TRIGGER_GLOBAL_FILTER:
- return changeFilters(newState, action);
-
- case CHANGE_PATCH_SET_DETAIL_SYSTEMS_METADATA:
- newState.metadata = action.payload;
- return newState;
-
- case FETCH_PATCH_SET_SYSTEMS_NO_FILTERS + '_PENDING':
- return {
- ...state,
- templateHasSystemsLoading: true,
- templateHasSystems: null
- };
-
- case FETCH_PATCH_SET_SYSTEMS_NO_FILTERS + '_FULFILLED':
- return {
- ...state,
- templateHasSystemsLoading: false,
- templateHasSystems: action.payload.data.length > 0
- };
-
- case FETCH_PATCH_SET_SYSTEMS_NO_FILTERS + '_REJECTED':
- return {
- ...state,
- templateHasSystemsLoading: false,
- // if we set this to true, Inventory table is shown, which has better error handling
- templateHasSystems: true
- };
-
- default:
- return state;
- }
+ let newState = { ...state };
+
+ switch (action.type) {
+ case CHANGE_PATCH_SET_DETAIL_SYSTEMS_PARAMS:
+ newState.queryParams = changeListParams(newState.queryParams, action.payload);
+ return newState;
+
+ case TRIGGER_GLOBAL_FILTER:
+ return changeFilters(newState, action);
+
+ case CHANGE_PATCH_SET_DETAIL_SYSTEMS_METADATA:
+ newState.metadata = action.payload;
+ return newState;
+
+ case FETCH_PATCH_SET_SYSTEMS_NO_FILTERS + '_PENDING':
+ return {
+ ...state,
+ templateHasSystemsLoading: true,
+ templateHasSystems: null,
+ };
+
+ case FETCH_PATCH_SET_SYSTEMS_NO_FILTERS + '_FULFILLED':
+ return {
+ ...state,
+ templateHasSystemsLoading: false,
+ templateHasSystems: action.payload.data.length > 0,
+ };
+
+ case FETCH_PATCH_SET_SYSTEMS_NO_FILTERS + '_REJECTED':
+ return {
+ ...state,
+ templateHasSystemsLoading: false,
+ // if we set this to true, Inventory table is shown, which has better error handling
+ templateHasSystems: true,
+ };
+
+ default:
+ return state;
+ }
};
-
diff --git a/src/store/Reducers/PatchSetsReducer.js b/src/store/Reducers/PatchSetsReducer.js
index 89008597c..dec646bfc 100644
--- a/src/store/Reducers/PatchSetsReducer.js
+++ b/src/store/Reducers/PatchSetsReducer.js
@@ -1,59 +1,54 @@
import * as ActionTypes from '../ActionTypes';
-import {
- changeFilters,
- selectRows,
- fetchPending,
- fetchRejected
-} from './HelperReducers';
+import { changeFilters, selectRows, fetchPending, fetchRejected } from './HelperReducers';
// Initial State. It should not include page and perPage to persist them dynamically
export const initialState = {
- rows: [],
- selectedRows: [],
- queryParams: {
- page: 1,
- perPage: 20,
- offset: 0,
- filter: {}
- },
- status: { isLoading: true },
- metadata: {
- limit: 20,
- offset: 0,
- total_items: 0
- },
- error: {}
+ rows: [],
+ selectedRows: [],
+ queryParams: {
+ page: 1,
+ perPage: 20,
+ offset: 0,
+ filter: {},
+ },
+ status: { isLoading: true },
+ metadata: {
+ limit: 20,
+ offset: 0,
+ total_items: 0,
+ },
+ error: {},
};
export const PatchSetsReducer = (state = initialState, action) => {
- let newState = { ...state };
+ let newState = { ...state };
- switch (action.type) {
- case ActionTypes.FETCH_ALL_PATCH_SETS + '_FULFILLED':
- return {
- ...newState,
- rows: action.payload.data?.map(set => ({ ...set.attributes, id: set.id })),
- metadata: action.payload.meta || {},
- error: {},
- status: { code: action.payload.status, isLoading: false, hasError: false }
- };
+ switch (action.type) {
+ case ActionTypes.FETCH_ALL_PATCH_SETS + '_FULFILLED':
+ return {
+ ...newState,
+ rows: action.payload.data?.map((set) => ({ ...set.attributes, id: set.id })),
+ metadata: action.payload.meta || {},
+ error: {},
+ status: { code: action.payload.status, isLoading: false, hasError: false },
+ };
- case ActionTypes.FETCH_ALL_PATCH_SETS + '_PENDING':
- return fetchPending(newState);
+ case ActionTypes.FETCH_ALL_PATCH_SETS + '_PENDING':
+ return fetchPending(newState);
- case ActionTypes.FETCH_ALL_PATCH_SETS + '_REJECTED':
- return fetchRejected(newState, action);
+ case ActionTypes.FETCH_ALL_PATCH_SETS + '_REJECTED':
+ return fetchRejected(newState, action);
- case ActionTypes.CHANGE_PATCH_SET_PARAMS:
- return changeFilters(newState, action);
+ case ActionTypes.CHANGE_PATCH_SET_PARAMS:
+ return changeFilters(newState, action);
- case ActionTypes.SELECT_PATCH_SET_ROW:
- return selectRows(newState, action);
+ case ActionTypes.SELECT_PATCH_SET_ROW:
+ return selectRows(newState, action);
- case ActionTypes.CLEAR_PATCH_SETS:
- return initialState;
+ case ActionTypes.CLEAR_PATCH_SETS:
+ return initialState;
- default:
- return newState;
- }
+ default:
+ return newState;
+ }
};
diff --git a/src/store/Reducers/SpecificPatchSetReducer.js b/src/store/Reducers/SpecificPatchSetReducer.js
index d6bd990a6..414125c46 100644
--- a/src/store/Reducers/SpecificPatchSetReducer.js
+++ b/src/store/Reducers/SpecificPatchSetReducer.js
@@ -1,40 +1,40 @@
import * as ActionTypes from '../ActionTypes';
-import {
- fetchPending,
- fetchRejected
-} from './HelperReducers';
+import { fetchPending, fetchRejected } from './HelperReducers';
export const initialState = {
- patchSet: {
- config: {}
- },
- assignedSystems: [],
- status: {},
- error: {}
+ patchSet: {
+ config: {},
+ },
+ assignedSystems: [],
+ status: {},
+ error: {},
};
export const SpecificPatchSetReducer = (state = initialState, action) => {
- switch (action.type) {
- case ActionTypes.FETCH_PATCH_SET + '_FULFILLED': {
- const { attributes: { config, name, description }, id } = action.payload?.data || { config: {} };
+ switch (action.type) {
+ case ActionTypes.FETCH_PATCH_SET + '_FULFILLED': {
+ const {
+ attributes: { config, name, description },
+ id,
+ } = action.payload?.data || { config: {} };
- return {
- ...state,
- patchSet: { name, description, config: config || {}, id },
- status: { code: action.payload.status, isLoading: false, hasError: false }
- };
- }
+ return {
+ ...state,
+ patchSet: { name, description, config: config || {}, id },
+ status: { code: action.payload.status, isLoading: false, hasError: false },
+ };
+ }
- case ActionTypes.FETCH_PATCH_SET + '_PENDING':
- return fetchPending(state);
+ case ActionTypes.FETCH_PATCH_SET + '_PENDING':
+ return fetchPending(state);
- case ActionTypes.FETCH_PATCH_SET + '_REJECTED':
- return fetchRejected(state, action);
+ case ActionTypes.FETCH_PATCH_SET + '_REJECTED':
+ return fetchRejected(state, action);
- case ActionTypes.CLEAR_PATCH_SET:
- return initialState;
+ case ActionTypes.CLEAR_PATCH_SET:
+ return initialState;
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
diff --git a/src/store/Reducers/SystemAdvisoryListStore.js b/src/store/Reducers/SystemAdvisoryListStore.js
index 2401b4dd7..50b2cc936 100644
--- a/src/store/Reducers/SystemAdvisoryListStore.js
+++ b/src/store/Reducers/SystemAdvisoryListStore.js
@@ -1,42 +1,42 @@
import { storeListDefaults } from '../../Utilities/constants';
import * as ActionTypes from '../ActionTypes';
import {
- changeFilters,
- selectRows,
- fetchPending,
- fetchRejected,
- fetchFulfilled,
- expandRows
+ changeFilters,
+ selectRows,
+ fetchPending,
+ fetchRejected,
+ fetchFulfilled,
+ expandRows,
} from './HelperReducers';
export const SystemAdvisoryListStore = (state = storeListDefaults, action) => {
- let newState = { ...state };
- switch (action.type) {
- case ActionTypes.FETCH_APPLICABLE_SYSTEM_ADVISORIES + '_FULFILLED':
- return fetchFulfilled(newState, action);
+ let newState = { ...state };
+ switch (action.type) {
+ case ActionTypes.FETCH_APPLICABLE_SYSTEM_ADVISORIES + '_FULFILLED':
+ return fetchFulfilled(newState, action);
- case ActionTypes.FETCH_APPLICABLE_SYSTEM_ADVISORIES + '_PENDING':
- return fetchPending(newState);
+ case ActionTypes.FETCH_APPLICABLE_SYSTEM_ADVISORIES + '_PENDING':
+ return fetchPending(newState);
- case ActionTypes.FETCH_APPLICABLE_SYSTEM_ADVISORIES + '_REJECTED':
- return fetchRejected(newState, action);
+ case ActionTypes.FETCH_APPLICABLE_SYSTEM_ADVISORIES + '_REJECTED':
+ return fetchRejected(newState, action);
- case ActionTypes.CHANGE_SYSTEM_ADVISORY_LIST_PARAMS:
- return changeFilters(newState, action);
+ case ActionTypes.CHANGE_SYSTEM_ADVISORY_LIST_PARAMS:
+ return changeFilters(newState, action);
- case ActionTypes.EXPAND_SYSTEM_ADVISORY_ROW:
- return expandRows(newState, action);
+ case ActionTypes.EXPAND_SYSTEM_ADVISORY_ROW:
+ return expandRows(newState, action);
- case ActionTypes.SELECT_SYSTEM_ADVISORY_ROW:
- return selectRows(newState, action);
+ case ActionTypes.SELECT_SYSTEM_ADVISORY_ROW:
+ return selectRows(newState, action);
- case ActionTypes.CLEAR_SYSTEM_ADVISORIES:
- return storeListDefaults;
+ case ActionTypes.CLEAR_SYSTEM_ADVISORIES:
+ return storeListDefaults;
- case ActionTypes.TRIGGER_GLOBAL_FILTER:
- return changeFilters(newState, action);
+ case ActionTypes.TRIGGER_GLOBAL_FILTER:
+ return changeFilters(newState, action);
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
diff --git a/src/store/Reducers/SystemAdvisoryListStore.test.js b/src/store/Reducers/SystemAdvisoryListStore.test.js
index c982db9e9..c62d9088e 100644
--- a/src/store/Reducers/SystemAdvisoryListStore.test.js
+++ b/src/store/Reducers/SystemAdvisoryListStore.test.js
@@ -1,25 +1,35 @@
-
/* eslint-disable */
-import { STATUS_LOADING, STATUS_REJECTED, STATUS_RESOLVED, storeListDefaults } from '../../Utilities/constants';
-import { CHANGE_SYSTEM_ADVISORY_LIST_PARAMS, CLEAR_SYSTEM_ADVISORIES, EXPAND_SYSTEM_ADVISORY_ROW, FETCH_APPLICABLE_SYSTEM_ADVISORIES, SELECT_SYSTEM_ADVISORY_ROW } from '../ActionTypes';
+import {
+ STATUS_LOADING,
+ STATUS_REJECTED,
+ STATUS_RESOLVED,
+ storeListDefaults,
+} from '../../Utilities/constants';
+import {
+ CHANGE_SYSTEM_ADVISORY_LIST_PARAMS,
+ CLEAR_SYSTEM_ADVISORIES,
+ EXPAND_SYSTEM_ADVISORY_ROW,
+ FETCH_APPLICABLE_SYSTEM_ADVISORIES,
+ SELECT_SYSTEM_ADVISORY_ROW,
+} from '../ActionTypes';
import { SystemAdvisoryListStore } from './SystemAdvisoryListStore';
const action_fulfilled = FETCH_APPLICABLE_SYSTEM_ADVISORIES + '_FULFILLED';
const action_rejected = FETCH_APPLICABLE_SYSTEM_ADVISORIES + '_REJECTED';
const action_pending = FETCH_APPLICABLE_SYSTEM_ADVISORIES + '_PENDING';
const fulfilled_payload = {
- data: [],
- meta: {
- limit: 50,
- offset: 25,
- total_items: 50
- }
+ data: [],
+ meta: {
+ limit: 50,
+ offset: 25,
+ total_items: 50,
+ },
};
const error = 'Error';
describe('SystemAdvisoryListStore tests', () => {
- it.each`
+ it.each`
state | action | result
${storeListDefaults} | ${{ type: action_fulfilled, payload: fulfilled_payload }} | ${{ ...storeListDefaults, metadata: fulfilled_payload.meta, rows: fulfilled_payload.data, status: { code: undefined, isLoading: false, hasError: false }, error: {} }}
${storeListDefaults} | ${{ type: action_pending, payload: {} }} | ${{ ...storeListDefaults, status: { code: undefined, isLoading: true, hasError: false }, error: {} }}
@@ -28,11 +38,11 @@ describe('SystemAdvisoryListStore tests', () => {
${storeListDefaults} | ${{ type: SELECT_SYSTEM_ADVISORY_ROW, payload: { id: 1, selected: true } }} | ${{ ...storeListDefaults, selectedRows: { 1: true } }}
${storeListDefaults} | ${{ type: 'NONSENSE', payload: {} }} | ${storeListDefaults}
${undefined} | ${{ type: 'NONSENSE', payload: {} }} | ${storeListDefaults}
- ${storeListDefaults} | ${{type: CLEAR_SYSTEM_ADVISORIES, payload: {}}} | ${storeListDefaults}
- `('$action', ({ state, action: { type, payload }, result }) => {
+ ${storeListDefaults} | ${{ type: CLEAR_SYSTEM_ADVISORIES, payload: {} }} | ${storeListDefaults}
+ `('$action', ({ state, action: { type, payload }, result }) => {
const res = SystemAdvisoryListStore(state, { type, payload });
expect(res).toEqual(result);
-});
+ });
});
// ${ storeListDefaults } | ${ { type: action_rejected, payload: error } } | ${ { ...storeListDefaults, status: STATUS_REJECTED, error } }
diff --git a/src/store/Reducers/SystemDetailStore.js b/src/store/Reducers/SystemDetailStore.js
index 70a78a79e..e6a7cf2c8 100644
--- a/src/store/Reducers/SystemDetailStore.js
+++ b/src/store/Reducers/SystemDetailStore.js
@@ -1,36 +1,36 @@
let initialState = {
- loaded: false
+ loaded: false,
};
// Reducer
export const SystemDetailStore = (state = initialState, { type, payload }) => {
- switch (type) {
- case 'FETCH_SYSTEM_DETAIL_FULFILLED':
- return {
- ...state,
- hasThirdPartyRepo: payload.data?.attributes.third_party,
- satelliteManaged: payload.data?.attributes.satellite_managed,
- patchSetName: payload.data?.attributes.baseline_name,
- patchSetId: payload.data?.attributes.baseline_id,
- templateName: payload.data?.attributes.template_name,
- templateUUID: payload.data?.attributes.template_uuid
- };
- case 'LOAD_ENTITY_PENDING':
- return {
- ...state,
- loaded: false
- };
- case 'LOAD_ENTITY_FULFILLED':
- return {
- ...state,
- loaded: true
- };
- case 'LOAD_ENTITY_REJECTED':
- return {
- ...state,
- loaded: true
- };
- default:
- return state;
- }
+ switch (type) {
+ case 'FETCH_SYSTEM_DETAIL_FULFILLED':
+ return {
+ ...state,
+ hasThirdPartyRepo: payload.data?.attributes.third_party,
+ satelliteManaged: payload.data?.attributes.satellite_managed,
+ patchSetName: payload.data?.attributes.baseline_name,
+ patchSetId: payload.data?.attributes.baseline_id,
+ templateName: payload.data?.attributes.template_name,
+ templateUUID: payload.data?.attributes.template_uuid,
+ };
+ case 'LOAD_ENTITY_PENDING':
+ return {
+ ...state,
+ loaded: false,
+ };
+ case 'LOAD_ENTITY_FULFILLED':
+ return {
+ ...state,
+ loaded: true,
+ };
+ case 'LOAD_ENTITY_REJECTED':
+ return {
+ ...state,
+ loaded: true,
+ };
+ default:
+ return state;
+ }
};
diff --git a/src/store/Reducers/SystemDetailStore.test.js b/src/store/Reducers/SystemDetailStore.test.js
index dae148686..f8c4b225f 100644
--- a/src/store/Reducers/SystemDetailStore.test.js
+++ b/src/store/Reducers/SystemDetailStore.test.js
@@ -1,23 +1,19 @@
import { initialState, SystemDetailStore } from './SystemDetailStore';
-/* eslint-disable */
-
const stateAfterAction = {
- loaded: true
+ loaded: true,
};
describe('SystemDetailStore', () => {
- it.each`
- state | action | result
- ${initialState} | ${{ type: 'TEST_INVALID_ACTION', payload: {} }} | ${{ loaded: false }}
- ${undefined} | ${{ type: 'LOAD_ENTITY_FULFILLED', payload: {} }} | ${{ ...initialState, ...stateAfterAction }}
- ${{ ...initialState, test: 'testState' }} | ${{ type: 'LOAD_ENTITY_FULFILLED', payload: {} }} | ${{ ...initialState, ...stateAfterAction, test: 'testState' }}
- ${undefined} | ${{ type: 'LOAD_ENTITY_REJECTED', payload: {} }} | ${{ ...initialState, ...stateAfterAction, }}
- ${{ ...initialState, test: 'testState' }} | ${{ type: 'LOAD_ENTITY_REJECTED', payload: {} }} | ${{ ...initialState, ...stateAfterAction, test: 'testState' }}
- ${initialState} | ${{ type: 'LOAD_ENTITY_FULFILLED', payload: { data: { attributes: { third_party: true } } } }} | ${{ ...initialState, loaded: true }}
- `('$action', ({ state, action: { type, payload }, result }) => {
- const res = SystemDetailStore(state, { type, payload });
- expect(res).toEqual(result);
- });
+ it.each`
+ state | action | result
+ ${initialState} | ${{ type: 'TEST_INVALID_ACTION', payload: {} }} | ${{ loaded: false }}
+ ${undefined} | ${{ type: 'LOAD_ENTITY_FULFILLED', payload: {} }} | ${{ ...initialState, ...stateAfterAction }}
+ ${{ ...initialState, test: 'testState' }} | ${{ type: 'LOAD_ENTITY_FULFILLED', payload: {} }} | ${{ ...initialState, ...stateAfterAction, test: 'testState' }}
+ ${undefined} | ${{ type: 'LOAD_ENTITY_REJECTED', payload: {} }} | ${{ ...initialState, ...stateAfterAction }}
+ ${{ ...initialState, test: 'testState' }} | ${{ type: 'LOAD_ENTITY_REJECTED', payload: {} }} | ${{ ...initialState, ...stateAfterAction, test: 'testState' }}
+ ${initialState} | ${{ type: 'LOAD_ENTITY_FULFILLED', payload: { data: { attributes: { third_party: true } } } }} | ${{ ...initialState, loaded: true }}
+ `('$action', ({ state, action: { type, payload }, result }) => {
+ const res = SystemDetailStore(state, { type, payload });
+ expect(res).toEqual(result);
+ });
});
-
-/* eslint-enable */
diff --git a/src/store/Reducers/SystemPackageListStore.js b/src/store/Reducers/SystemPackageListStore.js
index 760a75285..03ee34e3e 100644
--- a/src/store/Reducers/SystemPackageListStore.js
+++ b/src/store/Reducers/SystemPackageListStore.js
@@ -1,34 +1,43 @@
import { storeListDefaults, systemPackagesDefaultFilters } from '../../Utilities/constants';
import * as ActionTypes from '../ActionTypes';
-import { changeFilters, fetchFulfilled, fetchPending, fetchRejected, selectRows } from './HelperReducers';
+import {
+ changeFilters,
+ fetchFulfilled,
+ fetchPending,
+ fetchRejected,
+ selectRows,
+} from './HelperReducers';
let initializeState = { queryParams: systemPackagesDefaultFilters };
-export const SystemPackageListStore = (state = { ...storeListDefaults, ...initializeState }, action) => {
- let newState = { ...state };
- switch (action.type) {
- case ActionTypes.FETCH_APPLICABLE_SYSTEM_PACKAGES + '_FULFILLED':
- return fetchFulfilled(newState, action);
+export const SystemPackageListStore = (
+ state = { ...storeListDefaults, ...initializeState },
+ action,
+) => {
+ let newState = { ...state };
+ switch (action.type) {
+ case ActionTypes.FETCH_APPLICABLE_SYSTEM_PACKAGES + '_FULFILLED':
+ return fetchFulfilled(newState, action);
- case ActionTypes.FETCH_APPLICABLE_SYSTEM_PACKAGES + '_PENDING':
- return fetchPending(newState);
+ case ActionTypes.FETCH_APPLICABLE_SYSTEM_PACKAGES + '_PENDING':
+ return fetchPending(newState);
- case ActionTypes.FETCH_APPLICABLE_SYSTEM_PACKAGES + '_REJECTED':
- return fetchRejected(newState, action);
+ case ActionTypes.FETCH_APPLICABLE_SYSTEM_PACKAGES + '_REJECTED':
+ return fetchRejected(newState, action);
- case ActionTypes.CHANGE_SYSTEM_PACKAGES_LIST_PARAMS:
- return changeFilters(newState, action);
+ case ActionTypes.CHANGE_SYSTEM_PACKAGES_LIST_PARAMS:
+ return changeFilters(newState, action);
- case ActionTypes.SELECT_SYSTEM_PACKAGES_ROW:
- return selectRows(newState, action);
+ case ActionTypes.SELECT_SYSTEM_PACKAGES_ROW:
+ return selectRows(newState, action);
- case ActionTypes.CLEAR_SYSTEM_PACKAGES:
- return { ...storeListDefaults, ...initializeState };
+ case ActionTypes.CLEAR_SYSTEM_PACKAGES:
+ return { ...storeListDefaults, ...initializeState };
- case ActionTypes.TRIGGER_GLOBAL_FILTER:
- return changeFilters(newState, action);
+ case ActionTypes.TRIGGER_GLOBAL_FILTER:
+ return changeFilters(newState, action);
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
diff --git a/src/store/Reducers/SystemPackageListStore.test.js b/src/store/Reducers/SystemPackageListStore.test.js
index bd8631b42..d0a9f0b97 100644
--- a/src/store/Reducers/SystemPackageListStore.test.js
+++ b/src/store/Reducers/SystemPackageListStore.test.js
@@ -1,42 +1,42 @@
-
-/* eslint-disable */
import { storeListDefaults } from '../../Utilities/constants';
-import { CHANGE_SYSTEM_PACKAGES_LIST_PARAMS, SELECT_SYSTEM_PACKAGES_ROW,
- FETCH_APPLICABLE_SYSTEM_PACKAGES, TRIGGER_GLOBAL_FILTER, CLEAR_SYSTEM_PACKAGES } from '../ActionTypes';
+import {
+ CHANGE_SYSTEM_PACKAGES_LIST_PARAMS,
+ SELECT_SYSTEM_PACKAGES_ROW,
+ FETCH_APPLICABLE_SYSTEM_PACKAGES,
+ TRIGGER_GLOBAL_FILTER,
+ CLEAR_SYSTEM_PACKAGES,
+} from '../ActionTypes';
import { SystemPackageListStore } from './SystemPackageListStore';
const action_fulfilled = FETCH_APPLICABLE_SYSTEM_PACKAGES + '_FULFILLED';
const action_rejected = FETCH_APPLICABLE_SYSTEM_PACKAGES + '_REJECTED';
const action_pending = FETCH_APPLICABLE_SYSTEM_PACKAGES + '_PENDING';
const fulfilled_payload = {
- data: [],
- meta: {
- limit: 50,
- offset: 25,
- total_items: 50
- }
+ data: [],
+ meta: {
+ limit: 50,
+ offset: 25,
+ total_items: 50,
+ },
};
const error = 'Error';
describe('SystemPackageListStore tests', () => {
- it.each`
- state | action | result
- ${storeListDefaults} | ${{ type: action_fulfilled, payload: fulfilled_payload }} | ${{ ...storeListDefaults, metadata: fulfilled_payload.meta, rows: fulfilled_payload.data, status: { code: undefined, isLoading: false, hasError: false }, error: {} }}
- ${storeListDefaults} | ${{ type: action_pending, payload: {} }} | ${{ ...storeListDefaults, status: { code: undefined, isLoading: true, hasError: false }, error: {} }}
- ${storeListDefaults} | ${{ type: CLEAR_SYSTEM_PACKAGES, payload: {} }} | ${{ ...storeListDefaults, status: { isLoading: true }, error: {}, queryParams: { filter: { update_status: ['Installable']} } }}
- ${ storeListDefaults} | ${{ type: action_rejected, payload: error }} | ${{ ...storeListDefaults, status: { isLoading: false, hasError: true, code: undefined }, error, metadata: {} }}
- ${storeListDefaults} | ${{ type: action_rejected, payload: error }} | ${{ ...storeListDefaults, status: { isLoading: false, hasError: true, code: undefined }, error, metadata: {} } }
-
+ it.each`
+ state | action | result
+ ${storeListDefaults} | ${{ type: action_fulfilled, payload: fulfilled_payload }} | ${{ ...storeListDefaults, metadata: fulfilled_payload.meta, rows: fulfilled_payload.data, status: { code: undefined, isLoading: false, hasError: false }, error: {} }}
+ ${storeListDefaults} | ${{ type: action_pending, payload: {} }} | ${{ ...storeListDefaults, status: { code: undefined, isLoading: true, hasError: false }, error: {} }}
+ ${storeListDefaults} | ${{ type: CLEAR_SYSTEM_PACKAGES, payload: {} }} | ${{ ...storeListDefaults, status: { isLoading: true }, error: {}, queryParams: { filter: { update_status: ['Installable'] } } }}
+ ${storeListDefaults} | ${{ type: action_rejected, payload: error }} | ${{ ...storeListDefaults, status: { isLoading: false, hasError: true, code: undefined }, error, metadata: {} }}
+ ${storeListDefaults} | ${{ type: action_rejected, payload: error }} | ${{ ...storeListDefaults, status: { isLoading: false, hasError: true, code: undefined }, error, metadata: {} }}
${storeListDefaults} | ${{ type: CHANGE_SYSTEM_PACKAGES_LIST_PARAMS, payload: { limit: 10 } }} | ${{ ...storeListDefaults, queryParams: { limit: 10, offset: 0, page: 1, page_size: 20 } }}
- ${storeListDefaults} | ${{ type: TRIGGER_GLOBAL_FILTER, payload: { limit: 10 } }} | ${{ ...storeListDefaults, queryParams: { limit: 10, offset: 0, page: 1, page_size: 20 } }}
+ ${storeListDefaults} | ${{ type: TRIGGER_GLOBAL_FILTER, payload: { limit: 10 } }} | ${{ ...storeListDefaults, queryParams: { limit: 10, offset: 0, page: 1, page_size: 20 } }}
${storeListDefaults} | ${{ type: SELECT_SYSTEM_PACKAGES_ROW, payload: { id: 1, selected: true } }} | ${{ ...storeListDefaults, selectedRows: { 1: true } }}
- ${storeListDefaults} | ${{ type: 'NONSENSE', payload: {} }} | ${storeListDefaults}
- ${undefined} | ${{ type: 'NONSENSE', payload: {} }} | ${{ ...storeListDefaults, queryParams: { filter: { update_status: ['Installable'] } } }}
- `('$action', ({ state, action: { type, payload }, result }) => {
- const res = SystemPackageListStore(state, { type, payload });
- expect(res).toEqual(result);
- });
+ ${storeListDefaults} | ${{ type: 'NONSENSE', payload: {} }} | ${storeListDefaults}
+ ${undefined} | ${{ type: 'NONSENSE', payload: {} }} | ${{ ...storeListDefaults, queryParams: { filter: { update_status: ['Installable'] } } }}
+ `('$action', ({ state, action: { type, payload }, result }) => {
+ const res = SystemPackageListStore(state, { type, payload });
+ expect(res).toEqual(result);
+ });
});
-
-/* eslint-enable */
diff --git a/src/store/Reducers/SystemsStore.js b/src/store/Reducers/SystemsStore.js
index 99d7fdf5f..412ed3c41 100644
--- a/src/store/Reducers/SystemsStore.js
+++ b/src/store/Reducers/SystemsStore.js
@@ -4,38 +4,37 @@ import { changeListParams } from '../../Utilities/Helpers';
// Initial State. It should not include page and perPage to persist them dynamically
const initialState = {
- selectedRows: {},
- queryParams: {
- page: 1,
- perPage: 20,
- filter: {
- stale: [true, false]
- }
- }
+ selectedRows: {},
+ queryParams: {
+ page: 1,
+ perPage: 20,
+ filter: {
+ stale: [true, false],
+ },
+ },
};
// Reducer
export const SystemsStore = (state = initialState, action) => {
- let newState = { ...state };
+ let newState = { ...state };
- switch (action.type) {
- case ActionTypes.CHANGE_SYSTEMS_PARAMS:
- newState.queryParams = changeListParams(newState.queryParams, action.payload);
- return newState;
+ switch (action.type) {
+ case ActionTypes.CHANGE_SYSTEMS_PARAMS:
+ newState.queryParams = changeListParams(newState.queryParams, action.payload);
+ return newState;
- case ActionTypes.TRIGGER_GLOBAL_FILTER:
- return changeFilters(newState, action);
+ case ActionTypes.TRIGGER_GLOBAL_FILTER:
+ return changeFilters(newState, action);
- case ActionTypes.CHANGE_SYSTEMS_METADATA:
- newState.metadata = action.payload;
- return newState;
+ case ActionTypes.CHANGE_SYSTEMS_METADATA:
+ newState.metadata = action.payload;
+ return newState;
- case ActionTypes.TOGGLE_ALL_SELECTED:
- newState.areAllSelected = action.payload;
- return newState;
+ case ActionTypes.TOGGLE_ALL_SELECTED:
+ newState.areAllSelected = action.payload;
+ return newState;
- default:
- return state;
- }
+ default:
+ return state;
+ }
};
-
diff --git a/src/store/index.js b/src/store/index.js
index 25b8069ae..89044cb5e 100644
--- a/src/store/index.js
+++ b/src/store/index.js
@@ -17,31 +17,36 @@ import { PatchSetsReducer } from './Reducers/PatchSetsReducer';
import { PatchSetDetailStore } from './Reducers/PatchSetDetailStore';
import { PatchSetDetailSystemsStore } from './Reducers/PatchSetDetailSystemsStore';
import { SpecificPatchSetReducer } from './Reducers/SpecificPatchSetReducer';
-import { legacy_createStore as createStore, applyMiddleware, combineReducers, compose } from 'redux';
+import {
+ legacy_createStore as createStore,
+ applyMiddleware,
+ combineReducers,
+ compose,
+} from 'redux';
const composeEnhancers = window.__REDUX_DEVTOOLS_EXTENSION_COMPOSE__ || compose;
export const defaultReducers = {
- AdvisoryListStore,
- SystemDetailStore,
- SystemAdvisoryListStore,
- AdvisoryDetailStore,
- SystemPackageListStore,
- PackagesListStore,
- PackageDetailStore,
- CvesListStore,
- SystemsStore,
- PackageSystemsStore,
- AdvisorySystemsStore,
- GlobalFilterStore,
- PatchSetsStore: PatchSetsReducer,
- PatchSetDetailStore,
- PatchSetDetailSystemsStore,
- SpecificPatchSetReducer,
- notifications: notificationsReducer
+ AdvisoryListStore,
+ SystemDetailStore,
+ SystemAdvisoryListStore,
+ AdvisoryDetailStore,
+ SystemPackageListStore,
+ PackagesListStore,
+ PackageDetailStore,
+ CvesListStore,
+ SystemsStore,
+ PackageSystemsStore,
+ AdvisorySystemsStore,
+ GlobalFilterStore,
+ PatchSetsStore: PatchSetsReducer,
+ PatchSetDetailStore,
+ PatchSetDetailSystemsStore,
+ SpecificPatchSetReducer,
+ notifications: notificationsReducer,
};
export const store = createStore(
- combineReducers(defaultReducers),
- composeEnhancers(applyMiddleware(promiseMiddleware, notificationsMiddleware()))
+ combineReducers(defaultReducers),
+ composeEnhancers(applyMiddleware(promiseMiddleware, notificationsMiddleware())),
);