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 suite failure with ghc 8.4 #32

Closed
DanBurton opened this issue Mar 14, 2018 · 4 comments
Closed

test suite failure with ghc 8.4 #32

DanBurton opened this issue Mar 14, 2018 · 4 comments

Comments

@DanBurton
Copy link

It is unclear to me whether this test failure has significance or not.

Failures:

  tests/GenericsSpec.hs:62:
  1) Generics.T# has a valid Read1 instance
       uncaught exception: ErrorCall (Prelude.read: no parse
       CallStack (from HasCallStack):
         error, called at tests/GenericsSpec.hs:49:18 in main:GenericsSpec) (after 4 tests)
       *** Failed! (after 4 tests):
       Exception:
         Prelude.read: no parse
         CallStack (from HasCallStack):
           error, called at tests/GenericsSpec.hs:49:18 in main:GenericsSpec
       MkT2# {getT2# = 1, (##) = -1}

Randomized with seed 1253242578

Finished in 0.0331 seconds
23 examples, 1 failure
@RyanGlScott
Copy link
Collaborator

Thank you for noticing this. It turns out this revealed a GHC 8.4.1 bug.

Until I get a better understanding of what's going on here, I think it's probably best to keep this excluded from Stackage.

(Also, I really should figure out how to run this test suite via Travis...)

@DanBurton
Copy link
Author

I think I'll go against your advice and keep transformers-compat in the stackage nightly build plan for now. With a few exceptions (of test suites marked as skipped or expected failures for other reasons), the following list of transformer-compat's revdeps are known to build and pass their test suites:

- Glob
- adjunctions
- aeson-better-errors
- bifunctors
- bound
- bytes
- comonad
- constraints
- contravariant
- criterion
- cron
- deriving-compat
- distributive
- errors
- exceptions
- free
- graphs
- invariant
- jose-jwt
- kan-extensions
- keys
- lens
- linear
- machines
- microlens-mtl
- mmorph
- monad-control
- monad-coroutine
- monad-parallel
- monadic-arrays
- optparse-applicative
- pointed
- prompt
- quickcheck-instances
- recursion-schemes
- semigroupoids
- tagged
- these
- transformers-base
- unbound-generics

But if we get reports on any of these of weird behavior, we'll keep this issue in mind.

@RyanGlScott
Copy link
Collaborator

Sorry, I should have been more clear. I meant to say "keep the test suite off of Stackage"—the library itself is fine!

And the test suite failure is only due to an infelicity in GHC itself—transformers-compat is doing everything it's supposed to correctly. Moreover, it's an extreme corner case, so it's pretty unlikely others would notice this. (Indeed, it went unnoticed until after 8.4.1's release.)

@DanBurton
Copy link
Author

Ahh. Got it. Thanks for the clarification.

RyanGlScott added a commit that referenced this issue Mar 15, 2018
RyanGlScott added a commit that referenced this issue Mar 15, 2018
* Migrate tests to separate .cabal file

This is a hack, but a serviceable one. Fixes #24.

* Work around #14918

Re #32

* Don't run the tests if -f-generic-deriving

* Ugly, ugly hackery

* Typofix

* A new hope

* Setup up CPP properly
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

2 participants