diff --git a/js/server/tests/aql-graph-traverser.js b/js/server/tests/aql-graph-traverser.js index ca1b8a8d0e..a7982524a9 100644 --- a/js/server/tests/aql-graph-traverser.js +++ b/js/server/tests/aql-graph-traverser.js @@ -445,7 +445,8 @@ function multiCollectionGraphSuite () { }, testNoBindParameterSingleForFilter: function () { - var query = "FOR s IN " + vn + " SORT s FOR x, e, p IN OUTBOUND s " + en + " FILTER p.vertices[1]._key == s._key SORT x RETURN x"; + var query = "FOR s IN " + vn + " SORT s FOR x, e, p IN OUTBOUND s " + + en + " FILTER p.vertices[1]._key == s._key SORT x RETURN x"; var result = db._query(query).toArray(); assertEqual(result.length, 0); var plans = AQL_EXPLAIN(query, { }, opts).plans; @@ -455,8 +456,9 @@ function multiCollectionGraphSuite () { }); }, - testNoBindParameterSingleForFilter: function () { - var query = "FOR s IN " + vn + " SORT s FOR x, e, p IN OUTBOUND s " + en + " FILTER p.vertices[1]._key == NOOPT(V8(RAND())) SORT x RETURN x"; + testNoBindParameterV8Function: function () { + var query = "FOR s IN " + vn + " SORT s FOR x, e, p IN OUTBOUND s " + + en + " FILTER p.vertices[1]._key == NOOPT(V8(RAND())) SORT x RETURN x"; var result = db._query(query).toArray(); assertEqual(result.length, 0); var plans = AQL_EXPLAIN(query, { }, opts).plans; @@ -785,16 +787,11 @@ function multiEdgeCollectionGraphSuite () { var g; const gn = "UnitTestGraph"; - const vn2 = "UnitTestVertexCollection2"; const en2 = "UnitTestEdgeCollection2"; var ruleName = "merge-traversal-filter"; var paramEnabled = { optimizer: { rules: [ "-all", "+" + ruleName ] } }; var opts = _.clone(paramEnabled); - // We always use the same query, the result should be identical. - var validateResult = function (result) { - }; - return { setUp: function() { @@ -809,7 +806,7 @@ function multiEdgeCollectionGraphSuite () { vc = db._create(vn, {numberOfShards: 4}); ec = db._createEdgeCollection(en, {numberOfShards: 4}); - ec2 = db._createEdgeCollection(en2, {numberOfShards: 4}); + var ec2 = db._createEdgeCollection(en2, {numberOfShards: 4}); g = gm._create(gn, [gm._relation(en, vn, vn), gm._relation(en2, vn, vn)]); @@ -835,7 +832,7 @@ function multiEdgeCollectionGraphSuite () { testTwoVertexCollectionsInOutbound: function () { /* this test is intended to trigger the clone functionality. */ - expectResult = ['B', 'C', 'D', 'E']; + var expectResult = ['B', 'C', 'D', 'E']; var query = "FOR x IN ANY @startId GRAPH @graph SORT x._id RETURN x._key"; var bindVars = { graph: gn, diff --git a/js/server/tests/aql-optimizer-rule-merge-traversal-filter.js b/js/server/tests/aql-optimizer-rule-merge-traversal-filter.js index 21d7f5f68e..ee3a553c07 100644 --- a/js/server/tests/aql-optimizer-rule-merge-traversal-filter.js +++ b/js/server/tests/aql-optimizer-rule-merge-traversal-filter.js @@ -1,5 +1,5 @@ /*jshint globalstrict:false, strict:false, maxlen: 500 */ -/*global assertEqual, assertTrue, assertNotEqual, AQL_EXPLAIN, AQL_EXECUTE */ +/*global assertEqual, assertTrue, assertNotEqual, AQL_EXPLAIN, AQL_EXECUTE, AQL_EXECUTEJSON */ //////////////////////////////////////////////////////////////////////////////// /// @brief tests for optimizer rules @@ -43,7 +43,6 @@ function optimizerRuleTestSuite () { var paramEnabled = { optimizer: { rules: [ "-all", "+" + ruleName ] } }; var paramDisabled = { optimizer: { rules: [ "+all", "-" + ruleName ] } }; var graphName = "myUnittestGraph"; - var ruleName = "merge-traversal-filter"; var opts = _.clone(paramEnabled); return { @@ -57,7 +56,7 @@ function optimizerRuleTestSuite () { opts.allPlans = true; opts.verbosePlans = true; - try { graph_module._drop(graphName, true); } catch (x) {}; + try { graph_module._drop(graphName, true); } catch (x) {} var graph = graph_module._create(graphName, [ graph_module._relation("edges", "circles", ["circles"])]);