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

t/ua-mode.t fails #28

Closed
jberger opened this issue Sep 6, 2016 · 6 comments

Comments

Projects
None yet
2 participants
@jberger
Copy link

commented Sep 6, 2016

As seen on cpan testers and I can report locally, t/ua-mode.t fails on the two user mode test attempts (4 total test failures, 2 per test attempt). I spent some time trying to debug but I couldn't figure out what was wrong.

@jberger

This comment has been minimized.

Copy link
Author

commented Sep 6, 2016

I can say for sure that the guard in Test::Mojo::IRC is firing its timeout.

@jhthorsen jhthorsen added the bug label Sep 6, 2016

@jhthorsen jhthorsen self-assigned this Sep 6, 2016

@jhthorsen

This comment has been minimized.

Copy link
Owner

commented Sep 6, 2016

Yeah, this one is tricky (at least for me), since it does not fail all_the_time

@jberger

This comment has been minimized.

Copy link
Author

commented Sep 6, 2016

It fails for me every time. Could it be a timing issue?

On Tue, Sep 6, 2016, 6:23 AM Jan Henning Thorsen notifications@github.com
wrote:

Yeah, this one is tricky (at least for me), since it does not fail
all_the_time


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#28 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAs6FWS5mqabRIIeq1A-EqOgHmGkYTZPks5qnU0ngaJpZM4J1csz
.

jhthorsen added a commit that referenced this issue Sep 6, 2016

jhthorsen added a commit that referenced this issue Sep 6, 2016

Released version 0.36
 - Fix t/ua-mode.t #28
@jhthorsen

This comment has been minimized.

Copy link
Owner

commented Sep 6, 2016

Soon available on CPAN. Thanks for reminding me 👍

@jhthorsen jhthorsen closed this Sep 6, 2016

@jhthorsen

This comment has been minimized.

Copy link
Owner

commented Sep 7, 2016

It's still failing, even if I can't make it fail locally:

http://www.cpantesters.org/cpan/report/0521b3d4-7481-11e6-8452-fb0924312fb7

while true; do prove -l t/ua-mode.t ; done

@jhthorsen jhthorsen reopened this Sep 7, 2016

jhthorsen added a commit that referenced this issue Sep 7, 2016

Released version 0.37
 - Skipping t/ua-mode.t for now. #28
@jberger

This comment has been minimized.

Copy link
Author

commented Sep 7, 2016

Honestly, give it a little bit too see if you get any more fails. In my
experience slaven's smokers do that sometimes. I suspect he downloads the
files as they are updated but reports them as they currently exist.

On Wed, Sep 7, 2016, 2:15 AM Jan Henning Thorsen notifications@github.com
wrote:

Reopened #28 #28.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#28 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAs6FaBnokzUDtRXncnVOCeeDD0roqH_ks5qnmSDgaJpZM4J1csz
.

@jhthorsen jhthorsen closed this Oct 30, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.