Skip to content

Fixed running Rust unit tests in CI#2511

Merged
lslezak merged 1 commit intomasterfrom
fix_rust_ci
Jun 25, 2025
Merged

Fixed running Rust unit tests in CI#2511
lslezak merged 1 commit intomasterfrom
fix_rust_ci

Conversation

@lslezak
Copy link
Contributor

@lslezak lslezak commented Jun 25, 2025

Problem

  • The CI tests actually do not run the Rust unit tests, only the documentation tests.

Solution

  • The --doc tarpaulin option means "Test only this library's documentation" so the unit tests were not executed
  • Use also the --all-targets option which means "Test all targets (excluding doctests)"
  • The --all option is deprecated alias for the --workspace option, use --workspace directly
  • Symlink all langtable data files (the complete directory)

Testing

  • Tested manually, all unit tests are executed

Notes

  • The CI failure here is expected and shows that the unit tests are really executed

- The --doc tarpaulin option means "Test only this library's documentation"
  so the unit tests where not executed
- Use also the --all-targets option which means
  "Test all targets (excluding doctests)"
- The --all option is deprecated alias for the --workspace option,
  use --workspace directly
- Symlink all langtable data files (the complete directory)
@lslezak lslezak merged commit af59f11 into master Jun 25, 2025
12 of 14 checks passed
@lslezak lslezak deleted the fix_rust_ci branch June 25, 2025 16:16
@imobachgs imobachgs mentioned this pull request Jun 30, 2025
imobachgs added a commit that referenced this pull request Jun 30, 2025
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 this pull request may close these issues.

2 participants