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 test coverage for special cases #18

Merged
merged 4 commits into from Jun 5, 2017

Conversation

Projects
None yet
2 participants
@simbabque
Contributor

simbabque commented Jun 4, 2017

This PR adds coverage for CODE reference relations, proxy relationships and a bunch small things. I tried adding tests for failing tests as well but ran into some trouble with Test::Builder::Tester. I might add more commits with a different approach later or create a second PR.

This is my entry for the pull request challenge 2017 in June.

simbabque added some commits Jun 4, 2017

test that we TODO correctly for CODE ref relations
This went into its own .t file because that seemed more clear than to hack it into the missing_methods test. If at some point this behavior gets added it will also be easier to adapt the test.

@chiselwright chiselwright merged commit c64becb into chiselwright:master Jun 5, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@simbabque simbabque deleted the simbabque:more-tests branch Jun 5, 2017

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