Merge pull request #17 from PolymerLabs/check-output

Check that test output compiles and lints

Author: aomarks

.eslintrc.json

@@ -9,6 +9,7 @@
   ],
   "rules": {
     "@typescript-eslint/no-use-before-define": ["error", "nofunc"],
-    "@typescript-eslint/explicit-function-return-type": "off"
+    "@typescript-eslint/explicit-function-return-type": "off",
+    "semi": "error"
   }
 }

.github/workflows/tests.yml

@@ -32,3 +32,9 @@ jobs:
 
       - name: AVA unit tests
         run: npx ava
+
+      - name: Check test output TypeScript
+        run: npm run test:check-tsc
+
+      - name: Check test output eslint
+        run: npm run test:check-eslint

package-lock.json

@@ -16,9 +16,11 @@
     "clean": "rimraf 'lib/*' 'lib/.*' 'testdata/*/output'",
     "build": "npm run clean && npm run generate-json-schema && tsc",
     "generate-json-schema": "typescript-json-schema src/config.ts Config --required --noExtraProps > config.schema.json",
-    "test": "npm run build && ava --verbose",
+    "test": "npm run build && ava --verbose && npm run test:check-tsc && npm run test:check-eslint",
     "test:update-goldens": "npm run build && UPDATE_TEST_GOLDENS=true ava --verbose",
-    "lint": "eslint 'src/**/*.ts' --max-warnings 0",
+    "test:check-tsc": "ls testdata/*/output/tsconfig.json | xargs -n 1 tsc --noEmit --project",
+    "test:check-eslint": "eslint 'testdata/*/output/*.ts' 'testdata/*/output/**/*.ts' --max-warnings 0",
+    "lint": "eslint 'src/*.ts' 'src/**/*.ts' --max-warnings 0",
     "prettier": "prettier 'src/**/*.ts'",
     "format:check": "npm run prettier -- --check",
     "format": "npm run prettier -- --write",
@@ -48,6 +50,7 @@
     "dir-compare": "^2.3.0",
     "eslint": "^6.8.0",
     "fs-extra": "^9.0.0",
+    "lit-html": "^1.2.1",
     "prettier": "^2.0.5",
     "rimraf": "^3.0.2",
     "typescript": "^3.8.3",

package.json

@@ -18,7 +18,8 @@ npx chokidar \
   "lib/**/*.js" \
   "config.schema.json" \
   "testdata/*/input/**/*" \
-  "testdata/*/golden/**/*" \
-  "ava.config.js" -c "ava" &
+  "testdata/*/goldens/" \
+  "ava.config.js" \
+  -c "ava; npm run test:check-tsc; npm run test:check-eslint" &
 
 wait