Merge branch 'merc-uniques' of https://github.com/Paliak/PathOfBuilding into merc-uniques

Author: Paliak

RELEASE.md

@@ -37,10 +37,12 @@ The JSON data and required skill tree assets should come in a `.zip` archive.
 Steps:
 1. Download the `.zip` archive.
 2. Create a new directory in `./src/TreeData` with the following schema:
-    `<major_league_version>_<minor_league_version>`.
+    `<major_league_version>_<minor_league_version>`. For alternate or ruthless trees, add the suffixing as appropriate.
     For 3.14, the correct directory name would be `3_14`.
+    For 3.25 Ruthless 'alternate' tree, the correct directory name would be `3_25_ruthless_alternate`.
 3. Copy the following file from the `.zip` archive root to the new directory:
    * `data.json`.
+   Note for Ruthless for example, the exported data from GGG will be `ruthless.json`, and this file should be copied into the new directory and renamed to `data.json` for the following steps to pick it up.
 4. Copy the following files from the `assets` subdirectory in the `.zip` archive to the
     new directory:
     * `mastery-active-effect-3.png`
@@ -54,7 +56,7 @@ Steps:
    according to the file's format. This is important, otherwise the JSON data converter
    won't trigger.
 7. Restart Path of Building Community. This should result in a new file `tree.lua`.
-8. Remove `data.json` and `sprites.json` from the new directory. Do not commit these files.
+8. Remove `data.json` and `sprites.json` from the new directories. Do not commit these files.
 
 ## Timeless Jewel updates
 

spec/System/TestDefence_spec.lua

@@ -457,7 +457,40 @@ describe("TestDefence", function()
 		assert.are.equals(1000, takenHitFromTypeMaxHit("Lightning"))
 		assert.are.equals(1250, build.calcsTab.calcsOutput.ChaosMaximumHitTaken)
 	end)
-	
+
+	it("energy shield increased by spell block chance if corresponding flag is present", function()
+		build.configTab.input.enemyIsBoss = "None"
+		build.configTab.input.customMods = "\z
+		You have no intelligence\n\z
+		100 Energy shield \n\z
+		energy shield is increased by chance to block spell damage\n\z
+		50% chance to block spell damage\n\z
+		"
+		build.configTab:BuildModList()
+		runCallback("OnFrame")
+
+		assert.are.equals(0, build.calcsTab.calcsOutput.SpellBlockChanceOverCap )
+		assert.are.equals(50, build.calcsTab.calcsOutput.SpellBlockChance)
+		assert.are.equals(150, build.calcsTab.calcsOutput.EnergyShield)
+	end)
+
+	it("energy shield increased by spell block chance if corresponding flag is present should be capped to max block", function()
+		build.configTab.input.enemyIsBoss = "None"
+
+		build.configTab.input.customMods = "\z
+		You have no intelligence\n\z
+		100 Energy shield \n\z
+		energy shield is increased by chance to block spell damage\n\z
+		100% chance to block spell damage\n\z
+		"
+		build.configTab:BuildModList()
+		runCallback("OnFrame")
+
+		assert.are.equals(25, build.calcsTab.calcsOutput.SpellBlockChanceOverCap )
+		assert.are.equals(75, build.calcsTab.calcsOutput.SpellBlockChance)
+		assert.are.equals(175, build.calcsTab.calcsOutput.EnergyShield)
+	end)
+
 	local function withinTenPercent(value, otherValue)
 		local ratio = otherValue / value
 		return 0.9 < ratio and ratio < 1.1
@@ -807,7 +840,7 @@ describe("TestDefence", function()
 		assert.are.equals(0, floor(poolsRemaining.Life))
 		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
 	end)
-
+	
 	it("Unreakable + Iron Reflexes", function()
 		build.configTab.input.customMods = [[
 			you have no dexterity
@@ -902,4 +935,188 @@ describe("TestDefence", function()
 		-- Oath Of Maji should apply only once
 		assert.are.equals(2*2*ironReflexesArmour + baseArmour + baseEvasion, build.calcsTab.mainOutput.Armour)
 	end)
+	
+	it("MoM + EB", function()
+		build.configTab.input.enemyIsBoss = "None"
+		-- enough mana and es, 0% and 100% bypass
+		build.configTab.input.customMods = [[
+			50% of damage is taken from mana before life
+			energy shield protects mana instead of life
+			+40 to maximum life
+			+1960 to mana
+			+2000 to energy shield
+			You have no intelligence
+			+60% to all resistances
+		]]
+		build.configTab:BuildModList()
+		runCallback("OnFrame")
+		assert.are.equals(200, build.calcsTab.calcsOutput.FireMaximumHitTaken)
+		assert.are.equals(200, build.calcsTab.calcsOutput.ChaosMaximumHitTaken)
+		local poolsRemaining = poolsRemainingAfterTypeMaxHit("Chaos")
+		assert.are.equals(2000, round(poolsRemaining.EnergyShield))
+		assert.are.equals(1900, round(poolsRemaining.Mana))
+		assert.are.equals(0, floor(poolsRemaining.Life))
+		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
+		poolsRemaining = poolsRemainingAfterTypeMaxHit("Fire")
+		assert.are.equals(1900, round(poolsRemaining.EnergyShield))
+		assert.are.equals(2000, round(poolsRemaining.Mana))
+		assert.are.equals(0, floor(poolsRemaining.Life))
+		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
+		
+		-- enough mana and es, 50% bypass
+		build.configTab.input.customMods = [[
+			50% of damage is taken from mana before life
+			energy shield protects mana instead of life
+			50% of non-chaos damage taken bypasses energy shield
+			+40 to maximum life
+			+1960 to mana
+			+2000 to energy shield
+			You have no intelligence
+			+60% to all resistances
+		]]
+		build.configTab:BuildModList()
+		runCallback("OnFrame")
+		assert.are.equals(200, build.calcsTab.calcsOutput.FireMaximumHitTaken)
+		poolsRemaining = poolsRemainingAfterTypeMaxHit("Fire")
+		assert.are.equals(1950, round(poolsRemaining.EnergyShield))
+		assert.are.equals(1950, round(poolsRemaining.Mana))
+		assert.are.equals(0, floor(poolsRemaining.Life))
+		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
+
+		-- es bottleneck, 0% and 100% bypass
+		build.configTab.input.customMods = [[
+			50% of damage is taken from mana before life
+			energy shield protects mana instead of life
+			+40 to maximum life
+			+1960 to mana
+			+50 to energy shield
+			You have no intelligence
+			+60% to all resistances
+		]]
+		build.configTab:BuildModList()
+		runCallback("OnFrame")
+		assert.are.equals(200, build.calcsTab.calcsOutput.FireMaximumHitTaken)
+		assert.are.equals(200, build.calcsTab.calcsOutput.ChaosMaximumHitTaken)
+		poolsRemaining = poolsRemainingAfterTypeMaxHit("Chaos")
+		assert.are.equals(50, round(poolsRemaining.EnergyShield))
+		assert.are.equals(1900, round(poolsRemaining.Mana))
+		assert.are.equals(0, floor(poolsRemaining.Life))
+		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
+		poolsRemaining = poolsRemainingAfterTypeMaxHit("Fire")
+		assert.are.equals(0, round(poolsRemaining.EnergyShield))
+		assert.are.equals(1950, round(poolsRemaining.Mana))
+		assert.are.equals(0, floor(poolsRemaining.Life))
+		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
+
+		-- es bottleneck, 50% bypass
+		build.configTab.input.customMods = [[
+			50% of damage is taken from mana before life
+			energy shield protects mana instead of life
+			50% of non-chaos damage taken bypasses energy shield
+			+40 to maximum life
+			+1960 to mana
+			+40 to energy shield
+			You have no intelligence
+			+60% to all resistances
+		]]
+		build.configTab:BuildModList()
+		runCallback("OnFrame")
+		assert.are.equals(200, build.calcsTab.calcsOutput.FireMaximumHitTaken)
+		poolsRemaining = poolsRemainingAfterTypeMaxHit("Fire")
+		assert.are.equals(0, round(poolsRemaining.EnergyShield))
+		assert.are.equals(1940, round(poolsRemaining.Mana))
+		assert.are.equals(0, floor(poolsRemaining.Life))
+		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
+		
+		-- mana bottleneck, 0% and 100% bypass
+		build.configTab.input.customMods = [[
+			50% of damage is taken from mana before life
+			energy shield protects mana instead of life
+			+40 to maximum life
+			+2000 to energy shield
+			You have no intelligence
+			+60% to all resistances
+		]]
+		build.configTab:BuildModList()
+		runCallback("OnFrame")
+		assert.are.equals(200, build.calcsTab.calcsOutput.FireMaximumHitTaken)
+		assert.are.equals(140, build.calcsTab.calcsOutput.ChaosMaximumHitTaken)
+		poolsRemaining = poolsRemainingAfterTypeMaxHit("Chaos")
+		assert.are.equals(2000, round(poolsRemaining.EnergyShield))
+		assert.are.equals(0, round(poolsRemaining.Mana))
+		assert.are.equals(0, floor(poolsRemaining.Life))
+		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
+		poolsRemaining = poolsRemainingAfterTypeMaxHit("Fire")
+		assert.are.equals(1900, round(poolsRemaining.EnergyShield))
+		assert.are.equals(40, round(poolsRemaining.Mana))
+		assert.are.equals(0, floor(poolsRemaining.Life))
+		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
+
+		-- mana bottleneck, 50% bypass
+		-- taking 160 damage in this scenario:
+		-- 160 damage is split to 80 damage straight to life, 80 damage to MoM pools
+		-- 50% of the 80 MoM pool damage is taken by ES and 50% bypasses
+		-- pool of 20 mana takes 40 damage, gets depleted and the remaining damage continues on to life, for a total of 100 damage to life
+		build.configTab.input.customMods = [[
+			50% of damage is taken from mana before life
+			energy shield protects mana instead of life
+			50% of non-chaos damage taken bypasses energy shield
+			+40 to maximum life
+			-20 to mana
+			+2000 to energy shield
+			You have no intelligence
+			+60% to all resistances
+		]]
+		build.configTab:BuildModList()
+		runCallback("OnFrame")
+		assert.are.equals(160, build.calcsTab.calcsOutput.FireMaximumHitTaken)
+		poolsRemaining = poolsRemainingAfterTypeMaxHit("Fire")
+		assert.are.equals(1960, round(poolsRemaining.EnergyShield))
+		assert.are.equals(0, round(poolsRemaining.Mana))
+		assert.are.equals(0, floor(poolsRemaining.Life))
+		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
+
+		-- mana+es bottleneck, 0% and 100% bypass
+		build.configTab.input.customMods = [[
+			50% of damage is taken from mana before life
+			energy shield protects mana instead of life
+			+940 to maximum life
+			+50 to energy shield
+			You have no intelligence
+			+60% to all resistances
+		]]
+		build.configTab:BuildModList()
+		runCallback("OnFrame")
+		assert.are.equals(1090, build.calcsTab.calcsOutput.FireMaximumHitTaken)
+		assert.are.equals(1040, build.calcsTab.calcsOutput.ChaosMaximumHitTaken)
+		poolsRemaining = poolsRemainingAfterTypeMaxHit("Chaos")
+		assert.are.equals(50, round(poolsRemaining.EnergyShield))
+		assert.are.equals(0, round(poolsRemaining.Mana))
+		assert.are.equals(0, floor(poolsRemaining.Life))
+		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
+		poolsRemaining = poolsRemainingAfterTypeMaxHit("Fire")
+		assert.are.equals(0, round(poolsRemaining.EnergyShield))
+		assert.are.equals(0, round(poolsRemaining.Mana))
+		assert.are.equals(0, floor(poolsRemaining.Life))
+		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
+
+		-- mana+es bottleneck, 50% bypass
+		build.configTab.input.customMods = [[
+			50% of damage is taken from mana before life
+			energy shield protects mana instead of life
+			50% of non-chaos damage taken bypasses energy shield
+			+940 to maximum life
+			+50 to energy shield
+			You have no intelligence
+			+60% to all resistances
+		]]
+		build.configTab:BuildModList()
+		runCallback("OnFrame")
+		assert.are.equals(1090, build.calcsTab.calcsOutput.FireMaximumHitTaken)
+		poolsRemaining = poolsRemainingAfterTypeMaxHit("Fire")
+		assert.are.equals(0, round(poolsRemaining.EnergyShield))
+		assert.are.equals(0, round(poolsRemaining.Mana))
+		assert.are.equals(0, floor(poolsRemaining.Life))
+		assert.are.equals(0, floor(poolsRemaining.OverkillDamage))
+	end)
 end)