@xenon-device-management/xenon 1.17.4 → 1.17.5

package/lib/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@xenon-device-management/xenon",
-    "version": "1.17.4",
+    "version": "1.17.5",
     "description": "Xenon - Intelligent Mobile Infrastructure. A self-healing device orchestration platform for Appium.",
     "main": "./lib/src/index.js",
     "exports": {

package/lib/src/device-managers/ios/IOSStreamService.js

@@ -692,6 +692,12 @@ let IOSStreamService = class IOSStreamService {
                     if (!fs_extra_1.default.existsSync(logDir))
                         fs_extra_1.default.mkdirSync(logDir);
                     const wdaRunLog = path_1.default.join(logDir, `runwda-${udid}.log`);
+                    // Truncated per run, because this file is also the evidence the failure
+                    // is classified from. Appending across runs meant a "Did not find test
+                    // app" line from a day earlier was still matched today, so every later
+                    // failure of any kind — including one where WDA was demonstrably
+                    // installed — was reported as "WebDriverAgent is not installed".
+                    fs_extra_1.default.writeFileSync(wdaRunLog, '');
                     (_a = session.wdaProcess.stdout) === null || _a === void 0 ? void 0 : _a.on('data', (d) => fs_extra_1.default.appendFileSync(wdaRunLog, d));
                     (_b = session.wdaProcess.stderr) === null || _b === void 0 ? void 0 : _b.on('data', (d) => fs_extra_1.default.appendFileSync(wdaRunLog, d));
                     // 4. Wait for WDA to be ready
@@ -770,11 +776,16 @@ let IOSStreamService = class IOSStreamService {
                         }
                         if (((_c = session.wdaProcess) === null || _c === void 0 ? void 0 : _c.exitCode) !== null) {
                             const logContent = fs_extra_1.default.existsSync(wdaRunLog) ? fs_extra_1.default.readFileSync(wdaRunLog, 'utf8') : '';
-                            // Missing WDA is permanent until the app is installed — surface an
-                            // actionable reason instead of a raw exit code + log tail.
+                            // Both of these are permanent until a human acts, and they call for
+                            // opposite actions — install the app, versus trust the app that is
+                            // already installed. Surface which, rather than a raw exit code and
+                            // a log tail.
                             if ((0, iosStreamDiagnostics_1.isMissingWdaError)(logContent)) {
                                 throw new Error((0, iosStreamDiagnostics_1.missingWdaMessage)(udid));
                             }
+                            if ((0, iosStreamDiagnostics_1.isWdaLaunchFailure)(logContent)) {
+                                throw new Error((0, iosStreamDiagnostics_1.wdaLaunchFailureMessage)(udid));
+                            }
                             throw new Error(`WDA process exited with code ${(_d = session.wdaProcess) === null || _d === void 0 ? void 0 : _d.exitCode}. Log: ${logContent.slice(-200)}`);
                         }
                         yield new Promise((resolve) => setTimeout(resolve, 1000));

package/lib/src/device-managers/ios/iosStreamDiagnostics.js

@@ -9,6 +9,8 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.isMissingWdaError = isMissingWdaError;
 exports.missingWdaMessage = missingWdaMessage;
+exports.isWdaLaunchFailure = isWdaLaunchFailure;
+exports.wdaLaunchFailureMessage = wdaLaunchFailureMessage;
 exports.classifyTunnelStderr = classifyTunnelStderr;
 exports.isOwnStreamProcess = isOwnStreamProcess;
 /**
@@ -27,6 +29,34 @@ function missingWdaMessage(udid) {
     return (`WebDriverAgent is not installed on ${udid}. ` +
         `Live streaming requires the WebDriverAgentRunner app — install WDA on this device to enable the mirror.`);
 }
+/**
+ * True when WDA is installed but the runner could not be *started*.
+ *
+ * go-ios reports the two conditions quite differently, and conflating them
+ * sends people to reinstall an app that is already there:
+ *
+ *   missing:  cannot get test app information: Did not find test app for '…'
+ *   present:  cannot start test runner: LaunchAppWithStdIo: failed to launch
+ *             app: launchApp: failed to get PID
+ *
+ * The second is nearly always the device refusing to run the binary — an
+ * untrusted developer certificate, a locked screen, or a lapsed signature —
+ * none of which reinstalling addresses on its own.
+ */
+function isWdaLaunchFailure(logContent) {
+    if (!logContent)
+        return false;
+    return (/failed to launch app/i.test(logContent) ||
+        /LaunchAppWithStdIo/i.test(logContent) ||
+        /cannot start test runner/i.test(logContent));
+}
+/** Human-facing, actionable message for the installed-but-unlaunchable case. */
+function wdaLaunchFailureMessage(udid) {
+    return (`WebDriverAgent is installed on ${udid} but would not start. ` +
+        `This is usually the device refusing to run it: trust the developer certificate under ` +
+        `Settings → General → VPN & Device Management, unlock the screen, and check the signing ` +
+        `profile has not expired. Launching WebDriverAgentRunner once by hand confirms it.`);
+}
 /**
  * Classify a chunk of go-ios `tunnel start` stderr.
  *

package/lib/test/unit/iosStreamDiagnostics.spec.js

@@ -2,6 +2,17 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const chai_1 = require("chai");
 const iosStreamDiagnostics_1 = require("../../src/device-managers/ios/iosStreamDiagnostics");
+/**
+ * Both verbatim from a real go-ios run log, and the whole point is that they
+ * mean opposite things: reinstall the app, versus trust the app that is
+ * already installed.
+ */
+const GO_IOS_MISSING = `{"level":"ERROR","msg":"Failed running WDA","error":"runXUITestWithBundleIdsXcode15Ctx: ` +
+    `cannot get test app information: Did not find test app for ` +
+    `'com.qasecret.WebDriverAgentRunner.xctrunner' on device. Is it installed?"}`;
+const GO_IOS_WONT_LAUNCH = `{"level":"ERROR","msg":"Failed running WDA","error":"runXUITestWithBundleIdsXcode15Ctx: ` +
+    `cannot start test runner: LaunchAppWithStdIo: failed to launch app: launchApp: ` +
+    `failed to get PID: pidFromResponse: could not get pid"}`;
 describe('iosStreamDiagnostics', () => {
     describe('isMissingWdaError', () => {
         it('detects the go-ios "Did not find test app" WDA crash', () => {
@@ -25,6 +36,44 @@ describe('iosStreamDiagnostics', () => {
             (0, chai_1.expect)(msg.toLowerCase()).to.contain('install');
         });
     });
+    describe('telling "not installed" apart from "would not start"', () => {
+        it('reads the real missing-app log as missing', () => {
+            (0, chai_1.expect)((0, iosStreamDiagnostics_1.isMissingWdaError)(GO_IOS_MISSING)).to.equal(true);
+            (0, chai_1.expect)((0, iosStreamDiagnostics_1.isWdaLaunchFailure)(GO_IOS_MISSING)).to.equal(false);
+        });
+        it('reads the real launch-failure log as a launch failure, not as missing', () => {
+            // This is the one that was reported as "WebDriverAgent is not installed"
+            // against a device where it demonstrably was installed, sending the
+            // reader off to reinstall an app that was already there.
+            (0, chai_1.expect)((0, iosStreamDiagnostics_1.isWdaLaunchFailure)(GO_IOS_WONT_LAUNCH)).to.equal(true);
+            (0, chai_1.expect)((0, iosStreamDiagnostics_1.isMissingWdaError)(GO_IOS_WONT_LAUNCH)).to.equal(false);
+        });
+        it('gives the two cases opposite advice', () => {
+            (0, chai_1.expect)((0, iosStreamDiagnostics_1.missingWdaMessage)('udid-1')).to.contain('not installed');
+            const launch = (0, iosStreamDiagnostics_1.wdaLaunchFailureMessage)('udid-1');
+            (0, chai_1.expect)(launch).to.contain('is installed');
+            (0, chai_1.expect)(launch).to.contain('VPN & Device Management');
+            (0, chai_1.expect)(launch).to.not.contain('not installed');
+        });
+        it('is safe on empty input', () => {
+            (0, chai_1.expect)((0, iosStreamDiagnostics_1.isWdaLaunchFailure)('')).to.equal(false);
+        });
+        it('is why the run log must be truncated per run', () => {
+            // These classifiers read a file, and the file used to be appended to
+            // across runs. A stale "Did not find test app" from a previous day still
+            // wins over today's launch failure — which is exactly how an installed
+            // WDA came to be reported as missing. IOSStreamService now empties the
+            // log before each spawn; this pins what happens if it stops.
+            const stale = `${GO_IOS_MISSING}\n${GO_IOS_WONT_LAUNCH}`;
+            (0, chai_1.expect)((0, iosStreamDiagnostics_1.isMissingWdaError)(stale)).to.equal(true);
+            (0, chai_1.expect)((0, iosStreamDiagnostics_1.isWdaLaunchFailure)(stale)).to.equal(true);
+        });
+        it('does not flag an unrelated crash as either', () => {
+            const crash = 'WDA crashed: signal 11 (SIGSEGV)';
+            (0, chai_1.expect)((0, iosStreamDiagnostics_1.isMissingWdaError)(crash)).to.equal(false);
+            (0, chai_1.expect)((0, iosStreamDiagnostics_1.isWdaLaunchFailure)(crash)).to.equal(false);
+        });
+    });
     describe('classifyTunnelStderr', () => {
         it('extracts the real target udid from the go-ios JSON payload', () => {
             const line = JSON.stringify({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xenon-device-management/xenon",
-  "version": "1.17.4",
+  "version": "1.17.5",
   "description": "Xenon - Intelligent Mobile Infrastructure. A self-healing device orchestration platform for Appium.",
   "main": "./lib/src/index.js",
   "exports": {