add tests

Author: Lms24

packages/astro/src/integration/index.ts

@@ -60,6 +60,7 @@ export const sentryAstro = (options: SentryOptions = {}): AstroIntegration => {
         if (pathToServerInit) {
           options.debug && console.log(`[sentry-astro] Using ${pathToServerInit} for server init.`);
           injectScript('page-ssr', buildSdkInitFileImportSnippet(pathToServerInit));
+        } else {
           options.debug && console.log('[sentry-astro] Using default server init.');
           injectScript('page-ssr', buildServerSnippet(options || {}));
         }

packages/astro/test/integration/index.test.ts

@@ -0,0 +1,104 @@
+import { vi } from 'vitest';
+
+import { sentryAstro } from '../../src/integration';
+
+const sentryVitePluginSpy = vi.fn(() => 'sentryVitePlugin');
+
+vi.mock('@sentry/vite-plugin', () => ({
+  // @ts-expect-error - just mocking around
+  sentryVitePlugin: vi.fn(args => sentryVitePluginSpy(args)),
+}));
+
+vi.mock('vite', () => ({
+  loadEnv: () => ({
+    SENTRY_AUTH_TOKEN: 'my-token',
+  }),
+}));
+
+describe('sentryAstro integration', () => {
+  afterEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it('has a name', () => {
+    const integration = sentryAstro({});
+    expect(integration.name).toBe('@sentry/astro');
+  });
+
+  it('enables source maps and adds the sentry vite plugin if an auth token is detected', async () => {
+    const integration = sentryAstro({
+      sourceMapsUploadOptions: { enabled: true, org: 'my-org', project: 'my-project', telemetry: false },
+    });
+    const updateConfig = vi.fn();
+    const injectScript = vi.fn();
+
+    expect(integration.hooks['astro:config:setup']).toBeDefined();
+    // @ts-expect-error - the hook exists and we only need to pass what we actually use
+    await integration.hooks['astro:config:setup']({ updateConfig, injectScript });
+
+    expect(updateConfig).toHaveBeenCalledTimes(1);
+    expect(updateConfig).toHaveBeenCalledWith({
+      vite: {
+        build: {
+          sourcemap: true,
+        },
+        plugins: ['sentryVitePlugin'],
+      },
+    });
+
+    expect(sentryVitePluginSpy).toHaveBeenCalledTimes(1);
+    expect(sentryVitePluginSpy).toHaveBeenCalledWith({
+      authToken: 'my-token',
+      org: 'my-org',
+      project: 'my-project',
+      telemetry: false,
+    });
+  });
+
+  it("doesn't enable source maps if `sourceMapsUploadOptions.enabled` is `false`", async () => {
+    const integration = sentryAstro({
+      sourceMapsUploadOptions: { enabled: false },
+    });
+    const updateConfig = vi.fn();
+    const injectScript = vi.fn();
+
+    expect(integration.hooks['astro:config:setup']).toBeDefined();
+    // @ts-expect-error - the hook exists and we only need to pass what we actually use
+    await integration.hooks['astro:config:setup']({ updateConfig, injectScript });
+
+    expect(updateConfig).toHaveBeenCalledTimes(0);
+    expect(sentryVitePluginSpy).toHaveBeenCalledTimes(0);
+  });
+
+  it('injects client and server init scripts', async () => {
+    const integration = sentryAstro({});
+    const updateConfig = vi.fn();
+    const injectScript = vi.fn();
+
+    expect(integration.hooks['astro:config:setup']).toBeDefined();
+    // @ts-expect-error - the hook exists and we only need to pass what we actually use
+    await integration.hooks['astro:config:setup']({ updateConfig, injectScript });
+
+    expect(injectScript).toHaveBeenCalledTimes(2);
+    expect(injectScript).toHaveBeenCalledWith('page', expect.stringContaining('Sentry.init'));
+    expect(injectScript).toHaveBeenCalledWith('page-ssr', expect.stringContaining('Sentry.init'));
+  });
+
+  it('injects client and server init scripts from custom paths', async () => {
+    const integration = sentryAstro({
+      clientInitPath: 'my-client-init-path.js',
+      serverInitPath: 'my-server-init-path.js',
+    });
+
+    const updateConfig = vi.fn();
+    const injectScript = vi.fn();
+
+    expect(integration.hooks['astro:config:setup']).toBeDefined();
+    // @ts-expect-error - the hook exists and we only need to pass what we actually use
+    await integration.hooks['astro:config:setup']({ updateConfig, injectScript });
+
+    expect(injectScript).toHaveBeenCalledTimes(2);
+    expect(injectScript).toHaveBeenCalledWith('page', expect.stringContaining('my-client-init-path.js'));
+    expect(injectScript).toHaveBeenCalledWith('page-ssr', expect.stringContaining('my-server-init-path.js'));
+  });
+});