Context
Current test naming is inconsistent — some use test_ prefix, some don't:
// With prefix
fn test_generate_config_contains_sections()
fn test_validate_bot_token_ok()
// Without prefix
fn estimate_tokens_empty()
fn pack_arrival_event_single()
Proposal
Standardize on no test_ prefix (Rust convention — #[test] + mod tests already provides context) with descriptive <subject>_<scenario>_<expected> format.
Example: estimate_tokens_empty_returns_zero()
Scope
All #[test] functions in src/.
Raised during PR #951 review.
Context
Current test naming is inconsistent — some use
test_prefix, some don't:Proposal
Standardize on no
test_prefix (Rust convention —#[test]+mod testsalready provides context) with descriptive<subject>_<scenario>_<expected>format.Example:
estimate_tokens_empty_returns_zero()Scope
All
#[test]functions insrc/.Raised during PR #951 review.