[INFO] fetching crate lib_battleship 2.1.0... [INFO] testing lib_battleship-2.1.0 against 1.60.0 for beta-1.61-1 [INFO] extracting crate lib_battleship 2.1.0 into /workspace/builds/worker-99/source [INFO] validating manifest of crates.io crate lib_battleship 2.1.0 on toolchain 1.60.0 [INFO] running `Command { std: "/workspace/cargo-home/bin/cargo" "+1.60.0" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking crates.io crate lib_battleship 2.1.0 [INFO] finished tweaking crates.io crate lib_battleship 2.1.0 [INFO] tweaked toml for crates.io crate lib_battleship 2.1.0 written to /workspace/builds/worker-99/source/Cargo.toml [INFO] running `Command { std: "/workspace/cargo-home/bin/cargo" "+1.60.0" "generate-lockfile" "--manifest-path" "Cargo.toml" "-Zno-index-update", kill_on_drop: false }` [INFO] running `Command { std: "/workspace/cargo-home/bin/cargo" "+1.60.0" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-99/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-99/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:b0c94ce3c1162fcb8e57cac5b65ec2f72eabb1eebea4fcc35e269e823f681646" "/opt/rustwide/cargo-home/bin/cargo" "+1.60.0" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] cfe48feab71f960f99836a6368f39797713e45815188477540869e6a6fde0108 [INFO] running `Command { std: "docker" "start" "-a" "cfe48feab71f960f99836a6368f39797713e45815188477540869e6a6fde0108", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "cfe48feab71f960f99836a6368f39797713e45815188477540869e6a6fde0108", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "cfe48feab71f960f99836a6368f39797713e45815188477540869e6a6fde0108", kill_on_drop: false }` [INFO] [stdout] cfe48feab71f960f99836a6368f39797713e45815188477540869e6a6fde0108 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-99/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-99/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=warn" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:b0c94ce3c1162fcb8e57cac5b65ec2f72eabb1eebea4fcc35e269e823f681646" "/opt/rustwide/cargo-home/bin/cargo" "+1.60.0" "build" "--frozen" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 1e0066e03251860b8ac08db80b62eaa85e3d53942a3d6df272820fc2fc7d033e [INFO] running `Command { std: "docker" "start" "-a" "1e0066e03251860b8ac08db80b62eaa85e3d53942a3d6df272820fc2fc7d033e", kill_on_drop: false }` [INFO] [stderr] Blocking waiting for file lock on package cache [INFO] [stderr] Compiling lib_battleship v2.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused return value of `std::mem::replace` that must be used [INFO] [stdout] --> src/battlefield/ship_status.rs:45:13 [INFO] [stdout] | [INFO] [stdout] 45 | mem::replace(&mut self.status_p1[ship_type_id], curr_val - 1); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_must_use)]` on by default [INFO] [stdout] = note: if you don't need the old value, you can just assign the new value directly [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused return value of `std::mem::replace` that must be used [INFO] [stdout] --> src/battlefield/ship_status.rs:49:13 [INFO] [stdout] | [INFO] [stdout] 49 | mem::replace(&mut self.status_p2[ship_type_id], curr_val - 1); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: if you don't need the old value, you can just assign the new value directly [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 2 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished dev [unoptimized + debuginfo] target(s) in 1.62s [INFO] running `Command { std: "docker" "inspect" "1e0066e03251860b8ac08db80b62eaa85e3d53942a3d6df272820fc2fc7d033e", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "1e0066e03251860b8ac08db80b62eaa85e3d53942a3d6df272820fc2fc7d033e", kill_on_drop: false }` [INFO] [stdout] 1e0066e03251860b8ac08db80b62eaa85e3d53942a3d6df272820fc2fc7d033e [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-99/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-99/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=warn" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:b0c94ce3c1162fcb8e57cac5b65ec2f72eabb1eebea4fcc35e269e823f681646" "/opt/rustwide/cargo-home/bin/cargo" "+1.60.0" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 6039f68f726e6e82204ce148be192696ae43fb9b6808aa93570c68677953cd2b [INFO] running `Command { std: "docker" "start" "-a" "6039f68f726e6e82204ce148be192696ae43fb9b6808aa93570c68677953cd2b", kill_on_drop: false }` [INFO] [stderr] Blocking waiting for file lock on package cache [INFO] [stdout] warning: unused return value of `std::mem::replace` that must be used [INFO] [stdout] --> src/battlefield/ship_status.rs:45:13 [INFO] [stdout] | [INFO] [stdout] 45 | mem::replace(&mut self.status_p1[ship_type_id], curr_val - 1); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_must_use)]` on by default [INFO] [stdout] = note: if you don't need the old value, you can just assign the new value directly [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Compiling lib_battleship v2.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused return value of `std::mem::replace` that must be used [INFO] [stdout] --> src/battlefield/ship_status.rs:49:13 [INFO] [stdout] | [INFO] [stdout] 49 | mem::replace(&mut self.status_p2[ship_type_id], curr_val - 1); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: if you don't need the old value, you can just assign the new value directly [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 2 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused return value of `std::mem::replace` that must be used [INFO] [stdout] --> src/battlefield/ship_status.rs:45:13 [INFO] [stdout] | [INFO] [stdout] 45 | mem::replace(&mut self.status_p1[ship_type_id], curr_val - 1); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_must_use)]` on by default [INFO] [stdout] = note: if you don't need the old value, you can just assign the new value directly [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused return value of `std::mem::replace` that must be used [INFO] [stdout] --> src/battlefield/ship_status.rs:49:13 [INFO] [stdout] | [INFO] [stdout] 49 | mem::replace(&mut self.status_p2[ship_type_id], curr_val - 1); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: if you don't need the old value, you can just assign the new value directly [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 2 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished test [unoptimized + debuginfo] target(s) in 1.79s [INFO] running `Command { std: "docker" "inspect" "6039f68f726e6e82204ce148be192696ae43fb9b6808aa93570c68677953cd2b", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "6039f68f726e6e82204ce148be192696ae43fb9b6808aa93570c68677953cd2b", kill_on_drop: false }` [INFO] [stdout] 6039f68f726e6e82204ce148be192696ae43fb9b6808aa93570c68677953cd2b [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-99/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-99/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=warn" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:b0c94ce3c1162fcb8e57cac5b65ec2f72eabb1eebea4fcc35e269e823f681646" "/opt/rustwide/cargo-home/bin/cargo" "+1.60.0" "test" "--frozen", kill_on_drop: false }` [INFO] [stdout] 98af6a34175ab13fc7071035b50cc4af0158753d377634e1fa2cbd6a587b8df2 [INFO] running `Command { std: "docker" "start" "-a" "98af6a34175ab13fc7071035b50cc4af0158753d377634e1fa2cbd6a587b8df2", kill_on_drop: false }` [INFO] [stderr] Blocking waiting for file lock on package cache [INFO] [stderr] warning: unused return value of `std::mem::replace` that must be used [INFO] [stderr] --> src/battlefield/ship_status.rs:45:13 [INFO] [stderr] | [INFO] [stderr] 45 | mem::replace(&mut self.status_p1[ship_type_id], curr_val - 1); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_must_use)]` on by default [INFO] [stderr] = note: if you don't need the old value, you can just assign the new value directly [INFO] [stderr] [INFO] [stderr] warning: unused return value of `std::mem::replace` that must be used [INFO] [stderr] --> src/battlefield/ship_status.rs:49:13 [INFO] [stderr] | [INFO] [stderr] 49 | mem::replace(&mut self.status_p2[ship_type_id], curr_val - 1); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: if you don't need the old value, you can just assign the new value directly [INFO] [stderr] [INFO] [stderr] warning: `lib_battleship` (lib) generated 2 warnings [INFO] [stderr] warning: `lib_battleship` (lib test) generated 2 warnings (2 duplicates) [INFO] [stderr] Finished test [unoptimized + debuginfo] target(s) in 0.65s [INFO] [stderr] Running unittests (/opt/rustwide/target/debug/deps/lib_battleship-bc9ef17393717e67) [INFO] [stdout] [INFO] [stdout] running 37 tests [INFO] [stdout] test battlefield::cell::tests::assert_shoot_sets_shot ... ok [INFO] [stdout] test game::test::destroying_last_ship_wins_game ... ok [INFO] [stdout] test game::test::owner_cell_status_doesnt_show_misses ... ok [INFO] [stdout] test game::test::destroying_a_ship_returns_ship_destroyed ... ok [INFO] [stdout] test game::test::a_hit_lets_you_shoot_again ... ok [INFO] [stdout] test common::player::test::should_alternate_between_players ... ok [INFO] [stdout] test game::test::should_not_allow_shots_after_game_ends ... ok [INFO] [stdout] test game::test::should_return_contained_ship_types ... ok [INFO] [stdout] test pregame::test::should_allow_adding_ship_types ... ok [INFO] [stdout] test battlefield::cell::tests::assert_shooting_works ... ok [INFO] [stdout] test pregame::test::should_disallow_too_long_ship_types ... ok [INFO] [stdout] test battlefield::battlefield::test::should_give_out_cell_references ... ok [INFO] [stdout] test pregame::test::should_disallow_placing_ships_on_top_of_each_other ... ok [INFO] [stdout] test pregame::test::should_disallow_placing_ships_out_of_bounds ... ok [INFO] [stdout] test battlefield::cell::tests::assert_new_cells_are_empty_and_not_shot ... ok [INFO] [stdout] test pregame::test::should_disallow_placing_ships_twice ... ok [INFO] [stdout] test battlefield::cell::tests::assert_ship_type_id_works ... ok [INFO] [stdout] test pregame::test::should_allow_placing_ships ... ok [INFO] [stdout] test game::test::shooting_at_empty_cells_is_a_miss ... ok [INFO] [stdout] test game::test::shooting_at_filled_cells_is_a_hit ... ok [INFO] [stdout] test game::test::shooting_out_of_bounds_is_an_error ... ok [INFO] [stdout] test game::test::can_get_cell_status_for_owner ... ok [INFO] [stdout] test pregame::test::can_get_cell_status ... ok [INFO] [stdout] test game::test::should_respect_order_of_play ... ok [INFO] [stdout] test battlefield::battlefield::test::should_return_width ... ok [INFO] [stdout] test battlefield::ship_status::test::should_sum_player_health ... ok [INFO] [stdout] test common::ship_type::test::constructor_should_work ... ok [INFO] [stdout] test game::test::should_tell_whos_turn_it_is ... ok [INFO] [stdout] test game::test::should_return_dimensions ... ok [INFO] [stdout] test game::test::can_get_cell_status_for_opponent ... ok [INFO] [stdout] test pregame::test::constructor_should_check_dimensions ... ok [INFO] [stdout] test pregame::test::should_disallow_placing_ships_of_unknown_type ... ok [INFO] [stdout] test pregame::test::should_disallow_zero_length_ship_types ... ok [INFO] [stdout] test pregame::test::should_start_game ... ok [INFO] [stdout] test pregame::test::should_not_start_when_no_ships_placed ... ok [INFO] [stdout] test pregame::test::should_return_dimensions ... ok [INFO] [stdout] test pregame::test::should_not_start_when_not_all_ships_placed ... ok [INFO] [stdout] [INFO] [stdout] test result: ok. 37 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.10s [INFO] [stdout] [INFO] [stderr] Doc-tests lib_battleship [INFO] [stdout] [INFO] [stdout] running 3 tests [INFO] [stdout] test src/pregame.rs - pregame::PreGame::place_ship (line 116) ... ok [INFO] [stdout] test src/game.rs - game::Game::shoot (line 62) ... ok [INFO] [stdout] test src/lib.rs - (line 6) ... ok [INFO] [stdout] [INFO] [stdout] test result: ok. 3 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.76s [INFO] [stdout] [INFO] running `Command { std: "docker" "inspect" "98af6a34175ab13fc7071035b50cc4af0158753d377634e1fa2cbd6a587b8df2", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "98af6a34175ab13fc7071035b50cc4af0158753d377634e1fa2cbd6a587b8df2", kill_on_drop: false }` [INFO] [stdout] 98af6a34175ab13fc7071035b50cc4af0158753d377634e1fa2cbd6a587b8df2