Merge pull request #707 from TEEZY234/add-csv-export-tests-692

test(frontend): add csvExport unit tests and update Vitest discovery …

Author: ogazboiz

frontend/src/components/streamCreationForm.test.tsx

@@ -1,5 +1,10 @@
 import { render, screen, fireEvent } from "@testing-library/react";
 import { test, expect, vi } from "vitest";
+
+vi.mock("next/navigation", () => ({
+  useRouter: vi.fn(() => ({ push: vi.fn() })),
+}));
+
 import { StreamCreationWizard } from "./stream-creation/StreamCreationWizard";
 import React from "react";
 
@@ -16,9 +21,9 @@ test("StreamCreationWizard — validation errors shown", () => {
   );
 
   // The wizard starts at Template step. We need to go to Recipient step to test validation.
-  // Actually, let's just click 'Next' and see if it goes to next step or shows errors if any.
-  fireEvent.click(screen.getByText(/next/i));
+  // Click the Next button explicitly instead of matching incidental text.
+  fireEvent.click(screen.getByRole('button', { name: /next/i }));
 
   // It should move to Recipient step.
-  expect(screen.getByText(/recipient/i)).toBeInTheDocument();
+  expect(screen.getByRole('heading', { name: /recipient address/i })).toBeInTheDocument();
 });

frontend/src/utils/csvExport.test.ts

@@ -0,0 +1,114 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { convertArrayToCSV, downloadCSV, escapeCsvCell } from './csvExport';
+
+describe('escapeCsvCell', () => {
+  it('returns empty string for null', () => {
+    expect(escapeCsvCell(null)).toBe('');
+  });
+
+  it('returns empty string for undefined', () => {
+    expect(escapeCsvCell(undefined)).toBe('');
+  });
+
+  it('returns plain strings unchanged', () => {
+    expect(escapeCsvCell('hello')).toBe('hello');
+  });
+
+  it('quotes values containing commas', () => {
+    expect(escapeCsvCell('hello,world')).toBe('"hello,world"');
+  });
+
+  it('quotes values containing newlines', () => {
+    expect(escapeCsvCell('hello\nworld')).toBe('"hello\nworld"');
+  });
+
+  it('doubles quotes inside values', () => {
+    expect(escapeCsvCell('She said "Hi"')).toBe('"She said ""Hi"""');
+  });
+
+  it('formats Date values via toLocaleString', () => {
+    const original = Date.prototype.toLocaleString;
+    Date.prototype.toLocaleString = function () {
+      return '2026-06-01 12:00:00';
+    };
+
+    try {
+      expect(escapeCsvCell(new Date('2026-06-01T12:00:00Z'))).toBe('2026-06-01 12:00:00');
+    } finally {
+      Date.prototype.toLocaleString = original;
+    }
+  });
+});
+
+describe('convertArrayToCSV', () => {
+  it('returns an empty string for an empty array', () => {
+    expect(convertArrayToCSV([])).toBe('');
+  });
+
+  it('serializes a single row correctly', () => {
+    const rows = [{ name: 'Alice', amount: 10 }];
+    expect(convertArrayToCSV(rows)).toBe('name,amount\nAlice,10');
+  });
+
+  it('serializes multiple rows with mixed values', () => {
+    const rows = [
+      { id: 1, value: 'hello,world' },
+      { id: 2, value: 'line\nbreak' },
+      { id: 3, value: 'quote"test' },
+    ];
+
+    expect(convertArrayToCSV(rows)).toBe(
+      'id,value\n1,"hello,world"\n2,"line\nbreak"\n3,"quote""test"'
+    );
+  });
+
+  it('preserves header order from the first row', () => {
+    const rows = [
+      { b: 1, a: 'first' },
+      { b: 2, a: 'second' },
+    ];
+
+    expect(convertArrayToCSV(rows)).toBe('b,a\n1,first\n2,second');
+  });
+});
+
+describe('downloadCSV', () => {
+  let originalURL: typeof URL;
+
+  beforeEach(() => {
+    originalURL = globalThis.URL;
+    document.body.innerHTML = '';
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+    Object.defineProperty(globalThis, 'URL', {
+      configurable: true,
+      writable: true,
+      value: originalURL,
+    });
+  });
+
+  it('creates a Blob and triggers a download', () => {
+    const createObjectURL = vi.fn(() => 'blob://test-url');
+    const revokeObjectURL = vi.fn();
+
+    Object.defineProperty(globalThis, 'URL', {
+      configurable: true,
+      writable: true,
+      value: {
+        ...globalThis.URL,
+        createObjectURL,
+        revokeObjectURL,
+      } as unknown as typeof URL,
+    });
+
+    const clickSpy = vi.spyOn(HTMLAnchorElement.prototype, 'click').mockImplementation(() => {});
+    downloadCSV([{ foo: 'bar' }], 'test.csv');
+
+    expect(createObjectURL).toHaveBeenCalledTimes(1);
+    expect(clickSpy).toHaveBeenCalledTimes(1);
+    expect(revokeObjectURL).toHaveBeenCalledWith('blob://test-url');
+    expect(document.querySelector('a[download="test.csv"]')).toBeNull();
+  });
+});

frontend/src/utils/csvExport.ts

@@ -2,7 +2,7 @@
 
 type CsvRow = Record<string, unknown>;
 
-const escapeCsvCell = (value: unknown): string => {
+export const escapeCsvCell = (value: unknown): string => {
     if (value === null || value === undefined) {
         return '';
     }

frontend/vitest.config.ts

@@ -7,7 +7,7 @@ export default defineConfig({
     environment: 'happy-dom',
     globals: true,
     setupFiles: ['./src/__tests__/setup.ts'],
-    include: ['src/__tests__/**/*.{test,spec}.{ts,tsx}'],
+    include: ['src/__tests__/**/*.{test,spec}.{ts,tsx}', 'src/**/*.{test,spec}.{ts,tsx}'],
     coverage: {
       reporter: ['text', 'json', 'html'],
       include: ['src/utils/**', 'src/hooks/**', 'src/components/**'],