diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2019-09-16 16:36:44 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-09-16 16:36:44 +0200 |
commit | 49a8abb5663a152df0cd97264bc9771c72cde0e2 (patch) | |
tree | 19b894a1ff82cf06b0add35772f3caf2d9057d7d /container-search/src/test | |
parent | a30ea6f8527a5d5cfacf84d4a576929b91d83153 (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.java | 42 | ||||
-rw-r--r-- | container-search/src/test/java/com/yahoo/search/dispatch/MockInvoker.java | 16 |
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; } |