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(common): disable deprecated date pipe tests on chrome mobile #21933
Conversation
b86622d
to
0c7ff58
Compare
@@ -12,7 +12,10 @@ import {JitReflector} from '@angular/platform-browser-dynamic/src/compiler_refle | |||
import {browserDetection} from '@angular/platform-browser/testing/src/browser_util'; | |||
|
|||
{ | |||
xdescribe('DeprecatedDatePipe', () => { | |||
// some tests are disabled on chrome mobile because of the following bug affecting the intl API |
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.
Can you please enclose the comment + test in a function and use this function so that we know why a test is disabled ?
function hasCromeBug796583(): boolean {
return ...;
}
if (!hasCromeBug796583 || ...) {
...
}
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.
see inline comments
Feel free to add the merge label once the requested change is implemented. |
0c7ff58
to
45a6295
Compare
45a6295
to
89da2c9
Compare
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. |
PR Type
What kind of change does this PR introduce?
What is the current behavior?
There's a bug in chrome mobile 63 with the intl API: https://bugs.chromium.org/p/chromium/issues/detail?id=796583
because of that, the deprecated date pipe tests are failing.
Issue Number: #21907
What is the new behavior?
Those tests are disabled on chrome mobile
Does this PR introduce a breaking change?