Charles Lyding 3dae3c03d1 fix(@schematics/angular): add less dependency in application migration if needed
The application migration schematic will now attempt to detect the usage
of Less stylesheets within a workspace and install the `less` dependency
if required. This will only occur if the migration analysis allows for
the conversion to use the `@angular/build` package instead of the
`@angular-devkit/build-angular` package. The Less usage detection may
not detect Less stylesheets within Node.js packages and currently does
not handle the case where a project only has inline Less stylesheets.
More complex analysis may be added in the future. However, a build time
error message will be presented with instructions to install `less` in
these cases if the stylesheet preprocessor is required and not present.
2024-04-29 14:05:36 -04:00

289 lines
11 KiB
TypeScript

/**
* @license
* Copyright Google LLC All Rights Reserved.
*
* Use of this source code is governed by an MIT-style license that can be
* found in the LICENSE file at https://angular.io/license
*/
import { EmptyTree } from '@angular-devkit/schematics';
import { SchematicTestRunner, UnitTestTree } from '@angular-devkit/schematics/testing';
import { Builders, ProjectType, WorkspaceSchema } from '../../utility/workspace-models';
function createWorkSpaceConfig(tree: UnitTestTree) {
const angularConfig: WorkspaceSchema = {
version: 1,
projects: {
app: {
root: '/project/app',
sourceRoot: 'src',
projectType: ProjectType.Application,
prefix: 'app',
architect: {
build: {
builder: Builders.Browser,
options: {
tsConfig: 'src/tsconfig.app.json',
main: 'src/main.ts',
polyfills: 'src/polyfills.ts',
outputPath: 'dist/project',
resourcesOutputPath: '/resources',
},
},
},
},
},
};
tree.create('/angular.json', JSON.stringify(angularConfig, undefined, 2));
tree.create('/tsconfig.json', JSON.stringify({}, undefined, 2));
tree.create('/package.json', JSON.stringify({}, undefined, 2));
}
describe(`Migration to use the application builder`, () => {
const schematicName = 'use-application-builder';
const schematicRunner = new SchematicTestRunner(
'migrations',
require.resolve('../migration-collection.json'),
);
let tree: UnitTestTree;
beforeEach(() => {
tree = new UnitTestTree(new EmptyTree());
createWorkSpaceConfig(tree);
});
it(`should replace 'outputPath' to string if 'resourcesOutputPath' is set to 'media'`, async () => {
// Replace resourcesOutputPath
tree.overwrite('angular.json', tree.readContent('angular.json').replace('/resources', 'media'));
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const {
projects: { app },
} = JSON.parse(newTree.readContent('/angular.json'));
const { outputPath, resourcesOutputPath } = app.architect['build'].options;
expect(outputPath).toEqual({
base: 'dist/project',
});
expect(resourcesOutputPath).toBeUndefined();
});
it(`should set 'outputPath.media' if 'resourcesOutputPath' is set and is not 'media'`, async () => {
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const {
projects: { app },
} = JSON.parse(newTree.readContent('/angular.json'));
const { outputPath, resourcesOutputPath } = app.architect['build'].options;
expect(outputPath).toEqual({
base: 'dist/project',
media: 'resources',
});
expect(resourcesOutputPath).toBeUndefined();
});
it(`should remove 'browser' portion from 'outputPath'`, async () => {
// Replace outputPath
tree.overwrite(
'angular.json',
tree.readContent('angular.json').replace('dist/project/', 'dist/project/browser/'),
);
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const {
projects: { app },
} = JSON.parse(newTree.readContent('/angular.json'));
const { outputPath } = app.architect['build'].options;
expect(outputPath).toEqual({
base: 'dist/project',
media: 'resources',
});
});
it('should remove tilde prefix from CSS @import specifiers', async () => {
// Replace outputPath
tree.create(
'/project/app/src/styles.css',
'@import "~@angular/material";\n@import "./abc.css"\n',
);
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const content = newTree.readText('/project/app/src/styles.css');
expect(content).toEqual('@import "@angular/material";\n@import "./abc.css"\n');
});
it('should remove caret prefix from CSS @import specifiers and as external dependency', async () => {
// Replace outputPath
tree.create(
'/project/app/src/styles.css',
'@import "^@angular/material";\n@import "./abc.css"\n',
);
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const content = newTree.readText('/project/app/src/styles.css');
expect(content).toEqual('@import "@angular/material";\n@import "./abc.css"\n');
const {
projects: { app },
} = JSON.parse(newTree.readContent('/angular.json'));
const { externalDependencies } = app.architect['build'].options;
expect(externalDependencies).toEqual(['@angular/material']);
});
it('should remove tilde prefix from SCSS @import specifiers', async () => {
// Replace outputPath
tree.create('/project/app/src/styles.scss', '@import "~@angular/material";\n@import "./abc"\n');
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const content = newTree.readText('/project/app/src/styles.scss');
expect(content).toEqual('@import "@angular/material";\n@import "./abc"\n');
});
it('should remove tilde prefix from SCSS @use specifiers', async () => {
// Replace outputPath
tree.create('/project/app/src/styles.scss', '@use "~@angular/material";\n@import "./abc"\n');
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const content = newTree.readText('/project/app/src/styles.scss');
expect(content).toEqual('@use "@angular/material";\n@import "./abc"\n');
});
it('should remove caret prefix from SCSS @import specifiers and as external dependency', async () => {
// Replace outputPath
tree.create('/project/app/src/styles.scss', '@import "^@angular/material";\n@import "./abc"\n');
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const content = newTree.readText('/project/app/src/styles.scss');
expect(content).toEqual('@import "@angular/material";\n@import "./abc"\n');
const {
projects: { app },
} = JSON.parse(newTree.readContent('/angular.json'));
const { externalDependencies } = app.architect['build'].options;
expect(externalDependencies).toEqual(['@angular/material']);
});
it('should remove caret prefix from SCSS @use specifiers and as external dependency', async () => {
// Replace outputPath
tree.create('/project/app/src/styles.scss', '@use "^@angular/material";\n@import "./abc"\n');
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const content = newTree.readText('/project/app/src/styles.scss');
expect(content).toEqual('@use "@angular/material";\n@import "./abc"\n');
const {
projects: { app },
} = JSON.parse(newTree.readContent('/angular.json'));
const { externalDependencies } = app.architect['build'].options;
expect(externalDependencies).toEqual(['@angular/material']);
});
it('should add SCSS workspace include path for root referenced @import specifiers', async () => {
// Replace outputPath
tree.create(
'/project/app/src/styles.scss',
'@use "@angular/material";\n@import "some/path/abc"\n',
);
tree.create('/some/path/abc.scss', '');
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const content = newTree.readText('/project/app/src/styles.scss');
expect(content).toEqual('@use "@angular/material";\n@import "some/path/abc"\n');
const {
projects: { app },
} = JSON.parse(newTree.readContent('/angular.json'));
const { stylePreprocessorOptions } = app.architect['build'].options;
expect(stylePreprocessorOptions).toEqual({ includePaths: ['.'] });
});
it('should add SCSS workspace include path for root referenced @use specifiers', async () => {
// Replace outputPath
tree.create(
'/project/app/src/styles.scss',
'@use "@angular/material";\n@use "some/path/abc"\n',
);
tree.create('/some/path/abc.scss', '');
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const content = newTree.readText('/project/app/src/styles.scss');
expect(content).toEqual('@use "@angular/material";\n@use "some/path/abc"\n');
const {
projects: { app },
} = JSON.parse(newTree.readContent('/angular.json'));
const { stylePreprocessorOptions } = app.architect['build'].options;
expect(stylePreprocessorOptions).toEqual({ includePaths: ['.'] });
});
it('should not add SCSS workspace include path for root referenced @import specifiers with ".import" local file', async () => {
// Replace outputPath
tree.create(
'/project/app/src/styles.scss',
'@use "@angular/material";\n@import "some/path/abc"\n',
);
tree.create('/some/path/abc.scss', '');
tree.create('/project/app/src/some/path/abc.import.scss', '');
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const content = newTree.readText('/project/app/src/styles.scss');
expect(content).toEqual('@use "@angular/material";\n@import "some/path/abc"\n');
const {
projects: { app },
} = JSON.parse(newTree.readContent('/angular.json'));
const { stylePreprocessorOptions } = app.architect['build'].options;
expect(stylePreprocessorOptions).toBeUndefined();
});
it('should add SCSS workspace include path for root referenced @use specifiers with ".import" local file', async () => {
// Replace outputPath
tree.create(
'/project/app/src/styles.scss',
'@use "@angular/material";\n@use "some/path/abc"\n',
);
tree.create('/some/path/abc.scss', '');
tree.create('/project/app/src/some/path/abc.import.scss', '');
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const content = newTree.readText('/project/app/src/styles.scss');
expect(content).toEqual('@use "@angular/material";\n@use "some/path/abc"\n');
const {
projects: { app },
} = JSON.parse(newTree.readContent('/angular.json'));
const { stylePreprocessorOptions } = app.architect['build'].options;
expect(stylePreprocessorOptions).toEqual({ includePaths: ['.'] });
});
it('should add "less" dependency when converting to "@angular/build" and a ".less" file is present', async () => {
tree.create('/project/app/src/styles.less', '');
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const { devDependencies } = JSON.parse(newTree.readContent('/package.json'));
expect(devDependencies['less']).toBeDefined();
});
it('should not add "less" dependency when converting to "@angular/build" and a ".less" file is present', async () => {
const newTree = await schematicRunner.runSchematic(schematicName, {}, tree);
const { devDependencies } = JSON.parse(newTree.readContent('/package.json'));
expect(devDependencies['less']).toBeUndefined();
});
});