Skip to content

Commit

Permalink
Don't assert that macosx will have a JIT, becuase the brew package so…
Browse files Browse the repository at this point in the history
…mehow doesn't (#426)

See #426 for some meta, test isn't important enough to keep running it right now
  • Loading branch information
PragTob authored Feb 29, 2024
1 parent 2d56fba commit 3bcaade
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 0 deletions.
2 changes: 2 additions & 0 deletions test/benchee/system_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -143,6 +143,8 @@ defmodule Benchee.SystemTest do
end

# It may be compiled in a way that it doesn't but in the CI and dev machines it should be fine
# except for mac sometimes, where of course it isn't
@tag :guaranteed_jit
test ".jit_enabled? should say true for versions > 24.0.0" do
system_data = system(%Suite{}).system
jit_enabled? = system_data.jit_enabled?
Expand Down
10 changes: 10 additions & 0 deletions test/test_helper.exs
Original file line number Diff line number Diff line change
Expand Up @@ -54,4 +54,14 @@ exclusions =
exclusions
end

# somehow on CI macos doesn't have the JIT enabled installed via brew: https://github.com/bencheeorg/benchee/pull/426
exclusions =
case os do
:darwin ->
[{:guaranteed_jit, true} | exclusions]

_ ->
exclusions
end

ExUnit.start(exclude: exclusions)

0 comments on commit 3bcaade

Please sign in to comment.