Skip to content

Commit

Permalink
capricorn86#1144@patch: Run tests against newer Node.js versions
Browse files Browse the repository at this point in the history
Also fixed tests that were failing locally for me (running on Node.js 18), for various reasons.
  • Loading branch information
IGx89 committed Oct 26, 2023
1 parent 8ed0db0 commit a0a4954
Show file tree
Hide file tree
Showing 5 changed files with 13 additions and 10 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/pull_request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
node-version: [16]
node-version: [16, 18, 20]

steps:
- uses: actions/checkout@v3
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@ export default class HTMLInputElementDateUtility {
date.setUTCDate(date.getUTCDate() - day + 3);
const firstThursday = date.getTime();
date.setUTCMonth(0, 1);
if (date.getDay() !== 4) {
date.setUTCMonth(0, 1 + ((4 - date.getDay() + 7) % 7));
if (date.getUTCDay() !== 4) {
date.setUTCMonth(0, 1 + ((4 - date.getUTCDay() + 7) % 7));
}
return (
date.getUTCFullYear() +
Expand All @@ -37,7 +37,7 @@ export default class HTMLInputElementDateUtility {
}
const date = new Date(`${Y}-01-01T00:00Z`);
const jan4th = new Date(`${Y}-01-04T00:00Z`);
const jan4thDay = (jan4th.getDay() + 6) % 7;
const jan4thDay = (jan4th.getUTCDay() + 6) % 7;
const ordinalDate = 1 + (Number(W) - 1) * 7 - jan4thDay + 3;
date.setUTCDate(ordinalDate);
if (date.getUTCFullYear() > Number(Y)) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -390,6 +390,7 @@ describe('HTMLInputElement', () => {
{ type: 'time', value: '00:00', want: new Date('1970-01-01T00:00Z') },
{ type: 'time', value: '12:00', want: new Date('1970-01-01T12:00Z') },
{ type: 'time', value: '18:55', want: new Date('1970-01-01T18:55Z') },
{ type: 'week', value: '1981-W01', want: new Date('1980-12-29T00:00Z') },
{ type: 'week', value: '2023-W22', want: new Date('2023-05-29T00:00Z') }
])(`Should return valid date for type $type with valid value`, ({ type, value, want }) => {
element.type = type;
Expand Down
4 changes: 2 additions & 2 deletions packages/happy-dom/test/window/Window.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -874,7 +874,7 @@ describe('Window', () => {
expect((<ErrorEvent>(<unknown>errorEvent)).error?.message).toBe('Test error');
expect((<ErrorEvent>(<unknown>errorEvent)).message).toBe('Test error');
resolve(null);
}, 15);
}, 20);
});
});
});
Expand Down Expand Up @@ -1030,7 +1030,7 @@ describe('Window', () => {
expect((<ErrorEvent>(<unknown>errorEvent)).error?.message).toBe('Test error');
expect((<ErrorEvent>(<unknown>errorEvent)).message).toBe('Test error');
resolve(null);
}, 10);
}, 20);
});
});
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -70,15 +70,17 @@ async function itObservesUnhandledJavaScriptFetchRejections(): Promise<void> {
}

if (
errorEvent.error.message !==
'Fetch to "https://localhost:3000/404.js" failed. Error: connect ECONNREFUSED 127.0.0.1:3000'
!errorEvent.error.message.startsWith(
'Fetch to "https://localhost:3000/404.js" failed. Error: connect ECONNREFUSED'
)
) {
throw new Error('Error message not correct.');
}

if (
errorEvent.message !==
'Fetch to "https://localhost:3000/404.js" failed. Error: connect ECONNREFUSED 127.0.0.1:3000'
!errorEvent.message.startsWith(
'Fetch to "https://localhost:3000/404.js" failed. Error: connect ECONNREFUSED'
)
) {
throw new Error('Error message not correct.');
}
Expand Down

0 comments on commit a0a4954

Please sign in to comment.