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

Test failure on ghc 9.10 #155

Open
eldritch-cookie opened this issue Oct 8, 2024 · 8 comments
Open

Test failure on ghc 9.10 #155

eldritch-cookie opened this issue Oct 8, 2024 · 8 comments

Comments

@eldritch-cookie
Copy link

katip-testsuite.log

@MichaelXavier
Copy link
Collaborator

Can you try this with 0.8.8.2? I tried running the test suite a few dozeN times and it didn't fail. The test log didn't have a seed value or anything so I don't know how to reproduce this.

@eldritch-cookie
Copy link
Author

yes i can run the testsuite on 0.8.8.2 it fails again katip-0.8.8.2-build.log
the test that failed is the one on line 163

testCase "Katip.Monadic.logLocM" $ do
not a quickcheck test, the error is a call to head so it probably is caused by
let loc = _itemLoc (head loggedItems)
but i have no idea why loggedItems is an empty list.

@9999years
Copy link
Contributor

Comes from here:

xs -> Just . toLoc . head $ filter filterKatip xs

@9999years
Copy link
Contributor

9999years commented Oct 24, 2024

Cabal changed the package names, which getLoc uses to filter out katip traces. With cabal-3.12:

       name   | srcLocPackage
entry: getLoc | katip-0.8.8.2-inplace
entry: logLocM | katip-0.8.8.2-inplace-test

With cabal-3.10:

entry: getLoc | katip-0.8.8.2-inplace
entry: logLocM | main

It's actually not related to 9.10 at all, you can reproduce on 9.8 with Cabal 3.12 or have tests pass on 9.10 with Cabal 3.10...

@MichaelXavier
Copy link
Collaborator

@9999years Do you understand the issue well enough to write up a fix? I'd be more than happy to merge and release it but I don't have much time to chase this down.

@9999years
Copy link
Contributor

@MichaelXavier I think factoring the tests out into a separate package whose name doesn't start with katip- would work.

@maksar
Copy link

maksar commented Nov 21, 2024

Also worth noting, that some golden files are not in the repo:

    Text-golden:                                                        OK
      Golden file did not exist; created
    Json-golden:                                                        OK
      Golden file did not exist; created

@juhp
Copy link

juhp commented Dec 11, 2024

Also failed for Stackage Nightly

juhp added a commit to commercialhaskell/stackage that referenced this issue Dec 11, 2024
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

5 participants