Make -unused-code-warning tests actual tests

In their previous form they were not executed as part of the default build or as part of the runtest alias. They relied on a deriving_inline workflow which wasn't great for tests and it lacked a bit of context on what was being tested so I converted them to cram tests. It's a bit harder to read the generated code than it was but the whole setup is not spread across 3 different directories anymore and the tests are now documented. Signed-off-by: Nathan Rebours <[email protected]>
ocaml-ppx · May 28, 2024 · a20f5a8 · a20f5a8
1 parent 5bf8c9c
commit a20f5a8
Show file tree

Hide file tree

Showing 11 changed files with 205 additions and 193 deletions.
diff --git a/test/deriving_warning/disable/dune b/test/deriving_warning/disable/dune
diff --git a/test/deriving_warning/disable/ppx_warning_flags_test.ml b/test/deriving_warning/disable/ppx_warning_flags_test.ml
diff --git a/test/deriving_warning/disable/ppx_warning_flags_test.mli b/test/deriving_warning/disable/ppx_warning_flags_test.mli
diff --git a/...deriving_warning/ppx/ppx_warning_flags.ml → test/deriving_warning/driver.ml b/...deriving_warning/ppx/ppx_warning_flags.ml → test/deriving_warning/driver.ml
@@ -93,3 +93,5 @@ let () =
   (* The derivers are added from right to left *)
   Deriving.add_alias "alias_warn" [ alias_no_warn; alias_do_warn ]
   |> Deriving.ignore
+
+let () = Driver.standalone ()
diff --git a/test/deriving_warning/dune b/test/deriving_warning/dune
@@ -0,0 +1,8 @@
+(executable
+ (name driver)
+ (libraries ppxlib)
+ (preprocess
+  (pps ppxlib.metaquot)))
+
+(cram
+ (deps driver.exe))
diff --git a/test/deriving_warning/enable/dune b/test/deriving_warning/enable/dune
diff --git a/test/deriving_warning/enable/ppx_warning_flags_test.ml b/test/deriving_warning/enable/ppx_warning_flags_test.ml
diff --git a/test/deriving_warning/enable/ppx_warning_flags_test.mli b/test/deriving_warning/enable/ppx_warning_flags_test.mli
diff --git a/test/deriving_warning/ppx/dune b/test/deriving_warning/ppx/dune
diff --git a/test/deriving_warning/ppx/ppx_warning_flags.mli b/test/deriving_warning/ppx/ppx_warning_flags.mli