Skip to content

Commit ac09df8

Browse files
danielmx-devdanielmx-dev
authored
Dev - Upgrade to Node v20 (#9121)
1 parent 1927b0b commit ac09df8

File tree

91 files changed

+30288
-56544
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

91 files changed

+30288
-56544
lines changed

‎.nvmrc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
16.17
1+
v20

‎.puppeteerrc.cjs

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
const {join} = require('path');
2+
3+
/**
4+
* @type {import("puppeteer").Configuration}
5+
*/
6+
module.exports = {
7+
// Changes the cache location for Puppeteer.
8+
cacheDirectory: join(__dirname, 'node_modules', '.cache', 'puppeteer'),
9+
};

‎changelog/dev-upgrade-support-to-node-20

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
Significance: minor
2+
Type: dev
3+
4+
Update node to v20

‎client/additional-methods-setup/upe-preview-methods-selector/test/add-payment-methods-task.test.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -270,10 +270,13 @@ describe( 'AddPaymentMethodsTask', () => {
270270

271271
userEvent.click( screen.getByText( 'Continue' ) );
272272

273+
await jest.runAllTimersAsync();
274+
273275
expect( updateEnabledPaymentMethodsMock ).toHaveBeenCalledWith( [
274276
'card',
275277
'p24',
276278
] );
279+
277280
await waitFor( () =>
278281
expect( setCompletedMock ).toHaveBeenCalledWith(
279282
{ initialMethods: [ 'card' ] },
@@ -335,6 +338,8 @@ describe( 'AddPaymentMethodsTask', () => {
335338

336339
userEvent.click( screen.getByText( 'Continue' ) );
337340

341+
await jest.runAllTimersAsync();
342+
338343
// Methods are removed.
339344
expect( updateEnabledPaymentMethodsMock ).toHaveBeenCalledWith( [
340345
'card',

‎client/components/banner-notice/tests/index.test.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44
import React from 'react';
55
import { render, screen } from '@testing-library/react';
66
import user from '@testing-library/user-event';
7-
import { mocked } from 'ts-jest/utils';
87
import { speak } from '@wordpress/a11y';
98

109
/**
@@ -16,7 +15,7 @@ jest.mock( '@wordpress/a11y', () => ( { speak: jest.fn() } ) );
1615

1716
describe( 'BannerNotice', () => {
1817
beforeEach( () => {
19-
mocked( speak ).mockClear();
18+
jest.mocked( speak ).mockClear();
2019
} );
2120

2221
it( 'should match snapshot', () => {

‎client/components/loadable-checkbox/test/__snapshots__/index.test.js.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
// Jest Snapshot v1, https://goo.gl/fbAQLP
22

33
exports[`Loadable checkbox renders correctly 1`] = `
4-
Object {
4+
{
55
"asFragment": [Function],
66
"baseElement": <body>
77
<p

‎client/components/loadable-checkbox/test/index.test.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -100,13 +100,13 @@ describe( 'Loadable checkbox', () => {
100100
jest.useFakeTimers();
101101
act( () => {
102102
userEvent.click( container.queryByLabelText( 'Foo' ) );
103-
jest.runTimersToTime( 400 );
103+
jest.advanceTimersByTime( 400 );
104104
} );
105105
container.rerender( getLoadableCheckbox( false, 1500, 0 ) );
106106
expect( mockOnChangeEvent ).not.toHaveBeenCalled();
107107

108108
act( () => {
109-
jest.runTimersToTime( 1200 );
109+
jest.advanceTimersByTime( 1200 );
110110
} );
111111
container.rerender( getLoadableCheckbox( false, 1500, 0 ) );
112112
expect( mockOnChangeEvent ).toHaveBeenCalled();
@@ -122,12 +122,12 @@ describe( 'Loadable checkbox', () => {
122122
jest.useFakeTimers();
123123
act( () => {
124124
userEvent.click( container.queryByLabelText( 'Foo' ) );
125-
jest.runTimersToTime( 400 );
125+
jest.advanceTimersByTime( 400 );
126126
} );
127127
container.rerender( getLoadableCheckbox( false, 0, 1500 ) );
128128
expect( mockOnChangeEvent ).not.toHaveBeenCalled();
129129
act( () => {
130-
jest.runTimersToTime( 1200 );
130+
jest.advanceTimersByTime( 1200 );
131131
} );
132132
container.rerender( getLoadableCheckbox( false, 0, 1500 ) );
133133
expect( mockOnChangeEvent ).toHaveBeenCalledWith( false );

‎client/components/tooltip/test/index.test.tsx

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -193,6 +193,9 @@ describe( 'ClickTooltip', () => {
193193

194194
act( () => {
195195
userEvent.click( screen.getByText( 'Trigger element' ) );
196+
} );
197+
198+
act( () => {
196199
jest.runAllTimers();
197200
} );
198201

‎client/connect-account-page/test/index.test.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import React from 'react';
66
import { render, screen, waitFor } from '@testing-library/react';
77
import userEvent from '@testing-library/user-event';
88
import apiFetch from '@wordpress/api-fetch';
9-
import { mocked } from 'ts-jest/utils';
109

1110
/**
1211
* Internal dependencies
@@ -103,7 +102,7 @@ describe( 'ConnectAccountPage', () => {
103102
render( <ConnectAccountPage /> );
104103

105104
// mockApiFetch.mockRejectedValueOnce( {} );
106-
mocked( apiFetch ).mockRejectedValueOnce( new Error() );
105+
jest.mocked( apiFetch ).mockRejectedValueOnce( new Error() );
107106

108107
userEvent.click(
109108
screen.getByRole( 'button', { name: 'Verify business details' } )

‎client/deposits/filters/test/index.js

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,18 @@ import { DepositsFilters } from '../';
1515
// TODO: this is a bit of a hack as we're mocking an old version of WC, we should relook at this.
1616
jest.mock( '@woocommerce/settings', () => ( {
1717
...jest.requireActual( '@woocommerce/settings' ),
18-
getSetting: jest.fn( ( key ) => ( key === 'wcVersion' ? 7.7 : '' ) ),
18+
getSetting: jest.fn( ( key ) => ( key === 'wcVersion' ? 7.8 : '' ) ),
1919
} ) );
2020

2121
describe( 'Deposits filters', () => {
22+
beforeAll( () => {
23+
jest.useFakeTimers();
24+
} );
25+
26+
afterAll( () => {
27+
jest.useRealTimers();
28+
} );
29+
2230
beforeEach( () => {
2331
// the query string is preserved across tests, so we need to reset it
2432
updateQueryString( {}, '/', {} );

0 commit comments

Comments
 (0)