Skip to content

digital-credentials: test conditional mediation rejects with TypeError#59087

Closed
marcoscaceres wants to merge 1 commit into
masterfrom
dc-conditional-mediation-typeerror
Closed

digital-credentials: test conditional mediation rejects with TypeError#59087
marcoscaceres wants to merge 1 commit into
masterfrom
dc-conditional-mediation-typeerror

Conversation

@marcoscaceres

Copy link
Copy Markdown
Contributor

Per Credential Management spec step 8a, conditional mediation is not supported for digital credentials and must reject with TypeError.

This PR:

  • Removes "conditional" from the mediation-ignored test loop (it's not ignored — it's rejected)
  • Adds a dedicated test verifying that navigator.credentials.get() with mediation: "conditional" and a digital member rejects with TypeError

Related spec PR: w3c-fedid/digital-credentials#419

Per Credential Management spec step 8a, conditional mediation is not
supported for digital credentials and must reject with TypeError.

Remove 'conditional' from the mediation-ignored test loop and add a
dedicated test that verifies the TypeError rejection.
@marcoscaceres

Copy link
Copy Markdown
Contributor Author

Closing this PR. The underlying spec conflict between Credential Management (throws TypeError for conditional + unsupporting type) and Digital Credentials (says MUST NOT throw) is being resolved upstream:

Once both land, the correct behavior is: no TypeError for mediation: "conditional" with digital credentials (mediation is implicitly required regardless). A new WPT test reflecting the updated behavior can be written then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants