[INFO] fetching crate lib_battleship 2.1.0...
[INFO] testing lib_battleship-2.1.0 against master#99e3aef02079e9c10583638520cd0c134dc3a01d for pr-85530
[INFO] extracting crate lib_battleship 2.1.0 into /workspace/builds/worker-2/source
[INFO] validating manifest of crates.io crate lib_battleship 2.1.0 on toolchain 99e3aef02079e9c10583638520cd0c134dc3a01d
[INFO] running `Command { std: "/workspace/cargo-home/bin/cargo" "+99e3aef02079e9c10583638520cd0c134dc3a01d" "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-2/source/Cargo.toml
[INFO] running `Command { std: "/workspace/cargo-home/bin/cargo" "+99e3aef02079e9c10583638520cd0c134dc3a01d" "generate-lockfile" "--manifest-path" "Cargo.toml" "-Zno-index-update", kill_on_drop: false }`
[INFO] running `Command { std: "/workspace/cargo-home/bin/cargo" "+99e3aef02079e9c10583638520cd0c134dc3a01d" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2/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:20528e8c29e4536546f0102ce88568b16ea7538393a33fcca951030f7af10457" "/opt/rustwide/cargo-home/bin/cargo" "+99e3aef02079e9c10583638520cd0c134dc3a01d" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] 3ca5e4f8c32389597086203fc35b03c3de802aff73270b2ff9adfdf399c9c495
[INFO] [stderr] WARNING: Your kernel does not support swap limit capabilities or the cgroup is not mounted. Memory limited without swap.
[INFO] running `Command { std: "docker" "start" "-a" "3ca5e4f8c32389597086203fc35b03c3de802aff73270b2ff9adfdf399c9c495", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "3ca5e4f8c32389597086203fc35b03c3de802aff73270b2ff9adfdf399c9c495", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "3ca5e4f8c32389597086203fc35b03c3de802aff73270b2ff9adfdf399c9c495", kill_on_drop: false }`
[INFO] [stdout] 3ca5e4f8c32389597086203fc35b03c3de802aff73270b2ff9adfdf399c9c495
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2/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=forbid" "-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:20528e8c29e4536546f0102ce88568b16ea7538393a33fcca951030f7af10457" "/opt/rustwide/cargo-home/bin/cargo" "+99e3aef02079e9c10583638520cd0c134dc3a01d" "build" "--frozen" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr] WARNING: Your kernel does not support swap limit capabilities or the cgroup is not mounted. Memory limited without swap.
[INFO] [stdout] a3e0ea2abbe2909af9b130549e74d0d810efad3cb1abff451f31fecb05374985
[INFO] running `Command { std: "docker" "start" "-a" "a3e0ea2abbe2909af9b130549e74d0d810efad3cb1abff451f31fecb05374985", kill_on_drop: false }`
[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 0.85s
[INFO] running `Command { std: "docker" "inspect" "a3e0ea2abbe2909af9b130549e74d0d810efad3cb1abff451f31fecb05374985", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "a3e0ea2abbe2909af9b130549e74d0d810efad3cb1abff451f31fecb05374985", kill_on_drop: false }`
[INFO] [stdout] a3e0ea2abbe2909af9b130549e74d0d810efad3cb1abff451f31fecb05374985
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2/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=forbid" "-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:20528e8c29e4536546f0102ce88568b16ea7538393a33fcca951030f7af10457" "/opt/rustwide/cargo-home/bin/cargo" "+99e3aef02079e9c10583638520cd0c134dc3a01d" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr] WARNING: Your kernel does not support swap limit capabilities or the cgroup is not mounted. Memory limited without swap.
[INFO] [stdout] f0e46555800df657bed940c2d8c351d96bf63aaf54a2c1efaa623253974bf9db
[INFO] running `Command { std: "docker" "start" "-a" "f0e46555800df657bed940c2d8c351d96bf63aaf54a2c1efaa623253974bf9db", kill_on_drop: false }`
[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] [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.32s
[INFO] running `Command { std: "docker" "inspect" "f0e46555800df657bed940c2d8c351d96bf63aaf54a2c1efaa623253974bf9db", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "f0e46555800df657bed940c2d8c351d96bf63aaf54a2c1efaa623253974bf9db", kill_on_drop: false }`
[INFO] [stdout] f0e46555800df657bed940c2d8c351d96bf63aaf54a2c1efaa623253974bf9db
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2/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=forbid" "-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:20528e8c29e4536546f0102ce88568b16ea7538393a33fcca951030f7af10457" "/opt/rustwide/cargo-home/bin/cargo" "+99e3aef02079e9c10583638520cd0c134dc3a01d" "test" "--frozen", kill_on_drop: false }`
[INFO] [stderr] WARNING: Your kernel does not support swap limit capabilities or the cgroup is not mounted. Memory limited without swap.
[INFO] [stdout] 5c0b39dca022f99c675b3368ca77cf4a064980a6d0aedd276d9811d9b6d8a71e
[INFO] running `Command { std: "docker" "start" "-a" "5c0b39dca022f99c675b3368ca77cf4a064980a6d0aedd276d9811d9b6d8a71e", kill_on_drop: false }`
[INFO] [stdout] 
[INFO] [stderr] warning: unused return value of `std::mem::replace` that must be used
[INFO] [stdout] running 37 tests
[INFO] [stderr]   --> src/battlefield/ship_status.rs:45:13
[INFO] [stdout] test battlefield::battlefield::test::should_give_out_cell_references ... ok
[INFO] [stderr]    |
[INFO] [stdout] test battlefield::ship_status::test::should_sum_player_health ... ok
[INFO] [stderr] 45 |             mem::replace(&mut self.status_p1[ship_type_id], curr_val - 1);
[INFO] [stdout] test game::test::shooting_at_empty_cells_is_a_miss ... ok
[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: 2 warnings emitted
[INFO] [stderr] 
[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: 2 warnings emitted
[INFO] [stderr] 
[INFO] [stderr]     Finished test [unoptimized + debuginfo] target(s) in 0.01s
[INFO] [stderr]      Running unittests (/opt/rustwide/target/debug/deps/lib_battleship-87b6a5a11ee0fab9)
[INFO] [stdout] test game::test::shooting_at_filled_cells_is_a_hit ... ok
[INFO] [stdout] test game::test::a_hit_lets_you_shoot_again ... ok
[INFO] [stdout] test game::test::shooting_out_of_bounds_is_an_error ... ok
[INFO] [stdout] test battlefield::cell::tests::assert_shooting_works ... ok
[INFO] [stdout] test common::player::test::should_alternate_between_players ... ok
[INFO] [stdout] test common::ship_type::test::constructor_should_work ... ok
[INFO] [stdout] test game::test::can_get_cell_status_for_opponent ... ok
[INFO] [stdout] test game::test::destroying_last_ship_wins_game ... ok
[INFO] [stdout] test game::test::can_get_cell_status_for_owner ... ok
[INFO] [stdout] test game::test::should_not_allow_shots_after_game_ends ... ok
[INFO] [stdout] test game::test::destroying_a_ship_returns_ship_destroyed ... ok
[INFO] [stdout] test battlefield::cell::tests::assert_shoot_sets_shot ... ok
[INFO] [stdout] test pregame::test::should_allow_placing_ships ... ok
[INFO] [stdout] test game::test::owner_cell_status_doesnt_show_misses ... ok
[INFO] [stdout] test battlefield::cell::tests::assert_ship_type_id_works ... ok
[INFO] [stdout] test battlefield::battlefield::test::should_return_width ... ok
[INFO] [stdout] test pregame::test::should_disallow_zero_length_ship_types ... ok
[INFO] [stdout] test game::test::should_respect_order_of_play ... ok
[INFO] [stdout] test pregame::test::should_disallow_too_long_ship_types ... ok
[INFO] [stdout] test pregame::test::can_get_cell_status ... 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::constructor_should_check_dimensions ... ok
[INFO] [stdout] test pregame::test::should_start_game ... 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_on_top_of_each_other ... ok
[INFO] [stdout] test pregame::test::should_disallow_placing_ships_twice ... ok
[INFO] [stdout] test pregame::test::should_not_start_when_not_all_ships_placed ... ok
[INFO] [stdout] test game::test::should_tell_whos_turn_it_is ... ok
[INFO] [stdout] test pregame::test::should_allow_adding_ship_types ... ok
[INFO] [stdout] test game::test::should_return_contained_ship_types ... ok
[INFO] [stdout] test game::test::should_return_dimensions ... ok
[INFO] [stdout] test pregame::test::should_disallow_placing_ships_of_unknown_type ... ok
[INFO] [stdout] test pregame::test::should_disallow_placing_ships_out_of_bounds ... ok
[INFO] [stdout] 
[INFO] [stdout] test result: ok. 37 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.01s
[INFO] [stdout] 
[INFO] [stderr]    Doc-tests lib_battleship
[INFO] [stdout] 
[INFO] [stdout] running 3 tests
[INFO] [stdout] test src/lib.rs - (line 6) ... ok
[INFO] [stdout] test src/game.rs - game::Game::shoot (line 62) ... ok
[INFO] [stdout] test src/pregame.rs - pregame::PreGame::place_ship (line 116) ... ok
[INFO] [stdout] 
[INFO] [stdout] test result: ok. 3 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.65s
[INFO] [stdout] 
[INFO] running `Command { std: "docker" "inspect" "5c0b39dca022f99c675b3368ca77cf4a064980a6d0aedd276d9811d9b6d8a71e", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "5c0b39dca022f99c675b3368ca77cf4a064980a6d0aedd276d9811d9b6d8a71e", kill_on_drop: false }`
[INFO] [stdout] 5c0b39dca022f99c675b3368ca77cf4a064980a6d0aedd276d9811d9b6d8a71e