thunderstrike/puppeteer
thunderstrike/puppeteer
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

chore: tidy up TODO from TS tests migration (#6090)

Browse Source 
Deleting `Node` errors in strict mode; we don't need to have this test
any more.
This commit is contained in:
Jack Franklin 2020-06-24 11:18:37 +01:00 committed by Mathias Bynens
parent f481922175
commit 785551997f
1 changed files with 0 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
test/jshandle.spec.ts
View File

@ -195,18 +195,6 @@ describe('JSHandle', function () {
await page.evaluate((e) => e.nodeType === Node.TEXT_NODE, element) await page.evaluate((e) => e.nodeType === Node.TEXT_NODE, element)
); );
}); });
it('should work with nullified Node', async () => {
const { page } = getTestState();
await page.setContent('<section>test</section>');
// TODO (@jackfranklin): this line doesn't do anything. What was intended here?
// await page.evaluate(() => delete Node);
const handle = await page.evaluateHandle(() =>
document.querySelector('section')
);
const element = handle.asElement();
expect(element).not.toBe(null);
});
}); });
describe('JSHandle.toString', function () { describe('JSHandle.toString', function () {