mirror of
https://github.com/dart-lang/sdk
synced 2024-09-05 16:41:07 +00:00
[web] delete notification_permission_test.
This test provides very little value. It requests the notification permissions and waits for an answer. What's happening is that the test launches a prompt and then waiting for a user to click on the browser dialog to accept or reject the request. On the bots, this is consistently timing out in chrome and safari. Turns out this issue was [identified][1] 5 years ago and the test was marked as Timeout in the status files for that reason (dart2js) or skipped (DDC). Back then there were discussions about whether we could make chrome automatically accept or reject the request, but there was no traction on that. If we thought this test was providing value, we could explore whether there is a way to make bots automatically accept/reject the prompt by saving some preferences on the test machines. That said, I'm honestly not seeing much value here. For that reason, I'm inclined to delete the test instead. Other reasons to delete mentioned in the review: * this API may require a secure context, but our test runner doesn't default to use https yet * this API requires a [user gesture in FF][2] [1]: https://github.com/dart-lang/sdk/issues/32002 [2]: https://developer.mozilla.org/en-US/docs/Web/API/Notification/requestPermission_static Change-Id: Ib8d372b2ebfb5859a0a9bc82d50a14ad8badeab1 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/332320 Reviewed-by: Srujan Gaddam <srujzs@google.com> Commit-Queue: Sigmund Cherem <sigmund@google.com>
This commit is contained in:
parent
c439c97b80
commit
e0112afc70
|
@ -1,14 +0,0 @@
|
|||
// Copyright (c) 2017, the Dart project authors. Please see the AUTHORS file
|
||||
// for details. All rights reserved. Use of this source code is governed by a
|
||||
// BSD-style license that can be found in the LICENSE file.
|
||||
|
||||
import 'dart:html';
|
||||
|
||||
import 'package:async_helper/async_minitest.dart';
|
||||
|
||||
main() {
|
||||
test('Notification.requestPermission', () async {
|
||||
String permission = await Notification.requestPermission();
|
||||
expect(permission, isNotNull);
|
||||
});
|
||||
}
|
|
@ -21,7 +21,6 @@ html/custom_element_name_clash_test: Skip # Issue 29922
|
|||
html/custom_elements_23127_test: Skip # Issue 29922
|
||||
html/custom_elements_test: Skip # Issue 29922
|
||||
html/interactive_geolocation_test: Skip # Requires allowing geo location.
|
||||
html/notification_permission_test: Skip # Issue 32002
|
||||
isolate/*: SkipByDesign # No support for dart:isolate in dart4web (http://dartbug.com/30538)
|
||||
js/js_util/js_prefix_test: SkipByDesign # JS$ prefix not implemented on ddc.
|
||||
mirrors/*: SkipByDesign # Mirrors not supported on web in Dart 2.0.
|
||||
|
|
|
@ -1,16 +0,0 @@
|
|||
// Copyright (c) 2017, the Dart project authors. Please see the AUTHORS file
|
||||
// for details. All rights reserved. Use of this source code is governed by a
|
||||
// BSD-style license that can be found in the LICENSE file.
|
||||
|
||||
// @dart = 2.9
|
||||
|
||||
import 'dart:html';
|
||||
|
||||
import 'package:async_helper/async_minitest.dart';
|
||||
|
||||
main() {
|
||||
test('Notification.requestPermission', () async {
|
||||
String permission = await Notification.requestPermission();
|
||||
expect(permission, isNotNull);
|
||||
});
|
||||
}
|
|
@ -21,7 +21,6 @@ html/custom_element_name_clash_test: Skip # Issue 29922
|
|||
html/custom_elements_23127_test: Skip # Issue 29922
|
||||
html/custom_elements_test: Skip # Issue 29922
|
||||
html/interactive_geolocation_test: Skip # Requires allowing geo location.
|
||||
html/notification_permission_test: Skip # Issue 32002
|
||||
isolate/*: SkipByDesign # No support for dart:isolate in dart4web (http://dartbug.com/30538)
|
||||
js/js_util/js_prefix_test: SkipByDesign # JS$ prefix not implemented on ddc.
|
||||
mirrors/*: SkipByDesign # Mirrors not supported on web in Dart 2.0.
|
||||
|
|
Loading…
Reference in a new issue