Skip to content

Commit f60509d

Browse files
committed
feat: export ensureBrowser function and add test for API tools
1 parent c6d73be commit f60509d

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

src/__tests__/toolHandler.test.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -391,4 +391,11 @@ describe('Tool Handler', () => {
391391
// Clean up
392392
delete process.env.CHROME_EXECUTABLE_PATH;
393393
});
394+
395+
test('should not launch browser for API tools', async () => {
396+
const ensureBrowser = jest.spyOn(require('../toolHandler'), 'ensureBrowser');
397+
await handleToolCall('playwright_get', { url: 'https://api.restful-api.dev/objects' }, mockServer);
398+
expect(ensureBrowser).not.toHaveBeenCalled();
399+
ensureBrowser.mockRestore();
400+
});
394401
});

src/toolHandler.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ async function registerConsoleMessage(page) {
155155
/**
156156
* Ensures a browser is launched and returns the page
157157
*/
158-
async function ensureBrowser(browserSettings?: BrowserSettings) {
158+
export async function ensureBrowser(browserSettings?: BrowserSettings) {
159159
try {
160160
// Check if browser exists but is disconnected
161161
if (browser && !browser.isConnected()) {

0 commit comments

Comments
 (0)