testeranto 0.48.0 → 0.48.1

package/src/Project.ts

@@ -96,7 +96,6 @@ export class ITProject {
     const testPath = `${process.cwd()}/${config.tests}`;
     const featurePath = `${process.cwd()}/${config.features}`;
 
-    // q to quit, x to kill
     process.stdin.on('keypress', (str, key) => {
       if (key.name === 'q') {
         this.initiateShutdown("'q' command")
@@ -108,9 +107,6 @@ export class ITProject {
         process.exit(-1)
       }
     });
-    // process.on('SIGINT', () => this.initiateShutdown("CTRL+C"));
-    // process.on('SIGQUIT', () => this.initiateShutdown("Keyboard quit"));
-    // process.on('SIGTERM', () => this.initiateShutdown("'kill' command"));
 
     import(testPath).then((tests) => {
       this.tests = tests.default;
@@ -118,7 +114,6 @@ export class ITProject {
       import(featurePath).then((features) => {
         this.features = features.default;
 
-        // Make html files for webtests. Is this necessary?
         Promise.resolve(Promise.all(
           [
             ...this.getSecondaryEndpointsPoints("web")
@@ -184,18 +179,16 @@ export class ITProject {
           },
           plugins: [
             ...(config.nodePlugins || []),
-            // this should npt be necessary
-            // {
-            //   name: 'rebuild-notify',
-            //   setup(build) {
-            //     build.onEnd(result => {
-            //       console.log(`node build ended with ${result.errors.length} errors`);
-            //       console.log(result)
-            //       result.errors.length !== 0 && process.exit(-1);
-            //       // HERE: somehow restart the server from here, e.g., by sending a signal that you trap and react to inside the server.
-            //     })
-            //   }
-            // },
+            {
+              name: 'rebuild-notify',
+              setup(build) {
+                build.onEnd(result => {
+                  console.log(`node build ended with ${result.errors.length} errors`);
+                  console.log(result)
+                  result.errors.length !== 0 && process.exit(-1);
+                })
+              }
+            },
           ],
         };
         const esbuildConfigWeb: BuildOptions = {
@@ -209,14 +202,15 @@ export class ITProject {
           },
 
           external: [
+            "tests.test.js",
+            "features.test.js",
             // "url", 
+            // "react",
             "electron",
             "path",
             "fs",
-            // "react",
             "stream",
-            "tests.test.js",
-            "features.test.js"],
+          ],
           platform: "browser",
 
           outbase: config.outbase,
@@ -238,19 +232,16 @@ export class ITProject {
           },
           plugins: [
             ...(config.webPlugins || []),
-
-            // This should not be necessary
-            // {
-            //   name: 'rebuild-notify',
-            //   setup(build) {
-            //     build.onEnd(result => {
-            //       console.log(`web build ended with ${result.errors.length} errors`);
-            //       console.log(result)
-            //       result.errors.length !== 0 && process.exit(-1);
-            //       // HERE: somehow restart the server from here, e.g., by sending a signal that you trap and react to inside the server.
-            //     })
-            //   }
-            // },
+            {
+              name: 'rebuild-notify',
+              setup(build) {
+                build.onEnd(result => {
+                  console.log(`web build ended with ${result.errors.length} errors`);
+                  console.log(result)
+                  result.errors.length !== 0 && process.exit(-1);
+                })
+              }
+            },
           ],
         };
 
@@ -299,24 +290,33 @@ export class ITProject {
         `)
 
         Promise.all([
-          esbuild.context(esbuildConfigNode).then(async (nodeContext) => {
-            await nodeContext.watch();
-          }),
-          esbuild.context(esbuildConfigWeb).then(async (esbuildWeb) => {
-            await esbuildWeb.watch();
-          })
+          esbuild.context(esbuildConfigNode)
+            .then(async (nodeContext) => {
+              await nodeContext.watch();
+            }),
+
+          esbuild.context(esbuildConfigWeb)
+            .then(async (esbuildWeb) => {
+              await esbuildWeb.watch();
+            })
+
         ]).then(() => {
+
           if (config.devMode === false) {
             console.log("Your tests were built but not run because devMode was false. Exiting gracefully");
             process.exit(0);
+
           } else {
+
             pm2.connect(async (err) => {
+
               if (err) {
                 console.error(err);
                 process.exit(-1);
               } else {
                 console.log(`pm2 is connected`);
               }
+
               // run a websocket as an alternative to node IPC
               webSocketServer = new WebSocketServer({
                 port: 8080,
@@ -325,36 +325,19 @@ export class ITProject {
 
               webSocketServer.on('open', () => {
                 console.log('open');
-                // process.exit()
               });
 
               webSocketServer.on('close', (data) => {
                 console.log('webSocketServer close: %s', data);
-                // process.exit()
               });
 
               webSocketServer.on('listening', () => {
                 console.log("webSocketServer listening", webSocketServer.address());
-                // process.exit()
               });
 
               webSocketServer.on('connection', (webSocket: WebSocket) => {
-                console.log('webSocketServer connection');
-
                 webSocket.on('message', (webSocketData) => {
-                  // console.log('webSocket message: %s', webSocketData);
                   const payload = JSON.parse(webSocketData.valueOf().toString() as any);
-
-                  // console.log('webSocket payload', JSON.stringify(payload.data.testResourceConfiguration.name, null, 2));
-                  //  as {
-                  //   type: string,
-                  //   data: ITTestResourceRequirement & {
-                  //     testResourceConfiguration: {
-                  //       name: string;
-                  //     }
-                  //   }
-                  // };
-
                   const messageType = payload.type;
 
                   if (messageType === "testeranto:hola") {
@@ -401,7 +384,6 @@ export class ITProject {
 
                   pm2.launchBus((err, pm2_bus) => {
                     pm2_bus.on("testeranto:hola", (packet: { data: { requirement: ITTestResourceRequirement } }) => {
-                      console.log("hola IPC", packet);
                       this.requestResource(
                         packet.data.requirement,
                         'ipc'