From e6fc39f15aa05e4b92fd5dc11364fc7fc72b0818 Mon Sep 17 00:00:00 2001 From: Willi Goesgens Date: Wed, 17 Sep 2014 16:28:57 +0200 Subject: [PATCH] Enable first tests for indexes using DESC --- js/server/tests/aql-optimizer-rule-use-index-for-sort.js | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/js/server/tests/aql-optimizer-rule-use-index-for-sort.js b/js/server/tests/aql-optimizer-rule-use-index-for-sort.js index a0cad92cfa..fd03701ba9 100644 --- a/js/server/tests/aql-optimizer-rule-use-index-for-sort.js +++ b/js/server/tests/aql-optimizer-rule-use-index-for-sort.js @@ -188,9 +188,6 @@ function optimizerRuleTestSuite() { var queries = [ ["FOR v IN " + colName + " SORT v.c RETURN [v.a, v.b]", true], - // todo: we use an index anyways right now. - // currently only ASC supported. - // "FOR v IN " + colName + " SORT v.a DESC RETURN [v.a, v.b]", true], ["FOR v IN " + colName + " SORT v.b, v.a RETURN [v.a]", true], ["FOR v IN " + colName + " SORT v.c RETURN [v.a, v.b]", true], ["FOR v IN " + colName + " SORT v.a + 1 RETURN [v.a]", false],// this will throw... @@ -227,9 +224,8 @@ function optimizerRuleTestSuite() { testRuleHasEffect : function () { var allresults; var queries = [ - // currently only ASC supported, but we use the index range anyways. - // todo: this may change. "FOR v IN " + colName + " SORT v.d DESC RETURN [v.d]", + "FOR v IN " + colName + " SORT v.d ASC RETURN [v.d]", "FOR v IN " + colName + " SORT v.d FILTER v.a > 2 LIMIT 3 RETURN [v.d] ", "FOR v IN " + colName + " FOR w IN 1..10 SORT v.d RETURN [v.d]",