fix relative path

facebook · Jan 14, 2025 · 6a77536 · 6a77536
1 parent 0f630a7
commit 6a77536
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 26 deletions.
diff --git a/packages/cli/__tests__/compile-stylex-folder-test.js b/packages/cli/__tests__/compile-stylex-folder-test.js
@@ -258,6 +258,11 @@ describe('cache mechanism works as expected', () => {
       copiedNodeModules: false,
     },
   };
+
+  beforeAll(async () => {
+    await fs.rm(cachePath, { recursive: true, force: true });
+  });
+
   beforeEach(() => {
     writeSpy = jest.spyOn(cacheModule, 'writeCache');
   });
@@ -327,7 +332,8 @@ describe('cache mechanism works as expected', () => {
 
 describe('CLI works with a custom cache path', () => {
   let writeSpy;
-  const customCachePath = path.join(__dirname, '__custom_cache__');
+  const projectRoot = path.resolve(__dirname, '../../../');
+  const customCachePath = path.join(projectRoot, '__custom_cache__');
   const config: TransformConfig = {
     input: path.resolve('./source'),
     output: path.resolve('./src'),
@@ -369,31 +375,20 @@ describe('CLI works with a custom cache path', () => {
   test('uses the custom cache path for caching', async () => {
     await compileDirectory(config);
 
-    const customFilePath = path.join(config.input, 'index.js');
-
-    const cacheFilePath = path.join(
-      customCachePath,
-      path.relative(config.input, customFilePath) + '.json',
-    );
-
-    expect(
-      await fs
-        .access(customCachePath)
-        .then(() => true)
-        .catch(() => false),
-    ).toBe(true);
-    expect(
-      await fs
-        .access(cacheFilePath)
-        .then(() => true)
-        .catch(() => false),
-    ).toBe(true);
+    const cacheFiles = await fs.readdir(customCachePath);
+    expect(cacheFiles.length).toEqual(3);
 
-    const cacheData = JSON.parse(await fs.readFile(cacheFilePath, 'utf-8'));
-    expect(cacheData).toHaveProperty('inputHash');
-    expect(cacheData).toHaveProperty('outputHash');
-    expect(cacheData).toHaveProperty('collectedCSS');
+    for (const cacheFile of cacheFiles) {
+      const cacheFilePath = path.join(customCachePath, cacheFile);
+      const cacheContent = JSON.parse(
+        await fs.readFile(cacheFilePath, 'utf-8'),
+      );
+      expect(cacheContent).toHaveProperty('inputHash');
+      expect(cacheContent).toHaveProperty('outputHash');
+      expect(cacheContent).toHaveProperty('collectedCSS');
+    }
   });
+
   test('skips transformation when cache is valid', async () => {
     await compileDirectory(config);
 

diff --git a/packages/cli/src/cache.js b/packages/cli/src/cache.js
@@ -18,8 +18,9 @@ export function getDefaultCachePath() {
 }
 
 export async function getCacheFilePath(cachePath, filePath) {
-  const projectRoot = path.join(__dirname, '../../../');
-  const relativePath = path.relative(projectRoot, filePath);
+  const projectRoot = path.resolve(__dirname, '../../../');
+  const absoluteFilePath = path.resolve(filePath);
+  const relativePath = path.relative(projectRoot, absoluteFilePath);
   const fileName = relativePath.replace(/[\\/]/g, '__');
   return path.join(cachePath, `${fileName}.json`);
 }