Skip to content

/ browserid-local-verify

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

write (t)ests to (d)rive (d)evelopment #1

Closed
lloyd opened this issue Nov 15, 2013 · 1 comment
Closed

write (t)ests to (d)rive (d)evelopment #1

lloyd opened this issue Nov 15, 2013 · 1 comment

Comments

@lloyd
Copy link
Contributor

@lloyd lloyd commented Nov 15, 2013

  • rsa: RSA various lengths: verify a properly formulated assertion from a supported secondary
  • dsa: DSA various lengths: verify a properly formulated assertion from a supported secondary
  • assertion verification: basic IdP / Fallback issued assertions should verify
  • secondaries: verify failure of an assertion from an unsupported secondary
  • time checking: expired or future issued assertions and certificates should be detected
  • domain authority: a fallback may not speak for a domain that supports persona (in the absence of trustedIssuers)
  • audience-matching: audience matching tests
  • lookup: using built in HTTP
  • lookup: over-riding request function
  • lookup: HTTP redirect handling
  • lookup: HTTP timeout
  • delegation: too many redirects
  • delegation: cycle detection
  • delegation: basic success case
  • well-known: malformed public key
  • well-known: missing required fields
  • well-known: parsing disabled: true
  • regression: regression tests from a set of assertions issued in the past
  • domain GET param: should always pass in domain= get param for dynamic well-knowns (required for bridging support)
  • idp-claims: extraction of idp custom extensions - new fields like uid
  • unverified email: (specific case of idp-claims) test to prove the library is flexible enough to support an extension like unverified-email
@lloyd
Copy link
Contributor Author

@lloyd lloyd commented Nov 22, 2013

initial test implementation is complete, at 90% coverage with all known required features implemented.
@lloyd lloyd closed this Nov 22, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
@lloyd