Skip to content

Commit 1d75fa2

Browse files
AlexMikhalevclaude
andcommitted
fix: resolve remaining clippy errors in router examples
- Replace CostOptimized::default() with CostOptimized (unit struct) - Replace remaining vec! with arrays in strategy.rs tests Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent dcabb53 commit 1d75fa2

2 files changed

Lines changed: 7 additions & 7 deletions

File tree

crates/terraphim_router/examples/unified_routing.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
127127
)
128128
.with_cost(CostLevel::Cheap),
129129
);
130-
let router_cost = router_cost.with_strategy(Box::new(CostOptimized::default()));
130+
let router_cost = router_cost.with_strategy(Box::new(CostOptimized));
131131

132132
let decision3 = router_cost.route(task2, &RoutingContext::default())?;
133133
println!(

crates/terraphim_router/src/strategy.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -358,7 +358,7 @@ mod tests {
358358
fn test_cost_optimized() {
359359
let strategy = CostOptimized;
360360

361-
let providers = vec![
361+
let providers = [
362362
create_test_provider("expensive", CostLevel::Expensive, Latency::Medium),
363363
create_test_provider("cheap", CostLevel::Cheap, Latency::Medium),
364364
create_test_provider("moderate", CostLevel::Moderate, Latency::Medium),
@@ -374,7 +374,7 @@ mod tests {
374374
fn test_latency_optimized() {
375375
let strategy = LatencyOptimized;
376376

377-
let providers = vec![
377+
let providers = [
378378
create_test_provider("slow", CostLevel::Moderate, Latency::Slow),
379379
create_test_provider("fast", CostLevel::Moderate, Latency::Fast),
380380
create_test_provider("medium", CostLevel::Moderate, Latency::Medium),
@@ -390,7 +390,7 @@ mod tests {
390390
fn test_round_robin() {
391391
let strategy = RoundRobin::new();
392392

393-
let providers = vec![
393+
let providers = [
394394
create_test_provider("a", CostLevel::Cheap, Latency::Fast),
395395
create_test_provider("b", CostLevel::Cheap, Latency::Fast),
396396
create_test_provider("c", CostLevel::Cheap, Latency::Fast),
@@ -422,7 +422,7 @@ mod tests {
422422
1.0, // Always use A
423423
);
424424

425-
let providers = vec![
425+
let providers = [
426426
create_test_provider("cheap-slow", CostLevel::Cheap, Latency::Slow),
427427
create_test_provider("expensive-fast", CostLevel::Expensive, Latency::Fast),
428428
];
@@ -443,7 +443,7 @@ mod tests {
443443
0.0, // Always use B
444444
);
445445

446-
let providers = vec![
446+
let providers = [
447447
create_test_provider("cheap-slow", CostLevel::Cheap, Latency::Slow),
448448
create_test_provider("expensive-fast", CostLevel::Expensive, Latency::Fast),
449449
];
@@ -464,7 +464,7 @@ mod tests {
464464
0.5, // 50/50 split
465465
);
466466

467-
let providers = vec![
467+
let providers = [
468468
create_test_provider("cheap-slow", CostLevel::Cheap, Latency::Slow),
469469
create_test_provider("expensive-fast", CostLevel::Expensive, Latency::Fast),
470470
];

0 commit comments

Comments
 (0)