Merge pull request #5232 from Agoric/ta/swingset-peer-deps

chore(swingset-vat): declare deps of testing tools
Agoric · Apr 28, 2022 · 4723487 · 4723487
2 parents cb701ec + 7a6c796
commit 4723487
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 10 deletions.
diff --git a/packages/SwingSet/package.json b/packages/SwingSet/package.json
@@ -22,9 +22,7 @@
   },
   "devDependencies": {
     "@agoric/vat-data": "^0.2.0",
-    "@endo/ses-ava": "^0.2.25",
     "@types/tmp": "^0.2.0",
-    "ava": "^3.12.1",
     "tmp": "^0.2.1"
   },
   "dependencies": {
@@ -50,6 +48,10 @@
     "node-lmdb": "^0.9.5",
     "semver": "^6.3.0"
   },
+  "peerDependencies": {
+    "@endo/ses-ava": "^0.2.25",
+    "ava": "^3.12.1"
+  },
   "files": [
     "bin/vat",
     "src/**/*.js",

diff --git a/packages/SwingSet/tools/prepare-test-env-ava.js b/packages/SwingSet/tools/prepare-test-env-ava.js
@@ -8,17 +8,9 @@ import '@endo/init/pre-bundle-source.js';
 
 import './prepare-test-env.js';
 
-// eslint-disable-next-line import/no-extraneous-dependencies
 import '@endo/ses-ava/exported.js';
 
-// eslint thinks these are extraneous dependencies because this file
-// is in the tools/ directory rather than the test/ directory.
-// TODO How do we tell eslint that tools/ is dev-only? Either
-// that, or should we just move tools/* into test/ ?
-//
-// eslint-disable-next-line import/no-extraneous-dependencies
 import { wrapTest } from '@endo/ses-ava';
-// eslint-disable-next-line import/no-extraneous-dependencies
 import rawTest from 'ava';
 
 /** @type {typeof rawTest} */