fix: resolve data race in snapshot directory across concurrent tests

Author: diegosantdev

src/differ.js

@@ -1,22 +1,25 @@
 const fs = require('fs').promises;
 const path = require('path');
 
-const SNAPSHOTS_DIR = '.licensepulse/snapshots';
+function getSnapshotsDir() {
+  return process.env.LICENSEPULSE_SNAPSHOTS_DIR || '.licensepulse/snapshots';
+}
 
 function getSnapshotFilename(repoId) {
   return repoId.replace('/', '-') + '.json';
 }
 
 function getSnapshotPath(repoId) {
-  return path.join(SNAPSHOTS_DIR, getSnapshotFilename(repoId));
+  return path.join(getSnapshotsDir(), getSnapshotFilename(repoId));
 }
 
 async function ensureSnapshotsDir() {
   try {
     const path = require('path');
-    const baseDir = path.dirname(SNAPSHOTS_DIR);
+    const dir = getSnapshotsDir();
+    const baseDir = path.dirname(dir);
     await fs.mkdir(baseDir, { recursive: true });
-    await fs.mkdir(SNAPSHOTS_DIR, { recursive: true });
+    await fs.mkdir(dir, { recursive: true });
   } catch (error) {
     // Ignore errors if directory already exists
   }
@@ -57,12 +60,13 @@ async function saveSnapshot(repoId, licenseData) {
 async function getAllSnapshots() {
   try {
     await ensureSnapshotsDir();
-    const files = await fs.readdir(SNAPSHOTS_DIR);
+    const dir = getSnapshotsDir();
+    const files = await fs.readdir(dir);
 
     const snapshots = [];
     for (const file of files) {
       if (file.endsWith('.json')) {
-        const filePath = path.join(SNAPSHOTS_DIR, file);
+        const filePath = path.join(dir, file);
         const data = await fs.readFile(filePath, 'utf8');
         snapshots.push(JSON.parse(data));
       }

tests/integration/workflow.test.js

@@ -1,3 +1,4 @@
+process.env.LICENSEPULSE_SNAPSHOTS_DIR = '.licensepulse/snapshots-integration';
 const Watchlist = require('../../src/watchlist');
 const { fetchLicense } = require('../../src/watcher');
 const { checkAndUpdate, loadSnapshot } = require('../../src/differ');
@@ -9,7 +10,7 @@ const axios = require('axios');
 
 describe('Integration: Complete Workflow', () => {
   const testWatchlistPath = 'test-integration-watchlist.json';
-  const snapshotsDir = '.licensepulse/snapshots';
+  const snapshotsDir = process.env.LICENSEPULSE_SNAPSHOTS_DIR;
 
   beforeAll(() => {
     process.env.GITHUB_TOKEN = 'test-token';

tests/unit/differ.test.js

@@ -1,3 +1,4 @@
+process.env.LICENSEPULSE_SNAPSHOTS_DIR = '.licensepulse/snapshots-unit';
 const fs = require('fs').promises;
 const path = require('path');
 const {
@@ -9,7 +10,7 @@ const {
   getSnapshotPath
 } = require('../../src/differ');
 
-const SNAPSHOTS_DIR = '.licensepulse/snapshots';
+const SNAPSHOTS_DIR = process.env.LICENSEPULSE_SNAPSHOTS_DIR;
 
 describe('Differ', () => {
   beforeEach(async () => {