-
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
feat(e2e): Take screenshots for grid-list, list, icon, menu, radio, tabs, slide-toggle #3098
Conversation
tinayuangao
commented
Feb 15, 2017
- Updated button and checkbox e2e test so the screenshot wait for ripple animation to finish
- Added screenshots for grid-list, list, icon, menu, radio, tabs, slide-toggle
- Skipped some tests which UI is already tested. Also skipped e2e tests with animation.
6104026
to
25a168d
Compare
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.
It looks like there's a minor lint issue, otherwise good. Is there a way we can see the screenshots generated by this?
|
||
describe('grid-list', () => { | ||
beforeEach(() => browser.get('/grid-list')); | ||
|
||
it('should render a grid list container', () => { | ||
expectToExist('md-grid-list'); | ||
screenshot(''); |
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 could be improved by either having the id
be an optional param, or making it default to ''
.
Made Currently there's no way to see the screenshots. Once #2774 get in we can view the screenshots from https://material2-screenshots.firebaseapp.com/PR# |
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. |