Skip to content

Commit 2ce4c42

Browse files
alan-agius4mgechev
authored andcommitted
build: only run ngcc hook for large tests
1 parent be1bcba commit 2ce4c42

File tree

1 file changed

+11
-10
lines changed

1 file changed

+11
-10
lines changed

scripts/test.ts

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -89,17 +89,18 @@ export default function(args: ParsedArgs, logger: logging.Logger) {
8989
const specGlob = args.large ? '*_spec_large.ts' : '*_spec.ts';
9090
const regex = args.glob ? args.glob : `packages/**/${specGlob}`;
9191

92-
if (args['ve']) {
93-
// tslint:disable-next-line:no-console
94-
console.warn('********* VE Enabled ***********');
95-
} else {
96-
// CI is really flaky with NGCC
97-
// This is a working around test order and isolation issues.
98-
console.warn('********* Running ngcc ***********');
99-
execSync('yarn ngcc', { stdio: 'inherit' });
100-
}
101-
10292
if (args.large) {
93+
if (args['ve']) {
94+
// tslint:disable-next-line:no-console
95+
console.warn('********* VE Enabled ***********');
96+
} else {
97+
console.warn('********* Ivy Enabled ***********');
98+
// CI is really flaky with NGCC
99+
// This is a working around test order and isolation issues.
100+
console.warn('********* Running ngcc ***********');
101+
execSync('yarn ngcc', { stdio: 'inherit' });
102+
}
103+
103104
// Default timeout for large specs is 2.5 minutes.
104105
jasmine.DEFAULT_TIMEOUT_INTERVAL = 150000;
105106
}

0 commit comments

Comments
 (0)