Skip to content

Commit 760a859

Browse files
committed
Library tests: lighter conditions
1 parent fbe22bf commit 760a859

File tree

2 files changed

+17
-19
lines changed

2 files changed

+17
-19
lines changed

lib/tests/dune.inc

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,10 @@
3232
(library
3333
;; lib/tests/test_fun_call.ml
3434
(name test_fun_call_75)
35-
(enabled_if (and (<> %{profile} using-effects) (<> %{profile} wasm) (<> %{profile} wasm-effects)))
35+
(enabled_if (<> %{profile} using-effects))
3636
(modules test_fun_call)
3737
(libraries js_of_ocaml unix)
38-
(inline_tests (modes js wasm))
38+
(inline_tests (modes js))
3939
(preprocess
4040
(pps ppx_js_internal ppx_expect)))
4141

@@ -52,10 +52,10 @@
5252
(library
5353
;; lib/tests/test_json.ml
5454
(name test_json_75)
55-
(enabled_if (and (<> %{profile} wasm) (<> %{profile} wasm-effects)))
55+
(enabled_if true)
5656
(modules test_json)
5757
(libraries js_of_ocaml unix)
58-
(inline_tests (modes js wasm))
58+
(inline_tests (modes js))
5959
(preprocess
6060
(pps ppx_js_internal ppx_expect)))
6161

@@ -72,10 +72,10 @@
7272
(library
7373
;; lib/tests/test_poly_compare.ml
7474
(name test_poly_compare_75)
75-
(enabled_if (and (<> %{profile} wasm) (<> %{profile} wasm-effects)))
75+
(enabled_if true)
7676
(modules test_poly_compare)
7777
(libraries js_of_ocaml unix)
78-
(inline_tests (modes js wasm))
78+
(inline_tests (modes js))
7979
(preprocess
8080
(pps ppx_js_internal ppx_expect)))
8181

@@ -102,10 +102,10 @@
102102
(library
103103
;; lib/tests/test_sys.ml
104104
(name test_sys_75)
105-
(enabled_if (and (>= %{ocaml_version} 5) (<> %{profile} wasm) (<> %{profile} wasm-effects)))
105+
(enabled_if (>= %{ocaml_version} 5))
106106
(modules test_sys)
107107
(libraries js_of_ocaml unix)
108-
(inline_tests (modes js wasm))
108+
(inline_tests (modes js))
109109
(preprocess
110110
(pps ppx_js_internal ppx_expect)))
111111

lib/tests/gen-rules/gen.ml

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ let () =
7171
(enabled_if %s)
7272
(modules %s)
7373
(libraries js_of_ocaml unix)
74-
(inline_tests (modes js wasm))
74+
(inline_tests (modes js%s))
7575
(preprocess
7676
(pps ppx_js_internal ppx_expect)))
7777
|}
@@ -80,13 +80,11 @@ let () =
8080
basename
8181
(Hashtbl.hash prefix mod 100)
8282
(match enabled_if basename with
83-
| Any -> "true"
84-
| GE5 ->
85-
(* ZZZ /static not yet implemented *)
86-
"(and (>= %{ocaml_version} 5) (<> %{profile} wasm) (<> %{profile} \
87-
wasm-effects))"
88-
| Not_wasm -> "(and (<> %{profile} wasm) (<> %{profile} wasm-effects))"
89-
| No_effects_not_wasm ->
90-
"(and (<> %{profile} using-effects) (<> %{profile} wasm) (<> %{profile} \
91-
wasm-effects))")
92-
basename)
83+
| Any | Not_wasm -> "true"
84+
| GE5 -> "(>= %{ocaml_version} 5)"
85+
| No_effects_not_wasm -> "(<> %{profile} using-effects)")
86+
basename
87+
(match enabled_if basename with
88+
| Any -> " wasm"
89+
| GE5 -> "" (* ZZZ /static not yet implemented *)
90+
| Not_wasm | No_effects_not_wasm -> ""))

0 commit comments

Comments
 (0)