I think we should keep this... I think `for num_bottles in std::iter::range_step_inclusive(99, 1, -1)` would look much better without `std::iter` before it. You can add `#[cfg(not(test))]` to avoid the compiler warning when compiling the test.