Skip to content
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

unreliable tests #14

Closed
p-alik opened this issue Apr 12, 2017 · 4 comments
Closed

unreliable tests #14

p-alik opened this issue Apr 12, 2017 · 4 comments
Assignees

Comments

@p-alik
Copy link
Owner

p-alik commented Apr 12, 2017

  1. t/14-sleep.t
    When I was installing/testing 2.004.001 using 'cpan', t/14-sleep.t hanged
    during either test back to gearman org #4 or Add missing LICENSE meta name in the Makefile.PL script #5 ("4/?" was displayed on the screen).... After I
    gave up and hit control-C, I ran the test again, and then it worked.
  2. t/50-wait_timeout.t
    http://www.cpantesters.org/cpan/report/5db73e0c-1589-11e7-807f-f398cc11fead
    #   Failed test 'timeout'
    #   at /home/sand/src/perl/repoperls/installed-perls/host/k93msid/v5.25.3/f11c/lib/site_perl/5.25.3/Test/Timer.pm line 24.
    # Test exceeded specified threshold
    # Looks like you failed 1 test of 10.

#   Failed test 'wait with timeout'
#   at t/50-wait_timeout.t line 65.
# Looks like you failed 1 test of 4.
t/50-wait_timeout.t ............. 
@p-alik p-alik self-assigned this Apr 12, 2017
@esabol
Copy link
Contributor

esabol commented Apr 13, 2017

Just some context for (1) in the above post: It occurred on CentOS 7.3 using /usr/bin/perl (v5.16.3) when tested using the cpan command. I could not reproduce it running "make test" from the command line outside of cpan, but I didn't try multiple times either.

@esabol
Copy link
Contributor

esabol commented Oct 26, 2017

These tests have been quite reliable lately. I'm not sure what caused them to be unreliable before, but I'm satisfied that this is likely no longer a problem. Are you still getting t/50-wait_timeout.t failures on cpantesters? If not, feel free to close this issue.

@p-alik
Copy link
Owner Author

p-alik commented Oct 27, 2017

http://matrix.cpantesters.org/?dist=Gearman%202.004.009 doesn't got some fail reports for 2.004.009. Here are latest reports with regard to 50-wait_timeout.t.

Gearman-2.004.007:

@p-alik
Copy link
Owner Author

p-alik commented Oct 27, 2017

we'll reopen the issue if necessary

@p-alik p-alik closed this as completed Oct 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants