-
Notifications
You must be signed in to change notification settings - Fork 5.9k
feat: add test for markdown webview #5740
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #5740 +/- ##
=======================================
Coverage 72.65% 72.65%
=======================================
Files 30 30
Lines 1682 1682
Branches 369 369
=======================================
Hits 1222 1222
Misses 397 397
Partials 63 63 Continue to review full report at Codecov.
|
66dfe33
to
b4a673b
Compare
// It's an iframe within an iframe | ||
// so we have to do .frameLocator twice | ||
const renderedText = await codeServerPage.page | ||
.frameLocator("iframe.webview.ready") | ||
.frameLocator("#active-frame") | ||
.locator("text=Hello world") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was the hardest part. I didn't realize it was an <iframe>
inside of an <iframe>
but some rubber-ducking with @code-asher helped me figure it out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉 🎉 🎉
This adds a test to ensure webviews work by rendering the Markdown Preview.