brave-real-browser-mcp-server 2.9.1 → 2.9.3

package/dist/handlers/navigation-handlers.test.js

@@ -9,6 +9,11 @@
  */
 import { describe, it, expect, beforeEach, vi } from 'vitest';
 import { handleNavigate, handleWait } from './navigation-handlers.js';
+// Test constants
+const TEST_URLS = {
+    BASIC: 'https://example.com',
+    FORM: 'https://httpbin.org/forms/post'
+};
 // Mock all external dependencies
 vi.mock('../browser-manager', () => ({
     getBrowserInstance: vi.fn(),
@@ -69,33 +74,33 @@ describe('Navigation Handlers', () => {
         describe('Successful Navigation', () => {
             it('should navigate to URL successfully', async () => {
                 // Arrange: Basic navigation args
-                const args = { url: 'https://example.com' };
+                const args = { url: TEST_URLS.BASIC };
                 mockPageInstance.goto.mockResolvedValue(undefined);
                 // Act: Navigate to URL
                 const result = await handleNavigate(args);
                 // Assert: Should navigate successfully
-                expect(mockPageInstance.goto).toHaveBeenCalledWith('https://example.com', { waitUntil: 'domcontentloaded', timeout: 60000 });
+                expect(mockPageInstance.goto).toHaveBeenCalledWith(TEST_URLS.BASIC, { waitUntil: 'domcontentloaded', timeout: 60000 });
                 expect(mockWorkflowValidation.validateWorkflow).toHaveBeenCalledWith('navigate', args);
                 expect(mockWorkflowValidation.recordExecution).toHaveBeenCalledWith('navigate', args, true);
                 expect(result).toHaveProperty('content');
                 expect(result.content[0].type).toBe('text');
-                expect(result.content[0].text).toContain('Successfully navigated to https://example.com');
+                expect(result.content[0].text).toContain(`Successfully navigated to ${TEST_URLS.BASIC}`);
                 expect(result.content[0].text).toContain('Workflow Status: Page loaded');
                 expect(result.content[0].text).toContain('Next step: Use get_content');
             });
             it('should navigate with custom waitUntil option', async () => {
                 // Arrange: Navigation with custom wait condition
-                const args = { url: 'https://example.com', waitUntil: 'load' };
+                const args = { url: TEST_URLS.BASIC, waitUntil: 'load' };
                 mockPageInstance.goto.mockResolvedValue(undefined);
                 // Act: Navigate with custom wait condition
                 const result = await handleNavigate(args);
                 // Assert: Should use custom waitUntil
-                expect(mockPageInstance.goto).toHaveBeenCalledWith('https://example.com', { waitUntil: 'load', timeout: 60000 });
-                expect(result.content[0].text).toContain('Successfully navigated to https://example.com');
+                expect(mockPageInstance.goto).toHaveBeenCalledWith(TEST_URLS.BASIC, { waitUntil: 'load', timeout: 60000 });
+                expect(result.content[0].text).toContain(`Successfully navigated to ${TEST_URLS.BASIC}`);
             });
             it('should include comprehensive workflow guidance', async () => {
                 // Arrange: Successful navigation
-                const args = { url: 'https://test.com' };
+                const args = { url: TEST_URLS.BASIC };
                 mockPageInstance.goto.mockResolvedValue(undefined);
                 // Act: Navigate
                 const result = await handleNavigate(args);
@@ -109,7 +114,7 @@ describe('Navigation Handlers', () => {
         describe('Navigation Retry Logic', () => {
             it('should retry navigation on failure and succeed', async () => {
                 // Arrange: Navigation fails first time, succeeds second time
-                const args = { url: 'https://retry.com' };
+                const args = { url: TEST_URLS.BASIC };
                 mockPageInstance.goto
                     .mockRejectedValueOnce(new Error('Network error'))
                     .mockResolvedValueOnce(undefined);
@@ -117,11 +122,11 @@ describe('Navigation Handlers', () => {
                 const result = await handleNavigate(args);
                 // Assert: Should retry and succeed
                 expect(mockPageInstance.goto).toHaveBeenCalledTimes(2);
-                expect(result.content[0].text).toContain('Successfully navigated to https://retry.com');
+                expect(result.content[0].text).toContain(`Successfully navigated to ${TEST_URLS.BASIC}`);
             });
             it('should retry navigation multiple times before giving up', async () => {
                 // Arrange: Navigation fails all attempts
-                const args = { url: 'https://fail.com' };
+                const args = { url: TEST_URLS.BASIC };
                 const networkError = new Error('Persistent network error');
                 mockPageInstance.goto.mockRejectedValue(networkError);
                 // Act & Assert: Should retry 3 times then fail
@@ -131,7 +136,7 @@ describe('Navigation Handlers', () => {
             });
             it('should use exponential backoff between retries', async () => {
                 // Arrange: Navigation fails with retries
-                const args = { url: 'https://backoff.com' };
+                const args = { url: TEST_URLS.BASIC };
                 mockPageInstance.goto.mockRejectedValue(new Error('Timeout'));
                 // Act: Attempt navigation (will fail after retries)
                 try {
@@ -148,14 +153,14 @@ describe('Navigation Handlers', () => {
         describe('Navigation Error Handling', () => {
             it('should throw error when browser not initialized', async () => {
                 // Arrange: No page instance
-                const args = { url: 'https://example.com' };
+                const args = { url: TEST_URLS.BASIC };
                 mockBrowserManager.getPageInstance.mockReturnValue(null);
                 // Act & Assert: Should throw browser not initialized error
                 await expect(handleNavigate(args)).rejects.toThrow('Browser not initialized. Call browser_init first.');
             });
             it('should handle workflow validation failure', async () => {
                 // Arrange: Invalid workflow state
-                const args = { url: 'https://example.com' };
+                const args = { url: TEST_URLS.BASIC };
                 mockWorkflowValidation.validateWorkflow.mockReturnValue({
                     isValid: false,
                     errorMessage: 'Cannot navigate in current state',
@@ -166,7 +171,7 @@ describe('Navigation Handlers', () => {
             });
             it('should handle timeout errors from withTimeout wrapper', async () => {
                 // Arrange: Navigation that times out
-                const args = { url: 'https://timeout.com' };
+                const args = { url: TEST_URLS.BASIC };
                 mockSystemUtils.withTimeout.mockImplementation(async (operation, timeout, context) => {
                     throw new Error(`Operation timed out after ${timeout}ms in context: ${context}`);
                 });
@@ -310,7 +315,7 @@ describe('Navigation Handlers', () => {
     describe('Workflow Validation Integration', () => {
         it('should validate workflow before navigation operations', async () => {
             // Arrange: Valid navigation request
-            const args = { url: 'https://example.com' };
+            const args = { url: TEST_URLS.BASIC };
             mockPageInstance.goto.mockResolvedValue(undefined);
             // Act: Execute navigation
             await handleNavigate(args);
@@ -328,7 +333,7 @@ describe('Navigation Handlers', () => {
         });
         it('should record successful executions', async () => {
             // Arrange: Successful operation
-            const args = { url: 'https://success.com' };
+            const args = { url: TEST_URLS.BASIC };
             mockPageInstance.goto.mockResolvedValue(undefined);
             // Act: Execute successful operation
             await handleNavigate(args);
@@ -352,7 +357,7 @@ describe('Navigation Handlers', () => {
     describe('System Integration', () => {
         it('should use error handling wrapper for navigation', async () => {
             // Arrange: Navigation operation
-            const args = { url: 'https://example.com' };
+            const args = { url: TEST_URLS.BASIC };
             mockPageInstance.goto.mockResolvedValue(undefined);
             // Act: Execute navigation
             await handleNavigate(args);
@@ -370,7 +375,7 @@ describe('Navigation Handlers', () => {
         });
         it('should use timeout wrapper for navigation', async () => {
             // Arrange: Navigation operation
-            const args = { url: 'https://example.com' };
+            const args = { url: TEST_URLS.BASIC };
             mockPageInstance.goto.mockResolvedValue(undefined);
             // Act: Execute navigation
             await handleNavigate(args);

package/dist/test-constants.js

@@ -0,0 +1,111 @@
+/**
+ * Central Test Configuration
+ * Use these constants across all test files for consistency
+ */
+export const TEST_URLS = {
+    // CAPTCHA Testing URLs
+    CAPTCHA: {
+        CLOUDFLARE: 'https://nopecha.com/demo/cloudflare',
+        ECOURTS_INDIA: 'https://services.ecourts.gov.in/ecourtindia_v6/?p=casestatus/index&app_token=22e7493ca224682349cf0986dd144491a950819d30918b8e319ab0d39618f847',
+        RECAPTCHA: 'https://nopecha.com/demo/recaptcha',
+        HCAPTCHA: 'https://nopecha.com/demo/hcaptcha',
+        TURNSTILE: 'https://nopecha.com/demo/turnstile',
+    },
+    // General Content Testing
+    GENERAL: {
+        WIKIPEDIA: 'https://en.wikipedia.org/wiki/Web_scraping',
+        IMDB: 'https://www.imdb.com/',
+        GITHUB: 'https://github.com/',
+        EXAMPLE: 'https://example.com',
+    },
+    // API Discovery Testing
+    API: {
+        JSONPLACEHOLDER: 'https://jsonplaceholder.typicode.com',
+        REQRES: 'https://reqres.in/',
+    },
+    // E-commerce Testing
+    ECOMMERCE: {
+        AMAZON: 'https://www.amazon.com',
+    },
+    // Local Testing
+    LOCAL: {
+        LOCALHOST: 'http://localhost:3000',
+        FILE: 'file:///test.html',
+    },
+};
+export const TEST_SELECTORS = {
+    // Common selectors for testing
+    WIKIPEDIA: {
+        HEADING: '#firstHeading',
+        CONTENT: '#mw-content-text',
+        TOC: '#toc',
+        LINKS: 'a[href]',
+        IMAGES: 'img',
+        TABLES: 'table.wikitable',
+    },
+    ECOURTS: {
+        CAPTCHA_IMAGE: 'img[src*="captcha"]',
+        CAPTCHA_INPUT: 'input[name*="captcha" i]',
+        STATE_SELECT: 'select[name="state"]',
+        SEARCH_BUTTON: 'button[type="submit"]',
+    },
+    CLOUDFLARE: {
+        CHALLENGE: 'div[id^="cf-chl-widget"]',
+        IFRAME: 'iframe[src*="challenges.cloudflare.com"]',
+        VERIFY_TEXT: 'p:contains("Verifying")',
+    },
+    COMMON: {
+        HEADING: 'h1',
+        PARAGRAPH: 'p',
+        LINK: 'a',
+        IMAGE: 'img',
+        BUTTON: 'button',
+        INPUT: 'input',
+    },
+};
+export const TEST_TIMEOUTS = {
+    SHORT: 5000,
+    MEDIUM: 10000,
+    LONG: 30000,
+    CAPTCHA: 60000,
+};
+export const TEST_EXPECTATIONS = {
+    WIKIPEDIA: {
+        MIN_IMAGES: 5,
+        MIN_LINKS: 100,
+        HAS_TOC: true,
+    },
+    ECOURTS: {
+        HAS_CAPTCHA: true,
+        HAS_STATE_SELECT: true,
+    },
+    CLOUDFLARE: {
+        HAS_CHALLENGE: true,
+        VERIFICATION_TEXT: 'Verifying you are human',
+    },
+};
+export const TEST_DATA = {
+    // Sample data for testing
+    SENTIMENT: {
+        POSITIVE: 'This is an amazing product! I absolutely love it. Highly recommended!',
+        NEGATIVE: 'Terrible experience. Very disappointed. Would not recommend.',
+        NEUTRAL: 'The product arrived on time. It works as described.',
+    },
+    TRANSLATION: {
+        FRENCH: 'Bonjour, comment allez-vous? Je suis très heureux de vous rencontrer.',
+        SPANISH: 'Hola, ¿cómo estás? Estoy muy feliz de conocerte.',
+        GERMAN: 'Hallo, wie geht es dir? Ich bin sehr glücklich, dich kennenzulernen.',
+    },
+    SEARCH: {
+        PARTY_NAME: 'Ramesh Kumar',
+        CASE_NUMBER: '123/2024',
+        YEAR: '2024',
+    },
+};
+export default {
+    TEST_URLS,
+    TEST_SELECTORS,
+    TEST_TIMEOUTS,
+    TEST_EXPECTATIONS,
+    TEST_DATA,
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "brave-real-browser-mcp-server",
-  "version": "2.9.1",
+  "version": "2.9.3",
   "description": "MCP server for brave-real-browser",
   "type": "module",
   "main": "dist/index.js",