@futdevpro/nts-dynamo 1.11.26 → 1.11.27

package/src/_modules/socket/_services/socket-server.service.ts

@@ -498,12 +498,21 @@ export abstract class DyNTS_SocketServerService<
         }
       });
         
-      newSocketServer.listen(this.params.port);
-      
-      DyFM_Log.success(
-        `\nsocket server setup finished: "${this.params.name}"` + 
-        `\nsocket server listening on port: "${this.params.port}"`
-      );
+      try {
+        newSocketServer.listen(this.params.port);
+        
+        DyFM_Log.success(
+          `\nsocket server setup finished: "${this.params.name}"` + 
+          `\nsocket server listening on port: "${this.params.port}"`
+        );
+      } catch (listenError) {
+        const errorMessage = `Failed to bind socket server "${this.params.name}" to port ${this.params.port}. ` +
+          `This could be due to port already in use or insufficient permissions. ` +
+          `Please ensure the port is available and try again.`;
+        
+        DyFM_Log.error(errorMessage, listenError);
+        throw new Error(errorMessage);
+      }
 
       successCallback();
       

package/src/_modules/socket/app-extended.server.spec.ts

@@ -9,17 +9,42 @@ describe('| DyNTS_AppExtended;', (): void => {
   process.setMaxListeners(20);
   let app: DyNTS_AppExtended;
 
+  // Helper function to safely stop and cleanup app
+  const safeStopApp = async (): Promise<void> => {
+    if (app) {
+      try {
+        await app.stop();
+      } catch (error) {
+        DyFM_Log.testWarn('Error stopping app:', error);
+      }
+      app = null;
+    }
+  };
+
+  // Helper function to wait for app to be ready with timeout
+  const waitForAppReady = async (appInstance: DyNTS_AppExtended, timeoutMs: number = 10000): Promise<void> => {
+    const startTime = Date.now();
+    while (!appInstance.started && (Date.now() - startTime) < timeoutMs) {
+      await DyFM_delay(100);
+    }
+    
+    if (!appInstance.started) {
+      throw new Error(`App failed to start within ${timeoutMs}ms`);
+    }
+  };
+
   describe('| a extended-base instance;', (): void => {
     beforeEach(async (): Promise<void> => {
       try {
         await DyFM_waitUntil((): boolean => !app, second);
+        await DyFM_delay(100); // Ensure previous instance is fully cleaned up
 
         app = new DyNTS_AppExtendedBase_Mock();
-
         await DyFM_delay(100);
         
         await app.ready();
-
+        await waitForAppReady(app);
+        
         await DyFM_delay(100);
       } catch (error) {
         DyFM_Log.testError(
@@ -35,16 +60,15 @@ describe('| DyNTS_AppExtended;', (): void => {
           (new Error()).stack,
           (error as DyFM_Error)?.additionalContent?.errors?.reverse()?.unshift()?.stack
         );
+        throw error; // Re-throw to fail the test
       }
     });
 
     afterEach(async (): Promise<void> => {
       try {
         await DyFM_delay(100);
-        
-        await app.stop();
-
-        app = null;
+        await safeStopApp();
+        await DyFM_delay(100);
       } catch (error) {
         DyFM_Log.testError(
           '| DyNTS_AppExtended; | a extended-base instance; test afterEach error:\n', 
@@ -64,10 +88,12 @@ describe('| DyNTS_AppExtended;', (): void => {
 
     it('| should be created', (): void => {
       expect(app).toBeDefined();
+      expect(app).toBeInstanceOf(DyNTS_AppExtendedBase_Mock);
     });
 
-    it('| should be started', (): void => {
+    it('| should be started', async (): Promise<void> => {
       expect(app.started).toBeTrue();
+      expect(app).toBeDefined();
     });
   });
 
@@ -75,13 +101,14 @@ describe('| DyNTS_AppExtended;', (): void => {
     beforeEach(async (): Promise<void> => {
       try {
         await DyFM_waitUntil((): boolean => !app, second);
+        await DyFM_delay(100); // Ensure previous instance is fully cleaned up
 
         app = new DyNTS_AppExtendedFull_Mock();
-
         await DyFM_delay(100);
         
         await app.ready();
-
+        await waitForAppReady(app);
+        
         await DyFM_delay(100);
       } catch (error) {
         DyFM_Log.testError(
@@ -97,16 +124,15 @@ describe('| DyNTS_AppExtended;', (): void => {
           (new Error()).stack,
           (error as DyFM_Error)?.additionalContent?.errors?.reverse()?.unshift()?.stack
         );
+        throw error; // Re-throw to fail the test
       }
     });
 
     afterEach(async (): Promise<void> => {
       try {
         await DyFM_delay(100);
-        
-        await app.stop();
-
-        app = null;
+        await safeStopApp();
+        await DyFM_delay(100);
       } catch (error) {
         DyFM_Log.testError(
           '| DyNTS_AppExtended; | a extended-full instance; test afterEach error:\n', 
@@ -126,10 +152,12 @@ describe('| DyNTS_AppExtended;', (): void => {
 
     it('| should be created', async (): Promise<void> => {
       expect(app).toBeDefined();
+      expect(app).toBeInstanceOf(DyNTS_AppExtendedFull_Mock);
     });
       
     it('| should be started', async (): Promise<void> => {
       expect(app.started).toBeTrue();
+      expect(app).toBeDefined();
     });
   });
 
@@ -137,12 +165,14 @@ describe('| DyNTS_AppExtended;', (): void => {
     beforeEach(async (): Promise<void> => {
       try {
         await DyFM_waitUntil((): boolean => !app, second);
+        await DyFM_delay(100); // Ensure previous instance is fully cleaned up
 
         app = new DyNTS_AppWbMock_Mock();
         await DyFM_delay(100);
         
         await app.ready();
-
+        await waitForAppReady(app);
+        
         await DyFM_delay(100);
       } catch (error) {
         DyFM_Log.testError(
@@ -158,15 +188,15 @@ describe('| DyNTS_AppExtended;', (): void => {
           (new Error()).stack,
           (error as DyFM_Error)?.additionalContent?.errors?.reverse()?.unshift()?.stack
         );
+        throw error; // Re-throw to fail the test
       }
     });
+
     afterEach(async (): Promise<void> => {
       try {
         await DyFM_delay(100);
-        
-        await app.stop();
-
-        app = null;
+        await safeStopApp();
+        await DyFM_delay(100);
       } catch (error) {
         DyFM_Log.testError(
           '| DyNTS_AppExtended; | a wb-mock instance; test afterEach error:\n', 
@@ -186,10 +216,12 @@ describe('| DyNTS_AppExtended;', (): void => {
 
     it('| should be created', async (): Promise<void> => {
       expect(app).toBeDefined();
+      expect(app).toBeInstanceOf(DyNTS_AppWbMock_Mock);
     });
       
     it('| should be started', async (): Promise<void> => {
       expect(app.started).toBeTrue();
+      expect(app).toBeDefined();
     });
   });
 });