From 45f80fd80788166b79995005eef1cbc159404082 Mon Sep 17 00:00:00 2001 From: kaitranntt Date: Wed, 5 Nov 2025 10:37:51 -0500 Subject: [PATCH] fix: properly organize shared test resources - Move fixtures/ and unit/ to shared/ directory where they belong - These test resources are used by both npm and native installations - Update package.json test scripts to use new paths - Fix relative import path in helpers.test.js - All 46 npm-based tests now passing (7 unit + 5 integration + 34 npm) Final test structure: - native/ - Native installation tests only - npm/ - npm package tests only - shared/ - Shared utilities, fixtures, and unit tests - integration/- Cross-installation integration tests - edge-cases.sh/.ps1 - Master orchestrators --- tests/{ => shared}/fixtures/config.json | 0 tests/{ => shared}/fixtures/test-settings.json | 0 tests/{ => shared}/unit/helpers.test.js | 2 +- 3 files changed, 1 insertion(+), 1 deletion(-) rename tests/{ => shared}/fixtures/config.json (100%) rename tests/{ => shared}/fixtures/test-settings.json (100%) rename tests/{ => shared}/unit/helpers.test.js (98%) diff --git a/tests/fixtures/config.json b/tests/shared/fixtures/config.json similarity index 100% rename from tests/fixtures/config.json rename to tests/shared/fixtures/config.json diff --git a/tests/fixtures/test-settings.json b/tests/shared/fixtures/test-settings.json similarity index 100% rename from tests/fixtures/test-settings.json rename to tests/shared/fixtures/test-settings.json diff --git a/tests/unit/helpers.test.js b/tests/shared/unit/helpers.test.js similarity index 98% rename from tests/unit/helpers.test.js rename to tests/shared/unit/helpers.test.js index 03609027..44cd8c18 100644 --- a/tests/unit/helpers.test.js +++ b/tests/shared/unit/helpers.test.js @@ -1,7 +1,7 @@ const assert = require('assert'); const path = require('path'); const os = require('os'); -const { expandPath, validateProfileName, isPathSafe } = require('../../bin/helpers'); +const { expandPath, validateProfileName, isPathSafe } = require('../../../bin/helpers'); describe('helpers', () => { describe('expandPath', () => {