Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expand regression test suite #523

Merged
merged 3 commits into from
Aug 7, 2022
Merged

Conversation

@codecov
Copy link

codecov bot commented Aug 7, 2022

Codecov Report

Merging #523 (3b0419d) into master (0ae6a84) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #523   +/-   ##
=======================================
  Coverage   97.66%   97.66%           
=======================================
  Files          14       14           
  Lines        5390     5390           
=======================================
  Hits         5264     5264           
  Misses        126      126           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 7, 2022

Repo Comparison Test

Error when running latest on BlizzardInterfaceCode:

error: could not format file ./Interface/FrameXML/FlowContainer.lua: error parsing: error occurred while creating ast: unexpected token `)`. (starting from line 136, character 56 and ending on line 136, character 57)
additional information: expected 'then'
error: could not format file ./Interface/FrameXML/SecureGroupHeaders.lua: error parsing: error occurred while creating ast: unexpected token `then`. (starting from line 472, character 4 and ending on line 472, character 8)
additional information: expected ')'

diff --git ORI/luvit/deps/http-codec.lua ALT/luvit/deps/http-codec.lua
index 82caa66..d98936d 100644
--- ORI/luvit/deps/http-codec.lua
+++ ALT/luvit/deps/http-codec.lua
@@ -90,7 +90,7 @@ local STATUS_CODES = {
 	[507] = "Insufficient Storage", -- RFC 4918
 	[509] = "Bandwidth Limit Exceeded",
 	[510] = "Not Extended", -- RFC 2774
-	[511] = "Network Authentication Required",     -- RFC 6585
+	[511] = "Network Authentication Required", -- RFC 6585
 }
 
 local function encoder()
diff --git ORI/luvit/examples/http-connect.lua ALT/luvit/examples/http-connect.lua
index 866bbae..3d76276 100644
--- ORI/luvit/examples/http-connect.lua
+++ ALT/luvit/examples/http-connect.lua
@@ -6,7 +6,7 @@ local options = {
 	path = "www.google.com:80", -- what to proxy to
 	method = "CONNECT",
 	headers = {
-		{ "host", "proxy" },          -- proxy vhost on host
+		{ "host", "proxy" }, -- proxy vhost on host
 	},
 }
 
diff --git ORI/luvit/tests/test-url.lua ALT/luvit/tests/test-url.lua
index 105e660..88a1867 100644
--- ORI/luvit/tests/test-url.lua
+++ ALT/luvit/tests/test-url.lua
@@ -86,7 +86,11 @@ local relativeTests = {
 		"https://u:[email protected]/p/a/t/h?s#hash2",
 	},
 	{ "http://example.com/b//c//d;p?q#blarg", "https://a/b/c/d", "https://a/b/c/d" },
-	{ "http://example.com/b//c//d;p?q#blarg", "http://u:[email protected]/p/a/t/h?s#hash2", "http://u:[email protected]/p/a/t/h?s#hash2" },
+	{
+		"http://example.com/b//c//d;p?q#blarg",
+		"http://u:[email protected]/p/a/t/h?s#hash2",
+		"http://u:[email protected]/p/a/t/h?s#hash2",
+	},
 	{ "/foo/bar/baz", "/../etc/passwd", "/etc/passwd" },
 	{ "http://localhost", "file://foo/Users", "file://foo/Users" },
 	-- from node
diff --git ORI/luvit/tests/test-ustring.lua ALT/luvit/tests/test-ustring.lua
index 2595b6f..20348c6 100644
--- ORI/luvit/tests/test-ustring.lua
+++ ALT/luvit/tests/test-ustring.lua
@@ -64,7 +64,7 @@ require("tap")(function(test)
 			"😘",
 			"😄",
 			"💎",
-			"💛",-- Emoji
+			"💛", -- Emoji
 		}
 		assert(compareTable(u, e))
 		assert(tostring(u) == r) -- __tostring
diff --git ORI/lit/commands/serve.lua ALT/lit/commands/serve.lua
index 38e6078..e453aa3 100644
--- ORI/lit/commands/serve.lua
+++ ALT/lit/commands/serve.lua
@@ -37,11 +37,10 @@ return function()
 		end
 	end)
 
-	app.use(require("weblit-auto-headers"))
-.route({ method = "GET", path = "/snapshots" }, require("snapshots")).route(
-		{ method = "GET", path = "/stats" },
-		require("stats")
-	)
+	app
+		.use(require("weblit-auto-headers"))
+		.route({ method = "GET", path = "/snapshots" }, require("snapshots"))
+		.route({ method = "GET", path = "/stats" }, require("stats"))
 
 	-- Handle websocket clients
 	app.websocket({
diff --git ORI/lit/deps/http-codec.lua ALT/lit/deps/http-codec.lua
index a728bdb..6bb2f4b 100644
--- ORI/lit/deps/http-codec.lua
+++ ALT/lit/deps/http-codec.lua
@@ -90,7 +90,7 @@ local STATUS_CODES = {
 	[507] = "Insufficient Storage", -- RFC 4918
 	[509] = "Bandwidth Limit Exceeded",
 	[510] = "Not Extended", -- RFC 2774
-	[511] = "Network Authentication Required",     -- RFC 6585
+	[511] = "Network Authentication Required", -- RFC 6585
 }
 
 local function encoder()
diff --git ORI/lit/deps/ssh-rsa.lua ALT/lit/deps/ssh-rsa.lua
index 17bc6bf..ac1f3ae 100644
--- ORI/lit/deps/ssh-rsa.lua
+++ ALT/lit/deps/ssh-rsa.lua
@@ -45,25 +45,27 @@ do
 	function dec(data)
 		data = string.gsub(data, "[^" .. chars .. "=]", "")
 		return (
-			data:gsub(".", function(x)
-				if x == "=" then
-					return ""
-				end
-				local r, f = "", (chars:find(x) - 1)
-				for i = 6, 1, -1 do
-					r = r .. (f % 2 ^ i - f % 2 ^ (i - 1) > 0 and "1" or "0")
-				end
-				return r
-			end):gsub("%d%d%d?%d?%d?%d?%d?%d?", function(x)
-				if #x ~= 8 then
-					return ""
-				end
-				local c = 0
-				for i = 1, 8 do
-					c = c + (x:sub(i, i) == "1" and 2 ^ (8 - i) or 0)
-				end
-				return string.char(c)
-			end)
+			data
+				:gsub(".", function(x)
+					if x == "=" then
+						return ""
+					end
+					local r, f = "", (chars:find(x) - 1)
+					for i = 6, 1, -1 do
+						r = r .. (f % 2 ^ i - f % 2 ^ (i - 1) > 0 and "1" or "0")
+					end
+					return r
+				end)
+				:gsub("%d%d%d?%d?%d?%d?%d?%d?", function(x)
+					if #x ~= 8 then
+						return ""
+					end
+					local c = 0
+					for i = 1, 8 do
+						c = c + (x:sub(i, i) == "1" and 2 ^ (8 - i) or 0)
+					end
+					return string.char(c)
+				end)
 		)
 	end
 end
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_AuctionHouseUI/Blizzard_AuctionHouseCategoriesList.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_AuctionHouseUI/Blizzard_AuctionHouseCategoriesList.lua
index 845d009..e7f9062 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_AuctionHouseUI/Blizzard_AuctionHouseCategoriesList.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_AuctionHouseUI/Blizzard_AuctionHouseCategoriesList.lua
@@ -15,16 +15,13 @@ function AuctionFrameFilters_UpdateCategories(categoriesList, forceSelectionInto
 		local selected = selectedCategoryIndex and selectedCategoryIndex == categoryIndex
 		local isToken = categoryInfo:HasFlag("WOW_TOKEN_FLAG")
 
-		tinsert(
-			EXPANDED_FILTERS,
-			{
-				name = categoryInfo.name,
-				type = "category",
-				categoryIndex = categoryIndex,
-				selected = selected,
-				isToken = isToken,
-			}
-		)
+		tinsert(EXPANDED_FILTERS, {
+			name = categoryInfo.name,
+			type = "category",
+			categoryIndex = categoryIndex,
+			selected = selected,
+			isToken = isToken,
+		})
 
 		if selected then
 			AuctionFrameFilters_AddSubCategories(categoriesList, categoryInfo.subCategories)
@@ -52,15 +49,12 @@ function AuctionFrameFilters_AddSubCategories(categoriesList, subCategories)
 		for subCategoryIndex, subCategoryInfo in ipairs(subCategories) do
 			local selected = select(2, categoriesList:GetSelectedCategory()) == subCategoryIndex
 
-			tinsert(
-				EXPANDED_FILTERS,
-				{
-					name = subCategoryInfo.name,
-					type = "subCategory",
-					subCategoryIndex = subCategoryIndex,
-					selected = selected,
-				}
-			)
+			tinsert(EXPANDED_FILTERS, {
+				name = subCategoryInfo.name,
+				type = "subCategory",
+				subCategoryIndex = subCategoryIndex,
+				selected = selected,
+			})
 
 			if selected then
 				AuctionFrameFilters_AddSubSubCategories(categoriesList, subCategoryInfo.subCategories)
@@ -75,16 +69,13 @@ function AuctionFrameFilters_AddSubSubCategories(categoriesList, subSubCategorie
 			local selected = select(3, categoriesList:GetSelectedCategory()) == subSubCategoryIndex
 			local isLast = subSubCategoryIndex == #subSubCategories
 
-			tinsert(
-				EXPANDED_FILTERS,
-				{
-					name = subSubCategoryInfo.name,
-					type = "subSubCategory",
-					subSubCategoryIndex = subSubCategoryIndex,
-					selected = selected,
-					isLast = isLast,
-				}
-			)
+			tinsert(EXPANDED_FILTERS, {
+				name = subSubCategoryInfo.name,
+				type = "subSubCategory",
+				subSubCategoryIndex = subSubCategoryIndex,
+				selected = selected,
+				isLast = isLast,
+			})
 		end
 	end
 end
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_BattlefieldMap/Blizzard_BattlefieldMap.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_BattlefieldMap/Blizzard_BattlefieldMap.lua
index c46f765..723ec09 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_BattlefieldMap/Blizzard_BattlefieldMap.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_BattlefieldMap/Blizzard_BattlefieldMap.lua
@@ -289,7 +289,7 @@ function BattlefieldMapMixin:OnUpdate(elapsed)
 	-- tick mouse hover time for tab
 	if self.hover then
 		local xPos, yPos = GetCursorPosition()
-		if (self.oldX == xPos and self.oldy == yPos) then
+		if self.oldX == xPos and self.oldy == yPos then
 			self.hoverTime = self.hoverTime + elapsed
 		else
 			self.hoverTime = 0
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_ChallengesUI/Blizzard_ChallengesUI.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_ChallengesUI/Blizzard_ChallengesUI.lua
index 8702d3a..8830be0 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_ChallengesUI/Blizzard_ChallengesUI.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_ChallengesUI/Blizzard_ChallengesUI.lua
@@ -906,23 +906,20 @@ function ChallengeModeCompleteBannerMixin:OnEvent(event, ...)
 			C_ChallengeMode.GetCompletionInfo()
 
 		if not practiceRun then
-			TopBannerManager_Show(
-				self,
-				{
-					mapID = mapID,
-					level = level,
-					time = time,
-					onTime = onTime,
-					oldDungeonScore = oldDungeonScore,
-					newDungeonScore = newDungeonScore,
-					keystoneUpgradeLevels = keystoneUpgradeLevels,
-					isMapRecord = isMapRecord,
-					isAffixRecord = isAffixRecord,
-					primaryAffix = primaryAffix,
-					isEligibleForScore = isEligibleForScore,
-					upgradeMembers = upgradeMembers,
-				}
-			)
+			TopBannerManager_Show(self, {
+				mapID = mapID,
+				level = level,
+				time = time,
+				onTime = onTime,
+				oldDungeonScore = oldDungeonScore,
+				newDungeonScore = newDungeonScore,
+				keystoneUpgradeLevels = keystoneUpgradeLevels,
+				isMapRecord = isMapRecord,
+				isAffixRecord = isAffixRecord,
+				primaryAffix = primaryAffix,
+				isEligibleForScore = isEligibleForScore,
+				upgradeMembers = upgradeMembers,
+			})
 		end
 	end
 end
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Collections/Blizzard_Wardrobe.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Collections/Blizzard_Wardrobe.lua
index a788669..243b800 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Collections/Blizzard_Wardrobe.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Collections/Blizzard_Wardrobe.lua
@@ -1520,7 +1520,7 @@ function WardrobeCollectionFrameMixin:OnEvent(event, ...)
 		end
 	elseif event == "UNIT_MODEL_CHANGED" then
 		local hasAlternateForm, inAlternateForm = C_PlayerInfo.GetAlternateFormInfo()
-		if (self.inAlternateForm ~= inAlternateForm or self.updateOnModelChanged) then
+		if self.inAlternateForm ~= inAlternateForm or self.updateOnModelChanged then
 			if self.activeFrame:OnUnitModelChangedEvent() then
 				self.inAlternateForm = inAlternateForm
 				self.updateOnModelChanged = nil
@@ -3884,16 +3884,13 @@ function WardrobeSetsDataProviderMixin:GetSortedSetSources(setID)
 		local sourceInfo = C_TransmogCollection.GetSourceInfo(sourceID)
 		if sourceInfo then
 			local sortOrder = EJ_GetInvTypeSortOrder(sourceInfo.invType)
-			tinsert(
-				returnTable,
-				{
-					sourceID = sourceID,
-					collected = primaryAppearance.collected,
-					sortOrder = sortOrder,
-					itemID = sourceInfo.itemID,
-					invType = sourceInfo.invType,
-				}
-			)
+			tinsert(returnTable, {
+				sourceID = sourceID,
+				collected = primaryAppearance.collected,
+				sortOrder = sortOrder,
+				itemID = sourceInfo.itemID,
+				invType = sourceInfo.invType,
+			})
 		end
 	end
 
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_CombatLog/Blizzard_CombatLog.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_CombatLog/Blizzard_CombatLog.lua
index 3c33ffd..7d5866e 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_CombatLog/Blizzard_CombatLog.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_CombatLog/Blizzard_CombatLog.lua
@@ -3449,7 +3449,7 @@ function CombatLog_OnEvent(
 		else
 			if (settings.lineColorPriority == 1 and sourceName) or not destName then
 				lineColor = CombatLog_Color_ColorArrayByUnitType(sourceFlags, filterSettings)
-			elseif (settings.lineColorPriority == 2 and destName) then
+			elseif settings.lineColorPriority == 2 and destName then
 				lineColor = CombatLog_Color_ColorArrayByUnitType(destFlags, filterSettings)
 			else
 				lineColor = CombatLog_Color_ColorArrayByUnitType(sourceFlags, filterSettings)
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_CombatText/Blizzard_CombatText.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_CombatText/Blizzard_CombatText.lua
index 0386b17..7e2a82f 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_CombatText/Blizzard_CombatText.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_CombatText/Blizzard_CombatText.lua
@@ -431,9 +431,8 @@ function CombatText_OnUpdate(self, elapsed)
 			if value.scrollTime >= COMBAT_TEXT_FADEOUT_TIME then
 				alpha = 1
 					- (
-						(value.scrollTime - COMBAT_TEXT_FADEOUT_TIME) / (
-							COMBAT_TEXT_SCROLLSPEED - COMBAT_TEXT_FADEOUT_TIME
-						)
+						(value.scrollTime - COMBAT_TEXT_FADEOUT_TIME)
+						/ (COMBAT_TEXT_SCROLLSPEED - COMBAT_TEXT_FADEOUT_TIME)
 					)
 				alpha = max(alpha, 0)
 				value:SetAlpha(alpha)
@@ -695,11 +694,8 @@ function CombatText_StandardScroll(value)
 	-- Calculate x and y positions
 	local xPos = value.startX
 		+ ((COMBAT_TEXT_LOCATIONS.endX - COMBAT_TEXT_LOCATIONS.startX) * value.scrollTime / COMBAT_TEXT_SCROLLSPEED)
-	local yPos = value.startY + (
-			(value.endY - COMBAT_TEXT_LOCATIONS.startY)
-			* value.scrollTime
-			/ COMBAT_TEXT_SCROLLSPEED
-		)
+	local yPos = value.startY
+		+ ((value.endY - COMBAT_TEXT_LOCATIONS.startY) * value.scrollTime / COMBAT_TEXT_SCROLLSPEED)
 	return xPos, yPos
 end
 
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/ClubFinder.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/ClubFinder.lua
index 8eaf00a..4de1692 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/ClubFinder.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/ClubFinder.lua
@@ -670,9 +670,9 @@ function LookingForDropdownMixin:IsSpecInList(specID)
 end
 
 function LookingForDropdownMixin:ModifyTrackedSpecList(specName, className, specID, shouldAdd)
-	if (shouldAdd and not self.checkedList[specID]) then
+	if shouldAdd and not self.checkedList[specID] then
 		self.checkedCount = self.checkedCount + 1
-	elseif (not shouldAdd and self.checkedList[specID]) then
+	elseif not shouldAdd and self.checkedList[specID] then
 		self.checkedCount = self.checkedCount - 1
 	end
 	self.checkedList[specID] = shouldAdd and { specID = specID, specName = specName, className = className } or nil
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/CommunitiesFrame.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/CommunitiesFrame.lua
index d852f96..71eabb6 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/CommunitiesFrame.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/CommunitiesFrame.lua
@@ -140,7 +140,7 @@ function CommunitiesFrameMixin:OnShow()
 
 	self:UpdatePortrait()
 
-	self:RequestSubscribedClubFinderPostingInfo() -- In case something has changed in the status of our posting since loading in. 
+	self:RequestSubscribedClubFinderPostingInfo() -- In case something has changed in the status of our posting since loading in.
 
 	FrameUtil.RegisterFrameForEvents(self, COMMUNITIES_FRAME_EVENTS)
 	FrameUtil.RegisterFrameForEvents(self, CLUB_FINDER_APPLICANT_LIST_EVENTS)
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/CommunitiesMemberList.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/CommunitiesMemberList.lua
index d1a15c5..25afc9f 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/CommunitiesMemberList.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/CommunitiesMemberList.lua
@@ -1343,7 +1343,7 @@ function CommunitiesMemberListEntryMixin:RefreshExpandedColumns()
 			end
 			self.GuildInfo:SetText(color:WrapTextInColorCode(memberInfo.overallDungeonScore))
 		else
-			self.GuildInfo:SetText(NO_ROSTER_ACHIEVEMENT_POINTS) -- Display - if there is no dungeon score. 
+			self.GuildInfo:SetText(NO_ROSTER_ACHIEVEMENT_POINTS) -- Display - if there is no dungeon score.
 		end
 	end
 end
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/CommunitiesSettings.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/CommunitiesSettings.lua
index 0039483..df11da9 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/CommunitiesSettings.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Communities/CommunitiesSettings.lua
@@ -251,7 +251,7 @@ function CommunitiesSettingsDialogMixin:ResetClubFinderSettings()
 	self.ShouldListClub.Button:SetChecked(false)
 
 	self.ClubFocusDropdown:Initialize()
-	C_ClubFinder.SetRecruitmentSettings(Enum.ClubFinderSettingFlags.Social, true) --Initialize to this being on as default. 
+	C_ClubFinder.SetRecruitmentSettings(Enum.ClubFinderSettingFlags.Social, true) --Initialize to this being on as default.
 	UIDropDownMenu_Initialize(self.ClubFocusDropdown, ClubFocusClubDropdownInitialize)
 	self.ClubFocusDropdown:UpdateDropdownText()
 
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_GarrisonUI/Blizzard_GarrisonShipyardUI.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_GarrisonUI/Blizzard_GarrisonShipyardUI.lua
index 064e8c4..be67ad2 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_GarrisonUI/Blizzard_GarrisonShipyardUI.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_GarrisonUI/Blizzard_GarrisonShipyardUI.lua
@@ -1039,10 +1039,28 @@ function GarrisonShipyardMap_OnUpdate(self)
 end
 
 local fogData = {
-	["NavalMap-Alliance"] = { anchor = "BOTTOMRIGHT", leftOffset = 48, rightOffset = 0, topOffset = -36, bottomOffset = 0 },
+	["NavalMap-Alliance"] = {
+		anchor = "BOTTOMRIGHT",
+		leftOffset = 48,
+		rightOffset = 0,
+		topOffset = -36,
+		bottomOffset = 0,
+	},
 	["NavalMap-Horde"] = { anchor = "TOPLEFT", leftOffset = 0, rightOffset = -52, topOffset = 0, bottomOffset = 45 },
-	["NavalMap-IronHorde"] = { anchor = "TOPRIGHT", leftOffset = 70, rightOffset = 0, topOffset = 0, bottomOffset = 42 },
-	["NavalMap-OpenWaters"] = { anchor = "BOTTOMLEFT", leftOffset = 0, rightOffset = -82, topOffset = -70, bottomOffset = 0 },
+	["NavalMap-IronHorde"] = {
+		anchor = "TOPRIGHT",
+		leftOffset = 70,
+		rightOffset = 0,
+		topOffset = 0,
+		bottomOffset = 42,
+	},
+	["NavalMap-OpenWaters"] = {
+		anchor = "BOTTOMLEFT",
+		leftOffset = 0,
+		rightOffset = -82,
+		topOffset = -70,
+		bottomOffset = 0,
+	},
 }
 
 function GarrisonShipyardMap_SetupFog(self, siegeBreakerFrame, offeredGarrMissionTextureID)
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_NewPlayerExperience/Blizzard_TutorialSteps.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_NewPlayerExperience/Blizzard_TutorialSteps.lua
index f8e23aa..e91c2a1 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_NewPlayerExperience/Blizzard_TutorialSteps.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_NewPlayerExperience/Blizzard_TutorialSteps.lua
@@ -905,8 +905,7 @@ function Class_ItemUpgradeCheckingService:Start()
 end
 
 function Class_ItemUpgradeCheckingService:STRUCT_ItemContainer(itemLink, characterSlot, container, containerSlot)
-	return
-{
+	return {
 		ItemLink = itemLink,
 		Container = container,
 		ContainerSlot = containerSlot,
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_PTRFeedback/Blizzard_Reports.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_PTRFeedback/Blizzard_Reports.lua
index 6c956ee..8adb093 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_PTRFeedback/Blizzard_Reports.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_PTRFeedback/Blizzard_Reports.lua
@@ -94,13 +94,10 @@ function PTR_IssueReporter.AttachDefaultCollectionToSurvey(survey, ignoreTypeQue
 	survey:AddDataCollection(collector.RunFunction, GetCurrentConduits)
 	survey:AddDataCollection(collector.RunFunction, GetCurrentSoulbindTraits)
 	if not ignoreTypeQuestion then
-		survey:AddDataCollection(
-			collector.SelectOne_MessageKeyUpdater,
-			{
-				{ Choice = "Bug", Key = PTR_IssueReporter.Data.Message_Key },
-				{ Choice = "Feedback", Key = PTR_IssueReporter.Data.Feedback_Message_Key },
-			}
-		)
+		survey:AddDataCollection(collector.SelectOne_MessageKeyUpdater, {
+			{ Choice = "Bug", Key = PTR_IssueReporter.Data.Message_Key },
+			{ Choice = "Feedback", Key = PTR_IssueReporter.Data.Feedback_Message_Key },
+		})
 	end
 end
 --------------------------------------------------------------------------------------------------------
@@ -912,61 +909,41 @@ function PTR_IssueReporter.CreateNPEReports() -- These surveys are for the New P
 		collector.TextBlock,
 		"|cFFFFFFFFThe goal of this experience is to teach new\nplayers about the basics of World of Warcraft.\n\nThe next set of questions will ask about a few different aspects that the team is trying to teach players. Please rate each based on how well you felt like you learned them by the end of the experience.\n\nPlease be honest - if you don't feel like you learned how to do these things it means that our experience is not working, not that you missed anything!"
 	)
-	whatWasLearnedReport:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Moving Your Camera",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
-	whatWasLearnedReport:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Moving Your Character",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
-	whatWasLearnedReport:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Obtaining & Completing Quests",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
-	whatWasLearnedReport:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Fighting One Enemy",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
-	whatWasLearnedReport:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Fighting Multiple Enemies",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
+	whatWasLearnedReport:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Moving Your Camera", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
+	whatWasLearnedReport:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Moving Your Character", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
+	whatWasLearnedReport:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Obtaining & Completing Quests", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
+	whatWasLearnedReport:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Fighting One Enemy", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
+	whatWasLearnedReport:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Fighting Multiple Enemies", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
 
 	local whatWasLearnedReport2 =
 		PTR_IssueReporter.CreateSurvey(URSurveyIndexOffset + 24, "Overall Experience - Tutorials")
@@ -993,98 +970,66 @@ function PTR_IssueReporter.CreateNPEReports() -- These surveys are for the New P
 			"Learned\nExtremely\nWell",
 		}
 	)
-	whatWasLearnedReport2:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Obtaining & Equipping New Gear",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
-	whatWasLearnedReport2:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Quest Objectives",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
-	whatWasLearnedReport2:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Using Items To Complete Quests",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
+	whatWasLearnedReport2:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Obtaining & Equipping New Gear", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
+	whatWasLearnedReport2:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Quest Objectives", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
+	whatWasLearnedReport2:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Using Items To Complete Quests", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
 
 	local whatWasLearnedReport3 =
 		PTR_IssueReporter.CreateSurvey(URSurveyIndexOffset + 25, "Overall Experience - Tutorials")
 	PTR_IssueReporter.AttachDefaultCollectionToSurvey(whatWasLearnedReport3)
-	whatWasLearnedReport3:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Buying & Selling Items",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
-	whatWasLearnedReport3:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Mini-Map User Interface",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
-	whatWasLearnedReport3:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Large Map User Interface",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
-	whatWasLearnedReport3:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Interactable Objects",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
-	whatWasLearnedReport3:AddDataCollection(
-		collector.SelectOne_MultipleChoiceQuestion,
-		"Finding A Group For Activities",
-		{
-			"Did Not\nLearn At All",
-			"Learned\nSlightly Well",
-			"Learned\nModerately Well",
-			"Learned\nVery Well",
-			"Learned\nExtremely\nWell",
-		}
-	)
+	whatWasLearnedReport3:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Buying & Selling Items", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
+	whatWasLearnedReport3:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Mini-Map User Interface", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
+	whatWasLearnedReport3:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Large Map User Interface", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
+	whatWasLearnedReport3:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Interactable Objects", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
+	whatWasLearnedReport3:AddDataCollection(collector.SelectOne_MultipleChoiceQuestion, "Finding A Group For Activities", {
+		"Did Not\nLearn At All",
+		"Learned\nSlightly Well",
+		"Learned\nModerately Well",
+		"Learned\nVery Well",
+		"Learned\nExtremely\nWell",
+	})
 
 	local endOfSurvey = PTR_IssueReporter.CreateSurvey(URSurveyIndexOffset + 26, "Thank you!")
 	PTR_IssueReporter.AttachDefaultCollectionToSurvey(endOfSurvey)
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_RaidUI/Blizzard_RaidUI.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_RaidUI/Blizzard_RaidUI.lua
index 8cccf67..37ea48a 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_RaidUI/Blizzard_RaidUI.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_RaidUI/Blizzard_RaidUI.lua
@@ -1128,18 +1128,14 @@ function RaidPullout_SaveFrames(pullOutFrame)
 			end
 
 			-- Save its position and settings
-			tinsert(
-				RAID_SINGLE_POSITIONS,
-				1,
-				{
-					point = point,
-					relativePoint = relativePoint,
-					x = offsetX,
-					y = offsetY,
-					name = pullOutFrame.filterID,
-					["settings"] = settings,
-				}
-			)
+			tinsert(RAID_SINGLE_POSITIONS, 1, {
+				point = point,
+				relativePoint = relativePoint,
+				x = offsetX,
+				y = offsetY,
+				name = pullOutFrame.filterID,
+				["settings"] = settings,
+			})
 		else
 			if not RAID_PULLOUT_POSITIONS[filterID] then
 				RAID_PULLOUT_POSITIONS[filterID] = {}
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_StoreUI/Blizzard_FullProductCardTemplates.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_StoreUI/Blizzard_FullProductCardTemplates.lua
index efd30e7..f73a541 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_StoreUI/Blizzard_FullProductCardTemplates.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_StoreUI/Blizzard_FullProductCardTemplates.lua
@@ -74,7 +74,7 @@ function FullStoreCardMixin:ShowTooltip()
 
 		local entryInfo = self:GetEntryInfo()
 		local description = self:AppendBundleInformationToTooltipDescription(entryInfo, self.productTooltipDescription)
-		description = strtrim(description, "\n\r") -- Ensure we don't end the description with a new line.		
+		description = strtrim(description, "\n\r") -- Ensure we don't end the description with a new line.
 		StoreTooltip_Show(self.productTooltipTitle, description)
 	end
 end
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_StoreUI/Blizzard_SmallProductCardTemplates.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_StoreUI/Blizzard_SmallProductCardTemplates.lua
index 41fc8c4..ed54ab5 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_StoreUI/Blizzard_SmallProductCardTemplates.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_StoreUI/Blizzard_SmallProductCardTemplates.lua
@@ -330,7 +330,7 @@ function MediumStoreCardWithBuyButtonMixin:ShowTooltip()
 
 		local entryInfo = self:GetEntryInfo()
 		local description = self:AppendBundleInformationToTooltipDescription(entryInfo, self.productTooltipDescription)
-		description = strtrim(description, "\n\r") -- Ensure we don't end the description with a new line.		
+		description = strtrim(description, "\n\r") -- Ensure we don't end the description with a new line.
 		StoreTooltip_Show(self.productTooltipTitle, description)
 	end
 end
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_TorghastLevelPicker/Blizzard_TorghastLevelPicker.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_TorghastLevelPicker/Blizzard_TorghastLevelPicker.lua
index 0d5b7f7..3615bba 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_TorghastLevelPicker/Blizzard_TorghastLevelPicker.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_TorghastLevelPicker/Blizzard_TorghastLevelPicker.lua
@@ -371,30 +371,24 @@ function TorghastLevelPickerRewardCircleMixin:SetSortedRewards()
 			if IsCurrencyContainer then
 				local name, texture, quantity, quality =
 					CurrencyContainerUtil.GetCurrencyContainerInfo(reward.id, quantity)
-				table.insert(
-					self.currencyRewards,
-					{
-						id = reward.id,
-						texture = texture,
-						quantity = quantity,
-						quality = quality,
-						name = name,
-						isCurrencyContainer = true,
-					}
-				)
+				table.insert(self.currencyRewards, {
+					id = reward.id,
+					texture = texture,
+					quantity = quantity,
+					quality = quality,
+					name = name,
+					isCurrencyContainer = true,
+				})
 			else
 				local currencyInfo = C_CurrencyInfo.GetCurrencyInfo(reward.id)
-				table.insert(
-					self.currencyRewards,
-					{
-						id = reward.id,
-						texture = currencyInfo.iconFileID,
-						quantity = reward.quantity,
-						quality = currencyInfo.quality,
-						name = currencyInfo.name,
-						isCurrencyContainer = false,
-					}
-				)
+				table.insert(self.currencyRewards, {
+					id = reward.id,
+					texture = currencyInfo.iconFileID,
+					quantity = reward.quantity,
+					quality = currencyInfo.quality,
+					name = currencyInfo.name,
+					isCurrencyContainer = false,
+				})
 			end
 		end
 	end
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Tutorial/Blizzard_TutorialLogic.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Tutorial/Blizzard_TutorialLogic.lua
index 8494477..f516163 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Tutorial/Blizzard_TutorialLogic.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_Tutorial/Blizzard_TutorialLogic.lua
@@ -1085,8 +1085,7 @@ function Class_EquipFirstItemWatcher:UNIT_INVENTORY_CHANGED()
 end
 
 function Class_EquipFirstItemWatcher:STRUCT_ItemContainer(itemID, characterSlot, container, containerSlot)
-	return
-{
+	return {
 		ItemID = itemID,
 		Container = container,
 		ContainerSlot = containerSlot,
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_UIWidgets/Blizzard_UIWidgetTemplateBase.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_UIWidgets/Blizzard_UIWidgetTemplateBase.lua
index 92a5317..135237f 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_UIWidgets/Blizzard_UIWidgetTemplateBase.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_UIWidgets/Blizzard_UIWidgetTemplateBase.lua
@@ -755,9 +755,8 @@ function UIWidgetBaseTextureAndTextTemplateMixin:Setup(
 
 	if layoutIndex then
 		if
-			frameTextureKit and C_Texture.GetAtlasInfo(
-				GetFinalNameFromTextureKit("%s_" .. layoutIndex, frameTextureKit)
-			)
+			frameTextureKit
+			and C_Texture.GetAtlasInfo(GetFinalNameFromTextureKit("%s_" .. layoutIndex, frameTextureKit))
 		then
 			bgTextureKitFmt = "%s_" .. layoutIndex
 		end
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_UIWidgets/Blizzard_UIWidgetTemplateDiscreteProgressSteps.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_UIWidgets/Blizzard_UIWidgetTemplateDiscreteProgressSteps.lua
index 3051b19..073f1c8 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_UIWidgets/Blizzard_UIWidgetTemplateDiscreteProgressSteps.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_UIWidgets/Blizzard_UIWidgetTemplateDiscreteProgressSteps.lua
@@ -162,7 +162,7 @@ function UIWidgetTemplateDiscreteProgressStepsMixin:Setup(widgetInfo, widgetCont
 	SetupTextureKitsFromRegionInfo(textureKit, self.Steps, stepsTextureKitRegionInfo)
 	self:SetAnchors(numSteps - 1, deadSpacePercentage)
 	self:SetTooltip(widgetInfo.tooltip)
-	if (progMin >= progMax) then
+	if progMin >= progMax then
 		-- Show an obvious error state.
 		self.Steps.Step1Disabled:Show()
 		self.Steps.Step1Enabled:Hide()
diff --git ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_VoidStorageUI/Blizzard_VoidStorageUI.lua ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_VoidStorageUI/Blizzard_VoidStorageUI.lua
index f7ad36b..a25dc0b 100644
--- ORI/BlizzardInterfaceCode/Interface/AddOns/Blizzard_VoidStorageUI/Blizzard_VoidStorageUI.lua
+++ ALT/BlizzardInterfaceCode/Interface/AddOns/Blizzard_VoidStorageUI/Blizzard_VoidStorageUI.lua
@@ -163,18 +163,13 @@ function VoidStorageFrame_OnEvent(self, event, ...)
 		local itemName, _, itemQuality, _, _, _, _, _, _, texture = GetItemInfo(itemLink)
 		local r, g, b = GetItemQualityColor(itemQuality or 1)
 		self.dropWarningItem = true
-		StaticPopup_Show(
-			"VOID_DEPOSIT_CONFIRM",
-			nil,
-			nil,
-			{
-				["texture"] = texture,
-				["name"] = itemName,
-				["color"] = { r, g, b, 1 },
-				["link"] = itemLink,
-				["slot"] = slot,
-			}
-		)
+		StaticPopup_Show("VOID_DEPOSIT_CONFIRM", nil, nil, {
+			["texture"] = texture,
+			["name"] = itemName,
+			["color"] = { r, g, b, 1 },
+			["link"] = itemLink,
+			["slot"] = slot,
+		})
 		VoidStorageTransferButton:Disable()
 	end
 end
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/AlternatePowerBar.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/AlternatePowerBar.lua
index 303981e..dfdf848 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/AlternatePowerBar.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/AlternatePowerBar.lua
@@ -35,14 +35,14 @@ function AlternatePowerBar_OnEvent(self, event, ...)
 	elseif event == "PLAYER_ENTERING_WORLD" then
 		AlternatePowerBar_UpdateMaxValues(self)
 		AlternatePowerBar_UpdatePowerType(self)
-	elseif (event == "UNIT_MAXPOWER") then
+	elseif event == "UNIT_MAXPOWER" then
 		local unitTag = ...
 		local parent = self:GetParent()
 		if unitTag == parent.unit then
 			AlternatePowerBar_UpdateMaxValues(self)
 		end
 	elseif self:IsShown() then
-		if (event == "UNIT_POWER_UPDATE") then
+		if event == "UNIT_POWER_UPDATE" then
 			local unitTag = ...
 			local parent = self:GetParent()
 			if unitTag == parent.unit then
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/BuffFrame.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/BuffFrame.lua
index 0100fd5..5c68640 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/BuffFrame.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/BuffFrame.lua
@@ -97,33 +97,27 @@ function BuffFrameMixin:UpdateWithSlots(unit, filter, maxCount)
 		else
 			local deadlyDebuffInfo = C_SpellBook.GetDeadlyDebuffInfo(spellID)
 			if deadlyDebuffInfo then
-				table.insert(
-					self.deadlyDebuffInfo,
-					{
-						index = index,
-						texture = texture,
-						count = count,
-						debuffType = debuffType,
-						duration = duration,
-						expirationTime = expirationTime,
-						timeMod = timeMod,
-						warningText = deadlyDebuffInfo.warningText,
-						soundKitID = deadlyDebuffInfo.soundKitID,
-					}
-				)
+				table.insert(self.deadlyDebuffInfo, {
+					index = index,
+					texture = texture,
+					count = count,
+					debuffType = debuffType,
+					duration = duration,
+					expirationTime = expirationTime,
+					timeMod = timeMod,
+					warningText = deadlyDebuffInfo.warningText,
+					soundKitID = deadlyDebuffInfo.soundKitID,
+				})
 			else
-				table.insert(
-					self.debuffInfo,
-					{
-						index = index,
-						texture = texture,
-						count = count,
-						debuffType = debuffType,
-						duration = duration,
-						expirationTime = expirationTime,
-						timeMod = timeMod,
-					}
-				)
+				table.insert(self.debuffInfo, {
+					index = index,
+					texture = texture,
+					count = count,
+					debuffType = debuffType,
+					duration = duration,
+					expirationTime = expirationTime,
+					timeMod = timeMod,
+				})
 			end
 		end
 		index = index + 1
@@ -153,7 +147,7 @@ function BuffFrameMixin:ManageDeadlyDebuffs()
 	end
 	if mostCriticalDebuffIndex then
 		DeadlyDebuffFrame:Setup(self.deadlyDebuffInfo[mostCriticalDebuffIndex])
-		table.remove(self.deadlyDebuffInfo, mostCriticalDebuffIndex) --Only want to show the debuff in one place. 
+		table.remove(self.deadlyDebuffInfo, mostCriticalDebuffIndex) --Only want to show the debuff in one place.
 	else
 		DeadlyDebuffFrame:Hide()
 	end
@@ -376,7 +370,7 @@ function AuraButtonMixin:OnUpdate()
 	end
 
 	-- Update duration
-	securecall(self.UpdateDuration, self, self.timeLeft) -- Taint issue with SecondsToTimeAbbrev 
+	securecall(self.UpdateDuration, self, self.timeLeft) -- Taint issue with SecondsToTimeAbbrev
 
 	-- Update our timeLeft
 	local timeLeft = self.buttonInfo.expirationTime - GetTime()
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/ContainerFrame.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/ContainerFrame.lua
index ffe815d..92c4283 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/ContainerFrame.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/ContainerFrame.lua
@@ -1455,19 +1455,14 @@ function ContainerFrame_GetExtendedPriceString(itemButton, isEquipped, quantity)
 		textLine2 = "\n\n" .. CONFIRM_REFUND_ITEM_ENHANCEMENTS_LOST
 	end
 	StaticPopupDialogs["CONFIRM_REFUND_TOKEN_ITEM"].hasMoneyFrame = nil
-	StaticPopup_Show(
-		"CONFIRM_REFUND_TOKEN_ITEM",
-		itemsString,
-		textLine2,
-		{
-			["texture"] = refundItemTexture,
-			["name"] = itemName,
-			["color"] = { r, g, b, 1 },
-			["link"] = refundItemLink,
-			["index"] = nil,
-			["count"] = count * quantity,
-		}
-	)
+	StaticPopup_Show("CONFIRM_REFUND_TOKEN_ITEM", itemsString, textLine2, {
+		["texture"] = refundItemTexture,
+		["name"] = itemName,
+		["color"] = { r, g, b, 1 },
+		["link"] = refundItemLink,
+		["index"] = nil,
+		["count"] = count * quantity,
+	})
 	return true
 end
 
@@ -1513,7 +1508,8 @@ function ContainerFrameItemButton_OnClick(self, button)
 		else
 			local itemLocation = ItemLocation:CreateFromBagAndSlot(self:GetBagID(), self:GetID())
 			local itemIsValidAuctionItem = (
-				itemLocation:IsValid() and C_AuctionHouse.IsSellItemValid(
+				itemLocation:IsValid()
+				and C_AuctionHouse.IsSellItemValid(
 					itemLocation, --[[ displayError = ]]
 					false
 				)
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/EditModeManager.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/EditModeManager.lua
index 3ba53ff..4d9f9dd 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/EditModeManager.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/EditModeManager.lua
@@ -1478,8 +1478,7 @@ function EditModeSystemSettingsDialogMixin:UpdateSettings(systemFrame)
 							displayInfo = displayInfo,
 							currentValue = self.attachedToSystem:GetSettingValue(displayInfo.setting),
 							settingName = settingName,
-						},
-						settingFrame:Show()
+						}, settingFrame:Show()
 				end
 			end
 		end
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/LFGList.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/LFGList.lua
index 30be6fb..be3aef7 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/LFGList.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/LFGList.lua
@@ -1382,7 +1382,7 @@ function LFGListEntryCreation_SetEditMode(self, editMode)
 			if activityID then
 				LFGListEntryCreation_Select(self, self.selectedFilters, self.selectedCategory, groupID, activityID)
 			else
-				local activityID, groupID = C_LFGList.GetOwnedKeystoneActivityAndGroupAndLevel(true) -- Check for a timewalking keystone. 
+				local activityID, groupID = C_LFGList.GetOwnedKeystoneActivityAndGroupAndLevel(true) -- Check for a timewalking keystone.
 				if activityID then
 					LFGListEntryCreation_Select(self, self.selectedFilters, self.selectedCategory, groupID, activityID)
 				end
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/LevelUpDisplay.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/LevelUpDisplay.lua
index cbfb68b..7b1c8d3 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/LevelUpDisplay.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/LevelUpDisplay.lua
@@ -534,16 +534,13 @@ function LevelUpDisplay_BuildGarrisonAbilityList(self)
 	local abilityText = GARRISON_ABILITY_HACKS[self.buildingID].Subtext
 	local name, _, texture = GetSpellInfo(spellID)
 	local spellLink = GetSpellLink(spellID)
-	tinsert(
-		self.unlockList,
-		{
-			text = name,
-			subText = abilityText,
-			icon = texture,
-			subIcon = SUBICON_TEXCOOR_BOOK,
-			link = LEVEL_UP_ABILITY2 .. " " .. spellLink,
-		}
-	)
+	tinsert(self.unlockList, {
+		text = name,
+		subText = abilityText,
+		icon = texture,
+		subIcon = SUBICON_TEXCOOR_BOOK,
+		link = LEVEL_UP_ABILITY2 .. " " .. spellLink,
+	})
 
 	self.currSpell = 1
 end
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/MailFrame.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/MailFrame.lua
index 6dc3db1..d80d2a5 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/MailFrame.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/MailFrame.lua
@@ -104,18 +104,13 @@ function MailFrame_OnEvent(self, event, ...)
 		local itemName, itemLink, itemRarity, itemLevel, itemMinLevel, itemType, itemSubType, itemStackCount, itemEquipLoc, itemTexture =
 			GetItemInfo(itemLink)
 		local r, g, b = GetItemQualityColor(itemRarity)
-		StaticPopup_Show(
-			"CONFIRM_MAIL_ITEM_UNREFUNDABLE",
-			nil,
-			nil,
-			{
-				["texture"] = itemTexture,
-				["name"] = itemName,
-				["color"] = { r, g, b, 1 },
-				["link"] = itemLink,
-				["slot"] = slotNum,
-			}
-		)
+		StaticPopup_Show("CONFIRM_MAIL_ITEM_UNREFUNDABLE", nil, nil, {
+			["texture"] = itemTexture,
+			["name"] = itemName,
+			["color"] = { r, g, b, 1 },
+			["link"] = itemLink,
+			["slot"] = slotNum,
+		})
 	elseif event == "MAIL_UNLOCK_SEND_ITEMS" then
 		SendMailFrameLockSendMail:Hide()
 		StaticPopup_Hide("CONFIRM_MAIL_ITEM_UNREFUNDABLE")
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/MoneyFrame.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/MoneyFrame.lua
index a23fc8e..8a6b9db 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/MoneyFrame.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/MoneyFrame.lua
@@ -428,7 +428,8 @@ function MoneyFrame_Update(frameName, money, forceShow)
 	silverButton:ClearAllPoints()
 	local hideCopper = true
 	if
-		(copper > 0 or showLowerDenominations or info.showSmallerCoins == "Backpack" or forceShow) and not truncateCopper
+		(copper > 0 or showLowerDenominations or info.showSmallerCoins == "Backpack" or forceShow)
+		and not truncateCopper
 	then
 		hideCopper = false
 		-- Exception if showLowerDenominations and fixedWidth
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/PvpPopup.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/PvpPopup.lua
index 2fb26c7..1a6fecd 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/PvpPopup.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/PvpPopup.lua
@@ -39,8 +39,8 @@ function PVPReadyPopupMixin:GetCenterOffsetBasedOffNumRoles(roles)
 
 	local totalWidth = self:GetWidth()
 	local widthOfRoles = ROLE_BUTTON_WIDTH * countRoles
-	local usedRoleWidth = (ROLE_BUTTON_BASE_XOFFSET * (countRoles - 1)) + widthOfRoles --The total used space of the roles buttons (Including paddng in between) 
-	local centerOffset = (totalWidth - usedRoleWidth) / 2 --Trying to get the offset for just one side. 
+	local usedRoleWidth = (ROLE_BUTTON_BASE_XOFFSET * (countRoles - 1)) + widthOfRoles --The total used space of the roles buttons (Including paddng in between)
+	local centerOffset = (totalWidth - usedRoleWidth) / 2 --Trying to get the offset for just one side.
 	return centerOffset
 end
 
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/SpellBookFrame.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/SpellBookFrame.lua
index d956bd7..99fea2b 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/SpellBookFrame.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/SpellBookFrame.lua
@@ -712,16 +712,11 @@ function SpellButtonMixin:OnClick(button)
 						{ name = GetCurrentGlyphNameForSpell(spellID), id = spellID }
 					)
 				else
-					StaticPopup_Show(
-						"CONFIRM_GLYPH_PLACEMENT",
-						nil,
-						nil,
-						{
-							name = GetPendingGlyphName(),
-							currentName = GetCurrentGlyphNameForSpell(spellID),
-							id = spellID,
-						}
-					)
+					StaticPopup_Show("CONFIRM_GLYPH_PLACEMENT", nil, nil, {
+						name = GetPendingGlyphName(),
+						currentName = GetCurrentGlyphNameForSpell(spellID),
+						id = spellID,
+					})
 				end
 			else
 				AttachGlyphToSpell(spellID)
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/SpellFlyout.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/SpellFlyout.lua
index eef6ffc..b098f03 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/SpellFlyout.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/SpellFlyout.lua
@@ -29,16 +29,11 @@ function SpellFlyoutButton_OnClick(self)
 						{ name = GetCurrentGlyphNameForSpell(self.spellID), id = self.spellID }
 					)
 				else
-					StaticPopup_Show(
-						"CONFIRM_GLYPH_PLACEMENT",
-						nil,
-						nil,
-						{
-							name = GetPendingGlyphName(),
-							currentName = GetCurrentGlyphNameForSpell(self.spellID),
-							id = self.spellID,
-						}
-					)
+					StaticPopup_Show("CONFIRM_GLYPH_PLACEMENT", nil, nil, {
+						name = GetPendingGlyphName(),
+						currentName = GetCurrentGlyphNameForSpell(self.spellID),
+						id = self.spellID,
+					})
 				end
 			else
 				AttachGlyphToSpell(self.spellID)
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/StaticPopup.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/StaticPopup.lua
index 26d4a9b..5f487bb 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/StaticPopup.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/StaticPopup.lua
@@ -4840,9 +4840,8 @@ function StaticPopup_Show(which, text_arg1, text_arg2, data, insertedFrame)
 
 		if
 			not (
-				info["button" .. index] and (
-					not info["DisplayButton" .. index] or info["DisplayButton" .. index](dialog)
-				)
+				info["button" .. index]
+				and (not info["DisplayButton" .. index] or info["DisplayButton" .. index](dialog))
 			)
 		then
 			table.remove(buttons, index)
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/TutorialFrame.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/TutorialFrame.lua
index 276eb11..7c6c7d1 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/TutorialFrame.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/TutorialFrame.lua
@@ -232,7 +232,12 @@ local DISPLAY_DATA = {
 	[41] = { --TUTORIAL_ELITE_QUESTS
 		tileHeight = 21,
 		anchorData = { align = "LEFT", xOff = 15, yOff = 30 },
-		imageData1 = { file = "Interface\\TutorialFrame\\UI-TutorialFrame-Elite", align = "TOP", xOff = 10, yOff = -50 },
+		imageData1 = {
+			file = "Interface\\TutorialFrame\\UI-TutorialFrame-Elite",
+			align = "TOP",
+			xOff = 10,
+			yOff = -50,
+		},
 		textBox = { topLeft_xOff = 33, topLeft_yOff = -160, bottomRight_xOff = -29, bottomRight_yOff = 35 },
 	},
 
@@ -864,7 +869,7 @@ function TutorialFrame_NewTutorial(tutorialID, forceShow)
 		return
 	end
 	for index, value in pairs(TUTORIALFRAME_QUEUE) do
-		if (value == tutorialID) then
+		if value == tutorialID then
 			return
 		end
 	end
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/UnitPopupButtons.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/UnitPopupButtons.lua
index bf59ab1..788d876 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/UnitPopupButtons.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/UnitPopupButtons.lua
@@ -272,7 +272,7 @@ function UnitPopupInviteButtonMixin:CanShow()
 		if not UnitPopupSharedUtil.CanCooperate(dropdownMenu) or UnitIsUnit("player", dropdownMenu.unit) then
 			return false
 		end
-	elseif (dropdownMenu == ChannelRosterDropDown) then
+	elseif dropdownMenu == ChannelRosterDropDown then
 		if UnitInRaid(dropdownMenu.name) ~= nil then
 			return false
 		end
diff --git ORI/BlizzardInterfaceCode/Interface/FrameXML/UnitPopupSharedButtonMixins.lua ALT/BlizzardInterfaceCode/Interface/FrameXML/UnitPopupSharedButtonMixins.lua
index ac1da53..24dafd8 100644
--- ORI/BlizzardInterfaceCode/Interface/FrameXML/UnitPopupSharedButtonMixins.lua
+++ ALT/BlizzardInterfaceCode/Interface/FrameXML/UnitPopupSharedButtonMixins.lua
@@ -2721,17 +2721,12 @@ end
 
 function UnitPopupCommunitiesKickFriendButtonMixin:OnClick()
 	local dropdownMenu = UnitPopupSharedUtil.GetCurrentDropdownMenu()
-	StaticPopup_Show(
-		"CONFIRM_REMOVE_COMMUNITY_MEMBER",
-		nil,
-		nil,
-		{
-			clubType = dropdownMenu.clubInfo.clubType,
-			name = dropdownMenu.clubMemberInfo.name,
-			clubId = dropdownMenu.clubInfo.clubId,
-			memberId = dropdownMenu.clubMemberInfo.memberId,
-		}
-	)
+	StaticPopup_Show("CONFIRM_REMOVE_COMMUNITY_MEMBER", nil, nil, {
+		clubType = dropdownMenu.clubInfo.clubType,
+		name = dropdownMenu.clubMemberInfo.name,
+		clubId = dropdownMenu.clubInfo.clubId,
+		memberId = dropdownMenu.clubMemberInfo.memberId,
+	})
 end
 
 UnitPopupCommunitiesMemberNoteButtonMixin = CreateFromMixins(UnitPopupButtonBaseMixin)

@JohnnyMorganz JohnnyMorganz merged commit 86fae14 into master Aug 7, 2022
@JohnnyMorganz JohnnyMorganz deleted the expand-regression-suite branch August 7, 2022 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add additional external lua codebases to regression suite
2 participants