Details
Description
Right now we have this logic:
public MiniHBaseCluster getMiniHBaseCluster() { if (this.hbaseCluster instanceof MiniHBaseCluster) { return (MiniHBaseCluster)this.hbaseCluster; } throw new RuntimeException(hbaseCluster + " not an instance of " + MiniHBaseCluster.class.getName()); }
Since null is not actually an instance of MiniHBaseCluster we'll throw the RuntimeException rather than returning null.
Some tests call this method and check for null, which is pointless.