-
Notifications
You must be signed in to change notification settings - Fork 170
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
[t] use Test::APIcast from CPAN #528
Conversation
8bae095
to
205ef08
Compare
b1ba5ae
to
757d164
Compare
@@ -1,5 +1,5 @@ | |||
use lib 't'; | |||
use TestAPIcast 'no_plan'; | |||
use Test::APIcast 'no_plan'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This namespace looks a bit strange to me. Is it common in Perl to start testing libraries with Test::
?
I guess that's it, because we also have Test::Nginx
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep. And there is Test::Simple and Test::More.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
👍 |
757d164
to
521ae49
Compare
* install it with cpanminus * can be used outside APIcast repository
521ae49
to
5230f6e
Compare
be aware this increases the CI build time (s2i builder+runtime) significantly as there are dependencies that have to be pulled