From 438379f91382b120456057b56c27934dcb677d02 Mon Sep 17 00:00:00 2001
From: Dave Oster <dave@dealmachine.com>
Date: Sat, 10 Oct 2026 08:59:32 -0500
Subject: [PATCH] Prune evidenced redundant tests for staging review

---
 tests/lib/config.test.ts | 42 +---------------------------------------
 tests/lib/output.test.ts | 15 +-------------
 2 files changed, 2 insertions(+), 55 deletions(-)

diff --git a/tests/lib/config.test.ts b/tests/lib/config.test.ts
index acfa9f2..6b03a73 100644
--- a/tests/lib/config.test.ts
+++ b/tests/lib/config.test.ts
@@ -16,7 +16,7 @@ vi.mock('node:child_process', () => ({
   }),
 }));
 
-import { readConfig, writeConfig, deleteConfig, configExists, getConfigValue, setConfigValue } from '../../src/lib/config';
+import { readConfig, writeConfig, deleteConfig, setConfigValue } from '../../src/lib/config';
 
 const CONFIG_DIR = path.join('/tmp/test-home', '.dealmachine');
 const CONFIG_FILE = path.join(CONFIG_DIR, 'config.json');
@@ -85,20 +85,6 @@ describe('CLI config', () => {
       expect(writtenConfig.credentialStore).toBe('encrypted-file');
       expect(writtenConfig.credentialFile).toBe(CREDENTIAL_FILE);
     });
-
-    it('skips mkdir when directory exists', () => {
-      vi.mocked(fs.existsSync).mockReturnValue(true);
-      writeConfig(sampleConfig);
-      expect(fs.mkdirSync).not.toHaveBeenCalled();
-    });
-
-    it('falls back when the OS credential store is unavailable', () => {
-      vi.mocked(fs.existsSync).mockImplementation((file) => file === CONFIG_DIR);
-
-      writeConfig(sampleConfig);
-
-      expect(getWrittenConfig().credentialStore).toBe('encrypted-file');
-    });
   });
 
   describe('deleteConfig', () => {
@@ -114,32 +100,6 @@ describe('CLI config', () => {
     });
   });
 
-  describe('configExists', () => {
-    it('returns true when file exists', () => {
-      vi.mocked(fs.existsSync).mockReturnValue(true);
-      expect(configExists()).toBe(true);
-    });
-
-    it('returns false when file does not exist', () => {
-      vi.mocked(fs.existsSync).mockReturnValue(false);
-      expect(configExists()).toBe(false);
-    });
-  });
-
-  describe('getConfigValue', () => {
-    it('returns specific config value', () => {
-      vi.mocked(fs.existsSync).mockReturnValue(true);
-      vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify(sampleConfig));
-      expect(getConfigValue('apiKey')).toBe('dm_sk_live_test123');
-      expect(getConfigValue('organizationId')).toBe(7);
-    });
-
-    it('returns null when no config', () => {
-      vi.mocked(fs.existsSync).mockReturnValue(false);
-      expect(getConfigValue('apiKey')).toBeNull();
-    });
-  });
-
   describe('setConfigValue', () => {
     it('updates a single config value', () => {
       vi.mocked(fs.existsSync).mockReturnValue(true);
diff --git a/tests/lib/output.test.ts b/tests/lib/output.test.ts
index d0113f0..634d703 100644
--- a/tests/lib/output.test.ts
+++ b/tests/lib/output.test.ts
@@ -9,7 +9,7 @@ vi.mock('chalk', () => {
   return { default: new Proxy({}, handler) };
 });
 
-import { truncate, printTable, printJson, printKeyValue } from '../../src/lib/output';
+import { truncate, printTable, printJson } from '../../src/lib/output';
 
 describe('output helpers', () => {
   beforeEach(() => {
@@ -45,12 +45,6 @@ describe('output helpers', () => {
       expect(console.log).toHaveBeenCalledWith(expect.stringContaining('no results'));
     });
 
-    it('prints header and rows', () => {
-      printTable([{ name: 'Alice', age: 30 }], ['name', 'age']);
-      // At least header + separator + 1 row = 3 calls
-      expect(vi.mocked(console.log).mock.calls.length).toBeGreaterThanOrEqual(3);
-    });
-
     it('auto-detects columns from first row', () => {
       printTable([{ x: 1, y: 2 }]);
       const allOutput = vi.mocked(console.log).mock.calls.map(c => c[0]).join('\n');
@@ -58,11 +52,4 @@ describe('output helpers', () => {
       expect(allOutput).toContain('y');
     });
   });
-
-  describe('printKeyValue', () => {
-    it('prints key-value pairs', () => {
-      printKeyValue({ name: 'Test', count: 42 });
-      expect(vi.mocked(console.log).mock.calls.length).toBe(2);
-    });
-  });
 });