From 76f287582e2f7bf38c74f4a0e9382870475417b5 Mon Sep 17 00:00:00 2001 From: Leijurv Date: Mon, 24 Sep 2018 15:58:02 -0700 Subject: [PATCH] overhaul betterblockpostest --- .../utils/pathing/BetterBlockPosTest.java | 29 +++++++++++++++++-- 1 file changed, 27 insertions(+), 2 deletions(-) diff --git a/src/test/java/baritone/utils/pathing/BetterBlockPosTest.java b/src/test/java/baritone/utils/pathing/BetterBlockPosTest.java index 2f0ab6889..459593b0a 100644 --- a/src/test/java/baritone/utils/pathing/BetterBlockPosTest.java +++ b/src/test/java/baritone/utils/pathing/BetterBlockPosTest.java @@ -17,14 +17,16 @@ package baritone.utils.pathing; +import net.minecraft.util.EnumFacing; import net.minecraft.util.math.BlockPos; import org.junit.Test; -import static org.junit.Assert.assertTrue; +import static org.junit.Assert.assertEquals; public class BetterBlockPosTest { - @Test + // disabled since this is a benchmark, not really a test. also including it makes the tests take 50 seconds for no reason + /*@Test public void benchMulti() { System.out.println("Benching up()"); for (int i = 0; i < 10; i++) { @@ -37,7 +39,30 @@ public class BetterBlockPosTest { benchN(); assertTrue(i<10); } + }*/ + /** + * Make sure BetterBlockPos behaves just like BlockPos + */ + @Test + public void testSimple() { + BlockPos pos = new BlockPos(1, 2, 3); + BetterBlockPos better = new BetterBlockPos(1, 2, 3); + assertEquals(pos, better); + assertEquals(pos.up(), better.up()); + assertEquals(pos.down(), better.down()); + assertEquals(pos.north(), better.north()); + assertEquals(pos.south(), better.south()); + assertEquals(pos.east(), better.east()); + assertEquals(pos.west(), better.west()); + for (EnumFacing dir : EnumFacing.values()) { + assertEquals(pos.offset(dir), better.offset(dir)); + assertEquals(pos.offset(dir, 0), pos); + assertEquals(better.offset(dir, 0), better); + for (int i = -10; i < 10; i++) { + assertEquals(pos.offset(dir, i), better.offset(dir, i)); + } + } } public void benchOne() {