kasei / attean

A Perl Semantic Web Framework
19 stars 10 forks source link

Test failure in term-map.t #102

Closed kjetilk closed 7 years ago

kjetilk commented 7 years ago

Hi!

I just did an pull on Attean and I get test failures:

kjetil@heffalump:/mnt/robin/kjetil/dev/attean$ prove -lv t/term-map.t 

ok 1 - short blank node label mapping

    #   Failed test at t/term-map.t line 54.
    #                   '_:0x6c43bdfd172e4a93a63c5492d6ebf69d'
    #     doesn't match '(?^u:^_:0x[0-9A-Z]{32}$)'

    #   Failed test at t/term-map.t line 55.
    #                   '_:0xd87d70c4fc0f40b7bda0cb872d07d70a'
    #     doesn't match '(?^u:^_:0x[0-9A-Z]{32}$)'

    #   Failed test at t/term-map.t line 63.
    #                   '_:0x225a00f54ee2452ba43e20988237155b'
    #     doesn't match '(?^u:^_:0x[0-9A-Z]{32}$)'

    #   Failed test at t/term-map.t line 64.
    #                   '_:0x39b5ffe4988f45c78406b669ddcf7d69'
    #     doesn't match '(?^u:^_:0x[0-9A-Z]{32}$)'

    #   Failed test at t/term-map.t line 73.
    #                   '_:0x225a00f54ee2452ba43e20988237155b'
    #     doesn't match '(?^u:^_:0x[0-9A-Z]{32}$)'

    #   Failed test at t/term-map.t line 74.
    #                   '_:0x39b5ffe4988f45c78406b669ddcf7d69'
    #     doesn't match '(?^u:^_:0x[0-9A-Z]{32}$)'
    # Looks like you failed 6 tests of 6.
not ok 2 - UUID blank node label mapping

#   Failed test 'UUID blank node label mapping'
#   at t/term-map.t line 76.
# Looks like you failed 1 test of 2.
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/2 subtests 

Test Summary Report
-------------------
t/term-map.t (Wstat: 256 Tests: 2 Failed: 1)
  Failed test:  2
  Non-zero exit status: 1
Files=1, Tests=2,  0 wallclock secs ( 0.02 usr  0.00 sys +  0.34 cusr  0.02 csys =  0.38 CPU)
Result: FAIL
kasei commented 7 years ago

Weird. What version of perl is this?

kjetilk commented 7 years ago

On Wednesday 27. July 2016 09.26.23 Gregory Todd Williams wrote:

Weird. What version of perl is this?

Both: This is perl 5, version 20, subversion 2 (v5.20.2) built for x86_64-linux- gnu-thread-multi and This is perl 5, version 18, subversion 2 (v5.18.2)

Debian Jessie and Linux Mint 17.1 respectively.

Kjetil

kasei commented 7 years ago

I haven't fully nailed this down yet, but it looks like it might be due to a difference in case of the hex digits in those UUIDs. I get all uppercase UUIDs on perl v5.22.2 on ubuntu.

kjetilk commented 7 years ago

Hmmmm, so it might be a UUID library or something on my side that's old...?

kasei commented 7 years ago

Maybe. I think I can just relax the regex. I don't actually care about the case...

kasei commented 7 years ago

Oops. Those regexen allow A-Z instead of A-F. Still, give that a try and see if it fixes your failures.

kjetilk commented 7 years ago

On Thursday 28. July 2016 09.20.02 Gregory Todd Williams wrote:

Oops. Those regexen allow A-Z instead of A-F. Still, give that a try and see if it fixes your failures.

Yes, it does! Thanks!

Kjetil