Skip to content

Commit e1e6bc7

Browse files
test(maestro-sample): Await the mocked relay start before starting the test (#4791)
1 parent 432a4cb commit e1e6bc7

File tree

4 files changed

+12
-5
lines changed

4 files changed

+12
-5
lines changed

.github/workflows/sample-application.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,7 @@ jobs:
208208
path: ${{ env.REACT_NATIVE_SAMPLE_PATH }}/${{ matrix.platform }}/*.log
209209

210210
test:
211-
name: Test ${{ matrix.platform }} ${{ matrix.build-type }}
211+
name: Test ${{ matrix.platform }} ${{ matrix.build-type }} REV2
212212
runs-on: ${{ matrix.runs-on }}
213213
needs: [diff_check, build]
214214
if: ${{ needs.diff_check.outputs.skip_ci != 'true' }}

samples/react-native/e2e/captureErrorsScreenTransaction.test.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,13 @@ import { maestro } from './utils/maestro';
1010

1111
describe('Capture Errors Screen Transaction', () => {
1212
let sentryServer = createSentryServer();
13-
sentryServer.start();
1413

1514
const getErrorsEnvelope = () =>
1615
sentryServer.getEnvelope(containingTransactionWithName('Errors'));
1716

1817
beforeAll(async () => {
18+
await sentryServer.start();
19+
1920
const waitForErrorsTx = sentryServer.waitForEnvelope(
2021
containingTransactionWithName('Errors'), // The last created and sent transaction
2122
);

samples/react-native/e2e/captureSpaceflightNewsScreenTransaction.test.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import { maestro } from './utils/maestro';
1212

1313
describe('Capture Spaceflight News Screen Transaction', () => {
1414
let sentryServer = createSentryServer();
15-
sentryServer.start();
1615

1716
let newsEnvelopes: Envelope[] = [];
1817
let allTransactionEnvelopes: Envelope[] = [];
@@ -24,6 +23,8 @@ describe('Capture Spaceflight News Screen Transaction', () => {
2423
getItemOfTypeFrom<EventItem>(newsEnvelopes[1], 'transaction');
2524

2625
beforeAll(async () => {
26+
await sentryServer.start();
27+
2728
const containingNewsScreen = containingTransactionWithName(
2829
'SpaceflightNewsScreen',
2930
);

samples/react-native/e2e/utils/mockedSentryServer.ts

+7-2
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ export function createSentryServer({ port = 8961 } = {}): {
1616
predicate: (envelope: Envelope) => boolean,
1717
) => Promise<Envelope>;
1818
close: () => Promise<void>;
19-
start: () => void;
19+
start: () => Promise<void>;
2020
getEnvelope: (predicate: (envelope: Envelope) => boolean) => Envelope;
2121
getAllEnvelopes: (predicate: (envelope: Envelope) => boolean) => Envelope[];
2222
} {
@@ -64,7 +64,12 @@ export function createSentryServer({ port = 8961 } = {}): {
6464

6565
return {
6666
start: () => {
67-
server.listen(port);
67+
return new Promise<void>((resolve, _reject) => {
68+
server.listen(port, () => {
69+
console.log(`Sentry server listening on port ${port}`);
70+
resolve();
71+
});
72+
});
6873
},
6974
waitForEnvelope: async (
7075
predicate: (envelope: Envelope) => boolean,

0 commit comments

Comments
 (0)