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

Elm-test hangs indefinitely #295

Closed
smucode opened this issue Oct 1, 2018 · 5 comments
Closed

Elm-test hangs indefinitely #295

smucode opened this issue Oct 1, 2018 · 5 comments
Labels

Comments

@smucode
Copy link

smucode commented Oct 1, 2018

Perhaps every 20th build on our CI, elm-test (0.19.0-beta9) hangs. It just stops right after these messages:

elm-test --verbose
Running elm make /builds/<snip path>/FooTest.elm <snip more tests>/builds/<snip path>/BarTest.elm --output /dev/null
Running elm make /builds/<snip path>/elm-stuff/generated-code/elm-explorations/test/src/Test/Generated/Main2122303807.elm --output /builds/<snip path>/elm-stuff/generated-code/elm-explorations/test/elmTestOutput.js

Aborting and retrying the tests works.

This was not a problem with 0.18 and the previous elm-test version.

We have no more than 100 test cases in 10 files.

@rtfeldman
Copy link
Owner

Thanks! I'm assuming this is a duplicate of #288 but I appreciate having a separate issue - more data is better! 👍

@rtfeldman
Copy link
Owner

I think c123c8 introduced this, but I've had no luck isolating why. It only seems to happen on Travis CI, not on local Linux systems. I'm gonna try rewriting that commit incrementally, pushing to Travis whenever I get it to a state where it works locally again, and see if I can isolate the part of that refactor that introduced the problem.

@bruckner
Copy link

I'm hitting this while running locally (not in CI) with elm-test 0.19.0-beta9 on macos 10.14 Mojave with npm 5.4.2.

Fresh install, ran a half dozen times no problem, now hanging every run. Small test suite with two tests, was running sub-second.

@rtfeldman
Copy link
Owner

@Brucker would it be possible for you to share the tests in question?

@bruckner
Copy link

@rtfeldman It was the tests copied below. I downgraded to beta8 and that worked. Just upgraded to beta10 and so far so good.

suite : Test
suite =
    describe "RelativeTime"
        [ describe "formatTime"
            [ test "Formats midnight correctly" <|
                \_ ->
                    let
                        midnight =
                            millisToPosix 0

                        result =
                            RelativeTime.formatTime utc midnight
                    in
                    Expect.equal "12:00 am" result
            , test "Formats noon correctly" <|
                \_ ->
                    let
                        noon =
                            millisToPosix (12 * 3600 * 1000)

                        result =
                            RelativeTime.formatTime utc noon
                    in
                    Expect.equal "12:00 pm" result
            ]
        ]

mostlyobvious added a commit to RailsEventStore/rails_event_store that referenced this issue Jan 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants