-
Notifications
You must be signed in to change notification settings - Fork 2.3k
feat(example): add examples of usage protractor framework with angular-material components; #4891
feat(example): add examples of usage protractor framework with angular-material components; #4891
Conversation
Examples for #4883 |
jasmineNodeOpts: { | ||
defaultTimeoutInterval: 30000 | ||
} | ||
}; |
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.
End Of File (EOF
). Or it is not necessary?
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.
Hi, @CrispusDH ! Thanks for the review) Actually, I tried to follow code style of examples. But I can add it if necessary.
example/angular_material/conf.js
Outdated
], | ||
|
||
// Disable promise manager because we are going to use async/await | ||
SELENIUM_PROMISE_MANAGER: false, |
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: un-indented this line
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.
Done
Looks good to me. Could you please update the commit title based on guideline: |
Hi, @qiyigg ! Thanks for the review) I added mentioned fixes. |
merged, thanks |
Added examples of usage protractor for testing angular-material components.