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

Tests fail (with YAML 1.21?) #7

Open
eserte opened this issue Jan 7, 2017 · 5 comments · May be fixed by #8
Open

Tests fail (with YAML 1.21?) #7

eserte opened this issue Jan 7, 2017 · 5 comments · May be fixed by #8

Comments

@eserte
Copy link

eserte commented Jan 7, 2017

The test suite started to fail on my smokers:

#   Failed test at t/02_diff_fully-pp.t line 16.
#     Structures begin differing at:
#          $got->[1] = undef
#     $expected->[1] = ARRAY(0x22d44e0)
# Looks like you failed 1 test of 13.
t/02_diff_fully-pp.t .... 
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/13 subtests 

#   Failed test at t/02_diff_fully.t line 15.
#     Structures begin differing at:
#          $got->[1] = undef
#     $expected->[1] = ARRAY(0x26818b8)
# Looks like you failed 1 test of 13.
t/02_diff_fully.t ....... 
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/13 subtests 

Statistical analysis suggests that the problem happens with YAML 1.21:

****************************************************************
Regression 'mod:YAML'
****************************************************************
Name           	       Theta	      StdErr	 T-stat
[0='const']    	      1.0000	      0.0000	354079998524074496.00
[1='eq_1.17']  	      0.0000	      0.0000	   1.71
[2='eq_1.18']  	     -0.0000	      0.0000	  -3.91
[3='eq_1.19']  	     -0.0000	      0.0000	  -1.00
[4='eq_1.20']  	      0.0000	      0.0000	   3.42
[5='eq_1.21']  	     -1.0000	      0.0000	-155665306853043200.00

R^2= 1.000, N= 144, K= 6
****************************************************************
@ambs
Copy link

ambs commented Jan 31, 2017

I can confirm this issue.
Correction is welcome. :-)

charsbar added a commit to charsbar/p5-String-Diff that referenced this issue Apr 5, 2017
@jkeenan
Copy link

jkeenan commented Apr 4, 2018

These test failures persist when you attempt to install the library against perl-5.27.10 on FreeBSD-11.0, using cpanm as the installer. Here is date from the build.log:

{
  author => "YAPPO",
  dist => "String-Diff",
  distname => "String-Diff-0.07",
  distversion => 0.07,
  grade => "FAIL",
  prereqs => undef,
  test_output => [
    "Building and testing String-Diff-0.07",
    "cp lib/String/Diff.pm blib/lib/String/Diff.pm",
    "t/00_compile-pp.t ....... ok",
    "t/00_compile.t .......... ok",
    "t/01_export-pp.t ........ ok",
    "t/01_export.t ........... ok",
    "",
    "#   Failed test at t/02_diff_fully-pp.t line 16.",
    "#     Structures begin differing at:",
    "#          \$got->[1] = undef",
    "#     \$expected->[1] = ARRAY(0x8036854f8)",
    "# Looks like you failed 1 test of 13.",
    "t/02_diff_fully-pp.t .... ",
    "Dubious, test returned 1 (wstat 256, 0x100)",
    "Failed 1/13 subtests ",
    "",
    "#   Failed test at t/02_diff_fully.t line 15.",
    "#     Structures begin differing at:",
    "#          \$got->[1] = undef",
    "#     \$expected->[1] = ARRAY(0x80367e4c8)",
    "# Looks like you failed 1 test of 13.",
    "t/02_diff_fully.t ....... ",
    "Dubious, test returned 1 (wstat 256, 0x100)",
    "Failed 1/13 subtests ",
    "t/03_diff-pp.t .......... ok",
    "t/03_diff.t ............. ok",
    "t/04_diff_merge-pp.t .... ok",
    "t/04_diff_merge.t ....... ok",
    "t/05_diff_regexp-pp.t ... ok",
    "t/05_diff_regexp.t ...... ok",
    "t/06_escape.t ........... ok",
    "t/release-pod-syntax.t .. skipped: these tests are for release candidate testing",
    "",
    "Test Summary Report",
    "-------------------",
    "t/02_diff_fully-pp.t  (Wstat: 256 Tests: 13 Failed: 1)",
    "  Failed test:  2",
    "  Non-zero exit status: 1",
    "t/02_diff_fully.t     (Wstat: 256 Tests: 13 Failed: 1)",
    "  Failed test:  2",
    "  Non-zero exit status: 1",
    "Files=14, Tests=137,  1 wallclock secs ( 0.05 usr  0.02 sys +  1.11 cusr  0.27 csys =  1.45 CPU)",
    "Result: FAIL",
  ],
  via => "App::cpanminus::reporter 0.17 (1.7043)",
}

Other CPAN distributions depend on getting this one installed. We will be unable to assess the perl-5.28-readiness of those distributions until this one is itself perl-5.28-ready.

Thank you very much.
Jim Keenan

@neo1ite
Copy link

neo1ite commented Oct 13, 2020

I fix it in my fork: neo1ite@83f67e5

@jkeenan
Copy link

jkeenan commented Oct 13, 2020

I fix it in my fork: neo1ite@83f67e5

Thanks. When can we expect a new CPAN release?

Thank you very much.
Jim Keenan

@Sadrak
Copy link

Sadrak commented Mar 25, 2022

@yappo please release a new version with a fix.

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

Successfully merging a pull request may close this issue.

5 participants