Skip to content

Commit fe26f56

Browse files
committed
test(evm-wallet-experiment): improve delegation twin e2e failure messages
Include log file path in assertion failure messages so the relevant service log is immediately identifiable without digging through output.
1 parent 6743217 commit fe26f56

1 file changed

Lines changed: 17 additions & 5 deletions

File tree

packages/evm-wallet-experiment/test/e2e/docker/docker-e2e.test.ts

Lines changed: 17 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -336,11 +336,23 @@ describe('Docker E2E', () => {
336336
});
337337
const scriptPath =
338338
'/app/packages/evm-wallet-experiment/test/e2e/docker/run-delegation-twin-e2e.mjs';
339-
const output = dockerExec(
340-
kernelServices.away,
341-
`node --conditions development ${scriptPath} ${delegationMode} ${homeResult.kref} ${awayResult.kref} ${delegate}`,
342-
);
343-
expect(output).toContain('All delegation twin tests passed');
339+
const logFile = `logs/${kernelServices.away}.log`;
340+
let output = '';
341+
try {
342+
output = dockerExec(
343+
kernelServices.away,
344+
`node --conditions development ${scriptPath} ${delegationMode} ${homeResult.kref} ${awayResult.kref} ${delegate}`,
345+
);
346+
} catch (error) {
347+
throw new Error(
348+
`Delegation twin e2e script failed — see ${logFile}\n` +
349+
`${error instanceof Error ? error.message : String(error)}`,
350+
);
351+
}
352+
expect(
353+
output,
354+
`Assertions failed — see ${logFile} and logs/test-results.json`,
355+
).toContain('All delegation twin tests passed');
344356
}, 180_000);
345357
});
346358
},

0 commit comments

Comments
 (0)