Skip to content

Commit 485d50e

Browse files
JPeer264isaacs
andauthored
Update dev-packages/test-utils/src/cli.ts
Co-authored-by: isaacs <i@izs.me>
1 parent e709084 commit 485d50e

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

  • dev-packages/test-utils/src

dev-packages/test-utils/src/cli.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ export function fetchTrace(traceId: string): TraceItem[] {
3636
const target = traceTarget(traceId);
3737
const result = spawnSync('pnpm', ['exec', 'sentry', 'trace', 'view', target, '--json', '--fresh'], {
3838
encoding: 'utf8',
39+
maxBuffer: 64 * 1024 * 1024,
3940
env: {
4041
...process.env,
4142
// The E2E token is the only credential CI has. Locally the CLI would prefer a stored login
@@ -44,6 +45,13 @@ export function fetchTrace(traceId: string): TraceItem[] {
4445
SENTRY_FORCE_ENV_TOKEN: '1',
4546
},
4647
});
48+
49+
if (result.error) {
50+
throw new Error(
51+
`Could not run \`pnpm exec sentry trace view\`: ${result.error.message}. ` +
52+
'The test app needs `sentry` as a dev dependency.',
53+
);
54+
}
4755

4856
if (result.status === 0) {
4957
return (JSON.parse(result.stdout) as { spans?: TraceItem[] }).spans ?? [];

0 commit comments

Comments
 (0)