@afixt/test-utils 2.1.0 → 2.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@afixt/test-utils",
-    "version": "2.1.0",
+    "version": "2.1.1",
     "description": "Various utilities for accessibility testing",
     "main": "src/index.js",
     "scripts": {

package/src/getImageText.js

@@ -20,7 +20,8 @@ const _internal = {
  * @returns {Promise<string|false>} Extracted text if found, otherwise false.
  */
 async function getImageText(imagePath, options = {}) {
-    const { lang = 'eng', logger = m => console.log(m), ...tesseractOptions } = options;
+    const { lang = 'eng', logger: _logger, ...tesseractOptions } = options;
+    const logger = _logger === null ? () => {} : (_logger ?? (m => console.log(m)));
 
     try {
         const {

package/test/getImageText.test.js

@@ -121,16 +121,28 @@ describe('getImageText', () => {
         );
     });
 
-    it('should accept null logger to disable logging', async () => {
+    it('should convert null logger to a no-op function instead of passing null to Tesseract', async () => {
         recognizeSpy.mockResolvedValueOnce({ data: { text: 'hello' } });
 
         await getImageText('image.jpg', { logger: null });
 
-        expect(recognizeSpy).toHaveBeenCalledWith(
-            'image.jpg',
-            'eng',
-            expect.objectContaining({ logger: null })
-        );
+        const callArgs = recognizeSpy.mock.calls[0][2];
+        expect(typeof callArgs.logger).toBe('function');
+        // Verify it's a no-op (doesn't throw, returns undefined)
+        expect(callArgs.logger('test')).toBeUndefined();
+    });
+
+    it('should not crash when logger is null', async () => {
+        recognizeSpy.mockImplementation((path, lang, opts) => {
+            // Simulate Tesseract calling the logger
+            if (opts.logger) {
+                opts.logger({ status: 'recognizing text' });
+            }
+            return Promise.resolve({ data: { text: 'hello' } });
+        });
+
+        const result = await getImageText('image.jpg', { logger: null });
+        expect(result).toBe('hello');
     });
 
     it('should pass through additional Tesseract options', async () => {