Skip to content

test: include_subdirs unqualified duplicates cram rules #12142

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Alizter
Copy link
Collaborator

@Alizter Alizter commented Aug 7, 2025

This regression was observed in ocaml/opam-repository#28302. It is due to a mix of (include_subdirs unqualified) and directory cram tests in the same directory.

I've changed an existing test to highlight two issues with the (include_subdirs unqualified) stanza and cram tests.

  1. The cram test in the subdirectory is no longer being picked up.
  2. The regression above: Directory cram tests are generating multiple rules.

@Alizter Alizter force-pushed the opam-repo-regression-3.20 branch from 1d85099 to 4d04238 Compare August 7, 2025 17:35
@maiste maiste added the bug label Aug 8, 2025
@maiste maiste added this to the 3.20 milestone Aug 8, 2025
@maiste maiste mentioned this pull request Aug 8, 2025
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants