Skip to content

enable debug-asserts and overflow checks#742

Merged
arvidn merged 2 commits intomainfrom
release-asserts
Mar 12, 2026
Merged

enable debug-asserts and overflow checks#742
arvidn merged 2 commits intomainfrom
release-asserts

Conversation

@arvidn
Copy link
Contributor

@arvidn arvidn commented Mar 12, 2026

when running tests in release mode

Documented here https://doc.rust-lang.org/cargo/reference/environment-variables.html


Note

Low Risk
Low risk: changes are limited to CI test configuration and assertion/test gating, with no functional runtime behavior changes outside debug/assert-enabled builds.

Overview
CI unit_tests now runs cargo test --release with debug assertions and overflow checks enabled via Cargo profile env vars, and executes ignored tests by adding -- --include-ignored.

The extremely slow test_very_long_list is switched from #[cfg(not(debug_assertions))] to #[ignore] with guidance to run it explicitly, and TreeCache::restore()’s stack invariant check is changed from a release-only assert_eq! to a debug_assert_eq! so it’s enforced when debug assertions are enabled.

Written by Cursor Bugbot for commit 80aa800. This will update automatically on new commits. Configure here.

@arvidn arvidn requested a review from AmineKhaldi March 12, 2026 09:08
@coveralls-official
Copy link

coveralls-official bot commented Mar 12, 2026

Pull Request Test Coverage Report for Build 22997261819

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • 17 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.2%) to 87.921%

Files with Coverage Reduction New Missed Lines %
src/serde/object_cache.rs 17 90.48%
Totals Coverage Status
Change from base Build 22870033048: -0.2%
Covered Lines: 7119
Relevant Lines: 8097

💛 - Coveralls

AmineKhaldi
AmineKhaldi previously approved these changes Mar 12, 2026
@arvidn arvidn merged commit 71ddd66 into main Mar 12, 2026
31 checks passed
@arvidn arvidn deleted the release-asserts branch March 12, 2026 12:39
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