8000 Fix AuthenticationExtensionsClientInputs Example json prop name for conformance tests by yedidyas · Pull Request #404 · passwordless-lib/fido2-net-lib · GitHub
[go: up one dir, main page]

Skip to content

Conversation

yedidyas
Copy link
Contributor

expected value:
{"example.extension.bool":true}

Actual current value:
{"example.extension":true}

This fixes the first test:
[Server-ServerPublicKeyCredentialCreationOptions-Req-1 Test server generating ServerPublicKeyCredentialCreationOptionsRequest](https://github.com/Applications/FIDO%20Alliance%20-%20Certification%20Conformance%20Testing%20Tools.app/Contents/Resources/app.asar/app/index.html?grep=%0A%0A%20%20%20%20%20%20%20%20Server-ServerPublicKeyCredentialCreationOptions-Req-1%0A%0A%20%20%20%20%20%20%20%20Test%20server%20generating%20ServerPublicKeyCredentialCreationOptionsRequest%0A%0A%20%20%20%20)

@codecov-commenter
Copy link

Codecov Report

Merging #404 (ee0815f) into master (53caf81) will not change coverage.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##           master     #404   +/-   ##
=======================================
  Coverage   74.61%   74.61%           
=======================================
  Files          92       92           
  Lines        2714     2714           
  Branches      453      453           
=======================================
  Hits         2025     2025           
  Misses        575      575           
  Partials      114      114           
Impacted Files Coverage Δ
...ls/Objects/AuthenticationExtensionsClientInputs.cs 14.28% <ø> (ø)

@abergs abergs merged commit 02772cc into passwordless-lib:master Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0