From 074e8b294d9760f51e3bd9081b704ed0caddf5cf Mon Sep 17 00:00:00 2001
From: Paulo Santos <GitPaulo@users.noreply.github.com>
Date: Fri, 13 Feb 2026 12:31:45 +0000
Subject: [PATCH] copilot review: add test for coverage of no params passed

---
 __tests__/inference.test.ts | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/__tests__/inference.test.ts b/__tests__/inference.test.ts
index 4775ee7..3f78e12 100644
--- a/__tests__/inference.test.ts
+++ b/__tests__/inference.test.ts
@@ -663,5 +663,34 @@ describe('inference.ts', () => {
       expect(mockCreate.mock.calls[0][0]).toHaveProperty('max_tokens', 100)
       expect(mockCreate.mock.calls[0][0]).not.toHaveProperty('max_completion_tokens')
     })
+
+    it('sends neither token param when both are undefined', async () => {
+      const requestWithNoTokens = {
+        ...mockRequest,
+        maxCompletionTokens: undefined,
+        maxTokens: undefined,
+      }
+
+      const mockResponse = {
+        choices: [
+          {
+            message: {
+              content: 'No token limit response',
+            },
+          },
+        ],
+      }
+
+      mockCreate.mockResolvedValueOnce(mockResponse)
+
+      const result = await simpleInference(requestWithNoTokens)
+
+      expect(result).toBe('No token limit response')
+      expect(mockCreate).toHaveBeenCalledTimes(1)
+
+      const params = mockCreate.mock.calls[0][0]
+      expect(params).not.toHaveProperty('max_tokens')
+      expect(params).not.toHaveProperty('max_completion_tokens')
+    })
   })
 })