Add respawn-entity test #89

johanhelsing · 2023-11-08T21:51:07Z

No description provided.

nezuo

It might be useful to add comment to the top of the test that says it's purpose is to make sure despawning and rollback don't cause a panic.

nezuo · 2023-11-09T18:27:25Z

examples/manual_tests/respawn_entity.rs

+        // each frame, roll back and resimulate 5 frames back in time, and compare checksums
+        .with_check_distance(2);


Won't a with_check_distance of 2 only resimulate 5 frames?

It's better to increase it yes. I set it to to for a very minimal repro of the issue, but should revert that last commit.

I just realized I mistyped my comment. I meant to say: "Won't a with_check_distance of 2 only resimulate 2 frames?" The comment says 5.

johanhelsing added 2 commits November 8, 2023 22:47

Add respawn-entity test

d8de917

Reduce values for less noisy logs

e3a4aa6

nezuo reviewed Nov 9, 2023

View reviewed changes

Provide feedback

		// each frame, roll back and resimulate 5 frames back in time, and compare checksums
		.with_check_distance(2);