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 coverage enhancements #6

Open
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@openstrike

openstrike commented Jul 23, 2015

There are some small changes to the test suite here which improve the coverage a little. The major part of this is the creation of t/compat.t based on the existing compat_pl script. The PR doesn't remove compat_pl yet to make it really easy to compare the two. If/when you're happy that none of the functionality has been lost there's probably no need to keep compat_pl in the dist.

The other changes are minor or cosmetic and are hopefully fairly self-explanatory but please ask if you are in any doubt about them.

openstrike added some commits Jul 9, 2015

Added "compat" tests.
These test the methods marked "Compat" in Authen/SASL.pm and replace the
compat_pl script.
Remove double declaration of $cb
This is a cosmetic change to remove the warning generated by t/server/login.t
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment