opendev/gerrit
Code Issues Proposed changes

Query tests: Use query string in name for assertions

Browse Source 
This was accidentally changed in I21c43496 to repeat the actual list
rather than the query that generated it.

Change-Id: I2e37451e5f4645bd00ab14f78f9a8748764ce2f4
This commit is contained in:
Dave Borowitz
2015-05-29 12:11:45 -07:00
parent ed49f92c3b
commit c462706d93
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/test/java/com/google/gerrit/server/query/change/AbstractQueryChangesTest.java
View File

@@ -1219,7 +1219,7 @@ public abstract class AbstractQueryChangesTest {
throws Exception { throws Exception {
List<ChangeInfo> result = query.get(); List<ChangeInfo> result = query.get();
Iterable<Integer> ids = ids(result); Iterable<Integer> ids = ids(result);
assertThat(ids).named(ids.toString()) assertThat(ids).named(query.getQuery())
.containsExactlyElementsIn(ids(changes)).inOrder(); .containsExactlyElementsIn(ids(changes)).inOrder();
return result; return result;
} }