Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

Commit

Permalink
Exclude new test files from linter and npm packaging.
Browse files Browse the repository at this point in the history
  • Loading branch information
joyzhong committed Dec 3, 2019
1 parent 89fd463 commit e8fe73d
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 8 deletions.
4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,12 @@
"dist": "npm run build && npm run build:min && npm run build:esmodules",
"docgen": "npm run clean && npm run build:esmodules && tsc ./scripts/documentation/tsdoc.ts --resolveJsonModule && node ./scripts/documentation/tsdoc.js",
"fix:js": "eslint --fix test scripts webpack.config.js karma.conf.js",
"fix:ts": "tslint --exclude \"**/*.d.ts\" --fix \"packages/**/*.ts\" \"test/**/*.ts\" \"scripts/**/*.ts\"",
"fix:ts": "tslint --exclude \"**/*.d.ts\" --exclude \"packages/**/test/*.ts\" --fix \"packages/test/*.ts\" \"test/**/*.ts\" \"scripts/**/*.ts\"",
"fix:css": "stylelint --fix \"packages/**/*.scss\"; stylelint --fix --config=test/screenshot/.stylelintrc.yaml \"test/screenshot/**/*.scss\"",
"fix": "npm-run-all --parallel fix:*",
"lint:css": "stylelint \"packages/**/*.scss\" && stylelint --config=test/screenshot/.stylelintrc.yaml \"test/screenshot/**/*.scss\"",
"lint:js": "eslint test scripts webpack.config.js karma.conf.js",
"lint:ts": "tslint --exclude \"**/*.d.ts\" \"packages/**/*.ts\" \"test/**/*.ts\" \"scripts/**/*.ts\"",
"lint:ts": "tslint --exclude \"**/*.d.ts\" --exclude \"packages/**/test/*.ts\" \"packages/test/*.ts\" \"test/**/*.ts\" \"scripts/**/*.ts\"",
"lint:html": "find test/screenshot/spec/ -name '*.html' | grep -v 'index.html$' | xargs htmllint",
"lint:mdc": "ts-node --project scripts/tsconfig.json -- scripts/lint-mdc.ts",
"lint": "npm-run-all --parallel lint:*",
Expand Down
1 change: 1 addition & 0 deletions packages/mdc-checkbox/.npmignore
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
*.ts
!*.d.ts
*_test.d.ts
10 changes: 5 additions & 5 deletions packages/mdc-checkbox/test/foundation.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -191,11 +191,11 @@ describe('MDCCheckboxFoundation', () => {
jasmine.clock().install();
});

// it('exports constants', () => {
// expect(cssClasses).toEqual(MDCCheckboxFoundation.cssClasses);
// expect(numbers).toEqual(MDCCheckboxFoundation.numbers);
// expect(strings).toEqual(MDCCheckboxFoundation.strings);
// });
it('exports constants', () => {
expect(cssClasses).toEqual(MDCCheckboxFoundation.cssClasses);
expect(numbers).toEqual(MDCCheckboxFoundation.numbers);
expect(strings).toEqual(MDCCheckboxFoundation.strings);
});

it('defaultAdapter returns a complete adapter implementation', () => {
verifyDefaultAdapter(MDCCheckboxFoundation, [
Expand Down
2 changes: 1 addition & 1 deletion scripts/lint-mdc.ts
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ run();

function run() {
const inputFilePaths = glob.sync(path.join(PACKAGES_DIR_ABSOLUTE, '**/*.ts'), {
ignore: ['**/node_modules/**', '**/dist/**', '**/*.d.ts'],
ignore: ['**/node_modules/**', '**/dist/**', '**/*.d.ts', '**/test/*.ts'],
nodir: true,
});

Expand Down

0 comments on commit e8fe73d

Please sign in to comment.