Skip to content

Commit 79962ec

Browse files
committed
ci: updates tests & includes Node 16.14,18,20
1 parent 34b5bc5 commit 79962ec

File tree

3 files changed

+58
-33
lines changed

3 files changed

+58
-33
lines changed

.github/workflows/CI.yml

+54-27
Original file line numberDiff line numberDiff line change
@@ -11,102 +11,129 @@ on:
1111
tags:
1212
- "*"
1313

14+
concurrency:
15+
group: ci-${{ github.head_ref || github.ref }}
16+
cancel-in-progress: true
17+
1418
jobs:
1519
lint:
1620
name: "Linting"
1721
runs-on: ubuntu-latest
1822
steps:
19-
- uses: actions/checkout@v3
23+
- uses: actions/checkout@v4
2024
- run: yarn install --frozen-lockfile --non-interactive
21-
- run: yarn lint:js
25+
- run: yarn lint
2226

23-
ember-tests:
27+
ember-default-tests:
2428
needs: lint
2529
name: "Tests: ubuntu (node@${{ matrix.node-version }})"
2630
runs-on: ubuntu-latest
2731
strategy:
32+
fail-fast: false
2833
matrix:
29-
node-version: ["18"]
34+
node-version:
35+
- 16.14
36+
- 18
37+
- 20
3038

3139
steps:
32-
- uses: actions/checkout@v3
40+
- uses: actions/checkout@v4
3341
with:
3442
node-version: ${{ matrix.node-version }}
43+
cache: yarn
3544
- run: yarn install --frozen-lockfile --non-interactive
3645
- run: yarn test
3746

38-
prefer-native-tests:
39-
needs: ember-tests
47+
prefer-native-ember-default-tests:
48+
needs: ember-default-tests
4049
name: "PREFER_NATIVE=true Tests: ubuntu (node@${{ matrix.node-version }})"
4150
runs-on: ubuntu-latest
4251
strategy:
52+
fail-fast: false
4353
matrix:
44-
node-version: ["18"]
54+
node-version:
55+
- 16.14
56+
- 18
57+
- 20
4558

4659
steps:
47-
- uses: actions/checkout@v3
60+
- uses: actions/checkout@v4
4861
with:
4962
node-version: ${{ matrix.node-version }}
63+
cache: yarn
5064
- run: yarn install --frozen-lockfile --non-interactive
5165
- run: PREFER_NATIVE=true yarn test
5266

53-
native-promise-tests:
54-
needs: ember-tests
67+
native-promise-ember-default-tests:
68+
needs: ember-default-tests
5569
name: "NATIVE_PROMISE=true Tests: ubuntu (node@${{ matrix.node-version }})"
5670
runs-on: ubuntu-latest
5771
strategy:
72+
fail-fast: false
5873
matrix:
59-
node-version: ["18"]
74+
node-version:
75+
- 16.14
76+
- 18
77+
- 20
6078

6179
steps:
62-
- uses: actions/checkout@v3
80+
- uses: actions/checkout@v4
6381
with:
6482
node-version: ${{ matrix.node-version }}
6583
- run: yarn install --frozen-lockfile --non-interactive
6684
- run: NATIVE_PROMISE=true yarn test
6785

6886
node-tests:
69-
needs: ember-tests
87+
needs: ember-default-tests
7088
name: "Node Tests: ubuntu (node@${{ matrix.node-version }})"
7189
runs-on: ubuntu-latest
7290
strategy:
91+
fail-fast: false
7392
matrix:
74-
node-version: ["18"]
93+
node-version:
94+
- 16.14
95+
- 18
96+
- 20
7597

7698
steps:
77-
- uses: actions/checkout@v3
99+
- uses: actions/checkout@v4
78100
with:
79101
node-version: ${{ matrix.node-version }}
80102
- run: yarn install --frozen-lockfile --non-interactive
81103
- run: yarn run test:node
82-
83-
try-scenarios:
104+
105+
ember-try-scenarios:
84106
name: Tests - ${{ matrix.ember-try-scenario }}
85107
timeout-minutes: 60
86108
runs-on: ubuntu-latest
87109
continue-on-error: ${{ matrix.allow-failure }}
88-
needs: ember-tests
110+
needs: ember-default-tests
89111

90112
strategy:
91113
fail-fast: true
92114
matrix:
93115
ember-try-scenario:
94-
- ember-default
95116
- ember-lts-3.28
96117
- ember-lts-4.4
97118
- ember-lts-4.8
98119
- ember-release
99-
- ember-beta
100-
- ember-canary
101-
- embroider-safe
102-
- embroider-optimized
120+
allow-failure: [false]
121+
include:
122+
- ember-try-scenario: ember-beta
123+
allow-failure: true
124+
- ember-try-scenario: ember-canary
125+
allow-failure: true
126+
- ember-try-scenario: embroider-safe
127+
allow-failure: true
128+
- ember-try-scenario: embroider-optimized
129+
allow-failure: true
103130

104131
steps:
105-
- uses: actions/checkout@v3
132+
- uses: actions/checkout@v4
106133
- name: Install Node
107-
uses: actions/setup-node@v3
134+
uses: actions/setup-node@v4
108135
with:
109-
node-version: 18.x
136+
node-version: 16.14
110137
- name: install dependencies
111138
run: yarn install
112139
- name: test

package.json

+4-5
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,9 @@
2424
"lint:js": "eslint . --cache",
2525
"lint:js:fix": "eslint . --fix",
2626
"start": "ember serve",
27-
"test": "concurrently \"npm:lint\" \"npm:test:*\" --names \"lint,test:\"",
28-
"test:ember": "ember test",
29-
"test:ember-compatibility": "ember try:each",
27+
"test": "ember test",
3028
"test:node": "ember ts:precompile && mocha && ember ts:clean",
29+
"test:all": "yarn run test:node && ember try:each",
3130
"prepublishOnly": "ember ts:precompile",
3231
"postpublish": "ember ts:clean"
3332
},
@@ -112,7 +111,7 @@
112111
"ember-source": ">=3.28.0"
113112
},
114113
"engines": {
115-
"node": ">=16.13 || >= 18"
114+
"node": ">=16.14 || >= 18"
116115
},
117116
"publishConfig": {
118117
"registry": "https://registry.npmjs.org/"
@@ -130,7 +129,7 @@
130129
]
131130
},
132131
"volta": {
133-
"node": "10.24.1",
132+
"node": "16.20.2",
134133
"yarn": "1.22.10"
135134
}
136135
}

tests/dummy/config/ember-try.js

-1
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ module.exports = async function () {
4444
'ember-source': await getChannelURL('release'),
4545
},
4646
},
47-
allowedToFail: true,
4847
},
4948
{
5049
name: 'ember-beta',

0 commit comments

Comments
 (0)