Skip to content

Commit ad8ca91

Browse files
committed
Changed: Improve test coverage for agent parser and provider type mapping
- Always assert hidden field value in parse_agent_success_cases using unwrap_or(false) default - Add test case for unknown npm package mapping to ProviderType::Unknown
1 parent 725df1b commit ad8ca91

3 files changed

Lines changed: 3 additions & 5 deletions

File tree

src/AGENTS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,4 +67,4 @@ This is a high-performance library. Optimize aggressively.
6767

6868
# Post-Change Verification
6969

70-
After you make a change to source code, always run `.cargo/verify.sh` (`.cargo/verify.ps1` on Windows) before returning to the user.
70+
After code changes, always run `.cargo/verify` script before returning to the user. Search if not found.

src/llm-coding-tools-agents/src/parser/mod.rs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -372,10 +372,7 @@ mod tests {
372372
assert_eq!(result.content, expected_content);
373373
assert_eq!(&*result.data.description, expected_description);
374374
assert_eq!(result.data.model.as_deref(), expected_model);
375-
376-
if let Some(expected_hidden) = expected_hidden {
377-
assert_eq!(result.data.hidden, expected_hidden);
378-
}
375+
assert_eq!(result.data.hidden, expected_hidden.unwrap_or(false));
379376
}
380377

381378
#[rstest]

src/llm-coding-tools-models-dev/src/api/catalog_sources.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -575,6 +575,7 @@ mod tests {
575575
ProviderType::ClaudeCodeOAuth
576576
)]
577577
#[case::antigravity_package(Some("@ai-sdk/antigravity"), ProviderType::Antigravity)]
578+
#[case::unknown_package(Some("@unknown/package"), ProviderType::Unknown)]
578579
#[case::missing_package_unknown(None, ProviderType::Unknown)]
579580
fn npm_package_maps_to_correct_provider_type(
580581
#[case] npm_package: Option<&str>,

0 commit comments

Comments
 (0)