Alluxio / Community

New Contributor Tasks for Alluxio
20 stars 38 forks source link

[SMALLFIX] Use static imports for standard test utilities in /alluxio/core/common/src/test/java/alluxio/util/ShellUtilsTest.java #549

Closed newnius closed 5 years ago

newnius commented 5 years ago

Remove

import org.junit.Assert;

AND

Update all

Assert.{{ method }}*

to

{{ method }}*

For example

update

    Assert.assertTrue(mGraph.getRoots().isEmpty());

to

    assertTrue(mGraph.getRoots().isEmpty());
newnius commented 5 years ago

Put the static imports before non-static imports.

yangjf2019 commented 5 years ago

Hi, @newnius , Thank you very much . /assign @yangjf2019

yangjf2019 commented 5 years ago

Here is PR.