Skip to content

Commit 5892912

Browse files
committed
Update integration tests to not expect failure on ESM since it's now supported
1 parent 9835f05 commit 5892912

File tree

1 file changed

+7
-13
lines changed
  • dev-packages/node-integration-tests/suites/tracing/dataloader

1 file changed

+7
-13
lines changed

dev-packages/node-integration-tests/suites/tracing/dataloader/test.ts

Lines changed: 7 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -32,17 +32,11 @@ describe('dataloader auto-instrumentation', () => {
3232
]),
3333
};
3434

35-
createEsmAndCjsTests(
36-
__dirname,
37-
'scenario.mjs',
38-
'instrument.mjs',
39-
(createRunner, test) => {
40-
test('should auto-instrument `dataloader` package.', async () => {
41-
const runner = createRunner().expect({ transaction: EXPECTED_TRANSACTION }).start();
42-
runner.makeRequest('get', '/');
43-
await runner.completed();
44-
});
45-
},
46-
{ failsOnEsm: true },
47-
);
35+
createEsmAndCjsTests(__dirname, 'scenario.mjs', 'instrument.mjs', (createRunner, test) => {
36+
test('should auto-instrument `dataloader` package.', async () => {
37+
const runner = createRunner().expect({ transaction: EXPECTED_TRANSACTION }).start();
38+
runner.makeRequest('get', '/');
39+
await runner.completed();
40+
});
41+
});
4842
});

0 commit comments

Comments
 (0)