reuse cluster for TFsShellTest and made each test isolated

Cut TFsShellTest from 6 minutes to 30 seconds.  Same trick can be used for all long running tests.
Wasn't the point of moving to Google style to get rid of tachyon bad
habits? This should have been removed then...
On Dec 21, 2014 11:36 AM, "Haoyuan Li" <notifications@github.com> wrote:

> In core/src/test/java/tachyon/client/FileInStreamTest.java
> <https://github.com/amplab/tachyon/pull/570#discussion-diff-22149662>:
>
> >      System.setProperty("tachyon.user.quota.unit.bytes", "1000");
> >      System.setProperty("tachyon.user.default.block.size.byte", String.valueOf(BLOCK_SIZE));
> > -    mLocalTachyonCluster = new LocalTachyonCluster(10000);
> > -    mLocalTachyonCluster.start();
> > -    mTfs = mLocalTachyonCluster.getClient();
> > +    CLUSTER = new LocalTachyonCluster(10000);
> > +    CLUSTER.start();
> > +    TFS = CLUSTER.getClient();
> > +  }
> > +
>
> That rule was there before the Google Style. And we just carried on when
> we started to use Google Style. And there is no discussion around it as far
> as I remember.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/amplab/tachyon/pull/570/files#r22149662>.
>