From 10c34f82d10a53d9b0bb35d6fb9719db07ef43a2 Mon Sep 17 00:00:00 2001
From: Joe Danziger <joe@ticc.net>
Date: Sun, 11 May 2025 01:52:33 -0400
Subject: [PATCH] remove test

---
 tests/integration/mcp-server/rules.test.js | 78 ----------------------
 1 file changed, 78 deletions(-)
 delete mode 100644 tests/integration/mcp-server/rules.test.js

diff --git a/tests/integration/mcp-server/rules.test.js b/tests/integration/mcp-server/rules.test.js
deleted file mode 100644
index 5cbb1b92..00000000
--- a/tests/integration/mcp-server/rules.test.js
+++ /dev/null
@@ -1,78 +0,0 @@
-/**
- * Integration tests for the MCP server 'rules' tool (add/remove brand rules)
- */
-import { jest } from '@jest/globals';
-import { rulesDirect } from '../../../mcp-server/src/core/direct-functions/rules.js';
-import fs from 'fs';
-import path from 'path';
-
-// Mock logger
-const mockLogger = {
-	info: jest.fn(),
-	error: jest.fn(),
-	warn: jest.fn(),
-	debug: jest.fn()
-};
-
-// Use a temp directory for testing (simulate a project root)
-const tempProjectRoot = path.join(
-	__dirname,
-	'../../fixtures/temp-rules-project'
-);
-
-beforeAll(() => {
-	if (!fs.existsSync(tempProjectRoot))
-		fs.mkdirSync(tempProjectRoot, { recursive: true });
-});
-
-afterAll(() => {
-	if (fs.existsSync(tempProjectRoot))
-		fs.rmSync(tempProjectRoot, { recursive: true, force: true });
-});
-
-describe('rulesDirect (integration)', () => {
-	it('should add brand rules successfully', async () => {
-		const args = {
-			action: 'add',
-			rules: ['roo'],
-			projectRoot: tempProjectRoot,
-			yes: true
-		};
-		const result = await rulesDirect(args, mockLogger, {});
-		expect(result.success).toBe(true);
-		expect(result.data.output).toMatch(/add|roo/i);
-	});
-
-	it('should remove brand rules successfully', async () => {
-		const args = {
-			action: 'remove',
-			rules: ['roo'],
-			projectRoot: tempProjectRoot,
-			yes: true
-		};
-		const result = await rulesDirect(args, mockLogger, {});
-		expect(result.success).toBe(true);
-		expect(result.data.output).toMatch(/remove|roo/i);
-	});
-
-	it('should fail if missing required arguments', async () => {
-		const args = {
-			action: 'add',
-			rules: [], // missing brands
-			projectRoot: tempProjectRoot
-		};
-		const result = await rulesDirect(args, mockLogger, {});
-		expect(result.success).toBe(false);
-		expect(result.error.code).toBe('MISSING_ARGUMENT');
-	});
-
-	it('should fail if projectRoot is missing', async () => {
-		const args = {
-			action: 'add',
-			rules: ['roo']
-		};
-		const result = await rulesDirect(args, mockLogger, {});
-		expect(result.success).toBe(false);
-		expect(result.error.code).toBe('MISSING_ARGUMENT');
-	});
-});