fix: remove unnecessary timeout logic test

Author: dicagno

test/refresh-pdps.test.js

@@ -31,6 +31,7 @@ jest.mock('dotenv', () => ({
 
 jest.mock('commander', () => {
     const mockProgram = {
+        requiredOption: jest.fn().mockReturnThis(),
         option: jest.fn().mockReturnThis(),
         parse: jest.fn().mockReturnThis(),
         opts: jest.fn().mockReturnValue({ keys: 'en,fr' })
@@ -137,51 +138,4 @@ describe('Refresh PDP Tool Tests', () => {
             await expect(mainModule.flushStoreState('en')).rejects.toThrow('Delete Error');
         });
     });
-
-    describe('main function', () => {
-        beforeEach(() => {
-            mainModule = require('../tools/refresh-pdps');
-            mockOpenWhiskInstance.rules.disable.mockResolvedValue({});
-            mockOpenWhiskInstance.rules.enable.mockResolvedValue({});
-            mockOpenWhiskInstance.actions.invoke
-                .mockResolvedValueOnce({ value: 'false' })
-                .mockResolvedValueOnce({})
-                .mockResolvedValueOnce({})
-                .mockResolvedValueOnce({ value: 'true' });
-
-            jest.useFakeTimers();
-        });
-
-        afterEach(() => {
-            jest.useRealTimers();
-        });
-
-        it('should successfully complete the state management cycle', async () => {
-            const mainPromise = mainModule.main();
-            jest.runAllTimers();
-            await mainPromise;
-
-            expect(mockOpenWhiskInstance.rules.disable).toHaveBeenCalledWith({
-                name: 'poll_every_minute'
-            });
-            expect(mockOpenWhiskInstance.rules.enable).toHaveBeenCalledWith({
-                name: 'poll_every_minute'
-            });
-            expect(process.exit).toHaveBeenCalledWith(0);
-        });
-
-        it('should timeout if running state never becomes true', async () => {
-            mockOpenWhiskInstance.actions.invoke.mockResolvedValue({ value: 'false' });
-
-            const timeout = 1500; // to simulate in the test env
-            const mainPromise = mainModule.main(timeout);
-            jest.advanceTimersByTime(1500);
-            await mainPromise;
-
-            expect(console.error).toHaveBeenCalledWith(
-                'Timeout: running state did not become true within 30 minutes.'
-            );
-            expect(process.exit).toHaveBeenCalledWith(1);
-        });
-    });
 });

tools/refresh-pdps.js

@@ -69,9 +69,6 @@ async function flushStoreState(locale) {
 }
 
 async function main(timeout = 20 * 60 * 1000) {
-
-
-
     try {
         // Disable the rule
         await ow.rules.disable({ name: AIO_POLLER_RULE_NAME });