Skip to content

Dancer::Test and params()#962

Closed
tomhukins wants to merge 3 commits intoPerlDancer:develfrom
tomhukins:param_and_test
Closed

Dancer::Test and params()#962
tomhukins wants to merge 3 commits intoPerlDancer:develfrom
tomhukins:param_and_test

Conversation

@tomhukins
Copy link
Contributor

This helps people who mistakenly assume they can pass a source to param() as
permitted by params().
Despite what the documentation says, "body" can contain a hash reference as
well as a string.  See t/23_dancer_tests/02_tests_functions.t for an example
use.
@yanick yanick closed this in ec60c07 Sep 28, 2013
@yanick
Copy link
Contributor

yanick commented Sep 28, 2013

Merged. Thanks!

I've however didn't put in the param() check. As we don't do argument checking in the other functions, I just have to think a little longer on how we want to do that.

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.

2 participants