Try to avoid ignoring users fill argument

1 files changed, 11 insertions, 7 deletions

diff --git a/container-search/src/test/java/com/yahoo/search/grouping/vespa/GroupingExecutorTestCase.java b/container-search/src/test/java/com/yahoo/search/grouping/vespa/GroupingExecutorTestCase.java
index 386e8346cae..8d524ae011e 100644
--- a/container-search/src/test/java/com/yahoo/search/grouping/vespa/GroupingExecutorTestCase.java
+++ b/container-search/src/test/java/com/yahoo/search/grouping/vespa/GroupingExecutorTestCase.java
@@ -53,16 +53,10 @@ import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
 
 /**
- * @author <a href="mailto:simon@yahoo-inc.com">Simon Thoresen</a>
+ * @author Simon Thoresen
  */
 public class GroupingExecutorTestCase {
 
-    // --------------------------------------------------------------------------------
-    //
-    // Tests
-    //
-    // --------------------------------------------------------------------------------
-
     @Test
     public void requireThatNullRequestsPass() {
         Result res = newExecution(new GroupingExecutor()).search(newQuery());
@@ -262,12 +256,22 @@ public class GroupingExecutorTestCase {
                                               new GroupingListHit(Arrays.asList(grp1), null))),
                                       new FillRequestThrower());
         Result res = exec.search(query);
+        
+        // Fill with summary specified in grouping
         try {
             exec.fill(res);
             fail();
         } catch (FillRequestException e) {
             assertEquals("bar", e.summaryClass);
         }
+
+        // Fill again, with another summary
+        try {
+            exec.fill(res, "otherSummary");
+            fail();
+        } catch (FillRequestException e) {
+            assertEquals("otherSummary", e.summaryClass);
+        }
     }
 
     @Test