summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTomasz Zarna2013-07-19 10:55:06 (EDT)
committer Gerrit Code Review @ Eclipse.org2013-07-24 08:06:08 (EDT)
commit599b14be570b41b0ecb412b1b5efea76fce4294d (patch)
tree2655f7d4c0657ca3bb97da78d560ddb5823a7123
parent054f627e90dee5b2d5605258ec73c60b3bcc2018 (diff)
downloadorg.eclipse.mylyn.reviews-599b14be570b41b0ecb412b1b5efea76fce4294d.zip
org.eclipse.mylyn.reviews-599b14be570b41b0ecb412b1b5efea76fce4294d.tar.gz
org.eclipse.mylyn.reviews-599b14be570b41b0ecb412b1b5efea76fce4294d.tar.bz2
remove redundant assertions in GerritReviewRemoteFactoryTest#testUsers()refs/changes/05/14705/2
Change-Id: Ib1ac60cb27913cb0be6fee8f3489d651df765581 Signed-off-by: Tomasz Zarna <tomasz.zarna@tasktop.com>
-rw-r--r--org.eclipse.mylyn.gerrit.tests/src/org/eclipse/mylyn/internal/gerrit/core/remote/GerritReviewRemoteFactoryTest.java9
1 files changed, 1 insertions, 8 deletions
diff --git a/org.eclipse.mylyn.gerrit.tests/src/org/eclipse/mylyn/internal/gerrit/core/remote/GerritReviewRemoteFactoryTest.java b/org.eclipse.mylyn.gerrit.tests/src/org/eclipse/mylyn/internal/gerrit/core/remote/GerritReviewRemoteFactoryTest.java
index c3effa6..b23bbac 100644
--- a/org.eclipse.mylyn.gerrit.tests/src/org/eclipse/mylyn/internal/gerrit/core/remote/GerritReviewRemoteFactoryTest.java
+++ b/org.eclipse.mylyn.gerrit.tests/src/org/eclipse/mylyn/internal/gerrit/core/remote/GerritReviewRemoteFactoryTest.java
@@ -107,7 +107,6 @@ public class GerritReviewRemoteFactoryTest extends GerritRemoteTest {
@Test
public void testAccount() throws Exception {
- //Account
assertThat(reviewHarness.getRepository().getAccount(), notNullValue());
assertThat(reviewHarness.getRepository().getAccount().getDisplayName(), is("tests"));
assertThat(reviewHarness.getRepository().getAccount().getEmail(), is("tests@mylyn.eclipse.org"));
@@ -116,19 +115,13 @@ public class GerritReviewRemoteFactoryTest extends GerritRemoteTest {
@Test
public void testUsers() throws Exception {
- //Account
- assertThat(reviewHarness.getRepository().getAccount(), notNullValue());
- assertThat(reviewHarness.getRepository().getAccount().getDisplayName(), is("tests"));
- assertThat(reviewHarness.getRepository().getAccount().getEmail(), is("tests@mylyn.eclipse.org"));
-
- //Users
assertThat(reviewHarness.getRepository().getUsers().size(), is(1));
assertThat(reviewHarness.getRepository().getUsers().get(0).getDisplayName(), is("tests"));
+ assertThat(reviewHarness.getRepository().getUsers().get(0).getEmail(), is("tests@mylyn.eclipse.org"));
}
@Test
public void testApprovals() throws Exception {
- //Approvals
assertThat(reviewHarness.getRepository().getApprovalTypes().size(), is(2));
IApprovalType verifyApproval = reviewHarness.getRepository().getApprovalTypes().get(0);
assertThat(verifyApproval.getKey(), is("VRIF"));