Skip to content

Commit a17db00

Browse files
authored
Merge pull request #635 from simonihmig/fix-main
Fix CI for new Ember v6 canary and volta v2 releases
2 parents b423901 + f82d7b7 commit a17db00

27 files changed

+43
-41
lines changed

packages/addon-template/tests/dummy/config/environment.js

+1-4
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,7 @@ module.exports = function (environment) {
1111
// Here you can enable experimental features on an ember canary build
1212
// e.g. EMBER_NATIVE_DECORATOR_SUPPORT: true
1313
},
14-
EXTEND_PROTOTYPES: {
15-
// Prevent Ember Data from overriding Date.parse.
16-
Date: false,
17-
},
14+
EXTEND_PROTOTYPES: false,
1815
},
1916

2017
APP: {

packages/app-template/config/environment.js

+1-4
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,7 @@ module.exports = function (environment) {
1111
// Here you can enable experimental features on an ember canary build
1212
// e.g. EMBER_NATIVE_DECORATOR_SUPPORT: true
1313
},
14-
EXTEND_PROTOTYPES: {
15-
// Prevent Ember Data from overriding Date.parse.
16-
Date: false,
17-
},
14+
EXTEND_PROTOTYPES: false,
1815
},
1916

2017
APP: {

test-scenarios/addon-dev-dep-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ addonScenarios
2525
app = await scenario.prepare();
2626
});
2727
test('ensure build error', async function (assert) {
28-
let result = await app.execute('volta run npm -- run build');
28+
let result = await app.execute('volta run npm run build');
2929
assert.notEqual(result.exitCode, 0, result.output);
3030
assert.ok(
3131
/@ef4\/addon-template tried to import \"some-lib\" in \"@ef4\/addon-template\/index.js\" from addon code, but \"some-lib\" is a devDependency/.test(

test-scenarios/babel-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ appScenarios
100100
app = await scenario.prepare();
101101
});
102102
test('yarn test', async function (assert) {
103-
let result = await app.execute('volta run npm -- run test');
103+
let result = await app.execute('volta run npm run test');
104104
assert.equal(result.exitCode, 0, result.output);
105105
});
106106
});

test-scenarios/common-chunk-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ appScenarios
9595
});
9696

9797
test('npm run test', async function (assert) {
98-
let result = await app.execute('volta run npm -- run test');
98+
let result = await app.execute('volta run npm run test');
9999
assert.equal(result.exitCode, 0, result.output);
100100
});
101101
});

test-scenarios/conflict-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ appScenarios
4444
app = await scenario.prepare();
4545
});
4646
test('ensure build error', async function (assert) {
47-
let result = await app.execute('volta run npm -- run build');
47+
let result = await app.execute('volta run npm run build');
4848
assert.notEqual(result.exitCode, 0, result.output);
4949
assert.ok(
5050
/@ef4\/app-template needs inner-lib satisfying \^1.0.0, but we have version 2.3.4 because of @ef4\/addon-template/.test(

test-scenarios/csp-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ appScenarios
128128
app = await scenario.prepare();
129129
});
130130
test('npm run test', async function (assert) {
131-
let result = await app.execute('volta run npm -- run test');
131+
let result = await app.execute('volta run npm run test');
132132
assert.equal(result.exitCode, 0, result.output);
133133
});
134134
});

test-scenarios/custom-html-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ appScenarios
163163
});
164164

165165
test('npm run test', async function (assert) {
166-
let result = await app.execute('volta run npm -- run test');
166+
let result = await app.execute('volta run npm run test');
167167
assert.equal(result.exitCode, 0, result.output);
168168
});
169169
});

test-scenarios/customized-vendor-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ appScenarios
139139
app = await scenario.prepare();
140140
});
141141
test('npm run test', async function (assert) {
142-
let result = await app.execute('volta run npm -- run test');
142+
let result = await app.execute('volta run npm run test');
143143
assert.equal(result.exitCode, 0, result.output);
144144
});
145145
});

test-scenarios/doubly-indirect-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ appScenarios
165165
});
166166

167167
test('npm run test', async function (assert) {
168-
let result = await app.execute('volta run npm -- run test');
168+
let result = await app.execute('volta run npm run test');
169169
assert.equal(result.exitCode, 0, result.output);
170170
});
171171
});

test-scenarios/dynamic-import-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,7 @@ appScenarios
263263
});
264264

265265
test('npm run test', async function (assert) {
266-
let result = await app.execute('volta run npm -- run test');
266+
let result = await app.execute('volta run npm run test');
267267
assert.equal(result.exitCode, 0, result.output);
268268
});
269269

test-scenarios/ember-cli-typescript-2-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ appScenarios
5656
app = await scenario.prepare();
5757
});
5858
test('npm run test', async function (assert) {
59-
let result = await app.execute('volta run npm -- run test');
59+
let result = await app.execute('volta run npm run test');
6060
assert.equal(result.exitCode, 0, result.output);
6161
});
6262
});

test-scenarios/ember-cli-typescript-3-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ appScenarios
7070
app = await scenario.prepare();
7171
});
7272
test('npm run test', async function (assert) {
73-
let result = await app.execute('volta run npm -- run test');
73+
let result = await app.execute('volta run npm run test');
7474
assert.equal(result.exitCode, 0, result.output);
7575
});
7676
});

test-scenarios/ember-cli-typescript-4-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ appScenarios
4949
app = await scenario.prepare();
5050
});
5151
test('npm run test', async function (assert) {
52-
let result = await app.execute('volta run npm -- run test');
52+
let result = await app.execute('volta run npm run test');
5353
assert.equal(result.exitCode, 0, result.output);
5454
});
5555
});

test-scenarios/fastboot-helper.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import resolve from 'resolve';
44
import { realpathSync } from 'fs';
55

66
export async function setupFastboot(app: PreparedApp, environment = 'development') {
7-
let result = await app.execute(`volta run ember -- build --environment=${environment}`);
7+
let result = await app.execute(`volta run ember build --environment=${environment}`);
88
if (result.exitCode !== 0) {
99
throw new Error(`failed to build app for fastboot: ${result.output}`);
1010
}

test-scenarios/import-sync-test.ts

+4-3
Original file line numberDiff line numberDiff line change
@@ -83,12 +83,13 @@ appScenarios
8383
return { message: err.message }
8484
}
8585
},
86-
});`
86+
});`,
8787
},
8888
},
8989
tests: {
9090
acceptance: {
91-
'import-sync-test.js': `
91+
'import-sync-test.js':
92+
`
9293
import { module, test } from 'qunit';
9394
import { visit, currentURL } from '@ember/test-helpers';
9495
import { setupApplicationTest } from 'ember-qunit';
@@ -146,7 +147,7 @@ appScenarios
146147
});
147148

148149
test('npm run test', async function (assert) {
149-
let result = await app.execute('volta run npm -- run test');
150+
let result = await app.execute('volta run npm run test');
150151
assert.equal(result.exitCode, 0, result.output);
151152
});
152153

test-scenarios/indirect-test.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ appScenarios
203203
});
204204

205205
test('npm run test', async function (assert) {
206-
let result = await app.execute('volta run npm -- run test');
206+
let result = await app.execute('volta run npm run test');
207207
assert.equal(result.exitCode, 0, result.output);
208208
});
209209

@@ -248,7 +248,7 @@ Scenarios.fromProject(baseApp)
248248
});
249249

250250
test('npm run test', async function (assert) {
251-
let result = await app.execute('volta run npm -- run test');
251+
let result = await app.execute('volta run npm run test');
252252
assert.equal(result.exitCode, 0, result.output);
253253
});
254254
});

test-scenarios/layering-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ appScenarios
7272
});
7373

7474
test('npm run test', async function (assert) {
75-
let result = await app.execute('volta run npm -- run test');
75+
let result = await app.execute('volta run npm run test');
7676
assert.equal(result.exitCode, 0, result.output);
7777
});
7878
});

test-scenarios/leader-test.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ Scenarios.fromProject(baseApp)
7272
app = await scenario.prepare();
7373
});
7474
test('run tests', async function (assert) {
75-
let result = await app.execute('volta run npm -- run test');
75+
let result = await app.execute('volta run npm run test');
7676
assert.equal(result.exitCode, 0, result.output);
7777
});
7878
});
@@ -98,7 +98,7 @@ Scenarios.fromProject(baseApp)
9898
app = await scenario.prepare();
9999
});
100100
test('ensure error', async function (assert) {
101-
let result = await app.execute('volta run npm -- run build');
101+
let result = await app.execute('volta run npm run build');
102102
assert.notEqual(result.exitCode, 0, result.output);
103103
assert.ok(
104104
/To use these addons, your app needs ember-auto-import >= 2: .*other-problematic-addon.*problematic-addon/.test(
@@ -128,7 +128,7 @@ Scenarios.fromProject(baseApp)
128128
app = await scenario.prepare();
129129
});
130130
test('ensure error', async function (assert) {
131-
let result = await app.execute('volta run npm -- run build');
131+
let result = await app.execute('volta run npm run build');
132132
assert.notEqual(result.exitCode, 0, result.output);
133133
assert.ok(
134134
/To use these addons, your app needs ember-auto-import >= 2: .*other-problematic-addon.*problematic-addon/.test(

test-scenarios/merged-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,7 @@ appScenarios
226226
app = await scenario.prepare();
227227
});
228228
test('ensure build succeeds', async function (assert) {
229-
let result = await app.execute('volta run npm -- run test');
229+
let result = await app.execute('volta run npm run test');
230230
assert.equal(result.exitCode, 0, result.output);
231231
});
232232
});

test-scenarios/no-conflict-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ appScenarios
4343
app = await scenario.prepare();
4444
});
4545
test('ensure build succeeds', async function (assert) {
46-
let result = await app.execute('volta run npm -- run build');
46+
let result = await app.execute('volta run npm run build');
4747
assert.equal(result.exitCode, 0, result.output);
4848
});
4949
});

test-scenarios/node-es-modules.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ template
3737
});
3838
test('npm run test', async function (assert) {
3939
console.log(app.dir);
40-
let result = await app.execute('volta run npm -- run test');
40+
let result = await app.execute('volta run npm run test');
4141
assert.equal(result.exitCode, 0, result.output);
4242
});
4343
});

test-scenarios/sample-addon-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -220,7 +220,7 @@ addonScenarios
220220
app = await scenario.prepare();
221221
});
222222
test('yarn test', async function (assert) {
223-
let result = await app.execute('volta run npm -- run test');
223+
let result = await app.execute('volta run npm run test');
224224
assert.equal(result.exitCode, 0, result.output);
225225
});
226226
});

test-scenarios/static-import-test.ts

+9-2
Original file line numberDiff line numberDiff line change
@@ -116,6 +116,12 @@ function staticImportTest(project: Project) {
116116
}),
117117
});
118118
`,
119+
'hello-world.hbs': `
120+
<div class="hello-world">{{this.formattedDate}}</div>
121+
<div class="lodash">{{#if this.lodashPresent}}yes{{else}}no{{/if}}</div>
122+
<div class="aliased">{{this.aliasedResult}}</div>
123+
<div class="scoped">{{this.fromScoped}}</div>
124+
`,
119125
},
120126
lib: {
121127
'example1.js': 'export default function() { return "example1 worked" }',
@@ -151,6 +157,7 @@ function staticImportTest(project: Project) {
151157
templates: {
152158
'application.hbs': `{{hello-world}}`,
153159
components: {
160+
// Our "lts" scenario runs a very old version of Ember that does not support colocated components, that's why we intentionally keep this here
154161
'hello-world.hbs': `
155162
<div class="hello-world">{{this.formattedDate}}</div>
156163
<div class="lodash">{{#if this.lodashPresent}}yes{{else}}no{{/if}}</div>
@@ -216,7 +223,7 @@ function staticImportTest(project: Project) {
216223
let txt = await loadTxt();
217224
assert.equal(txt, 'here is some text');
218225
});
219-
});
226+
});
220227
`,
221228
'import-into-tests-test.js': `
222229
import { module, test } from 'qunit';
@@ -397,7 +404,7 @@ scenarios.forEachScenario(scenario => {
397404
});
398405

399406
test('npm run test', async function (assert) {
400-
let result = await app.execute('volta run npm -- run test');
407+
let result = await app.execute('volta run npm run test');
401408
assert.equal(result.exitCode, 0, result.output);
402409
});
403410
});

test-scenarios/targets-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ appScenarios
124124
app = await scenario.prepare();
125125
});
126126
test('yarn test', async function (assert) {
127-
let result = await app.execute('volta run npm -- run test');
127+
let result = await app.execute('volta run npm run test');
128128
assert.equal(result.exitCode, 0, result.output);
129129
});
130130
});

test-scenarios/v2-addon-test.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -387,7 +387,7 @@ scenarios.forEachScenario(scenario => {
387387
app = await scenario.prepare();
388388
});
389389
test('yarn test', async function (assert) {
390-
let result = await app.execute('volta run npm -- run test');
390+
let result = await app.execute('volta run npm run test');
391391
assert.equal(result.exitCode, 0, result.output);
392392
});
393393
});
@@ -454,7 +454,7 @@ Scenarios.fromProject(baseApp)
454454
app = await scenario.prepare();
455455
});
456456
test('ensure error', async function (assert) {
457-
let result = await app.execute('volta run npm -- run build');
457+
let result = await app.execute('volta run npm run build');
458458
assert.notEqual(result.exitCode, 0, result.output);
459459
assert.ok(
460460
/my-v1-addon needs to depend on ember-auto-import in order to use my-v2-addon/.test(result.stderr),
@@ -528,7 +528,7 @@ Scenarios.fromProject(baseApp)
528528
app = await scenario.prepare();
529529
});
530530
test('ensure success', async function (assert) {
531-
let result = await app.execute('volta run npm -- run test');
531+
let result = await app.execute('volta run npm run test');
532532
assert.strictEqual(result.exitCode, 0, result.output);
533533
});
534534
});

test-scenarios/webpack-config-test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ Scenarios.fromProject(baseApp)
112112
app = await scenario.prepare();
113113
});
114114
test('ensure build succeeds', async function (assert) {
115-
let result = await app.execute('volta run npm -- run test');
115+
let result = await app.execute('volta run npm run test');
116116
assert.equal(result.exitCode, 0, result.output);
117117
});
118118
});

0 commit comments

Comments
 (0)