Skip to content

Commit

Permalink
fix for
Browse files Browse the repository at this point in the history
feat(e2e): add support for puppteer v19 #3810
  • Loading branch information
rwaskiewicz committed Dec 2, 2022
1 parent d3ce762 commit b5e3cc6
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 13 deletions.
4 changes: 2 additions & 2 deletions src/sys/node/node-sys.ts
Expand Up @@ -607,8 +607,8 @@ export function createNodeSys(c: { process?: any } = {}): CompilerSystem {
'@types/jest': { minVersion: '24.9.1', recommendedVersion: '27.0.3', maxVersion: '27.0.0' },
jest: { minVersion: '24.9.1', recommendedVersion: '27.0.3', maxVersion: '27.0.0' },
'jest-cli': { minVersion: '24.9.0', recommendedVersion: '27.4.5', maxVersion: '27.0.0' },
puppeteer: { minVersion: '1.19.0', recommendedVersion: '10.0.0' },
'puppeteer-core': { minVersion: '1.19.0', recommendedVersion: '5.2.1' },
puppeteer: { minVersion: '10.0.0', recommendedVersion: '19' },
'puppeteer-core': { minVersion: '10.0.0', recommendedVersion: '19' },
'workbox-build': { minVersion: '4.3.1', recommendedVersion: '4.3.1' },
});

Expand Down
18 changes: 7 additions & 11 deletions src/testing/puppeteer/puppeteer-element.ts
Expand Up @@ -467,17 +467,13 @@ export class E2EElement extends MockHTMLElement implements pd.E2EElementInternal
}

async e2eSync() {
const executionContext = this._elmHandle.executionContext();

const { outerHTML, shadowRootHTML } = await executionContext.evaluate<{ outerHTML: any; shadowRootHTML: any }>(
(elm: HTMLElement) => {
return {
outerHTML: elm.outerHTML,
shadowRootHTML: elm.shadowRoot ? elm.shadowRoot.innerHTML : null,
};
},
this._elmHandle
);
const executionContext = getPuppeteerExecution(this._elmHandle);
const { outerHTML, shadowRootHTML } = await executionContext.evaluate((elm: Element) => {
return {
outerHTML: elm.outerHTML,
shadowRootHTML: elm.shadowRoot ? elm.shadowRoot.innerHTML : null,
};
}, this._elmHandle);

if (typeof shadowRootHTML === 'string') {
(this as any).shadowRoot = parseHtmlToFragment(shadowRootHTML) as any;
Expand Down

0 comments on commit b5e3cc6

Please sign in to comment.