ocaml-semver / ocaml-api-watch

Libraries and tools to keep watch on you OCaml lib's API changes
ISC License
21 stars 15 forks source link

Split the monolithic cram test into multiple files #86

Closed nebadesmondc closed 1 month ago

nebadesmondc commented 1 month ago

fixes #85 I have splitted the run.t file into the following:

@panglesd

panglesd commented 1 month ago

Hello @Dezzy12 and thanks for the PR!

Unfortunately, the tests are not passing anymore, as you can see from the result of the CI (the red cross next to "ocaml-ci", you can click on "details" to see what went wrong).

To test locally, you can run dune runtest. This will show you the difference between what is expected and what is the result of the test. If you want to change the expected value to what running the test produces, you can run dune promote. dune runtest --auto-promote is a shorthand for dune runtest followed by dune promote.

You can have more information about tests in the dune doc on writing and running tests, in particular its section on cram tests.

nebadesmondc commented 1 month ago

I have fixed the issue. Sorry for the Delay @panglesd

nebadesmondc commented 1 month ago

I have implemented the requested changes

panglesd commented 1 month ago

Thanks!