diff options
| author | Tomasz Zarna | 2013-07-19 10:55:06 (EDT) |
|---|---|---|
| committer | Gerrit Code Review @ Eclipse.org | 2013-07-24 08:06:08 (EDT) |
| commit | 599b14be570b41b0ecb412b1b5efea76fce4294d (patch) | |
| tree | 2655f7d4c0657ca3bb97da78d560ddb5823a7123 | |
| parent | 054f627e90dee5b2d5605258ec73c60b3bcc2018 (diff) | |
| download | org.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.java | 9 |
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")); |