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

Test fixes #10

Merged
merged 2 commits into from Jun 2, 2017

Conversation

Projects
None yet
2 participants
@mjg17

mjg17 commented May 25, 2017

I'm taking part in the 2017 CPAN Pull Request Challenge and my May assignment is POE-Component-MessageQueue.

I saw that some CPANTS test runs were failing on 05_storage_api.t because the port configured in the test is already in use on the test machine. I've fixed this.

I've also fixed a perl diagnostic warning which was being emitted during each run of 05_storage_api.t.

I hope these are useful.

All the best
Michael

@ironcamel

This comment has been minimized.

Show comment
Hide comment
@ironcamel

ironcamel Jun 2, 2017

Owner

This is great, thanks!

Owner

ironcamel commented Jun 2, 2017

This is great, thanks!

@ironcamel ironcamel merged commit 4f9c042 into ironcamel:master Jun 2, 2017

@mjg17

This comment has been minimized.

Show comment
Hide comment
@mjg17

mjg17 Jun 4, 2017

I'm glad it's useful, thanks for merging!

mjg17 commented Jun 4, 2017

I'm glad it's useful, thanks for merging!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment