Skip to content

Commit

Permalink
tests: don't use single-argument map (#291)
Browse files Browse the repository at this point in the history
Single-argument `map` is being removed from Julia after an analysis
of registered packages found that the change doesn't break the
functionality of any package. See JuliaLang/julia#35293 and
JuliaLang/julia#52631

That said, the Julia change breaks your test suite, so here's a PR that
fixes that.
  • Loading branch information
nsajko authored Jan 18, 2024
1 parent 5b3bd1d commit dbbf2f3
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions test/test_map.jl
Original file line number Diff line number Diff line change
Expand Up @@ -169,10 +169,8 @@ wp = WorkerPool(procs)
function constfun()
return 42
end
@test map(constfun) == @showprogress map(constfun)
@test broadcast(constfun) == @showprogress broadcast(constfun)
#@test mapreduce(constfun, error) == @showprogress mapreduce(constfun, error) # julia 1.2+
@showprogress foreach(printlnconstfun)


# #136: make sure mid progress shows up even without sleep
Expand Down

0 comments on commit dbbf2f3

Please sign in to comment.