Fix and enable specific ppxes tests in ci

ocaml · Sep 21, 2023 · a385cbb · a385cbb
1 parent 919aa3e
commit a385cbb
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -44,6 +44,10 @@ jobs:
     # The type of runner that the job will run on
     runs-on: ${{ matrix.os }}
 
+    # Some tests requiring specific ppxes are disabled by default
+    env:
+      MERLIN_TESTS: all
+
     # Steps represent a sequence of tasks that will be executed as part of the job
     steps:
       # Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
@@ -59,6 +63,7 @@ jobs:
         run: |
           opam depext conf-jq --yes # opam depext bug
           opam pin menhirLib 20201216 --no-action
+          opam install --yes ppx_string ppx_compare
           opam install . --deps-only --with-test --yes
 
       - name: Build and test in release mode (windows)

diff --git a/tests/test-dirs/with-ppx/issue1660-deriving-compare.t b/tests/test-dirs/with-ppx/issue1660-deriving-compare.t
@@ -28,9 +28,10 @@
   > (library
   >  (name main)
   >  (preprocess (pps ppx_compare)))
+  > EOF
 
   $ dune build
 
   $ $MERLIN single type-enclosing -position 2:7 \
   > -filename main.ml < main.ml |  jq '.value[0].type'
-  "type t =o | Bar"
+  "type t = Foo | Bar"