opendev/gerrit
Code Issues Proposed changes

AbstractSubmit: Add extra isNotNull assertion

Browse Source 
Change-Id: I04fd237736f1f060b362e9c0c21970af2dd3f98e
This commit is contained in:
Dave Borowitz 2016-01-13 16:17:38 -05:00
parent a3e3cafffd
commit fe2a4a34b7
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/AbstractSubmit.java
View File

@ -289,6 +289,7 @@ public abstract class AbstractSubmit extends AbstractDaemonTest {
getOnlyElement(queryProvider.get().byKeyPrefix(changeId)).change()); getOnlyElement(queryProvider.get().byKeyPrefix(changeId)).change());
PatchSetApproval submitter = approvalsUtil.getSubmitter( PatchSetApproval submitter = approvalsUtil.getSubmitter(
db, cn, new PatchSet.Id(cn.getChangeId(), psId)); db, cn, new PatchSet.Id(cn.getChangeId(), psId));
assertThat(submitter).isNotNull();
assertThat(submitter.isSubmit()).isTrue(); assertThat(submitter.isSubmit()).isTrue();
assertThat(submitter.getAccountId()).isEqualTo(admin.getId()); assertThat(submitter.getAccountId()).isEqualTo(admin.getId());
} }