summaryrefslogtreecommitdiffstats
path: root/container-search/src/test
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@yahoo-inc.com>2019-09-16 16:36:44 +0200
committerGitHub <noreply@github.com>2019-09-16 16:36:44 +0200
commit49a8abb5663a152df0cd97264bc9771c72cde0e2 (patch)
tree19b894a1ff82cf06b0add35772f3caf2d9057d7d /container-search/src/test
parenta30ea6f8527a5d5cfacf84d4a576929b91d83153 (diff)
Revert "Use a LeanHit until merging is done."
Diffstat (limited to 'container-search/src/test')
-rw-r--r--container-search/src/test/java/com/yahoo/search/dispatch/LeanHitTest.java42
-rw-r--r--container-search/src/test/java/com/yahoo/search/dispatch/MockInvoker.java16
2 files changed, 4 insertions, 54 deletions
diff --git a/container-search/src/test/java/com/yahoo/search/dispatch/LeanHitTest.java b/container-search/src/test/java/com/yahoo/search/dispatch/LeanHitTest.java
deleted file mode 100644
index 3478038e427..00000000000
--- a/container-search/src/test/java/com/yahoo/search/dispatch/LeanHitTest.java
+++ /dev/null
@@ -1,42 +0,0 @@
-package com.yahoo.search.dispatch;
-
-import org.junit.Test;
-
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
-
-public class LeanHitTest {
- byte [] gidA = {'a'};
- byte [] gidB = {'b'};
- byte [] gidC = {'c'};
- private void verifyTransitiveOrdering(LeanHit a, LeanHit b, LeanHit c) {
- assertTrue(a.compareTo(b) < 0);
- assertTrue(b.compareTo(c) < 0);
- assertTrue(a.compareTo(c) < 0);
- assertTrue(b.compareTo(a) > 0);
- assertTrue(c.compareTo(b) > 0);
- assertTrue(c.compareTo(a) > 0);
- }
- @Test
- public void testOrderingByRelevance() {
- assertEquals(0, new LeanHit(gidA, 0, 0, 1).compareTo(new LeanHit(gidA, 0, 0, 1)));
- verifyTransitiveOrdering(new LeanHit(gidA, 0, 0, 1),
- new LeanHit(gidA, 0, 0, 0),
- new LeanHit(gidA, 0, 0, -1));
- }
- @Test
- public void testOrderingByGid() {
- assertEquals(0, new LeanHit(gidA, 0, 0, 1).compareTo(new LeanHit(gidA, 0, 0, 1)));
-
- verifyTransitiveOrdering(new LeanHit(gidA, 0, 0, 1),
- new LeanHit(gidB, 0, 0, 1),
- new LeanHit(gidC, 0, 0, 1));
- }
- @Test
- public void testOrderingBySortData() {
- assertEquals(0, new LeanHit(gidA, 0, 0, gidA).compareTo(new LeanHit(gidA, 0, 0, gidA)));
- verifyTransitiveOrdering(new LeanHit(gidA, 0, 0, gidA),
- new LeanHit(gidA, 0, 0, gidB),
- new LeanHit(gidA, 0, 0, gidC));
- }
-}
diff --git a/container-search/src/test/java/com/yahoo/search/dispatch/MockInvoker.java b/container-search/src/test/java/com/yahoo/search/dispatch/MockInvoker.java
index 2fe434d6f3f..4b2f63d6b89 100644
--- a/container-search/src/test/java/com/yahoo/search/dispatch/MockInvoker.java
+++ b/container-search/src/test/java/com/yahoo/search/dispatch/MockInvoker.java
@@ -1,7 +1,6 @@
// Copyright 2019 Yahoo Holdings. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
package com.yahoo.search.dispatch;
-import com.yahoo.prelude.fastsearch.FastHit;
import com.yahoo.search.Query;
import com.yahoo.search.Result;
import com.yahoo.search.dispatch.searchcluster.Node;
@@ -38,20 +37,13 @@ class MockInvoker extends SearchInvoker {
}
@Override
- protected InvokerResult getSearchResult(Execution execution) throws IOException {
- InvokerResult ret = new InvokerResult(query, 10);
+ protected Result getSearchResult(Execution execution) throws IOException {
+ Result ret = new Result(query);
if (coverage != null) {
- ret.getResult().setCoverage(coverage);
+ ret.setCoverage(coverage);
}
if (hits != null) {
- for (Hit h : hits) {
- if (h instanceof FastHit) {
- FastHit fh = (FastHit) h;
- ret.getLeanHits().add(new LeanHit(fh.getRawGlobalId(), fh.getPartId(), fh.getDistributionKey(), fh.getRelevance().getScore()));
- } else {
- ret.getResult().hits().add(h);
- }
- }
+ ret.hits().addAll(hits);
}
return ret;
}