Replaced usages of test with it, and updated changelog

CHANGELOG.md

@@ -20,6 +20,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 ### Changed
 * [#408](https://github.com/shlinkio/shlink-web-client/issues/408) Updated to Chart.js 3.5
 * [#486](https://github.com/shlinkio/shlink-web-client/issues/486) Refactored components used to render visits charts, making them easier to maintain and understand.
+* [#409](https://github.com/shlinkio/shlink-web-client/issues/409) Increased required code coverage and added hard threshold check.
 
 ### Deprecated
 * *Nothing*

test/app/reducers/appUpdates.test.ts

@@ -17,13 +17,13 @@ describe('appUpdatesReducer', () => {
   });
 
   describe('appUpdateAvailable', () => {
-    test('creates expected action', () => {
+    it('creates expected action', () => {
       expect(appUpdateAvailable()).toEqual({ type: APP_UPDATE_AVAILABLE });
     });
   });
 
   describe('resetAppUpdate', () => {
-    test('creates expected action', () => {
+    it('creates expected action', () => {
       expect(resetAppUpdate()).toEqual({ type: RESET_APP_UPDATE });
     });
   });

test/common/MainHeader.test.tsx

@@ -21,24 +21,24 @@ describe('<MainHeader />', () => {
 
   afterEach(() => wrapper?.unmount());
 
-  test('ServersDropdown is rendered', () => {
+  it('renders ServersDropdown', () => {
     const wrapper = createWrapper();
 
     expect(wrapper.find(ServersDropdown)).toHaveLength(1);
   });
 
-  test.each([
+  it.each([
     [ '/foo', false ],
     [ '/bar', false ],
     [ '/settings', true ],
-  ])('link to settings is only active when current path is settings', (currentPath, isActive) => {
+  ])('sets link to settings as active only when current path is settings', (currentPath, isActive) => {
     const wrapper = createWrapper(currentPath);
     const settingsLink = wrapper.find(NavLink);
 
     expect(settingsLink.prop('active')).toEqual(isActive);
   });
 
-  test('expected class is rendered based on the nav bar state', () => {
+  it('renders expected class based on the nav bar state', () => {
     const wrapper = createWrapper();
 
     expect(wrapper.find(NavbarToggler).find(FontAwesomeIcon).prop('className')).toEqual('main-header__toggle-icon');
@@ -50,7 +50,7 @@ describe('<MainHeader />', () => {
     expect(wrapper.find(NavbarToggler).find(FontAwesomeIcon).prop('className')).toEqual('main-header__toggle-icon');
   });
 
-  test('opens Collapse when clicking toggle', () => {
+  it('opens Collapse when clicking toggle', () => {
     const wrapper = createWrapper();
 
     expect(wrapper.find(Collapse).prop('isOpen')).toEqual(false);

test/common/ShlinkVersionsContainer.test.tsx

@@ -14,7 +14,7 @@ describe('<ShlinkVersionsContainer />', () => {
 
   afterEach(() => wrapper?.unmount());
 
-  test.each([
+  it.each([
     [ null, 'text-center' ],
     [ Mock.of<NotFoundServer>({ serverNotFound: true }), 'text-center' ],
     [ Mock.of<NonReachableServer>({ serverNotReachable: true }), 'text-center' ],

test/common/services/ImageDownloader.test.ts

@@ -15,7 +15,7 @@ describe('ImageDownloader', () => {
     imageDownloader = new ImageDownloader(axios, windowMock);
   });
 
-  test('calls URL with response type blob', async () => {
+  it('calls URL with response type blob', async () => {
     get.mockResolvedValue({ data: {} });
 
     await imageDownloader.saveImage('/foo/bar.png', 'my-image.png');

test/settings/RealTimeUpdates.test.tsx

@@ -26,7 +26,7 @@ describe('<RealTimeUpdates />', () => {
   afterEach(jest.clearAllMocks);
   afterEach(() => wrapper?.unmount());
 
-  test('enabled real time updates are rendered as expected', () => {
+  it('renders enabled real time updates as expected', () => {
     const wrapper = createWrapper({ enabled: true });
     const toggle = wrapper.find(ToggleSwitch);
     const label = wrapper.find('label');
@@ -41,7 +41,7 @@ describe('<RealTimeUpdates />', () => {
     expect(small).toHaveLength(2);
   });
 
-  test('disabled real time updates are rendered as expected', () => {
+  it('renders disabled real time updates as expected', () => {
     const wrapper = createWrapper({ enabled: false });
     const toggle = wrapper.find(ToggleSwitch);
     const label = wrapper.find('label');
@@ -56,12 +56,12 @@ describe('<RealTimeUpdates />', () => {
     expect(small).toHaveLength(1);
   });
 
-  test.each([
+  it.each([
     [ 1, 'minute' ],
     [ 2, 'minutes' ],
     [ 10, 'minutes' ],
     [ 100, 'minutes' ],
-  ])('expected children are shown when interval is greater than 0', (interval, minutesWord) => {
+  ])('shows expected children when interval is greater than 0', (interval, minutesWord) => {
     const wrapper = createWrapper({ enabled: true, interval });
     const span = wrapper.find('span');
     const input = wrapper.find(Input);
@@ -73,7 +73,7 @@ describe('<RealTimeUpdates />', () => {
     expect(input.prop('value')).toEqual(`${interval}`);
   });
 
-  test.each([[ undefined ], [ 0 ]])('expected children are shown when interval is 0 or undefined', (interval) => {
+  it.each([[ undefined ], [ 0 ]])('shows expected children when interval is 0 or undefined', (interval) => {
     const wrapper = createWrapper({ enabled: true, interval });
     const span = wrapper.find('span');
     const small = wrapper.find('small').at(1);
@@ -84,7 +84,7 @@ describe('<RealTimeUpdates />', () => {
     expect(input.prop('value')).toEqual('');
   });
 
-  test('real time updates are updated on input change', () => {
+  it('updates real time updates on input change', () => {
     const wrapper = createWrapper();
     const input = wrapper.find(Input);
 
@@ -93,7 +93,7 @@ describe('<RealTimeUpdates />', () => {
     expect(setRealTimeUpdatesInterval).toHaveBeenCalledWith(10);
   });
 
-  test('real time updates are toggled on switch change', () => {
+  it('toggles real time updates on switch change', () => {
     const wrapper = createWrapper();
     const toggle = wrapper.find(ToggleSwitch);
 

test/settings/Settings.test.tsx

@@ -6,7 +6,7 @@ describe('<Settings />', () => {
   const Component = () => null;
   const Settings = createSettings(Component, Component, Component, Component);
 
-  test('a no-menu layout is renders with the four settings sections', () => {
+  it('renders a no-menu layout with the expected settings sections', () => {
     const wrapper = shallow(<Settings />);
     const layout = wrapper.find(NoMenuLayout);
     const sections = wrapper.find('SettingsSections');

test/short-urls/helpers/ShortUrlFormCheckboxGroup.test.tsx

@@ -4,7 +4,7 @@ import Checkbox from '../../../src/utils/Checkbox';
 import { InfoTooltip } from '../../../src/utils/InfoTooltip';
 
 describe('<ShortUrlFormCheckboxGroup />', () => {
-  test.each([
+  it.each([
     [ undefined, '', 0 ],
     [ 'This is the tooltip', 'mr-2', 1 ],
   ])('renders tooltip only when provided', (infoTooltip, expectedClassName, expectedAmountOfTooltips) => {

test/tags/helpers/EditTagModal.test.tsx

@@ -28,7 +28,7 @@ describe('<EditTagModal />', () => {
   afterEach(jest.clearAllMocks);
   afterEach(() => wrapper?.unmount());
 
-  test('modal can be toggled with different mechanisms', () => {
+  it('allows modal to be toggled with different mechanisms', () => {
     const wrapper = createWrapper();
     const modal = wrapper.find(Modal);
     const modalHeader = wrapper.find(ModalHeader);
@@ -45,10 +45,10 @@ describe('<EditTagModal />', () => {
     expect(tagEdited).not.toHaveBeenCalled();
   });
 
-  test.each([
+  it.each([
     [ true, 'Saving...' ],
     [ false, 'Save' ],
-  ])('submit button is rendered in expected state', (editing, expectedText) => {
+  ])('renders submit button in expected state', (editing, expectedText) => {
     const wrapper = createWrapper({ editing });
     const submitBtn = wrapper.find(Button).findWhere((btn) => btn.prop('color') === 'primary');
 
@@ -56,10 +56,10 @@ describe('<EditTagModal />', () => {
     expect(submitBtn.prop('disabled')).toEqual(editing);
   });
 
-  test.each([
+  it.each([
     [ true, 1 ],
     [ false, 0 ],
-  ])('error result is displayed in case of error', (error, expectedResultCount) => {
+  ])('displays error result in case of error', (error, expectedResultCount) => {
     const wrapper = createWrapper({ error, errorData: Mock.all<ProblemDetailsError>() });
     const result = wrapper.find(Result);
     const apiError = wrapper.find(ShlinkApiError);
@@ -68,7 +68,7 @@ describe('<EditTagModal />', () => {
     expect(apiError).toHaveLength(expectedResultCount);
   });
 
-  test('tag value is updated when text changes', () => {
+  it('updates tag value when text changes', () => {
     const wrapper = createWrapper();
 
     expect(wrapper.find(Input).prop('value')).toEqual('foo');
@@ -76,7 +76,7 @@ describe('<EditTagModal />', () => {
     expect(wrapper.find(Input).prop('value')).toEqual('bar');
   });
 
-  test('all functions are invoked on form submit', async () => {
+  it('invokes all functions on form submit', async () => {
     const wrapper = createWrapper();
     const form = wrapper.find('form');
 
@@ -89,7 +89,7 @@ describe('<EditTagModal />', () => {
     expect(tagEdited).toHaveBeenCalled();
   });
 
-  test('color is changed when changing on color picker', () => {
+  it('changes color when changing on color picker', () => {
     const wrapper = createWrapper();
 
     expect(wrapper.find(ChromePicker).prop('color')).toEqual('red');
@@ -97,7 +97,7 @@ describe('<EditTagModal />', () => {
     expect(wrapper.find(ChromePicker).prop('color')).toEqual('blue');
   });
 
-  test('popover can be toggled with different mechanisms', () => {
+  it('allows toggling popover with different mechanisms', () => {
     const wrapper = createWrapper();
 
     expect(wrapper.find(Popover).prop('isOpen')).toEqual(false);

test/utils/PaginationDropdown.test.tsx

@@ -13,19 +13,19 @@ describe('<PaginationDropdown />', () => {
   afterEach(jest.clearAllMocks);
   afterEach(() => wrapper?.unmount());
 
-  test('expected amount of items is rendered', () => {
+  it('renders expected amount of items', () => {
     const items = wrapper.find(DropdownItem);
 
     expect(items).toHaveLength(6);
   });
 
-  test.each([
+  it.each([
     [ 0, 10 ],
     [ 1, 50 ],
     [ 2, 100 ],
     [ 3, 200 ],
     [ 5, Infinity ],
-  ])('expected value is set when an item is clicked', (index, expectedValue) => {
+  ])('sets expected value when an item is clicked', (index, expectedValue) => {
     const item = wrapper.find(DropdownItem).at(index);
 
     expect(setValue).not.toHaveBeenCalled();

test/utils/dates/types/index.test.ts

@@ -10,7 +10,7 @@ import { parseDate } from '../../../../src/utils/helpers/date';
 
 describe('date-types', () => {
   describe('dateRangeIsEmpty', () => {
-    test.each([
+    it.each([
       [ undefined, true ],
       [{}, true ],
       [{ startDate: null }, true ],
@@ -24,24 +24,24 @@ describe('date-types', () => {
       [{ startDate: new Date() }, false ],
       [{ endDate: new Date() }, false ],
       [{ startDate: new Date(), endDate: new Date() }, false ],
-    ])('proper result is returned', (dateRange, expectedResult) => {
+    ])('returns proper result', (dateRange, expectedResult) => {
       expect(dateRangeIsEmpty(dateRange)).toEqual(expectedResult);
     });
   });
 
   describe('rangeIsInterval', () => {
-    test.each([
+    it.each([
       [ undefined, false ],
       [{}, false ],
       [ 'today' as DateInterval, true ],
       [ 'yesterday' as DateInterval, true ],
-    ])('proper result is returned', (range, expectedResult) => {
+    ])('returns proper result', (range, expectedResult) => {
       expect(rangeIsInterval(range)).toEqual(expectedResult);
     });
   });
 
   describe('rangeOrIntervalToString', () => {
-    test.each([
+    it.each([
       [ undefined, undefined ],
       [ 'today' as DateInterval, 'Today' ],
       [ 'yesterday' as DateInterval, 'Yesterday' ],
@@ -65,7 +65,7 @@ describe('date-types', () => {
         { startDate: parseDate('2020-01-01', 'yyyy-MM-dd'), endDate: parseDate('2021-02-02', 'yyyy-MM-dd') },
         '2020-01-01 - 2021-02-02',
       ],
-    ])('proper result is returned', (range, expectedValue) => {
+    ])('returns proper result', (range, expectedValue) => {
       expect(rangeOrIntervalToString(range)).toEqual(expectedValue);
     });
   });
@@ -75,7 +75,7 @@ describe('date-types', () => {
     const daysBack = (days: number) => subDays(new Date(), days);
     const formatted = (date?: Date | null): string | undefined => !date ? undefined : format(date, 'yyyy-MM-dd');
 
-    test.each([
+    it.each([
       [ undefined, undefined, undefined ],
       [ 'today' as DateInterval, now(), now() ],
       [ 'yesterday' as DateInterval, daysBack(1), daysBack(1) ],
@@ -84,7 +84,7 @@ describe('date-types', () => {
       [ 'last90Days' as DateInterval, daysBack(90), now() ],
       [ 'last180days' as DateInterval, daysBack(180), now() ],
       [ 'last365Days' as DateInterval, daysBack(365), now() ],
-    ])('proper result is returned', (interval, expectedStartDate, expectedEndDate) => {
+    ])('returns proper result', (interval, expectedStartDate, expectedEndDate) => {
       const { startDate, endDate } = intervalToDateRange(interval);
 
       expect(formatted(expectedStartDate)).toEqual(formatted(startDate));

test/utils/helpers/qrCodes.test.ts

@@ -2,7 +2,7 @@ import { buildQrCodeUrl, QrCodeFormat, QrErrorCorrection } from '../../../src/ut
 
 describe('qrCodes', () => {
   describe('buildQrCodeUrl', () => {
-    test.each([
+    it.each([
       [
         'foo.com',
         { size: 530, format: 'svg' as QrCodeFormat, margin: 0, errorCorrection: 'L' as QrErrorCorrection },

test/visits/charts/DoughnutChartLegend.test.tsx

@@ -15,7 +15,7 @@ describe('<DoughnutChartLegend />', () => {
     },
   });
 
-  test('renders the expected amount of items with expected colors and labels', () => {
+  it('renders the expected amount of items with expected colors and labels', () => {
     const wrapper = shallow(<DoughnutChartLegend chart={chart} />);
     const items = wrapper.find('li');
 

test/visits/types/helpers.test.ts

@@ -60,7 +60,7 @@ describe('visitsTypeHelpers', () => {
   });
 
   describe('toApiParams', () => {
-    test.each([
+    it.each([
       [ { page: 5, itemsPerPage: 100 } as VisitsParams, { page: 5, itemsPerPage: 100 } as ShlinkVisitsParams ],
       [
         {