@plugjs/plug 0.0.11 → 0.0.14

package/package.json

@@ -1,18 +1,18 @@
 {
   "name": "@plugjs/plug",
-  "version": "0.0.11",
+  "version": "0.0.14",
   "type": "commonjs",
   "main": "./dist/index.cjs",
   "module": "./dist/index.mjs",
-  "types": "./types/index.d.ts",
+  "types": "./dist/index.d.ts",
   "exports": {
     ".": {
       "require": {
-        "types": "./types/index.d.ts",
+        "types": "./dist/index.d.ts",
         "default": "./dist/index.cjs"
       },
       "import": {
-        "types": "./types/index.d.ts",
+        "types": "./dist/index.d.ts",
         "default": "./dist/index.mjs"
       }
     }
@@ -28,9 +28,9 @@
   "author": "",
   "license": "ISC",
   "dependencies": {
-    "@babel/parser": "^7.18.11",
-    "@babel/types": "^7.18.10",
-    "@plugjs/cov8-html": "^0.0.1",
+    "@babel/parser": "^7.18.13",
+    "@babel/types": "^7.18.13",
+    "@plugjs/cov8-html": "^0.0.2",
     "@types/node": "<17",
     "diff": "^5.1.0",
     "esbuild": "^0.15.5",
@@ -42,6 +42,7 @@
     "yargs-parser": "^21.1.1"
   },
   "devDependencies": {
+    "@plugjs/eslint-plugin": "^0.0.2",
     "@types/chai": "^4.3.3",
     "@types/diff": "^5.0.2",
     "@types/eslint": "^8.4.6",
@@ -49,8 +50,8 @@
     "@types/ms": "^0.7.31",
     "@types/picomatch": "^2.3.0",
     "@types/yargs-parser": "^21.0.0",
-    "@typescript-eslint/eslint-plugin": "^5.33.1",
-    "@typescript-eslint/parser": "^5.33.1",
+    "@typescript-eslint/eslint-plugin": "^5.34.0",
+    "@typescript-eslint/parser": "^5.34.0",
     "chai": "^4.3.6",
     "eslint-config-google": "^0.14.0"
   }

package/src/log/report.ts

@@ -354,8 +354,8 @@ class ReportImpl implements Report {
       if (file && file !== nul && annotation) {
         const { level, note } = annotation
         const $col = level === NOTICE ? $blu : level === WARN ? $ylw : $red
-        const ann = `${$gry('[')}${$col(note.padStart(aPad))}${$gry(']')}`
-        const pad = ''.padStart(fPad - file.length) // file is underlined
+        const ann = `${$gry('[')}${$col(note)}${$gry(']')}`
+        const pad = ''.padStart((fPad + aPad) - (file.length + note.length))
 
         this._emitter({ ...options, level }, [ $wht($und(file)), pad, ann ])
       } else if (file !== nul ) {

package/src/plugs/coverage/report.ts

@@ -329,7 +329,9 @@ export async function coverageReport(
 function updateNodeCoverageResult(result: NodeCoverageResult): void {
   const { coveredNodes, missingNodes, ignoredNodes } = result
   const totalNodes = result.totalNodes = coveredNodes + missingNodes + ignoredNodes
-  if (totalNodes - ignoredNodes) {
+  if (totalNodes === 0) {
+    result.coverage = 100 // No "total" nodes, means all ignored
+  } else if (totalNodes - ignoredNodes) {
     result.coverage = Math.floor((100 * coveredNodes) / (totalNodes - ignoredNodes))
   } else {
     result.coverage = 0 // No "infinity" on division by zero...

package/src/plugs/coverage.ts

@@ -98,10 +98,12 @@ export class Coverage<
     const _report = run.report('Coverage report')
 
     for (const [ _file, result ] of Object.entries(report.results)) {
-      const { coverage } = result.nodeCoverage
+      const { coverage, totalNodes } = result.nodeCoverage
       const file = _file as AbsolutePath
 
-      if (coverage < minimumFileCoverage) {
+      if (totalNodes === 0) {
+        _report.annotate(NOTICE, file, 'n/a')
+      } else if (coverage < minimumFileCoverage) {
         _report.annotate(ERROR, file, `${coverage} %`)
         fileErrors ++
       } else if (coverage < optimalFileCoverage) {

package/src/plugs/mocha/reporter.ts

@@ -39,6 +39,15 @@ export class PlugReporter extends RealMocha.reporters.Base {
       log.leave()
     })
 
+    runner.on('fail', (test) => {
+      // Hooks fail here (not on "hook end")
+      if (test.type as any === 'hook') {
+        const number = failures.push(test)
+        const tag = $gry('[') + $red('failed') + $gry('] [') + $red(number) + $gry(']')
+        log.error(`${$red(_failure)} ${test.title} ${tag}`)
+      }
+    })
+
     // Enter a test (increase indent)
     runner.on('test', (test) => {
       log.enter(NOTICE, `${$blu(_pending)} ${test.title}`)