Skip to content
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

OpenID spec is empty #6

Closed
jamesarosen opened this issue Jun 13, 2010 · 1 comment
Closed

OpenID spec is empty #6

jamesarosen opened this issue Jun 13, 2010 · 1 comment

Comments

@jamesarosen
Copy link
Contributor

No description provided.

@jamesarosen
Copy link
Contributor Author

First draft in 16ed90d. Still several failures because I'm not sure how to stub out the OpenID server properly.

Option 1: Stub the rack-openid portion

This seems like stubbing out too much and it ties us to using rack-openid as the consumer.

Option 2: Use rots

This is what rack-openid uses for testing. It seems to work well for that, but we would have to duplicate a lot of the test harness that test suite uses.

Option 3: use webmock

I like this because we're already using webmock and it seems to be the right amount to abstract. The hard part is getting the responses (bodies and header) all just right.

This issue was closed.
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

No branches or pull requests

1 participant