added unit tests

Author: barrymichaeldoyle

src/ui/display/__tests__/help.test.ts

@@ -0,0 +1,31 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { displayHelp } from '../help';
+import * as madeWithLove from '../madeWithLove';
+
+describe('displayHelp', () => {
+  let logSpy: ReturnType<typeof vi.spyOn>;
+  let madeWithLoveSpy: ReturnType<typeof vi.spyOn>;
+
+  beforeEach(() => {
+    logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+    madeWithLoveSpy = vi
+      .spyOn(madeWithLove, 'displayMadeWithLove')
+      .mockImplementation(() => {});
+  });
+
+  afterEach(() => {
+    logSpy.mockRestore();
+    madeWithLoveSpy.mockRestore();
+  });
+
+  it('prints the help message and calls displayMadeWithLove', () => {
+    displayHelp();
+    expect(logSpy).toHaveBeenCalled();
+    const output = logSpy.mock.calls.map(call => call[0]).join('\n');
+    expect(output).toContain('Patch Pulse CLI');
+    expect(output).toContain('Usage:');
+    expect(output).toContain('Options:');
+    expect(output).toContain('Configuration File:');
+    expect(madeWithLoveSpy).toHaveBeenCalled();
+  });
+});

src/ui/display/__tests__/license.test.ts

@@ -0,0 +1,63 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { displayLicense } from '../license';
+
+vi.mock('fs', () => ({
+  readFileSync: vi.fn(),
+}));
+
+describe('displayLicense', () => {
+  let logSpy: ReturnType<typeof vi.spyOn>;
+  let errorSpy: ReturnType<typeof vi.spyOn>;
+
+  beforeEach(() => {
+    logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+    errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+  });
+
+  afterEach(() => {
+    logSpy.mockRestore();
+    errorSpy.mockRestore();
+    vi.clearAllMocks();
+  });
+
+  it('prints the license content when LICENSE file is present', async () => {
+    const fs = await import('fs');
+    vi.mocked(fs.readFileSync).mockReturnValue('MIT License\nCopyright 2024');
+    const exitSpy = vi.spyOn(process, 'exit').mockImplementation(() => {
+      throw new Error('exit');
+    });
+    try {
+      displayLicense();
+    } catch {
+      // expected if process.exit is called
+    }
+    expect(fs.readFileSync).toHaveBeenCalled();
+    expect(logSpy).toHaveBeenCalled();
+    const output = logSpy.mock.calls.map(call => call[0]).join('\n');
+    expect(output).toContain('License');
+    expect(output).toContain('MIT License');
+    expect(output).toContain('Copyright 2024');
+    expect(errorSpy).not.toHaveBeenCalled();
+    exitSpy.mockRestore();
+  });
+
+  it('prints an error and fallback license if LICENSE file is missing', async () => {
+    const fs = await import('fs');
+    vi.mocked(fs.readFileSync).mockImplementation(() => {
+      throw new Error('File not found');
+    });
+    const exitSpy = vi.spyOn(process, 'exit').mockImplementation(() => {
+      throw new Error('exit');
+    });
+    try {
+      displayLicense();
+    } catch {
+      // expected due to process.exit mock
+    }
+    expect(errorSpy).toHaveBeenCalled();
+    expect(logSpy).toHaveBeenCalledWith(
+      expect.stringContaining('License: MIT')
+    );
+    exitSpy.mockRestore();
+  });
+});

src/ui/display/__tests__/madeWithLove.test.ts

@@ -0,0 +1,23 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { displayMadeWithLove } from '../madeWithLove';
+
+describe('displayMadeWithLove', () => {
+  let logSpy: ReturnType<typeof vi.spyOn>;
+
+  beforeEach(() => {
+    logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+  });
+
+  afterEach(() => {
+    logSpy.mockRestore();
+  });
+
+  it('prints the made with love message', () => {
+    displayMadeWithLove();
+    expect(logSpy).toHaveBeenCalledTimes(2);
+    const calls = logSpy.mock.calls.map(call => call[0]);
+    expect(calls[0]).toContain('─'.repeat(40));
+    expect(calls[1]).toContain('Made with ❤️  by');
+    expect(calls[1]).toContain('Barry Michael Doyle');
+  });
+});

src/ui/display/__tests__/thankYouMessage.test.ts

@@ -0,0 +1,24 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { displayThankYouMessage } from '../thankYouMessage';
+
+describe('displayThankYouMessage', () => {
+  let logSpy: ReturnType<typeof vi.spyOn>;
+
+  beforeEach(() => {
+    logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+  });
+
+  afterEach(() => {
+    logSpy.mockRestore();
+  });
+
+  it('prints the thank you message and help info', () => {
+    displayThankYouMessage();
+    expect(logSpy).toHaveBeenCalledTimes(3);
+    const calls = logSpy.mock.calls.map(call => call[0]);
+    expect(calls[0]).toBeUndefined(); // Empty line (console.log() with no arguments)
+    expect(calls[1]).toContain('🎉 Thank you for using Patch Pulse CLI!');
+    expect(calls[2]).toContain('💡 For more info:');
+    expect(calls[2]).toContain('npx patch-pulse --help');
+  });
+});

src/ui/display/__tests__/unknownArguments.test.ts

@@ -0,0 +1,47 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { displayUnknownArguments } from '../unknownArguments';
+
+describe('displayUnknownArguments', () => {
+  let errorSpy: ReturnType<typeof vi.spyOn>;
+  let logSpy: ReturnType<typeof vi.spyOn>;
+
+  beforeEach(() => {
+    errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+    logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+  });
+
+  afterEach(() => {
+    errorSpy.mockRestore();
+    logSpy.mockRestore();
+  });
+
+  it('displays unknown command error and help suggestions', () => {
+    displayUnknownArguments(['--unknown', '--flag']);
+    expect(errorSpy).toHaveBeenCalledWith(
+      expect.stringContaining('❌ Unknown command:')
+    );
+    expect(errorSpy).toHaveBeenCalledWith(
+      expect.stringContaining('--unknown --flag')
+    );
+    expect(logSpy).toHaveBeenCalled();
+    const output = logSpy.mock.calls.map(call => call[0]).join('\n');
+    expect(output).toContain('Available commands:');
+    expect(output).toContain('npx patch-pulse');
+    expect(output).toContain('npx patch-pulse --help');
+    expect(output).toContain('Configuration options:');
+    expect(output).toContain('For more information:');
+  });
+
+  it('handles single unknown argument', () => {
+    displayUnknownArguments(['--invalid']);
+    expect(errorSpy).toHaveBeenCalledWith(expect.stringContaining('--invalid'));
+  });
+
+  it('handles empty array of unknown arguments', () => {
+    displayUnknownArguments([]);
+    expect(errorSpy).toHaveBeenCalledWith(
+      expect.stringContaining('❌ Unknown command:')
+    );
+    expect(errorSpy).toHaveBeenCalledWith(expect.stringContaining(''));
+  });
+});

src/ui/display/__tests__/updateAvailable.test.ts

@@ -0,0 +1,41 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { displayUpdateAvailable } from '../updateAvailable';
+
+describe('displayUpdateAvailable', () => {
+  let logSpy: ReturnType<typeof vi.spyOn>;
+
+  beforeEach(() => {
+    logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+  });
+
+  afterEach(() => {
+    logSpy.mockRestore();
+  });
+
+  it('displays update available message with version information', () => {
+    displayUpdateAvailable('1.0.0', '2.0.0');
+    expect(logSpy).toHaveBeenCalled();
+    const output = logSpy.mock.calls.map(call => call[0]).join('\n');
+    expect(output).toContain('═'.repeat(50));
+    expect(output).toContain('🚀 UPDATE AVAILABLE!');
+    expect(output).toContain('Current Version:');
+    expect(output).toContain('1.0.0');
+    expect(output).toContain('Latest Version:');
+    expect(output).toContain('2.0.0');
+    expect(output).toContain('To update, run:');
+    expect(output).toContain('npx patch-pulse@latest');
+  });
+
+  it('handles different version formats', () => {
+    displayUpdateAvailable('1.2.3-beta.1', '1.2.3');
+    const output = logSpy.mock.calls.map(call => call[0]).join('\n');
+    expect(output).toContain('1.2.3-beta.1');
+    expect(output).toContain('1.2.3');
+  });
+
+  it('displays the correct number of lines', () => {
+    displayUpdateAvailable('1.0.0', '2.0.0');
+    // Should have 8 console.log calls: separator, title, separator, current version, latest version, update instruction, command, separator
+    expect(logSpy).toHaveBeenCalledTimes(8);
+  });
+});

src/ui/display/__tests__/version.test.ts

@@ -0,0 +1,40 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import * as madeWithLove from '../madeWithLove';
+import { displayVersion } from '../version';
+
+describe('displayVersion', () => {
+  let logSpy: ReturnType<typeof vi.spyOn>;
+  let madeWithLoveSpy: ReturnType<typeof vi.spyOn>;
+
+  beforeEach(() => {
+    logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+    madeWithLoveSpy = vi
+      .spyOn(madeWithLove, 'displayMadeWithLove')
+      .mockImplementation(() => {});
+  });
+
+  afterEach(() => {
+    logSpy.mockRestore();
+    madeWithLoveSpy.mockRestore();
+  });
+
+  it('prints version information and calls displayMadeWithLove', () => {
+    displayVersion();
+    expect(logSpy).toHaveBeenCalled();
+    const output = logSpy.mock.calls.map(call => call[0]).join('\n');
+    expect(output).toContain('Patch Pulse CLI');
+    expect(output).toContain('Version:');
+    expect(output).toContain('Author:');
+    expect(output).toContain('Repo:');
+    expect(output).toContain('License:');
+    expect(output).toContain('MIT');
+    expect(output).toContain('https://github.com/PatchPulse/cli');
+    expect(madeWithLoveSpy).toHaveBeenCalled();
+  });
+
+  it('displays the correct number of lines', () => {
+    displayVersion();
+    // Should have 1 console.log call for the version info, plus displayMadeWithLove will add more
+    expect(logSpy).toHaveBeenCalledTimes(1);
+  });
+});