@joystick.js/cli-canary 0.0.0-canary.122 → 0.0.0-canary.124

package/dist/lib/dev/runTests.js

@@ -50,8 +50,7 @@ const handleAvaSTDIO = (ava = {}, options = {}) => {
 };
 const runAva = (options = {}) => {
   try {
-    const packageRoot = options?.__dirname?.split("/dist")[0];
-    const avaPath = `${packageRoot}/node_modules/.bin/ava`;
+    const avaPath = `${process.cwd()}/node_modules/.bin/ava`;
     return new Promise((resolve, reject) => {
       const ava = child_process.exec(`${avaPath} --config ${options?.__dirname}/tests.config.js`, {
         stdio: "inherit",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@joystick.js/cli-canary",
-  "version": "0.0.0-canary.122",
+  "version": "0.0.0-canary.124",
   "type": "module",
   "description": "CLI for the Joystick JavaScript framework.",
   "main": "development.js",

package/src/lib/dev/runTests.js

@@ -64,13 +64,17 @@ const handleAvaSTDIO = (ava = {}, options = {}) => {
 
 const runAva = (options = {}) => {
   try {
-    // NOTE: __dirname includes the entire path to lib/dev/index.js but we only
-    // care about the root folder/package here (we need to reference the node_modules
-    // folder in the package).
-    const packageRoot = options?.__dirname?.split('/dist')[0];
-    const avaPath = `${packageRoot}/node_modules/.bin/ava`;
+    // NOTE: A little bananas to reason through this. In order for Ava to run w/o errors,
+    // the Ava binary being run here has to be identical to the one used in @joystick.js/test.
+    // That would equal the copy of Ava that's installed in a Joystick app's node_modules
+    // directory, not the node_modules directory of the CLI here. We can guarantee that will
+    // exist for the CLI here because a developer has to install @joystick.js/test which will
+    // add Ava as a dependency to their app in order to write tests.
+    const avaPath = `${process.cwd()}/node_modules/.bin/ava`;
     
     return new Promise((resolve, reject) => {
+      // NOTE: Despite using the app's node_modules path to reference Ava, we still want to reference
+      // the internal path here for the default test config in /lib/dev/tests.config.js.
       const ava = child_process.exec(`${avaPath} --config ${options?.__dirname}/tests.config.js`, {
         stdio: 'inherit',
         env: {