test(api): Improve error logging on integration test

stryker-mutator · Jul 7, 2018 · 701b36a · 701b36a
1 parent a820577
commit 701b36a
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 5 deletions.
diff --git a/packages/stryker-api/test/integration/install-module/install-module.ts b/packages/stryker-api/test/integration/install-module/install-module.ts
@@ -8,9 +8,19 @@ describe('we have a module using stryker', function () {
 
   const modulePath = path.resolve(__dirname, '../../../testResources/module');
 
-  const execInModule = (command: string) => {
-    console.log(`Exec '${command}' in ${modulePath}`);
-    return exec(command, { cwd: modulePath });
+  function execInModule (command: string): Promise<[string, string]> {
+    return new Promise((res, rej) => {
+      console.log(`Exec '${command}' in ${modulePath}`);
+      exec(command, { cwd: modulePath }, (error, stdout, stderr) => {
+        if (error) {
+          console.log(stdout);
+          console.error(stderr);
+          rej(error);
+        } else {
+          res([stdout, stderr]);
+        }
+      });
+    });
   };
 
   describe('after installing Stryker', () => {

diff --git a/packages/stryker/src/logging/LogConfigurator.ts b/packages/stryker/src/logging/LogConfigurator.ts
@@ -25,7 +25,7 @@ const layouts: { color: log4js.PatternLayout, noColor: log4js.PatternLayout } =
     type: 'pattern',
     pattern: '%r (%z) %p %c %m'
   }
-}
+};
 
 interface AppendersConfiguration {
   [name: string]: log4js.Appender;

diff --git a/packages/stryker/test/unit/logging/LogConfiguratorSpec.ts b/packages/stryker/test/unit/logging/LogConfiguratorSpec.ts
@@ -85,7 +85,7 @@ describe('LogConfigurator', () => {
     const notColoredLayout: log4js.PatternLayout  = {
       type: 'pattern',
       pattern: '%r (%z) %p %c %m'
-    }
+    };
     return {
       appenders: {
         console: { type: 'stdout', layout: coloredLayout },