Puppeteer v17 compatibility

[swissspidy]

In puppeteer/puppeteer#8844, which was released as part of Puppeteer v17, the execution context has been marked internal, so this access is no longer possible.

I see it's being used twice here:

pptr-testing-library/lib/index.ts

Line 92 in 860106b

.executionContext()

pptr-testing-library/lib/index.ts

Line 101 in 860106b

.executionContext()

[swissspidy]

The new recommended approach in v17.1.0 is to use ElementHandle.frame() directly, see puppeteer/puppeteer#8875. It should be released soon.

[patrickhulce]

Thanks for the note @swissspidy! We'll fix it 👍

[patrickhulce]

fix is up at #78 but getting this to work in a non-breaking way with typescript is...tricky :)

help with tests is welcome

[georgehb]

Is there any update on this?

If getting the types to work on both new and old versions of puppeteer is too much hassle, could the solution not be to drop support for old versions of puppeteer and release the changes as a major version bump?

[patrickhulce]

fixed by #78