Filip/playwright
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Revert "fix(playwright-test): prohibit async functions passed to describe (#14538)" (#14591)

Browse source 
This reverts commit bd0fe50a0e.
This commit is contained in:
Andrey Lushnikov 2022-06-02 11:31:10 -06:00 committed by GitHub
parent bd0fe50a0e
commit 8801f79742
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 1 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
packages/playwright-test/src/testType.ts
View file

@ -129,9 +129,7 @@ export class TestTypeImpl {
} }
setCurrentlyLoadingFileSuite(child); setCurrentlyLoadingFileSuite(child);
const result = fn(); fn();
if (result instanceof Promise)
throw errorWithLocation(location, 'describe cannot use async functions as callbacks');
setCurrentlyLoadingFileSuite(suite); setCurrentlyLoadingFileSuite(suite);
} }

11
tests/playwright-test/runner.spec.ts
View file

@ -48,17 +48,6 @@ test('it should enforce unique test names based on the describe block name', asy
expect(result.output).toContain(` - tests${path.sep}example.spec.js:8`); expect(result.output).toContain(` - tests${path.sep}example.spec.js:8`);
}); });
test('it should prohibit async functions in test.describe', async ({ runInlineTest }) => {
const result = await runInlineTest({
'tests/example.spec.js': `
const { test } = pwt;
test.describe('hello', async () => { test('my world', () => {}) });
`
});
expect(result.exitCode).toBe(1);
expect(result.output).toContain('describe cannot use async functions as callbacks');
});
test('it should not allow multiple tests with the same name in multiple files', async ({ runInlineTest }) => { test('it should not allow multiple tests with the same name in multiple files', async ({ runInlineTest }) => {
const result = await runInlineTest({ const result = await runInlineTest({
'tests/example1.spec.js': ` 'tests/example1.spec.js': `