remove spec from .mocharc.json (observablehq#978)

mbostock · web-flow · commit 845798a2e77f · 2022-07-12T10:45:07.000-04:00
* remove spec from .mocharc.json

* Update CONTRIBUTING.md
diff --git a/.mocharc.json b/.mocharc.json
@@ -1,14 +1,5 @@
 {
   "require": ["module-alias/register.js"],
   "loader": "./loader.js",
-  "extensions": ["js", "ts"],
-  "spec": [
-    "test/**/*-test.*",
-    "test/plot.js"
-  ],
-  "watch-files": [
-    "bundle.js",
-    "test",
-    "src"
-  ]
+  "extensions": ["js", "ts"]
 }
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -50,6 +50,12 @@ it("formatMonth(locale, format) does the right thing", () => {
 
 Plot&rsquo;s unit tests are written with [Mocha](https://mochajs.org).
 
+If you like, you can also run Mocha in watch mode for a specific file, so that unit tests re-run automatically when you make changes. For example:
+
+```bash
+yarn run mocha --conditions=mocha --parallel --watch test/marks/bar-test.js
+```
+
 ### Snapshot tests
 
 Snapshot tests live in `test/plots` and are registered in [`test/plots/index.js`](./test/plots/index.js); see [`test/plots/aapl-bollinger.js`](./test/plots/aapl-bollinger.js) for example. Unlike unit tests which only test individual methods, snapshot tests actually visualize data&mdash;they&rsquo;re more representative of how we expect people will use Plot. Snapshot tests can also serve as examples of how to use the Plot API, though note that some of the examples intentionally test edge case of the API and may not embody best practices. Each snapshot test defines a plot by exporting a default async function. For example, here&rsquo;s a line chart using BLS unemployment data:
diff --git a/package.json b/package.json
@@ -27,7 +27,7 @@
   ],
   "scripts": {
     "test": "yarn test:typecheck && yarn test:lint && yarn test:mocha",
-    "test:mocha": "mkdir -p test/output && mocha --conditions=mocha",
+    "test:mocha": "mkdir -p test/output && mocha --conditions=mocha 'test/**/*-test.*' 'test/plot.js'",
     "test:lint": "eslint src test",
     "test:typecheck": "tsc --noEmit",
     "prepublishOnly": "rm -rf dist && rollup -c && tsc",

-Original file line number
+Diff line change
 Plot’s unit tests are written with [Mocha](https://mochajs.org).
 +If you like, you can also run Mocha in watch mode for a specific file, so that unit tests re-run automatically when you make changes. For example:
++
 +```bash
 +yarn run mocha --conditions=mocha --parallel --watch test/marks/bar-test.js
 +```
++
 ### Snapshot tests
 Snapshot tests live in `test/plots` and are registered in [`test/plots/index.js`](./test/plots/index.js); see [`test/plots/aapl-bollinger.js`](./test/plots/aapl-bollinger.js) for example. Unlike unit tests which only test individual methods, snapshot tests actually visualize data—they’re more representative of how we expect people will use Plot. Snapshot tests can also serve as examples of how to use the Plot API, though note that some of the examples intentionally test edge case of the API and may not embody best practices. Each snapshot test defines a plot by exporting a default async function. For example, here’s a line chart using BLS unemployment data: