Added test re-runs. Made classes final to aid inlining

Author: dgant

pom.xml

@@ -63,6 +63,7 @@
                 <configuration>
 <!--                    <parallel>all</parallel> &lt;!&ndash; Run tests in parallel&ndash;&gt;-->
 <!--                    <useUnlimitedThreads>true</useUnlimitedThreads>-->
+                    <rerunFailingTestsCount>10</rerunFailingTestsCount>
                     <argLine>-Xms1g</argLine>
                     <argLine>-Xmx1g</argLine>
                 </configuration>

src/main/java/bwapi/Bullet.java

@@ -27,7 +27,7 @@
  * @see Game#getBullets
  * @see Bullet#exists
  */
-public class Bullet implements Comparable<Bullet> {
+public final class Bullet implements Comparable<Bullet> {
     private final BulletData bulletData;
     private final int id;
     private final Game game;

src/main/java/bwapi/Color.java

@@ -11,7 +11,7 @@
  * Starcraft uses a 256 color palette for rendering. Thus, the colors available are
  * limited to this palette.
  */
-public class Color {
+public final class Color {
     /**
      * The default color for Player 1.
      */

src/main/java/bwapi/Force.java

@@ -13,7 +13,7 @@
  * It is not called a team because players on the same force do not necessarily need
  * to be allied at the beginning of a match.
  */
-public class Force implements Comparable<Force> {
+public final class Force implements Comparable<Force> {
     private final Game game;
 
     private final int id;

src/main/java/bwapi/Player.java

@@ -18,7 +18,7 @@
  * @see PlayerType
  * @see Race
  */
-public class Player implements Comparable<Player> {
+public final class Player implements Comparable<Player> {
     private final PlayerData playerData;
     private final Game game;
     private final int id;

src/main/java/bwapi/Position.java

@@ -1,7 +1,7 @@
 package bwapi;
 
 
-public class Position extends Point<Position> {
+public final class Position extends Point<Position> {
 
     public static final int SIZE_IN_PIXELS = 1;
     public static final Position Invalid = new Position(32000 / SIZE_IN_PIXELS, 32000 / SIZE_IN_PIXELS);

src/main/java/bwapi/Region.java

@@ -21,7 +21,7 @@
  * @see Game#getRegionAt
  * @see Unit#getRegion
  */
-public class Region implements Comparable<Region> {
+public final class Region implements Comparable<Region> {
     private final RegionData regionData;
     private final Game game;
 

src/main/java/bwapi/TilePosition.java

@@ -1,6 +1,6 @@
 package bwapi;
 
-public class TilePosition extends Point<TilePosition> {
+public final class TilePosition extends Point<TilePosition> {
 
     public static final int SIZE_IN_PIXELS = 32;
     public static final TilePosition Invalid = new TilePosition(32000 / SIZE_IN_PIXELS, 32000 / SIZE_IN_PIXELS);

src/main/java/bwapi/Unit.java

@@ -39,7 +39,7 @@
  * However for units that were owned by the player, {@link #getPlayer} and {@link #getType} will continue to work for units
  * that have been destroyed.
  */
-public class Unit implements Comparable<Unit> {
+public final class Unit implements Comparable<Unit> {
     private static final Set<Order> gatheringGasOrders = EnumSet.of(
             Harvest1, Harvest2, MoveToGas, WaitForGas, HarvestGas, ReturnGas, ResetCollision);
     private static final Set<Order> gatheringMineralOrders = EnumSet.of(

src/main/java/bwapi/UnitCommand.java

@@ -6,7 +6,7 @@
 import static bwapi.TechType.*;
 import static bwapi.UnitCommandType.*;
 
-public class UnitCommand {
+public final class UnitCommand {
     Unit unit;
     UnitCommandType type;
     Unit target = null;