test: add tests for cleanTestResultPaths helper function (#1676)

Co-authored-by: Harsh Shukla <you@example.com>

Author: harshhgithub

packages/helpers/test/testing.test.js

@@ -1,5 +1,5 @@
-const { listFiles, buildParams, hasNestedConfig } = require('@asyncapi/generator-helpers');
-const fs = require('fs/promises');
+const { listFiles, buildParams, hasNestedConfig, cleanTestResultPaths } = require('@asyncapi/generator-helpers');
+const { rm, readdir } = require('fs/promises');
 
 jest.mock('fs/promises', () => ({
   rm: jest.fn(),
@@ -18,16 +18,16 @@ describe('listFiles', () => {
       { name: 'subdir', isFile: () => false },
     ];
 
-    fs.readdir.mockResolvedValue(mockDirents);
+    readdir.mockResolvedValue(mockDirents);
     const mockPath = '/mock/path';
 
     const result = await listFiles(mockPath);
-    expect(fs.readdir).toHaveBeenCalledWith(mockPath, { withFileTypes: true });
+    expect(readdir).toHaveBeenCalledWith(mockPath, { withFileTypes: true });
     expect(result).toEqual(['file1.txt', 'file2.js']);
   });
 
   it('should return an empty array if no files exist', async () => {
-    fs.readdir.mockResolvedValue([
+    readdir.mockResolvedValue([
       { name: 'folder', isFile: () => false },
     ]);
     const mockPath = '/mock/path';
@@ -36,6 +36,68 @@ describe('listFiles', () => {
   });
 });
 
+describe('cleanTestResultPaths', () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
+  it('should remove path when testResultPath exists', async () => {
+    const config = { testResultPath: '/tmp/test-results' };
+
+    await cleanTestResultPaths(config);
+
+    expect(rm).toHaveBeenCalledWith('/tmp/test-results', { recursive: true, force: true });
+  });
+
+  it('should not call rm when config is null/undefined', async () => {
+    await cleanTestResultPaths(null);
+    await cleanTestResultPaths(undefined);
+
+    expect(rm).not.toHaveBeenCalled();
+  });
+
+  it('should warn when testResultPath is missing and no nested config', async () => {
+    const consoleSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
+    const config = { someKey: 'value' };
+
+    await cleanTestResultPaths(config);
+
+    expect(consoleSpy).toHaveBeenCalledWith(
+      'Configuration missing testResultPath - no test results to clean:',
+      config
+    );
+
+    consoleSpy.mockRestore();
+  });
+
+  it('should recurse into nested configs and clean their paths', async () => {
+    const config = {
+      nested: {
+        testResultPath: '/tmp/nested-results',
+      },
+    };
+
+    await cleanTestResultPaths(config);
+
+    expect(rm).toHaveBeenCalledWith('/tmp/nested-results', { recursive: true, force: true });
+  });
+
+  it('should handle rm throwing an error gracefully', async () => {
+    rm.mockRejectedValueOnce(new Error('Permission denied'));
+    const consoleSpy = jest.spyOn(console, 'error').mockImplementation(() => {});
+
+    const config = { testResultPath: '/tmp/fail-results' };
+
+    await cleanTestResultPaths(config);
+
+    expect(consoleSpy).toHaveBeenCalledWith(
+      expect.stringContaining('Failed to clean /tmp/fail-results: Permission denied')
+    );
+
+    consoleSpy.mockRestore();
+  });
+});
+
 describe('hasNestedConfig', () => {
   it('should return false for an empty object', () => {
     expect(hasNestedConfig({})).toBe(false);
@@ -87,6 +149,7 @@ describe('hasNestedConfig', () => {
     expect(hasNestedConfig(config)).toBe(true);
   });
 });
+
 describe('buildParams', () => {
   it('should include clientFileName when language is not java', () => {
     const config = { clientFileName: 'myClient.js' };
@@ -139,4 +202,4 @@ describe('buildParams', () => {
       clientFileName: 'client.js',
     });
   });
-}); 
+});