diff options
author | 2017-05-30 12:35:33 +0200 | |
---|---|---|
committer | 2017-05-30 12:51:57 +0200 | |
commit | aea56b36af994b269800602e36000c293cabd00b (patch) | |
tree | 794f6b3b2528353cc39bd383730d408d4ff25233 /src/test/java/com/google/devtools/build/lib/runtime/TestSummaryTest.java | |
parent | 229f393bf460700594ae032a551879e026bd0b91 (diff) |
Migrate Java tests to Truth.
RELNOTES: None.
PiperOrigin-RevId: 157446717
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/runtime/TestSummaryTest.java')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/runtime/TestSummaryTest.java | 39 |
1 files changed, 18 insertions, 21 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/runtime/TestSummaryTest.java b/src/test/java/com/google/devtools/build/lib/runtime/TestSummaryTest.java index 6619d1f3d5..7ea22b3d65 100644 --- a/src/test/java/com/google/devtools/build/lib/runtime/TestSummaryTest.java +++ b/src/test/java/com/google/devtools/build/lib/runtime/TestSummaryTest.java @@ -14,10 +14,7 @@ package com.google.devtools.build.lib.runtime; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNotSame; -import static org.junit.Assert.assertSame; -import static org.junit.Assert.assertTrue; +import static com.google.common.truth.Truth.assertThat; import static org.mockito.AdditionalMatchers.find; import static org.mockito.AdditionalMatchers.not; import static org.mockito.Matchers.anyString; @@ -193,36 +190,36 @@ public class TestSummaryTest { // No need to copy if built twice in a row; no direct setters on the object. TestSummary summary = basicBuilder.build(); TestSummary sameSummary = basicBuilder.build(); - assertSame(summary, sameSummary); + assertThat(sameSummary).isSameAs(summary); basicBuilder.addTestTimes(ImmutableList.of(40L)); TestSummary summaryCopy = basicBuilder.build(); - assertEquals(summary.getTarget(), summaryCopy.getTarget()); - assertEquals(summary.getStatus(), summaryCopy.getStatus()); - assertEquals(summary.numCached(), summaryCopy.numCached()); - assertNotSame(summary, summaryCopy); - assertEquals(0, summary.totalRuns()); - assertEquals(1, summaryCopy.totalRuns()); + assertThat(summaryCopy.getTarget()).isEqualTo(summary.getTarget()); + assertThat(summaryCopy.getStatus()).isEqualTo(summary.getStatus()); + assertThat(summaryCopy.numCached()).isEqualTo(summary.numCached()); + assertThat(summaryCopy).isNotSameAs(summary); + assertThat(summary.totalRuns()).isEqualTo(0); + assertThat(summaryCopy.totalRuns()).isEqualTo(1); // Check that the builder can add a new warning to the copy, // despite the immutability of the original. basicBuilder.addTestTimes(ImmutableList.of(60L)); TestSummary fiftyCached = basicBuilder.setNumCached(50).build(); - assertEquals(summary.getStatus(), fiftyCached.getStatus()); - assertEquals(50, fiftyCached.numCached()); - assertEquals(2, fiftyCached.totalRuns()); + assertThat(fiftyCached.getStatus()).isEqualTo(summary.getStatus()); + assertThat(fiftyCached.numCached()).isEqualTo(50); + assertThat(fiftyCached.totalRuns()).isEqualTo(2); TestSummary sixtyCached = basicBuilder.setNumCached(60).build(); - assertEquals(60, sixtyCached.numCached()); - assertEquals(50, fiftyCached.numCached()); + assertThat(sixtyCached.numCached()).isEqualTo(60); + assertThat(fiftyCached.numCached()).isEqualTo(50); TestSummary failedCacheTemplate = TestSummary.newBuilderFromExisting(fiftyCached) .setStatus(BlazeTestStatus.FAILED) .build(); - assertEquals(50, failedCacheTemplate.numCached()); - assertEquals(BlazeTestStatus.FAILED, failedCacheTemplate.getStatus()); + assertThat(failedCacheTemplate.numCached()).isEqualTo(50); + assertThat(failedCacheTemplate.getStatus()).isEqualTo(BlazeTestStatus.FAILED); } @Test @@ -367,7 +364,7 @@ public class TestSummaryTest { TestSummary summaryFailed = createTestSummaryWithDetails( BlazeTestStatus.FAILED, Arrays.asList(detailPassed, detailFailed)); - assertEquals(BlazeTestStatus.FAILED, summaryFailed.getStatus()); + assertThat(summaryFailed.getStatus()).isEqualTo(BlazeTestStatus.FAILED); AnsiTerminalPrinter printerPassed = Mockito.mock(AnsiTerminalPrinter.class); TestSummaryPrinter.print(summaryPassed, printerPassed, true, true); @@ -428,8 +425,8 @@ public class TestSummaryTest { // This way we can make the test independent from the sort order of FAILEd // and PASSED. - assertTrue(summaryFailedCached.compareTo(summaryPassedNotCached) < 0); - assertTrue(summaryPassedCached.compareTo(summaryFailedNotCached) < 0); + assertThat(summaryFailedCached.compareTo(summaryPassedNotCached)).isLessThan(0); + assertThat(summaryPassedCached.compareTo(summaryFailedNotCached)).isLessThan(0); } @Test |