Add js_of_ocaml dep for tests, adapt test to ocaml.5.3

.github/workflows/workflow.yml

@@ -16,6 +16,7 @@ jobs:
         node-version:
           - 20.x
         ocaml-compiler:
+          - 5.3.x
           - 5.2.x
           - 4.12.x
           - 4.11.x
@@ -53,8 +54,6 @@ jobs:
 
       - run: opam install --deps-only --with-doc --with-test -y landmarks
 
-      - run: opam install js_of_ocaml
-
       - run: opam exec -- dune build -p landmarks
 
       - run: opam exec -- dune build -p landmarks @runtest

dune-project

@@ -20,7 +20,10 @@
     "\| instrumentation of the code may either done by hand, automatically or
     "\| semi-automatically using the ppx pepreprocessor (see landmarks-ppx package).
  )
- (depends (ocaml (>= 4.08)))
+ (depends
+   (ocaml (>= 4.08))
+   (js_of_ocaml (and :with-test (> 5)))
+ )
 )
 
 (package

landmarks.opam

@@ -16,6 +16,7 @@ bug-reports: "https://github.com/LexiFi/landmarks/issues"
 depends: [
   "dune" {>= "3.16"}
   "ocaml" {>= "4.08"}
+  "js_of_ocaml" {with-test & > "5"}
   "odoc" {with-doc}
 ]
 build: [

tests/poly-fail/dune

@@ -1,19 +1,11 @@
-(rule
- (targets test.out)
- (deps
-  (:ppx ../tools/standalone.exe)
-  (:tt ../../toplevel/tt.exe))
- (action
-  (with-stdout-to
-   test.out
-   (with-stdin-from
-    test.ml
-    (run %{tt} -color never -noprompt -no-version -ppx "%{ppx} --as-ppx")))))
-
 (rule
  (alias runtest)
  (package landmarks-ppx)
- (enabled_if
-  (>= %{ocaml_version} "5"))
+ (deps
+  test.ml
+  (:ppx ../tools/standalone.exe))
  (action
-  (diff test.out.expected test.out)))
+  (ignore-outputs
+   (with-accepted-exit-codes 2
+    (run ocaml -I ../../src -I ../../src/.landmark.objs/byte %{lib:landmarks:landmark.cma} -ppx "%{ppx} --as-ppx" test.ml)
+   ))))