Started by an SCM change341c6292f4b0c1fef620ce55a60082a3d69080fd459c6292f4b0c1fef620ce55a60082a3d69080fd459refs/remotes/origin/masterc6292f4b0c1fef620ce55a60082a3d69080fd459c6292f4b0c1fef620ce55a60082a3d69080fd459refs/remotes/origin/masterc6292f4b0c1fef620ce55a60082a3d69080fd459c6292f4b0c1fef620ce55a60082a3d69080fd459refs/remotes/origin/masterhttps://github.com/mcMMO-Dev/mcMMO.gitmcMMO.jarmcMMO.jartarget/mcMMO.jaroriginal-mcMMO.jaroriginal-mcMMO.jartarget/original-mcMMO.jarfalse#341160667135282mcMMO #341341falsefalse341214SUCCESS1783730948677https://popicraft.net/jenkins/job/mcMMO/341/Buildersrc/test/java/com/gmail/nossr50/skills/fishing/FishingManagerMasterAnglerTest.javasrc/test/java/com/gmail/nossr50/skills/fishing/FishingManagerAbilityGatesTest.javasrc/test/java/com/gmail/nossr50/skills/fishing/FishingManagerShakeTest.javasrc/test/java/com/gmail/nossr50/skills/fishing/FishingManagerProcessFishingTest.javasrc/test/java/com/gmail/nossr50/skills/fishing/FishingTest.javasrc/test/java/com/gmail/nossr50/skills/fishing/FishingDropsTest.javasrc/main/java/com/gmail/nossr50/skills/fishing/FishingManager.javasrc/test/java/com/gmail/nossr50/listeners/PlayerListenerFishingTest.javaChangelog.txtc6292f4b0c1fef620ce55a60082a3d69080fd4591783730108000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix McMMOPlayerShakeEvent never being fired and expand fishing test coverage The Shake ability constructed McMMOPlayerShakeEvent but never dispatched it through the plugin manager, a gap dating back to 2013. Plugins listening for the event never received it, and its cancel/setDrop API had no effect. The event now fires before the drop is spawned, so cancelling it suppresses the drop, damage, and XP, and drops replaced through the event are the ones that spawn. Also adds unit coverage across the fishing code: the overfishing detection state machine and its bounding-box boundary, Master Angler wait-time math and hook processing, the Shake drop pipeline including sheep shearing and inventory stealing, the processFishing reward pipeline, subskill eligibility gates, cumulative-chance drop selection, and the listener wiring for treasure override, vanilla XP boost, Shake, and Ice Fishing. 2026-07-10 17:35:08 -0700c6292f4b0c1fef620ce55a60082a3d69080fd459Fix McMMOPlayerShakeEvent never being fired and expand fishing test coverageaddsrc/test/java/com/gmail/nossr50/listeners/PlayerListenerFishingTest.javaaddsrc/test/java/com/gmail/nossr50/skills/fishing/FishingDropsTest.javaeditChangelog.txtaddsrc/test/java/com/gmail/nossr50/skills/fishing/FishingManagerMasterAnglerTest.javaaddsrc/test/java/com/gmail/nossr50/skills/fishing/FishingManagerProcessFishingTest.javaaddsrc/test/java/com/gmail/nossr50/skills/fishing/FishingManagerShakeTest.javaeditsrc/main/java/com/gmail/nossr50/skills/fishing/FishingManager.javaaddsrc/test/java/com/gmail/nossr50/skills/fishing/FishingManagerAbilityGatesTest.javaeditsrc/test/java/com/gmail/nossr50/skills/fishing/FishingTest.javagithttps://popicraft.net/jenkins/user/nossr50nossr50