fix(nuxt): Use absolute path for client config

getsentry · Sep 25, 2024 · a3b5dca · a3b5dca
1 parent 4ce48e9
commit a3b5dca
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/packages/nuxt/src/vite/utils.ts b/packages/nuxt/src/vite/utils.ts
@@ -24,5 +24,5 @@ export function findDefaultSdkInitFile(type: 'server' | 'client'): string | unde
 
   const filePath = filePaths.find(filename => fs.existsSync(filename));
 
-  return filePath ? path.basename(filePath) : undefined;
+  return filePath ? filePath : undefined;
 }
diff --git a/packages/nuxt/test/vite/utils.test.ts b/packages/nuxt/test/vite/utils.test.ts
@@ -10,26 +10,26 @@ describe('findDefaultSdkInitFile', () => {
   });
 
   it.each(['ts', 'js', 'mjs', 'cjs', 'mts', 'cts'])(
-    'should return the server file with .%s extension if it exists',
+    'should return the server file path with .%s extension if it exists',
     ext => {
       vi.spyOn(fs, 'existsSync').mockImplementation(filePath => {
         return !(filePath instanceof URL) && filePath.includes(`sentry.server.config.${ext}`);
       });
 
       const result = findDefaultSdkInitFile('server');
-      expect(result).toBe(`sentry.server.config.${ext}`);
+      expect(result).toMatch(`packages/nuxt/sentry.server.config.${ext}`);
     },
   );
 
   it.each(['ts', 'js', 'mjs', 'cjs', 'mts', 'cts'])(
-    'should return the client file with .%s extension if it exists',
+    'should return the client file path with .%s extension if it exists',
     ext => {
       vi.spyOn(fs, 'existsSync').mockImplementation(filePath => {
         return !(filePath instanceof URL) && filePath.includes(`sentry.client.config.${ext}`);
       });
 
       const result = findDefaultSdkInitFile('client');
-      expect(result).toBe(`sentry.client.config.${ext}`);
+      expect(result).toMatch(`packages/nuxt/sentry.client.config.${ext}`);
     },
   );
 
@@ -47,7 +47,7 @@ describe('findDefaultSdkInitFile', () => {
     expect(result).toBeUndefined();
   });
 
-  it('should return the server config file if server.config and instrument exist', () => {
+  it('should return the server config file path if server.config and instrument exist', () => {
     vi.spyOn(fs, 'existsSync').mockImplementation(filePath => {
       return (
         !(filePath instanceof URL) &&
@@ -56,6 +56,6 @@ describe('findDefaultSdkInitFile', () => {
     });
 
     const result = findDefaultSdkInitFile('server');
-    expect(result).toBe('sentry.server.config.js');
+    expect(result).toMatch('packages/nuxt/sentry.server.config.js');
   });
 });