Bug 182769

Summary: [WebAuthN] Improve testing configuration of WebAuthN
Product: WebKit Reporter: Jiewen Tan <jiewen_tan>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: alex.gaynor, jiewen_tan, jonathan, jschoi
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 191529, 191531, 205839, 189283, 189877, 191530, 202560, 202565    
Bug Blocks: 181943    

Description Jiewen Tan 2018-02-13 22:00:21 PST
Figure out a way to auto-test platform specific procedures in WebCredentialsMessengerProxy. Currently only MockWebCredentialsMessenger is crafted to test things within WebProcess. However, there is nothing out there to test the UIProcess corresponding parts.
Comment 1 Jiewen Tan 2018-03-09 15:06:01 PST
1. We will have to convert unit tests of LocalAuthenticator into proper API tests such that message parameters encoding and decoding between WebProcess and UIProcess can be properly tested as well.
Comment 2 Jiewen Tan 2018-03-09 15:06:51 PST
(In reply to Jiewen Tan from comment #0)
> Figure out a way to auto-test platform specific procedures in
> WebCredentialsMessengerProxy. Currently only MockWebCredentialsMessenger is
> crafted to test things within WebProcess. However, there is nothing out
> there to test the UIProcess corresponding parts.

UIProcess parts are partially covered by unit tests now
Comment 3 Jiewen Tan 2018-03-14 14:01:02 PDT
2. Needs manual tests for every step to determine if there is more prompt for user consent after the first one.
Comment 4 Jiewen Tan 2018-09-27 14:21:11 PDT
3. It might be good to import test cases from Chrome/Firefox as well.
Comment 5 Jiewen Tan 2018-12-20 16:22:50 PST
We should also try to improve numbers in:
https://w3c.github.io/test-results/webauthn/all.html.
Comment 6 Jiewen Tan 2019-10-09 15:28:15 PDT
We should probably gather all API tests into a single place.
Comment 7 Jiewen Tan 2019-10-09 15:29:04 PDT
We should visit the WebDriver virtual authenticator extension as well.