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/JvmPauseMonitorTest.java #547

Closed newnius closed 4 years ago

newnius commented 5 years ago

Change

import org.junit.Assert;

to

import static org.junit.Assert.assertNotEquals;

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