mojolicious / mojo-pg

Mojolicious :heart: PostgreSQL
https://metacpan.org/release/Mojo-Pg
Artistic License 2.0
101 stars 46 forks source link

t/database.t fails after update from 4.22 to 4.24 #76

Closed afresh1 closed 3 years ago

afresh1 commented 3 years ago

Packages that might matter:

Steps to reproduce the behavior

Updating the OpenBSD port from 4.22 to 4.24 (with a local update of Mojolicious from 8.22 to 8.73, although this test also fails with 8.22 as does migrations.t as "curfile" doesn't exist yet) and running make test in the port directory fails. It uses a helper to set up a Postgres instance to test against.

Expected behavior

Tests should pass.

Actual behavior

p5-Mojo-Pg-4.22-test.log p5-Mojo-Pg-4.24-test.log

I did test 4.23 and tests also passed there.

    #   Failed test 'right channel name'
    #   at t/database.t line 260.
    #          got: 'dbtest'
    #     expected: 'dbtest2'

    #   Failed test 'no payload'
    #   at t/database.t line 262.
    #          got: ''
    #     expected: 'bar'
    # Looks like you failed 2 tests of 19.

#   Failed test 'Notifications'
#   at t/database.t line 279.
# Looks like you failed 1 test of 21.
kraih commented 3 years ago

Can't replicate the problem.

afresh1 commented 3 years ago

It must be a local problem then. I will try again with a different setup.

afresh1 commented 3 years ago

Tested and it works fine on my OpenBSD current amd64 with perl v5.30.3. Sorry to be a bother.