Skip to content

Commit bb8688e

Browse files
committed
fix: tests
1 parent 6005e8a commit bb8688e

File tree

4 files changed

+67
-36
lines changed

4 files changed

+67
-36
lines changed

scripts/release/__tests__/createReleasePR.test.ts

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -283,22 +283,22 @@ describe('createReleasePR', () => {
283283
],
284284
});
285285

286-
expect(versions).toEqual({
287-
java: {
288-
current: '0.1.2',
289-
next: '0.2.0',
290-
releaseType: 'minor',
291-
},
292-
javascript: {
293-
current: '0.1.2',
294-
next: '0.2.0',
295-
releaseType: 'minor',
296-
},
297-
php: {
298-
current: '0.1.2',
299-
next: '0.2.0',
300-
releaseType: 'minor',
301-
},
286+
expect(versions.java).toEqual({
287+
current: '0.1.2',
288+
next: '0.2.0',
289+
releaseType: 'minor',
290+
});
291+
292+
expect(versions.javascript).toEqual({
293+
current: '0.1.2',
294+
next: '0.2.0',
295+
releaseType: 'minor',
296+
});
297+
298+
expect(versions.php).toEqual({
299+
current: '0.1.2',
300+
next: '0.2.0',
301+
releaseType: 'minor',
302302
});
303303
});
304304
});

scripts/release/__tests__/versionsHistory.test.ts

Lines changed: 45 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -20,43 +20,65 @@ describe('generateLanguageVersionsHistory', () => {
2020
expect(versions).toEqual({
2121
'1.2.4': {
2222
releaseDate: '2023-12-28',
23+
eligibilityDate: undefined,
24+
eligibilityStatus: 'eligible',
2325
},
2426
'1.2.5': {
2527
releaseDate: '2024-01-02',
28+
eligibilityDate: undefined,
29+
eligibilityStatus: 'eligible',
2630
},
2731
'1.2.6': {
2832
releaseDate: '2024-01-02',
33+
eligibilityDate: undefined,
34+
eligibilityStatus: 'eligible',
2935
},
3036
'1.2.7': {
3137
releaseDate: '2024-01-04',
38+
eligibilityDate: undefined,
39+
eligibilityStatus: 'eligible',
3240
},
3341
});
3442
});
3543

3644
it('parses version of different same minor', () => {
3745
const versions = generateLanguageVersionsHistory(
3846
[
39-
'1.1.4 Thu Dec 28 15:48:25 2023 +0000',
40-
'1.2.5 Tue Jan 2 14:17:11 2024 +0000',
41-
'1.3.6 Tue Jan 2 15:26:06 2024 +0000',
42-
'1.4.7 Thu Jan 4 15:09:11 2024 +0000',
47+
'1.1.4 Thu Dec 28 15:48:25 2022 +0000',
48+
'2.1.2 Tue Jan 2 14:17:11 2022 +0000',
49+
'2.2.5 Tue Jan 2 14:17:11 2024 +0000',
50+
'2.3.6 Tue Jan 2 15:26:06 2024 +0000',
51+
'3.4.7 Thu Jan 4 15:09:11 2024 +0000',
4352
],
4453
'javascript',
45-
{ current: '1.4.7', next: null, releaseType: null },
54+
{ current: '3.4.7', next: null, releaseType: null },
4655
);
4756

4857
expect(versions).toEqual({
4958
'1.1.4': {
50-
releaseDate: '2023-12-28',
59+
releaseDate: '2022-12-28',
60+
eligibilityDate: undefined,
61+
eligibilityStatus: 'not eligible',
5162
},
52-
'1.2.5': {
63+
'2.1.2': {
64+
releaseDate: '2022-01-02',
65+
eligibilityDate: '2026-08-14',
66+
eligibilityStatus: 'replaced',
67+
},
68+
'2.2.5': {
5369
releaseDate: '2024-01-02',
70+
eligibilityDate: '2026-08-14',
71+
eligibilityStatus: 'replaced',
5472
},
55-
'1.3.6': {
73+
'2.3.6': {
5674
releaseDate: '2024-01-02',
75+
eligibilityDate: '2026-08-14',
76+
eligibilityStatus: 'replaced',
5777
},
58-
'1.4.7': {
78+
'3.4.7': {
5979
releaseDate: '2024-01-04',
80+
eligibilityDate: undefined,
81+
eligibilityStatus: 'eligible',
6082
},
6183
});
6284
});
@@ -77,6 +99,8 @@ describe('generateLanguageVersionsHistory', () => {
7799
expect(versions).toEqual({
78100
'1.2.4': {
79101
releaseDate: '2023-12-28',
102+
eligibilityDate: undefined,
103+
eligibilityStatus: 'eligible',
80104
},
81105
});
82106
});
@@ -91,9 +115,13 @@ describe('generateLanguageVersionsHistory', () => {
91115
expect(versions).toEqual({
92116
'1.2.4': {
93117
releaseDate: '2023-12-28',
118+
eligibilityDate: undefined,
119+
eligibilityStatus: 'eligible',
94120
},
95121
'2.0.0': {
96122
releaseDate: start.toISOString().split('T')[0],
123+
eligibilityDate: undefined,
124+
eligibilityStatus: 'eligible',
97125
},
98126
});
99127
});
@@ -108,9 +136,13 @@ describe('generateLanguageVersionsHistory', () => {
108136
expect(versions).toEqual({
109137
'1.2.4': {
110138
releaseDate: '2023-12-28',
139+
eligibilityDate: undefined,
140+
eligibilityStatus: 'eligible',
111141
},
112142
'1.3.0': {
113143
releaseDate: start.toISOString().split('T')[0],
144+
eligibilityDate: undefined,
145+
eligibilityStatus: 'eligible',
114146
},
115147
});
116148
});
@@ -125,9 +157,13 @@ describe('generateLanguageVersionsHistory', () => {
125157
expect(versions).toEqual({
126158
'1.2.4': {
127159
releaseDate: '2023-12-28',
160+
eligibilityDate: undefined,
161+
eligibilityStatus: 'eligible',
128162
},
129163
'1.2.5': {
130164
releaseDate: start.toISOString().split('T')[0],
165+
eligibilityDate: undefined,
166+
eligibilityStatus: 'eligible',
131167
},
132168
});
133169
});

scripts/release/updateAPIVersions.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,10 @@ export async function updateAPIVersions(versions: Versions, changelog: Changelog
4646
await updateConfigFiles(versions);
4747

4848
for (const [lang, { current, releaseType, next }] of Object.entries(versions)) {
49+
if (!next) {
50+
continue;
51+
}
52+
4953
if (lang === 'dart') {
5054
await updateDartPackages(changelog[lang]!, next);
5155

scripts/release/versionsHistory.ts

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -46,17 +46,8 @@ function getEligibility(currentMajor: int, previousMajor: int, version: string,
4646
return { status: 'eligible' };
4747
}
4848

49-
if (versionMajor == previousMajor) {
50-
console.log(
51-
versionMajor,
52-
previousMajor,
53-
generatedReleaseDate.toISOString().split('T')[0],
54-
eligibilityEndDate.toISOString().split('T')[0],
55-
);
56-
57-
if (eligibilityEndDate >= new Date()) {
58-
return { status: 'replaced', date: eligibilityEndDate.toISOString().split('T')[0] };
59-
}
49+
if (versionMajor == previousMajor && eligibilityEndDate >= new Date()) {
50+
return { status: 'replaced', date: eligibilityEndDate.toISOString().split('T')[0] };
6051
}
6152

6253
return { status: 'not eligible' };

0 commit comments

Comments
 (0)