Skip to content

Commit d2c0e23

Browse files
committed
test: fix multiple configs e2e tests
Currently the options are incorrectly set under architect instead of the build target. (cherry picked from commit 0c2c8b9)
1 parent 3cc0f75 commit d2c0e23

File tree

1 file changed

+30
-27
lines changed

1 file changed

+30
-27
lines changed

tests/legacy-cli/e2e/tests/build/multiple-configs.ts

Lines changed: 30 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -9,34 +9,37 @@ export default async function () {
99
// These are the default options, that we'll overwrite in subsequent configs.
1010
// extractCss defaults to false
1111
// sourceMap defaults to true
12-
appArchitect['options'] = {
13-
outputPath: 'dist/latest-project',
14-
index: 'src/index.html',
15-
main: 'src/main.ts',
16-
polyfills: 'src/polyfills.ts',
17-
tsConfig: 'tsconfig.app.json',
18-
assets: [
19-
'src/favicon.ico',
20-
'src/assets',
21-
],
22-
'styles': [
23-
'src/styles.css',
24-
],
25-
'scripts': [],
26-
};
27-
const browserConfigs = appArchitect['build'].configurations;
28-
browserConfigs['production'] = {
29-
extractCss: true,
30-
};
31-
browserConfigs['one'] = {
32-
assets: [],
33-
};
34-
browserConfigs['two'] = {
35-
sourceMap: false,
36-
};
37-
browserConfigs['three'] = {
38-
extractCss: false, // Defaults to false when not set.
12+
appArchitect['build'] = {
13+
...appArchitect['build'],
14+
options: {
15+
...appArchitect['build'].options,
16+
extractCss: false,
17+
assets: [
18+
'src/favicon.ico',
19+
'src/assets',
20+
],
21+
styles: [
22+
'src/styles.css',
23+
],
24+
scripts: [],
25+
},
26+
configurations: {
27+
production: {
28+
extractCss: true,
29+
},
30+
one: {
31+
assets: [],
32+
},
33+
two: {
34+
sourceMap: false,
35+
},
36+
three: {
37+
extractCss: false, // Defaults to false when not set.
38+
},
39+
},
3940
};
41+
42+
return workspaceJson;
4043
});
4144

4245
// Test the base configuration.

0 commit comments

Comments
 (0)