Skip to content

fix(cram): stop truncating test names - #16146

Merged
rgrinberg merged 1 commit into
ocaml:mainfrom
rgrinberg:push-lzytvpnkmzll
Aug 25, 2026
Merged

fix(cram): stop truncating test names#16146
rgrinberg merged 1 commit into
ocaml:mainfrom
rgrinberg:push-lzytvpnkmzll

Conversation

@rgrinberg

@rgrinberg rgrinberg commented Aug 23, 2026

Copy link
Copy Markdown
Member

Preserve the full .t basename when deriving cram test names starting with Dune language 3.25, while retaining historical suffix-less names for older projects.

Use the project language version consistently for generated aliases, applies_to, and direct dune test requests. Update boundary coverage and migrate the repository cram stanzas and reference documentation to the 3.25 naming convention.

@rgrinberg
rgrinberg force-pushed the push-lzytvpnkmzll branch 7 times, most recently from f034b1f to 1a32b7c Compare August 25, 2026 20:10
@rgrinberg rgrinberg added cram Related to the cram test execution in Dune and removed no changelog labels Aug 25, 2026
@rgrinberg
rgrinberg force-pushed the push-lzytvpnkmzll branch 3 times, most recently from c4338cd to ddb7f23 Compare August 25, 2026 21:20
Preserve the full .t basename when deriving cram test names starting with Dune language 3.25, while retaining historical suffix-less names for older projects.

Use the project language version consistently when generating aliases and when resolving direct dune test requests. Update applies_to, alias, and direct test invocation coverage around the language-version boundary. Migrate the repository test stanzas and reference documentation to the 3.25 naming convention.

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
@rgrinberg
rgrinberg merged commit bf81328 into ocaml:main Aug 25, 2026
18 of 20 checks passed
@Alizter Alizter added this to the 3.25.0 milestone Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cram Related to the cram test execution in Dune

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants