Skip to content

Commit

Permalink
fix(compiler-cli): log ngcc skipping messages as debug instead of info (
Browse files Browse the repository at this point in the history
  • Loading branch information
filipesilva authored and kara committed May 6, 2019
1 parent 4537816 commit 60a8888
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion integration/ngcc/test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ if [[ $? != 0 ]]; then exit 1; fi

# Can it be safely run again (as a noop)?
# And check that it logged skipping compilation as expected
ivy-ngcc | grep 'Skipping'
ivy-ngcc -l debug | grep 'Skipping'
if [[ $? != 0 ]]; then exit 1; fi

# Check that running it with logging level error outputs nothing
Expand Down
6 changes: 3 additions & 3 deletions packages/compiler-cli/ngcc/src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ export function mainNgcc(
if (absoluteTargetEntryPointPath &&
hasProcessedTargetEntryPoint(
fs, absoluteTargetEntryPointPath, propertiesToConsider, compileAllFormats)) {
logger.info('The target entry-point has already been processed');
logger.debug('The target entry-point has already been processed');
return;
}

Expand Down Expand Up @@ -124,7 +124,7 @@ export function mainNgcc(

if (hasBeenProcessed(entryPointPackageJson, property)) {
compiledFormats.add(formatPath);
logger.info(`Skipping ${entryPoint.name} : ${property} (already compiled).`);
logger.debug(`Skipping ${entryPoint.name} : ${property} (already compiled).`);
continue;
}

Expand All @@ -147,7 +147,7 @@ export function mainNgcc(
`Skipping ${entryPoint.name} : ${format} (no valid entry point file for this format).`);
}
} else if (!compileAllFormats) {
logger.info(`Skipping ${entryPoint.name} : ${property} (already compiled).`);
logger.debug(`Skipping ${entryPoint.name} : ${property} (already compiled).`);
}

// Either this format was just compiled or its underlying format was compiled because of a
Expand Down
8 changes: 4 additions & 4 deletions packages/compiler-cli/ngcc/test/integration/ngcc_spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ describe('ngcc main()', () => {
basePath: '/node_modules',
targetEntryPointPath: '@angular/common/http/testing', logger,
});
expect(logger.logs.info).toContain(['The target entry-point has already been processed']);
expect(logger.logs.debug).toContain(['The target entry-point has already been processed']);
});

it('should process the target if any `propertyToConsider` is not marked as processed', () => {
Expand All @@ -97,7 +97,7 @@ describe('ngcc main()', () => {
targetEntryPointPath: '@angular/common/http/testing',
propertiesToConsider: ['fesm2015', 'esm5', 'esm2015'], logger,
});
expect(logger.logs.info).not.toContain([
expect(logger.logs.debug).not.toContain([
'The target entry-point has already been processed'
]);
});
Expand All @@ -115,7 +115,7 @@ describe('ngcc main()', () => {
compileAllFormats: false, logger,
});

expect(logger.logs.info).not.toContain([
expect(logger.logs.debug).not.toContain([
'The target entry-point has already been processed'
]);
});
Expand All @@ -132,7 +132,7 @@ describe('ngcc main()', () => {
compileAllFormats: false, logger,
});

expect(logger.logs.info).toContain([
expect(logger.logs.debug).toContain([
'The target entry-point has already been processed'
]);
});
Expand Down

0 comments on commit 60a8888

Please sign in to comment.