opendev/gerrit
Code Issues Proposed changes

ProjectIT: Fix assertions

Browse Source 
It should be:

  assertThat(actualValue).isEqualsTo(expectedValue)

Not the other way around.

Change-Id: I0e7db7543f88eb57ba33051bdd608189c6cadd49
Signed-off-by: Edwin Kempin <ekempin@google.com>
This commit is contained in:
Edwin Kempin
2017-10-13 14:20:32 +02:00
parent 5815a78397
commit 1616fdac53
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/project/ProjectIT.java
View File

@@ -41,7 +41,7 @@ public class ProjectIT extends AbstractDaemonTest {
@Test
public void createProject() throws Exception {
String name = name("foo");
assertThat(name).isEqualTo(gApi.projects().create(name).get().name);
assertThat(gApi.projects().create(name).get().name).isEqualTo(name);
RevCommit head = getRemoteHead(name, RefNames.REFS_CONFIG);
eventRecorder.assertRefUpdatedEvents(name, RefNames.REFS_CONFIG, null, head);
@@ -52,7 +52,7 @@ public class ProjectIT extends AbstractDaemonTest {
@Test
public void createProjectWithGitSuffix() throws Exception {
String name = name("foo");
assertThat(name).isEqualTo(gApi.projects().create(name + ".git").get().name);
assertThat(gApi.projects().create(name + ".git").get().name).isEqualTo(name);
RevCommit head = getRemoteHead(name, RefNames.REFS_CONFIG);
eventRecorder.assertRefUpdatedEvents(name, RefNames.REFS_CONFIG, null, head);
@@ -66,7 +66,7 @@ public class ProjectIT extends AbstractDaemonTest {
ProjectInput input = new ProjectInput();
input.name = name;
input.createEmptyCommit = true;
assertThat(name).isEqualTo(gApi.projects().create(input).get().name);
assertThat(gApi.projects().create(input).get().name).isEqualTo(name);
RevCommit head = getRemoteHead(name, RefNames.REFS_CONFIG);
eventRecorder.assertRefUpdatedEvents(name, RefNames.REFS_CONFIG, null, head);