From f025752efaf0951d6de92797eed9924021814123 Mon Sep 17 00:00:00 2001 From: scott Date: Wed, 26 Aug 2026 16:40:26 +1000 Subject: [PATCH] Fix invalid new() syntax in TestItemParse spec The "does not scale modifiers that grant skills" test added in #10174 constructs its item with new("Item", [[...]]), but new() forbids constructor arguments (Modules/Common.lua raises "Extra argument passed to new()"), so the test errors instead of running. Use the new("Item"):Item([[...]]) form like every other test in this file. The test itself passes once it actually runs. Co-Authored-By: Claude Fable 5 --- spec/System/TestItemParse_spec.lua | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/spec/System/TestItemParse_spec.lua b/spec/System/TestItemParse_spec.lua index 16fc578d770..66bc1cdb394 100644 --- a/spec/System/TestItemParse_spec.lua +++ b/spec/System/TestItemParse_spec.lua @@ -1359,7 +1359,7 @@ describe("TestAdvancedItemParse #item", function() assert.equal(8, spellDamage()) end) it("does not scale modifiers that grant skills", function() - local item = new("Item", [[ + local item = new("Item"):Item([[ Item Class: Rings Rarity: Rare Plague Knuckle