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

Add simulated mode. Closes #167 #168

Merged
merged 2 commits into from Nov 20, 2015
Merged

Add simulated mode. Closes #167 #168

merged 2 commits into from Nov 20, 2015

Conversation

@hueniverse
Copy link
Member

hueniverse commented Nov 20, 2015

No description provided.

@hueniverse hueniverse added the feature label Nov 20, 2015
API.md Outdated
it is in this mode will return the simulated credentials.

The `Bell.simulate(credetialsFunc)` takes a single argument:
- `credetialsFunc` - a function called for each incoming request to the protected resource with the signature `function(request, next)` where:

This comment has been minimized.

Copy link
@ldesplat

ldesplat Nov 20, 2015

Contributor

credentialsFunc

@ldesplat ldesplat added this to the 6.1.0 milestone Nov 20, 2015
@hueniverse

This comment has been minimized.

Copy link
Member Author

hueniverse commented Nov 20, 2015

Fixed, but I am not sure this makes sense anymore... :-)

I forgot I can just pass the credentials I want to server.inject(). Feel free to merge this but if you are testing your server with server.inject() you don't actually need it. If you need to start your server to test it with an actual connection, this might be useful.

@ldesplat

This comment has been minimized.

Copy link
Contributor

ldesplat commented Nov 20, 2015

I'll merge it since it sounds like it's useful. I won't make a release just yet unless you change your mind and need it right now :)

ldesplat added a commit that referenced this pull request Nov 20, 2015
Add simulated mode. Closes #167
@ldesplat ldesplat merged commit 343b617 into master Nov 20, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@ldesplat

This comment has been minimized.

Copy link
Contributor

ldesplat commented Nov 20, 2015

Thank you! Can't believe I forgot!

@hueniverse hueniverse deleted the test-mode branch Nov 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.