Skip to content

Commit

Permalink
Merge pull request #33273 from vespa-engine/arnej/test-yql-representa…
Browse files Browse the repository at this point in the history
…tion

test yqlRepresentation of order by "[rank]"
  • Loading branch information
geirst authored Feb 7, 2025
2 parents add10c8 + 09896e7 commit 9f53d71
Showing 1 changed file with 12 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -917,6 +917,18 @@ void testOrdering() {
assertEquals(Order.ASCENDING, parser.getSorting().fieldOrders().get(0).getSortOrder());
}

@Test
void testYqlRepresentationOfOrdering() {
var newTree = parse("select foo from bar where price < 100 order by \"[rank]\" limit 5");
var query = new Query();
query.getModel().getQueryTree().setRoot(newTree.getRoot());
query.setHits(parser.getHits());
query.getRanking().setSorting(parser.getSorting());
String got = query.yqlRepresentation(true);
// note: above code does not transfer selection or source, so we get '*' here:
assertEquals("select * from sources * where price < 100 order by \"[rank]\" limit 5", got);
}

@Test
void testAnnotatedOrdering() {
assertParse(
Expand Down

0 comments on commit 9f53d71

Please sign in to comment.