Skip test if there is no working spell checker #2

Merged
merged 1 commit into from Oct 27, 2011

Conversation

Projects
None yet
2 participants
Contributor

rwstauner commented Oct 26, 2011

Test::Spelling does this under the hood
which causes confusing behavior when Test::Tester doesn't know.

This fixes test failures like this one:
http://www.cpantesters.org/cpan/report/4a36b5f8-ffee-11e0-a9af-445633b12857

@rwstauner rwstauner Skip the test if there is no working spellchecker
Test::Spelling does this under the hood
which causes confusing behavior when Test::Tester doesn't know.
7ea4d76

@xenoterracide xenoterracide added a commit that referenced this pull request Oct 27, 2011

@xenoterracide xenoterracide Merge pull request #2 from rwstauner/has-spellchecker
Skip test if there is no working spell checker
b6e44a9

@xenoterracide xenoterracide merged commit b6e44a9 into xenoterracide:master Oct 27, 2011

Owner

xenoterracide commented Oct 27, 2011

would this be at all required in the generated pod-spell.t? or will that do fine on it's own.

Contributor

rwstauner commented Oct 27, 2011

No, the actual call to all_pod_files_ok does that check (according to the docs),
it just does it too late when wrapped in Test::Tester.

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