-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
test(cdk/testing): port the protractor tests to the webdriver env #22375
test(cdk/testing): port the protractor tests to the webdriver env #22375
Conversation
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.
Overall LGTM, but someone else should also do a review on this one
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.
Just saw this. LGTM. One nit for a follow-up.
* more pending tasks). | ||
* | ||
* For the implementation, see: | ||
* https://github.com/angular/angular/blob/master/packages/platform-browser/src/browser/testability.ts#L30-L49 |
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.
Nit: Use a perma link in case the location changes.
* feat(cdk/testing): create a webdriver harness environment (#22321) * test(cdk/testing): Get webdriver test suite running with dummy tests (#22344) Co-authored-by: Paul Gschwendtner <[email protected]> * test(cdk/testing): start the e2e-app and run webdriver tests in bazel (#22367) * test(cdk/testing): port the protractor tests to the webdriver env (#22375) * test(cdk/testing): enable webdriver tests on CI (#22379) Co-authored-by: Paul Gschwendtner <[email protected]> * fixup! test(cdk/testing): enable webdriver tests on CI (#22379) Co-authored-by: Paul Gschwendtner <[email protected]>
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
I've confirmed that the tests all pass when run manually, but I don't have them working on CI yet.
Thanks to @kyliau for creating the example that I based the wait for Angular logic off of: https://github.com/kyliau/angular-wait/blob/main/server/server.ts