Title: [238598] trunk/LayoutTests
Revision
238598
Author
jiewen_...@apple.com
Date
2018-11-27 18:16:25 -0800 (Tue, 27 Nov 2018)

Log Message

Unreviewed, test gardening

Add a time out parameter to the following test cases such that WebAuthN requests will timeout
themselves instead of letting the wpt wrapper or test runner kill them.

* http/wpt/webauthn/public-key-credential-create-success-hid.https.html:
* http/wpt/webauthn/public-key-credential-get-success-hid.https.html:
* platform/mac-wk2/TestExpectations:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (238597 => 238598)


--- trunk/LayoutTests/ChangeLog	2018-11-28 02:10:50 UTC (rev 238597)
+++ trunk/LayoutTests/ChangeLog	2018-11-28 02:16:25 UTC (rev 238598)
@@ -1,3 +1,14 @@
+2018-11-27  Jiewen Tan  <jiewen_...@apple.com>
+
+        Unreviewed, test gardening
+
+        Add a time out parameter to the following test cases such that WebAuthN requests will timeout
+        themselves instead of letting the wpt wrapper or test runner kill them.
+
+        * http/wpt/webauthn/public-key-credential-create-success-hid.https.html:
+        * http/wpt/webauthn/public-key-credential-get-success-hid.https.html:
+        * platform/mac-wk2/TestExpectations:
+
 2018-11-27  Chris Dumez  <cdu...@apple.com>
 
         Resync service-workers web platform tests from upstream

Modified: trunk/LayoutTests/http/wpt/webauthn/public-key-credential-create-success-hid.https.html (238597 => 238598)


--- trunk/LayoutTests/http/wpt/webauthn/public-key-credential-create-success-hid.https.html	2018-11-28 02:10:50 UTC (rev 238597)
+++ trunk/LayoutTests/http/wpt/webauthn/public-key-credential-create-success-hid.https.html	2018-11-28 02:16:25 UTC (rev 238598)
@@ -47,6 +47,7 @@
                 },
                 challenge: Base64URL.parse("MTIzNDU2"),
                 pubKeyCredParams: [{ type: "public-key", alg: -7 }],
+                timeout: 10
             }
         };
 
@@ -90,7 +91,8 @@
                 },
                 challenge: Base64URL.parse("MTIzNDU2"),
                 pubKeyCredParams: [{ type: "public-key", alg: -7 }],
-                authenticatorSelection: { requireResidentKey: false }
+                authenticatorSelection: { requireResidentKey: false },
+                timeout: 10
             }
         };
 
@@ -112,7 +114,8 @@
                 },
                 challenge: Base64URL.parse("MTIzNDU2"),
                 pubKeyCredParams: [{ type: "public-key", alg: -7 }],
-                authenticatorSelection: { userVerification: "preferred" }
+                authenticatorSelection: { userVerification: "preferred" },
+                timeout: 10
             }
         };
 
@@ -134,7 +137,8 @@
                 },
                 challenge: Base64URL.parse("MTIzNDU2"),
                 pubKeyCredParams: [{ type: "public-key", alg: -7 }],
-                authenticatorSelection: { userVerification: "discouraged" }
+                authenticatorSelection: { userVerification: "discouraged" },
+                timeout: 10
             }
         };
 
@@ -156,7 +160,8 @@
                 },
                 challenge: Base64URL.parse("MTIzNDU2"),
                 pubKeyCredParams: [{ type: "public-key", alg: -7 }],
-                authenticatorSelection: { authenticatorAttachment: "cross-platform", requireResidentKey: false, userVerification: "preferred" }
+                authenticatorSelection: { authenticatorAttachment: "cross-platform", requireResidentKey: false, userVerification: "preferred" },
+                timeout: 10
             }
         };
 

Modified: trunk/LayoutTests/http/wpt/webauthn/public-key-credential-get-success-hid.https.html (238597 => 238598)


--- trunk/LayoutTests/http/wpt/webauthn/public-key-credential-get-success-hid.https.html	2018-11-28 02:10:50 UTC (rev 238597)
+++ trunk/LayoutTests/http/wpt/webauthn/public-key-credential-get-success-hid.https.html	2018-11-28 02:16:25 UTC (rev 238598)
@@ -27,7 +27,8 @@
     promise_test(t => {
         const options = {
             publicKey: {
-                challenge: Base64URL.parse("MTIzNDU2")
+                challenge: Base64URL.parse("MTIzNDU2"),
+                timeout: 10
             }
         };
 
@@ -42,7 +43,8 @@
                 challenge: Base64URL.parse("MTIzNDU2"),
                 allowCredentials: [
                     { type: "public-key", id: Base64URL.parse(testHidCredentialIdBase64), transports: ["usb"] }
-                ]
+                ],
+                timeout: 10
             }
         };
 
@@ -55,7 +57,8 @@
         const options = {
             publicKey: {
                 challenge: Base64URL.parse("MTIzNDU2"),
-                userVerification: "preferred"
+                userVerification: "preferred",
+                timeout: 10
             }
         };
 
@@ -68,7 +71,8 @@
         const options = {
             publicKey: {
                 challenge: Base64URL.parse("MTIzNDU2"),
-                userVerification: "discouraged"
+                userVerification: "discouraged",
+                timeout: 10
             }
         };
 
@@ -84,7 +88,8 @@
                 allowCredentials: [
                     { type: "public-key", id: Base64URL.parse(testHidCredentialIdBase64), transports: ["usb"] }
                 ],
-                userVerification: "preferred"
+                userVerification: "preferred",
+                timeout: 10
             }
         };
 

Modified: trunk/LayoutTests/platform/mac-wk2/TestExpectations (238597 => 238598)


--- trunk/LayoutTests/platform/mac-wk2/TestExpectations	2018-11-28 02:10:50 UTC (rev 238597)
+++ trunk/LayoutTests/platform/mac-wk2/TestExpectations	2018-11-28 02:16:25 UTC (rev 238598)
@@ -906,8 +906,6 @@
 # The following tests require PublicKeyCredential objects. We can only produce them via local authenticators now.
 http/wpt/credential-management/credentialscontainer-store-basics.https.html [ Skip ]
 http/wpt/webauthn/idl.https.html [ Skip ]
-# Slow WebAuthN tests.
-http/wpt/webauthn/public-key-credential-get-success-hid.https.html [ Slow ]
 
 webkit.org/b/183878 media/modern-media-controls/tracks-panel/tracks-panel-up-click-over-media-does-not-dimiss-media-controls-when-media-is-playing.html [ Pass Timeout ] 
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to