opendev/gerrit
Code Issues Proposed changes

Merge "Migrate off the overload of StandardSubjectBuilder.fail(...) that accepts a message."

Browse Source
This commit is contained in:
Alice Kober-Sotzek
2019-06-21 08:01:18 +00:00
committed by Gerrit Code Review
parent 38e971f6dc cf284044a9
commit 5eb922ee2b
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
javatests/com/google/gerrit/acceptance/rest/change/AbstractSubmit.java
View File

@@ -17,7 +17,6 @@ package com.google.gerrit.acceptance.rest.change;
import static com.google.common.collect.Iterables.getOnlyElement; import static com.google.common.collect.Iterables.getOnlyElement;
import static com.google.common.truth.Truth.assertThat; import static com.google.common.truth.Truth.assertThat;
import static com.google.common.truth.Truth.assertWithMessage; import static com.google.common.truth.Truth.assertWithMessage;
import static com.google.common.truth.Truth.assert_;
import static com.google.common.truth.Truth8.assertThat; import static com.google.common.truth.Truth8.assertThat;
import static com.google.common.truth.TruthJUnit.assume; import static com.google.common.truth.TruthJUnit.assume;
import static com.google.gerrit.acceptance.testsuite.project.TestProjectUpdate.allow; import static com.google.gerrit.acceptance.testsuite.project.TestProjectUpdate.allow;
@@ -266,7 +265,7 @@ public abstract class AbstractSubmit extends AbstractDaemonTest {
break; break;
case CHERRY_PICK: case CHERRY_PICK:
default: default:
assert_().fail("Should not reach here."); assertWithMessage("Should not reach here.").fail();
break; break;
} }