refactor(clippy): apply lints

This commit is contained in:
LGUG2Z
2026-03-21 14:43:29 -07:00
parent e23cb3d495
commit cfa8db285b
2 changed files with 16 additions and 12 deletions

View File

@@ -199,8 +199,8 @@ mod columns_with_ratios_tests {
// First column: 20% = 200 // First column: 20% = 200
assert_eq!(layouts[0].right, 200); assert_eq!(layouts[0].right, 200);
// Remaining 800 split among 4 columns = 200 each // Remaining 800 split among 4 columns = 200 each
for i in 1..5 { for item in layouts.iter().take(5).skip(1) {
assert_eq!(layouts[i].right, 200); assert_eq!(item.right, 200);
} }
} }

View File

@@ -528,8 +528,8 @@ mod tests {
let ratios = opts.column_ratios.unwrap(); let ratios = opts.column_ratios.unwrap();
// Only MAX_RATIOS (5) values should be stored // Only MAX_RATIOS (5) values should be stored
for i in 0..MAX_RATIOS { for item in ratios.iter().take(MAX_RATIOS) {
assert_eq!(ratios[i], Some(0.1)); assert_eq!(*item, Some(0.1));
} }
} }
@@ -539,8 +539,8 @@ mod tests {
let opts: LayoutOptions = serde_json::from_str(json).unwrap(); let opts: LayoutOptions = serde_json::from_str(json).unwrap();
let ratios = opts.column_ratios.unwrap(); let ratios = opts.column_ratios.unwrap();
for i in 0..MAX_RATIOS { for item in ratios.iter().take(MAX_RATIOS) {
assert_eq!(ratios[i], None); assert_eq!(*item, None);
} }
} }
@@ -633,12 +633,16 @@ mod tests {
#[test] #[test]
fn test_constants_valid_ranges() { fn test_constants_valid_ranges() {
assert!(MIN_RATIO > 0.0); const {
assert!(MIN_RATIO < MAX_RATIO); assert!(MIN_RATIO > 0.0);
assert!(MAX_RATIO < 1.0); assert!(MIN_RATIO < MAX_RATIO);
assert!(DEFAULT_RATIO >= MIN_RATIO && DEFAULT_RATIO <= MAX_RATIO); assert!(MAX_RATIO < 1.0);
assert!(DEFAULT_SECONDARY_RATIO >= MIN_RATIO && DEFAULT_SECONDARY_RATIO <= MAX_RATIO); assert!(DEFAULT_RATIO >= MIN_RATIO && DEFAULT_RATIO <= MAX_RATIO);
assert!(MAX_RATIOS >= 1); assert!(
DEFAULT_SECONDARY_RATIO >= MIN_RATIO && DEFAULT_SECONDARY_RATIO <= MAX_RATIO
);
assert!(MAX_RATIOS >= 1);
}
} }
#[test] #[test]