unfunny/baibot-withmcp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix clippy warnings in tests

Browse Source 
- Avoid unwrap_or() on a statically-Some value by using the raw token count
- Use an array literal instead of vec! for the non-allocated test cases

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Slavi Pantaleev
2026-06-05 09:41:00 +03:00
parent 04e7667d11
commit f7cb38b620
2 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/controller/agent/create/tests.rs
View File

@@ -9,7 +9,7 @@ fn agent_config_parsing_works() {
let sample_config = crate::agent::default_config_for_provider(&provider);
let sample_config_pretty_yaml = serde_yaml_ng::to_string(&sample_config).unwrap();
let test_cases = vec![
let test_cases = [
// Invalid input
TestCase {
input: r#"Hello"#.to_owned(),

10
src/conversation/llm/tokenization.rs
View File

@@ -105,7 +105,8 @@ pub mod test {
let bpe = super::get_bpe_for_model(model);
let max_response_tokens: Option<u32> = Some(5);
let max_response_tokens_value: u32 = 5;
let max_response_tokens: Option<u32> = Some(max_response_tokens_value);
let prompt = super::Message {
author: super::Author::Prompt,
@@ -193,7 +194,7 @@ pub mod test {
&Some(prompt),
conversation_messages,
max_response_tokens,
prompt_length + max_response_tokens.unwrap_or(0) + forth_length + third_length,
prompt_length + max_response_tokens_value + forth_length + third_length,
);
assert_eq!(2, new_conversation_messages.len());
@@ -215,7 +216,8 @@ pub mod test {
let bpe = super::get_bpe_for_model(model);
let max_response_tokens: Option<u32> = Some(5);
let max_response_tokens_value: u32 = 5;
let max_response_tokens: Option<u32> = Some(max_response_tokens_value);
let prompt = super::Message {
author: super::Author::User,
@@ -303,7 +305,7 @@ pub mod test {
&Some(prompt),
conversation_messages,
max_response_tokens,
prompt_length + max_response_tokens.unwrap_or(0) + forth_length + third_length,
prompt_length + max_response_tokens_value + forth_length + third_length,
);
assert_eq!(2, new_conversation_messages.len());