Test the download race and cleared requests without inventories

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

plugins/player_sync/test/components.js

@@ -148,5 +148,14 @@ describe("player_sync components", function() {
 			const merged = mergePlayerData(storedSlots, withComponents(noSlots, []));
 			assert.equal("personal_logistic_slots" in merged, false);
 		});
+		it("should take cleared logistic requests from an upload without inventories", function() {
+			const storedSlots = { ...stored, personal_logistic_slots: [{ name: "stored" }] };
+			// Factorio 1.1 serializes a character without requests as an empty table
+			const cleared = { ...uploaded, personal_logistic_slots: {} };
+			delete cleared.character;
+			delete cleared.crafting_queue;
+			const merged = mergePlayerData(storedSlots, withComponents(cleared, ["inventories"]));
+			assert.deepEqual(merged.personal_logistic_slots, {});
+		});
 	});
 });

plugins/player_sync/test/instance.js

@@ -135,9 +135,14 @@ describe("player_sync", function() {
 			const expected = { generation: 1, name: "test", tag: "tag" };
 			assert.deepEqual(instance.server.rconCommands, [downloadCommand(expected)]);
 		});
-		it("should ignore config changes made after the download was started", async function() {
-			instance.mockConfigEntries.set("player_sync.sync_force", false);
-			respondWith({ generation: 1, name: "test", force: "enemy", tag: "tag" });
+		it("should ignore config changes made while the controller responds", async function() {
+			instance.connector.once("send", message => {
+				instance.mockConfigEntries.set("player_sync.sync_force", false);
+				instance.connector.emit("message", new lib.MessageResponse(
+					1, message.dst, message.src,
+					new DownloadResponse({ generation: 1, name: "test", force: "enemy", tag: "tag" }),
+				));
+			});
 			await download({ player_name: "test", components: { force: true } });
 			const expected = { generation: 1, name: "test", force: "enemy" };
 			assert.deepEqual(instance.server.rconCommands, [downloadCommand(expected)]);