zarro 1.202.0 → 1.203.0

package/gulp-tasks/modules/register-environment-variables.js

@@ -56,7 +56,7 @@
         });
         env.register({
             name: "MAX_CONCURRENCY",
-            default: os.cpus().length.toString(),
+            default: `${Math.floor(os.cpus().length / 2)}`,
             help: "Overrides other concurrency settings (BUILD_MAX_CPU_COUNT, MAX_NUNIT_AGENTS)"
         });
         env.register({

package/gulp-tasks/modules/test-dotnet-logic.js

@@ -183,7 +183,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
         });
         await runInParallel(concurrency, ...tasks);
         if (testResults.quackersEnabled) {
-            logOverallResults(testResults);
+            logOverallResults(testResults, testProcessResults);
         }
         else {
             console.log("If you install Quackers.TestLogger into your test projects, you'll get a lot more info here!");
@@ -231,7 +231,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
         }
         return args.map(quote).join(" ");
     }
-    function logOverallResults(testResults) {
+    function logOverallResults(testResults, testProcessResults) {
         const total = testResults.passed + testResults.skipped + testResults.failed, now = Date.now(), runTimeMs = now - testResults.started, runTime = nunitLikeTime(runTimeMs), darkerThemeSelected = (process.env["QUACKERS_THEME"] || "").toLowerCase() === "darker", red = darkerThemeSelected
             ? ansiColors.red.bind(ansiColors)
             : ansiColors.redBright.bind(ansiColors), cyan = darkerThemeSelected
@@ -239,6 +239,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
             : ansiColors.cyanBright.bind(ansiColors), yellow = darkerThemeSelected
             ? ansiColors.yellow.bind(ansiColors)
             : ansiColors.yellowBright.bind(ansiColors);
+        logTestSuiteTimes(testProcessResults, yellow);
         logFailures(testResults, red);
         logSlow(testResults, cyan);
         console.log(yellow(`
@@ -255,6 +256,41 @@ Test Run Summary
 `));
         console.log("\n");
     }
+    function logTestSuiteTimes(testProcessResults, yellow) {
+        if (!testProcessResults || testProcessResults.length === 0) {
+            return;
+        }
+        testProcessResults.sort((a, b) => {
+            if (a.runTimeMs === b.runTimeMs) {
+                return 0;
+            }
+            return a.runTimeMs > b.runTimeMs ? -1 : 1;
+        });
+        const assembliesAndTimes = testProcessResults.reduce((acc, cur) => {
+            const project = parseTestProjectFrom(cur.args);
+            acc.push({ project, runTimeMs: cur.runTimeMs });
+            return acc;
+        }, []);
+        console.log(yellow(`Test suite timings:`));
+        for (const r of assembliesAndTimes) {
+            console.log(yellow(`  ${r.project}: ${nunitLikeTime(r.runTimeMs)}`));
+        }
+    }
+    function parseTestProjectFrom(args) {
+        let next = false;
+        for (const arg of args) {
+            if (arg === "test") {
+                next = true;
+                continue;
+            }
+            if (next) {
+                return path.basename(arg)
+                    .replace(/\.dll$/i, "")
+                    .replace(/\.csproj$/i, "");
+            }
+        }
+        return `(project name parse failed for:) ${args.join(" ")}`;
+    }
     function logSlow(testResults, cyan) {
         logResultsSection(testResults.slowSummary, cyan("Slow tests:"), QUACKERS_SLOW_INDEX_PLACEHOLDER);
     }
@@ -328,7 +364,7 @@ Test Run Summary
         // addTrxLoggerTo(loggers, target);
         testResults.quackersEnabled = testResults.quackersEnabled || useQuackers;
         try {
-            const result = await test({
+            return await test({
                 target,
                 verbosity: finalVerbosity,
                 configuration,
@@ -342,12 +378,10 @@ Test Run Summary
                 env: testEnvironment,
                 label
             });
-            return result;
         }
         catch (e) {
             debug("WARN: catching SystemError instead of rethrowing it");
-            const err = e;
-            return err;
+            return e;
         }
     }
     function addTrxLoggerTo(loggers, target) {
@@ -557,6 +591,7 @@ Test Run Summary
         testWithNunitCli,
         shouldTestInParallel,
         testOneDotNetCoreProject,
-        testAsDotNetCore
+        testAsDotNetCore,
+        logTestSuiteTimes
     };
 })();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zarro",
-  "version": "1.202.0",
+  "version": "1.203.0",
   "description": "Some glue to make gulp easier, perhaps even zero- or close-to-zero-conf",
   "bin": {
     "zarro": "index.js"
@@ -87,6 +87,7 @@
     "sax": "^1.2.4",
     "semver": "^7.5.4",
     "simple-git": "^3.25.0",
+    "system-wrapper": "^1.8.0",
     "temp": "^0.9.4",
     "through2": "^3.0.2",
     "undertaker-forward-reference": "^1.0.2",

package/types.d.ts

@@ -276,12 +276,14 @@ declare global {
   }
 
   type DotNetTester = (configuration: string, source: string[]) => Promise<TestResults>;
+  type ResultOrError = SystemResult | SystemError;
 
   interface TestDotNetLogic {
     runTests: () => Promise<void>;
     testWithNunitCli: DotNetTester;
     testAsDotNetCore: DotNetTester;
     shouldTestInParallel: (testProjectPaths: string[]) => Promise<boolean>;
+    logTestSuiteTimes: (results: ResultOrError[], styleFn: StyleFunction) => void;
     testOneDotNetCoreProject: (
       target: string,
       configuration: string,