Skip to content

Commit e2f1226

Browse files
committed
fix(test): par_azip9 — drop approx-gated itertools helpers (CI rayon-only)
`tests/par_azip.rs` has `use itertools::{assert_equal, cloned, enumerate}` under `#[cfg(feature = "approx")]`, but `test_par_azip9` called `assert_equal(cloned(&a), x)` at line 85 unconditionally. With approx OFF the import is excluded and compile fails with E0425. Latent for as long as the file existed (since PR #73 era); never surfaced because no CI matrix combination ran `--features rayon` without approx. W-I4's new `hpc-stream-parallel` job exercises exactly that combination and tripped the failure. Fix: replace `assert_equal(cloned(&a), x)` with `assert_eq!(a, x)` — both `a` and `x` are `Array<i32, _>` and the file's other tests already use direct `assert_eq!`. Trim the now-dead `assert_equal, cloned` from the still-needed (test_par_azip3) `enumerate` import. Verified clean compile + 6/6 tests pass under both: cargo test --features rayon --test par_azip cargo test --features "rayon approx" --test par_azip https://claude.ai/code/session_01UwJuKqP828qyX1VkLgGJFS
1 parent 59e605c commit e2f1226

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

tests/par_azip.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#![cfg(feature = "rayon")]
22

33
#[cfg(feature = "approx")]
4-
use itertools::{assert_equal, cloned, enumerate};
4+
use itertools::enumerate;
55
use ndarray::parallel::prelude::*;
66
use ndarray::prelude::*;
77
use std::sync::atomic::{AtomicUsize, Ordering};
@@ -82,5 +82,5 @@ fn test_par_azip9() {
8282
*a = b + c + d + e + f + g + h + i;
8383
});
8484
let x = Array::from_shape_fn(a.dim(), |j| (j * 255) as i32);
85-
assert_equal(cloned(&a), x);
85+
assert_eq!(a, x);
8686
}

0 commit comments

Comments
 (0)