Alan Agius a144e82eb7 feat(@schematics/angular): add migration to update tslint to version 6
In tslint version 6, several recommanded rules have been removed in the following PRs:
https://github.com/palantir/tslint/pull/4871 https://github.com/palantir/tslint/pull/4312

With this migration we update the tslint depedency to 6+ and add back these rules in the user workspace `tslint.json` also we remove some deprecated rules.

Until version 10, this is an opt-in migration and users will need to run this manually with the below command:
```
ng update @angular/cli --migrate-only tslint-version-6
```

Closes: #17117
Reference: TOOL-1348
2020-03-16 21:41:48 -07:00

115 lines
3.8 KiB
TypeScript

/**
* @license
* Copyright Google Inc. 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 { TSLINT_VERSION } from './update-tslint';
describe('Migration of tslint to version 6', () => {
const schematicRunner = new SchematicTestRunner(
'migrations',
require.resolve('../migration-collection.json'),
);
let tree: UnitTestTree;
const TSLINT_PATH = '/tslint.json';
const PACKAGE_JSON_PATH = '/package.json';
const TSLINT_CONFIG = {
extends: 'tslint:recommended',
rules: {
'no-use-before-declare': true,
'arrow-return-shorthand': false,
'label-position': true,
},
};
const PACKAGE_JSON = {
devDependencies: {
tslint: '~5.1.0',
},
};
beforeEach(() => {
tree = new UnitTestTree(new EmptyTree());
tree.create(PACKAGE_JSON_PATH, JSON.stringify(PACKAGE_JSON, null, 2));
tree.create(TSLINT_PATH, JSON.stringify(TSLINT_CONFIG, null, 2));
});
it('should update tslint dependency', async () => {
const newTree = await schematicRunner.runSchematicAsync('tslint-version-6', {}, tree).toPromise();
const packageJson = JSON.parse(newTree.readContent(PACKAGE_JSON_PATH));
expect(packageJson.devDependencies.tslint).toBe(TSLINT_VERSION);
});
it('should remove old/deprecated rules', async () => {
const newTree = await schematicRunner.runSchematicAsync('tslint-version-6', {}, tree).toPromise();
const { rules } = JSON.parse(newTree.readContent(TSLINT_PATH));
expect(rules['no-use-before-declare']).toBeUndefined();
});
it('should add new rules', async () => {
const newTree = await schematicRunner.runSchematicAsync('tslint-version-6', {}, tree).toPromise();
const { rules } = JSON.parse(newTree.readContent(TSLINT_PATH));
expect(rules['eofline']).toBe(true);
});
it('should not update already present rules', async () => {
const newTree = await schematicRunner.runSchematicAsync('tslint-version-6', {}, tree).toPromise();
const { rules } = JSON.parse(newTree.readContent(TSLINT_PATH));
expect(rules['arrow-return-shorthand']).toBe(false);
});
it(`should not add new rules when not extending 'tslint:recommended'`, async () => {
tree.overwrite(
TSLINT_PATH,
JSON.stringify({
...TSLINT_CONFIG,
extends: 'tslint-config-prettier',
}, null, 2),
);
const newTree = await schematicRunner.runSchematicAsync('tslint-version-6', {}, tree).toPromise();
const { rules } = JSON.parse(newTree.readContent(TSLINT_PATH));
expect(rules['eofline']).toBeUndefined();
});
it(`should not add new rules when extending multiple configs`, async () => {
tree.overwrite(
TSLINT_PATH,
JSON.stringify({
...TSLINT_CONFIG,
extends: [
'tslint:recommended',
'tslint-config-prettier',
],
}, null, 2),
);
const newTree = await schematicRunner.runSchematicAsync('tslint-version-6', {}, tree).toPromise();
const { rules } = JSON.parse(newTree.readContent(TSLINT_PATH));
expect(rules['eofline']).toBeUndefined();
});
it(`should remove old/deprecated rules when extending multiple configs`, async () => {
tree.overwrite(
TSLINT_PATH,
JSON.stringify({
...TSLINT_CONFIG,
extends: [
'tslint:recommended',
'tslint-config-prettier',
],
}, null, 2),
);
const newTree = await schematicRunner.runSchematicAsync('tslint-version-6', {}, tree).toPromise();
const { rules } = JSON.parse(newTree.readContent(TSLINT_PATH));
expect(rules['no-use-before-declare']).toBeUndefined();
});
});