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_trap.t fails (if WWW::Mechanize::Cached is too old?) #76

Closed
eserte opened this issue Mar 23, 2017 · 6 comments
Closed

t/ua_trap.t fails (if WWW::Mechanize::Cached is too old?) #76

eserte opened this issue Mar 23, 2017 · 6 comments

Comments

@eserte
Copy link

eserte commented Mar 23, 2017

Some of my smokers show the following test failure:

Use of uninitialized value in concatenation (.) or string at /usr/perl5.18.4p/lib/site_perl/5.18.4/HTTP/Tiny.pm line 593.

#   Failed test 'No illegal methods called'
#   at t/ua_trap.t line 45.
#          got: 'Can't use string ("default_expires_in") as a HASH ref while "strict refs" in use at /usr/perl5.18.4p/lib/site_perl/5.18.4/Cache/BaseCache.pm line 430.
# '
#     expected: undef
# Can't use string ("default_expires_in") as a HASH ref while "strict refs" in use at /usr/perl5.18.4p/lib/site_perl/5.18.4/Cache/BaseCache.pm line 430.
# Looks like you failed 1 test of 1.
t/ua_trap.t ............. 
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/1 subtests 

This seems to happen if all prereqs for running ua_trap.t are met, but WWW::Mechanize::Cached is too old (<1.50?). Statistical analysis:

(1)
****************************************************************
Regression 'fail:t/ua_trap.t'
****************************************************************
Name                   Theta          StdErr     T-stat
[0='const']           1.0000          0.0000    38772244432185280.00
[1='eq_1']           -1.0000          0.0000    -7078810960203983.00

R^2= 1.000, N= 60, K= 2
****************************************************************
(2)
****************************************************************
Regression 'qr:(Failed test\s+\S+.*)'
****************************************************************
Name                   Theta          StdErr     T-stat
[0='const']           1.0000          0.0000    38772244432185280.00
[1='eq_Failed test 'No illegal methods called'']             -1.0000          0.0000    -7078810960203983.00

R^2= 1.000, N= 60, K= 2
****************************************************************
(3)
****************************************************************
Regression 'qr:WWW::Mechanize::Cached\s+\S+\s+(.*)'
****************************************************************
Name                   Theta          StdErr     T-stat
[0='const']           0.0000          0.0000       5.07
[1='eq_1.50']         1.0000          0.0000    6859754891225717.00
[2='eq_missing']              1.0000          0.0000    6668443905700334.00

R^2= 1.000, N= 60, K= 3
****************************************************************
oalders added a commit that referenced this issue Mar 23, 2017
@oalders
Copy link
Member

oalders commented Mar 23, 2017

Thanks @eserte!

@eserte
Copy link
Author

eserte commented Mar 25, 2017

Still I get failures. In the test log I see the following:

# === Test Recommends ===
# 
#     Module                     Want     Have
#     ---------------------- -------- --------
...
#     WWW::Mechanize::Cached     1.50     1.47

The prerequisite version is not met, but it's only a recommended module, so CPAN.pm does not do an upgrade. I think the test should additionally check for the version, i.e. adding 1.50 to the Test::Requires call in t/uaprof.t.

@mickeyn mickeyn reopened this Mar 25, 2017
@mickeyn
Copy link
Contributor

mickeyn commented Mar 28, 2017

@oalders ?

@oalders
Copy link
Member

oalders commented Mar 28, 2017

@mickeyn see #80

@mickeyn
Copy link
Contributor

mickeyn commented Mar 30, 2017

@eserte is this problem solved with the latest release?

@mickeyn
Copy link
Contributor

mickeyn commented Apr 27, 2017

Assuming this is no longer relevant (test is not failing anymore in new versions) - closing.
Please reopen if problem reappears.

@mickeyn mickeyn closed this as completed Apr 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

3 participants