Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable some tests in more-ci #23

Closed
mbarbin opened this issue Oct 27, 2024 · 0 comments · Fixed by #25
Closed

Enable some tests in more-ci #23

mbarbin opened this issue Oct 27, 2024 · 0 comments · Fixed by #25

Comments

@mbarbin
Copy link
Owner

mbarbin commented Oct 27, 2024

Proposal to split the tests into parts that can serve different purposes.

Explanation:

In #21 I enabled more checks in the ci to target more os and compiler versions.

However at this time all the doc and tests items are grouped together into a single package named provider-tests.opam, which causes some issues to be run on all matrix combinations:

  • it has too many dependencies (such as eio)
  • it has brittle expect-tests (e.g. it uses ls and wc in the mdx doc, with formatting diff between ubuntu and macos).

As a result, for now these ci check only check that the code builds, but do not run any tests.

I thought it would make sense to split the tests and extract one new entry point that can be made to run on all checks matrix combinations. This is an issue to track this work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant