-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Testsets for numbers #23862
Testsets for numbers #23862
Conversation
I think I managed to screw something up in this 3000+ line file... |
Looks correct to me. The failure is weird though:
|
Reduced to the following: @testset begin
@test fld(0.3,0.01) == 29.0
@test div(0.3,0.01) == 29.0
end The above two tests fail while inside a julia> @testset begin
@test fld(0.3,0.01) == 29.0
@test div(0.3,0.01) == 29.0
end
test set: Test Failed
Expression: fld(0.3, 0.01) == 29.0
Evaluated: 30.0 == 29.0
Stacktrace:
[1] macro expansion at ./REPL[2]:2 [inlined]
[2] macro expansion at ./test.jl:942 [inlined]
[3] anonymous at ./<missing>:?
test set: Test Failed
Expression: div(0.3, 0.01) == 29.0
Evaluated: 30.0 == 29.0
Stacktrace:
[1] macro expansion at ./REPL[2]:3 [inlined]
[2] macro expansion at ./test.jl:942 [inlined]
[3] anonymous at ./<missing>:?
Test Summary: | Fail Total
test set | 2 2
ERROR: Some tests did not pass: 0 passed, 2 failed, 0 errored, 0 broken. Without the testset they pass just fine: julia> @test fld(0.3,0.01) == 29.
Test Passed
julia> @test div(0.3,0.01) == 29.0
Test Passed |
lol wut |
Interesting fact: Adding the testsets in this file increased runtime with 14%, from 70 to 80 seconds. |
#23796 × 2 and also an (unrelated?) failure in ranges on circle CI i686:
What's up with that? |
Circle CI does not restart when closing/opening PRs? Relevant tests passed anyway. |
No description provided.