Skip to content
This repository has been archived by the owner on Oct 15, 2022. It is now read-only.

Remove failing tests, exception still causing failure #217

Merged
merged 1 commit into from Jan 29, 2015

Conversation

moollaza
Copy link
Member

I made a mistake putting the tests in a TODO, obviously an exception still get's thrown when the test runs, which causes the tests to end unexpectedly.

The failing tests are possibly related to dagolden/Path-Tiny#119 -- but I'm unable to reproduce the case where this fails right now. Will investigate further and bring back the tests with a fix.

For now this should get things working again for our devs

@moollaza
Copy link
Member Author

Managed to get a build log from a dev that shows the tests fail:

Skip blib/lib/auto/share/dist/App-DuckPAN/duckpan.js (unchanged)
PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-MTest::Harness" "-e" "undef *Test::Harness::Switches; test_harness(0, 'blib/lib', 'blib/arch')" t/*.t
t/app_duckpan.t ......... ok
t/load.t ................ ok
t/release-pod-syntax.t .. skipped: these tests are for release candidate testing
Error open (<) on '/tmp/TaUBtWJdBv/env.ini': No such file or directory at t/system_duckpan.t line 43.

#   Failed test 'env'
#   at /home/am/perl5/lib/perl5/Test/Builder.pm line 276.
# Tests were run but no plan was declared and done_testing() was not seen.
# Looks like your test exited with 2 just after 3.
t/system_duckpan.t ...... 
Dubious, test returned 2 (wstat 512, 0x200)
Failed 1/3 subtests 

Test Summary Report
-------------------
t/system_duckpan.t    (Wstat: 512 Tests: 3 Failed: 1)
  Failed test:  3
  Non-zero exit status: 2
  Parse errors: No plan found in TAP output
Files=4, Tests=40,  3 wallclock secs ( 0.05 usr  0.00 sys +  2.51 cusr  0.48 csys =  3.04 CPU)
Result: FAIL
Failed 1/4 test programs. 1/40 subtests failed.
make: *** [test_dynamic] Error 2
-> FAIL Installing http://duckpan.org/authors/id/M/MO/MOOLLAZA/App-DuckPAN-0.171.tar.gz failed. See /home/am/.cpanm/work/1422539169.2408/build.log for details. Retry with --force to force install it.

moollaza added a commit that referenced this pull request Jan 29, 2015
Remove failing tests, exception still causing failure
@moollaza moollaza merged commit e623b65 into master Jan 29, 2015
@moollaza moollaza deleted the zaahir/remove-tests branch January 29, 2015 14:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant