Add Loadout management UI (#1865)

* [ISSUE-9451] Loadout Management

* Transferred ConfigTab and ConfigSetListControl changes from PoB1

* [ISSUE-9451] Loadout Management

* Tranferred ItemSetListControl and ItemsTab changes from PoB1

* [ISSUE-9451] Loadout Management

* Transferred SkillSetListControl and SkillsTab changes from PoB1

* [ISSUE-9451] Loadout Management

* Transferred TreeTab and PassiveSpecListControl changes from PoB1

* [ISSUE-9451] Loadout Management

* Transferred Build and BuildSetListControl changes from PoB1

* [ISSUE-9451] Loadout Management

* Provide default "Default" title fallback when copying config sets with
  nil title
* Provide default "Default" title fallback when copying item sets with
  nil title
* Set newly created loadout as active immediately after creation in
  Build.lua

* Fix items getting deleted from shared loadouts

If an item belonged to 2 loadouts and 1 of them was deleted, it would wipe the items without checking to see if another loadout used them

* Pressing F2 to rename the loadout now grabs the right name

Renaming the loadout by pressing F2 would grab a table address instead of the name

* Importing a shared item list fix

When importing a shared item list it now uses CreateItemSet so it doesn't double add the set anymore

* Avoid reapplying active loadout sets

Only switch tree, item, skill, and config sets when the selected loadout
uses a different set from the currently active one.

This restores the previous loadout switching behaviour and prevents build
initialisation from reprocessing stale tab state from the prior build,
which caused the test suite to fail when resetting between skill tests.

---------

Co-authored-by: LocalIdentity <localidentity2@gmail.com>

Author: FenikSRT4

spec/System/TestConfigTab_spec.lua

@@ -0,0 +1,375 @@
+describe("TestSkillsTab", function()
+	before_each(function()
+		newBuild()
+		runCallback("OnFrame")
+	end)
+
+	describe("SkillsTab", function()
+		describe("NewSkillSet", function()
+			it("Creates a new skill set with specified ID", function()
+				local skillSetName = "New Skill Set"
+				local skillSetId = 2
+				build.skillsTab:NewSkillSet(skillSetId, skillSetName)
+
+				assert.are.equals(skillSetName, build.skillsTab.skillSets[skillSetId].title)
+				assert.are.equals(skillSetId, build.skillsTab.skillSets[skillSetId].id)
+			end)
+
+			it("Assigns auto-incremented ID when not specified", function()
+				local skillSetName = "Auto ID Skill Set"
+				local skillSet1 = build.skillsTab:NewSkillSet()
+				local skillSet2 = build.skillsTab:NewSkillSet(nil, skillSetName)
+
+				assert.are.equals(2, skillSet1.id)
+				assert.are.equals(3, skillSet2.id)
+				assert.are.equals(skillSetName, build.skillsTab.skillSets[skillSet2.id].title)
+			end)
+
+			it("Adds set to order list", function()
+				local newTitle = "New Skill"
+
+				build.skillsTab:NewSkillSet(nil, newTitle)
+
+				assert.are.equals(2, #build.skillsTab.skillSetOrderList)
+				assert.are.equals(2, build.skillsTab.skillSetOrderList[2])
+			end)
+
+			it("Updates the mod flag", function()
+				local newTitle = "New Skill"
+				build.skillsTab.modFlag = false
+
+				build.skillsTab:NewSkillSet(nil, newTitle)
+
+				assert.is_true(build.skillsTab.modFlag)
+			end)
+		end)
+
+		describe("CopySkillSet", function()
+			it("Copies a skill set with a new name", function()
+				local newTitle = "Copied Skill"
+				local newSkillSet = build.skillsTab:CopySkillSet(1, newTitle)
+
+				assert.is_not.same(build.skillsTab.skillSets[1], newSkillSet)
+				assert.are.equals(newTitle, newSkillSet.title)
+				assert.are.same(newSkillSet.socketGroupList, build.skillsTab.skillSets[1].socketGroupList)
+			end)
+
+			it("Deep copies all socket groups and gems", function()
+				local newTitle = "Copied Skill"
+
+				local testGem = { nameSpec = "TestGem", level = 20 }
+				build.skillsTab.skillSets[1].socketGroupList = { { gemList = { testGem } } }
+
+				local newSkillSet = build.skillsTab:CopySkillSet(1, newTitle)
+
+				assert.are.same("TestGem", build.skillsTab.skillSets[1].socketGroupList[1].gemList[1].nameSpec)
+				assert.are.same(20, build.skillsTab.skillSets[1].socketGroupList[1].gemList[1].level)
+				build.skillsTab.skillSets[1].socketGroupList[1].gemList[1].nameSpec = "Modified"
+
+				assert.are.equals("TestGem", newSkillSet.socketGroupList[1].gemList[1].nameSpec)
+				assert.are.equals(20, newSkillSet.socketGroupList[1].gemList[1].level)
+			end)
+
+			it("Returns copied skill set with next ID", function()
+				local newTitle = "Copied Skill"
+
+				local newSkillSet = build.skillsTab:CopySkillSet(1, newTitle)
+
+				assert.are.equals(2, newSkillSet.id)
+			end)
+
+			it("Adds copied skill set to order list", function()
+				local newTitle = "Copied Skill"
+
+				build.skillsTab:CopySkillSet(1, newTitle)
+
+				assert.are.equals(2, #build.skillsTab.skillSetOrderList)
+				assert.are.equals(2, build.skillsTab.skillSetOrderList[2])
+			end)
+
+			it("Updates the mod flag", function()
+				local newTitle = "Copied Skill"
+				build.skillsTab.modFlag = false
+
+				build.skillsTab:CopySkillSet(1, newTitle)
+
+				assert.is_true(build.skillsTab.modFlag)
+			end)
+		end)
+
+		describe("RenameSkillSet", function()
+			it("Renames a skill set", function()
+				local newTitle = "Renamed Skill"
+				build.skillsTab:RenameSkillSet(1, newTitle)
+
+				assert.are.equals(newTitle, build.skillsTab.skillSets[1].title)
+			end)
+
+			it("Does not rename non-existent skill set", function()
+				build.skillsTab:RenameSkillSet(999, "Non-existent")
+
+				assert.is_nil(build.skillsTab.skillSets[999])
+			end)
+
+			it("Updates the mod flag", function()
+				local newTitle = "Renamed Skill"
+				build.skillsTab.modFlag = false
+
+				build.skillsTab:RenameSkillSet(1, newTitle)
+
+				assert.is_true(build.skillsTab.modFlag)
+			end)
+		end)
+
+		describe("DeleteSkillSet", function()
+			it("Deletes a skill set and its order entry", function()
+				local skillSetName = "Skill To Delete"
+				build.skillsTab:NewSkillSet(2, skillSetName)
+
+				build.skillsTab:DeleteSkillSet(2, 2)
+
+				assert.is_nil(build.skillsTab.skillSets[2])
+				assert.are.equals(1, #build.skillsTab.skillSetOrderList)
+			end)
+
+			it("allows deletion of the last skill set", function()
+				local lastSkill = 1
+
+				assert.are.equals(1, #build.skillsTab.skillSetOrderList)
+				build.skillsTab:DeleteSkillSet(lastSkill, 1)
+
+				assert.are.equals(0, #build.skillsTab.skillSetOrderList)
+				assert.is_nil(build.skillsTab.skillSets[lastSkill])
+			end)
+		end)
+
+		describe("SetActiveSkillSet", function()
+			it("Switches to a valid skill set", function()
+				local skillSetName = "New Skill"
+				build.skillsTab:NewSkillSet(2, skillSetName)
+
+				build.skillsTab:SetActiveSkillSet(2)
+
+				assert.are.equals(2, build.skillsTab.activeSkillSetId)
+				assert.are.same(build.skillsTab.skillSets[2].socketGroupList, build.skillsTab.socketGroupList)
+			end)
+
+			it("Defaults to first skill set if invalid ID provided", function()
+				build.skillsTab:SetActiveSkillSet(999)
+
+				assert.are.equals(1, build.skillsTab.activeSkillSetId)
+			end)
+		end)
+	end)
+
+	describe("SkillsSetService", function()
+		local skillsSetService
+
+		before_each(function()
+			skillsSetService = new("SkillsSetService", build.skillsTab)
+		end)
+
+		describe("NewSkillSet", function()
+			it("Creates a new skill set via service", function()
+				local skillSetName = "Service New Skill"
+				skillsSetService:NewSkillSet(skillSetName)
+
+				assert.are.equals(2, #build.skillsTab.skillSetOrderList)
+				assert.are.equals(skillSetName, build.skillsTab.skillSets[2].title)
+			end)
+
+			it("Sets newly created skill as active", function()
+				skillsSetService:NewSkillSet("New From Service")
+
+				assert.are.equals(2, build.skillsTab.activeSkillSetId)
+			end)
+
+			it("Adds an undo state", function()
+				local newTitle = "New Skill"
+
+				local undoCountBefore = #build.skillsTab.undo
+				skillsSetService:NewSkillSet(newTitle)
+
+				assert.are.equals(#build.skillsTab.undo, undoCountBefore + 1)
+			end)
+		end)
+
+		describe("CopySkillSet", function()
+			it("Copies a skill set via service", function()
+				local skillSetName = "Copied via Service"
+				skillsSetService:CopySkillSet(1, skillSetName)
+				local skillSetId = build.skillsTab.skillSetOrderList[2]
+
+				assert.are.equals(skillSetName, build.skillsTab.skillSets[skillSetId].title)
+			end)
+
+			it("Sets copied skill set as active via service", function()
+				skillsSetService:NewSkillSet("Skill")
+				skillsSetService:CopySkillSet(1, "Service Copy")
+				local skillSetId = build.skillsTab.skillSetOrderList[3]
+
+				assert.are.equals(skillSetId, build.skillsTab.activeSkillSetId)
+			end)
+
+			it("Adds an undo state when copying a skill set", function()
+				local undoCountBefore = #build.skillsTab.undo
+				skillsSetService:CopySkillSet(1, "Copied Skill")
+
+				assert.are.equals(#build.skillsTab.undo, undoCountBefore + 1)
+			end)
+		end)
+
+		describe("RenameSkillSet", function()
+			it("Renames a skill set via service", function()
+				local skillSetName = "Original Name"
+				skillsSetService:NewSkillSet(skillSetName)
+				local skillSetId = build.skillsTab.skillSetOrderList[2]
+
+				skillsSetService:RenameSkillSet(skillSetId, "New Name")
+
+				assert.are.equals("New Name", build.skillsTab.skillSets[skillSetId].title)
+			end)
+
+			it("Does not rename non-existent skill set", function()
+				skillsSetService:RenameSkillSet(10, "Non-existent")
+
+				assert.is_nil(build.skillsTab.skillSets[10])
+			end)
+
+			it("Adds an undo state when renaming", function()
+				local skillSetName = "Skill To Rename"
+				skillsSetService:NewSkillSet(skillSetName)
+				local skillSetId = build.skillsTab.skillSetOrderList[2]
+
+				local undoCountBefore = #build.skillsTab.undo
+				skillsSetService:RenameSkillSet(skillSetId, "Renamed Skill")
+
+				assert.are.equals(#build.skillsTab.undo, undoCountBefore + 1)
+			end)
+		end)
+
+		describe("DeleteSkillSet", function()
+			it("Deletes a skill set via service", function()
+				skillsSetService:NewSkillSet("Service Delete Me")
+
+				skillsSetService:DeleteSkillSet(2, 2)
+
+				assert.is_nil(build.skillsTab.skillSets[2])
+				assert.is_nil(build.skillsTab.skillSetOrderList[2])
+			end)
+
+			it("Switches active skill set when deleting current", function()
+				skillsSetService:NewSkillSet("Service Delete Me")
+				local skillToKeep = 1
+				local skillToDelete = 2
+
+				assert.are.equals(skillToDelete, build.skillsTab.activeSkillSetId)
+
+				skillsSetService:DeleteSkillSet(skillToDelete, 2)
+
+				assert.are.equals(skillToKeep, build.skillsTab.activeSkillSetId)
+			end)
+
+			it("Does not delete last skill set via service", function()
+				skillsSetService:NewSkillSet("Last One")
+
+				skillsSetService:DeleteSkillSet(1, 1)
+
+				assert.are.equals(1, #build.skillsTab.skillSetOrderList)
+			end)
+
+			it("Adds an undo state when deleting", function()
+				local undoCountBefore = #build.skillsTab.undo
+				local undoCountAfterNew = #build.skillsTab.undo
+				skillsSetService:NewSkillSet("Undo Delete Test")
+
+				skillsSetService:DeleteSkillSet(2, 2)
+
+				assert.are.equals(#build.skillsTab.undo, undoCountBefore + 2)
+			end)
+		end)
+
+
+
+		describe("Integration", function()
+			it("Completes a skill set lifecycle", function()
+				local skill1Name = "First Skill"
+				local skill2Name = "Second Skill"
+
+				skillsSetService:NewSkillSet(skill1Name)
+				assert.are.equals("First Skill", build.skillsTab.skillSets[2].title)
+
+				skillsSetService:CopySkillSet(2, skill2Name)
+				assert.are.equals("Second Skill", build.skillsTab.skillSets[3].title)
+			end)
+		end)
+	end)
+
+	describe("SkillSetStateManagement", function()
+		local skillsSetService
+
+		before_each(function()
+			skillsSetService = new("SkillsSetService", build.skillsTab)
+		end)
+
+		describe("Socket group persistence", function()
+			it("Preserves socket groups across skill set switches", function()
+				skillsSetService:NewSkillSet("Set 1")
+				skillsSetService:NewSkillSet("Set 2")
+
+				local testSocketGroup = {
+					label = "Test Group",
+					enabled = true,
+					gemList = {
+						{ nameSpec = "TestGem", level = 20, quality = 0, qualityId = "Default", enabled = true, count = 1 }
+					}
+				}
+
+				build.skillsTab.skillSets[1].socketGroupList[1] = testSocketGroup
+				build.skillsTab:SetActiveSkillSet(1)
+
+				assert.are.same(testSocketGroup, build.skillsTab.skillSets[1].socketGroupList[1])
+
+				build.skillsTab:SetActiveSkillSet(3)
+				build.skillsTab.skillSets[3].socketGroupList[1] = { label = "Different" }
+
+				build.skillsTab:SetActiveSkillSet(1)
+				assert.are.same(testSocketGroup, build.skillsTab.skillSets[1].socketGroupList[1])
+			end)
+
+			it("Preserves nested gem data across skill set switches", function()
+				skillsSetService:NewSkillSet("Set 1")
+				skillsSetService:NewSkillSet("Set 2")
+				skillsSetService:NewSkillSet("Set 3")
+
+				local testGem = { nameSpec = "TestGem", level = 20, quality = 20 }
+				build.skillsTab.skillSets[3].socketGroupList = { { gemList = { testGem } } }
+
+				build.skillsTab:SetActiveSkillSet(3)
+				assert.are.same(testGem, build.skillsTab.skillSets[3].socketGroupList[1].gemList[1])
+			end)
+		end)
+
+		describe("Default values", function()
+			it("Initializes new skill sets with default structure", function()
+				assert.is_not_nil(build.skillsTab.skillSets[1])
+				assert.is_not_nil(build.skillsTab.skillSets[1].socketGroupList)
+				assert.is_not_nil(build.skillsTab.skillSets[1].id)
+			end)
+
+			it("Copies socket group structure when copying skill sets", function()
+				local testSocketGroup = {
+					label = "Original",
+					enabled = false,
+					gemList = {}
+				}
+				build.skillsTab.skillSets[1].socketGroupList[1] = testSocketGroup
+
+				local newSkillSet = build.skillsTab:CopySkillSet(1, "Copy Test")
+
+				assert.are.equals("Original", newSkillSet.socketGroupList[1].label)
+				assert.is_false(newSkillSet.socketGroupList[1].enabled)
+			end)
+		end)
+	end)
+end)