8000 Fixed array index test. It is now adapted to new definition of sparse · jameswei/arangodb@476d4e7 · GitHub
[go: up one dir, main page]
Skip to content

Commit 476d4e7

Browse files
mchackimchacki
committed
Fixed array index test. It is now adapted to new definition of sparse
1 parent 021eb6a commit 476d4e7

File tree

1 file changed

+10
-7
lines changed

1 file changed

+10
-7
lines changed

‎js/server/tests/shell-array-index-noncluster.js

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -194,6 +194,7 @@ function arrayHashIndexSuite () {
194194

195195
testInsertAndReadArrayCombinedSparse : function () {
196196
var idx = collection.ensureHashIndex("a[*]", "b[*]", {sparse: true}).id;
197+
// Sparse does not have any effect on array attributes.
197198

198199
var id = collection.save({a: [1, 2], b: ["a", "b"]})._id;
199200

@@ -226,13 +227,13 @@ function arrayHashIndexSuite () {
226227
assertEqual(res, ids);
227228

228229
res = collection.BY_EXAMPLE_HASH(idx, {a: "duplicate", b: null}, 0, null).documents;
229-
assertEqual(res.length, 0);
230+
assertEqual(res.length, 3);
230231

231232
res = collection.BY_EXAMPLE_HASH(idx, {a: null, b: "duplicate"}, 0, null).documents;
232-
assertEqual(res.length, 0);
233+
assertEqual(res.length, 3);
233234

234235
res = collection.BY_EXAMPLE_HASH(idx, {a: null, b: null}, 0, null).documents;
235-
assertEqual(res.length, 0);
236+
assertEqual(res.length, 3);
236237
},
237238

238239
////////////////////////////////////////////////////////////////////////////////
@@ -427,9 +428,10 @@ function arrayHashIndexSuite () {
427428
assertEqual(res.length, 1);
428429
assertEqual(res[0]._id, id7);
429430

430-
// But not like this
431+
// And this also
431432
res = collection.BY_EXAMPLE_HASH(idx, {a: null, b: 1}, 0, null).documents;
432-
assertEqual(res.length, 0);
433+
assertEqual(res.length, 1);
434+
assertEqual(res[0]._id, id7);
433435
},
434436

435437
/* TODO Decission required here
@@ -711,9 +713,10 @@ function arraySkiplistIndexSuite () {
711713
assertEqual(res.length, 1);
712714
assertEqual(res[0]._id, id7);
713715

714-
// But not like this
716+
// And this also
715717
res = collection.BY_EXAMPLE_SKIPLIST(idx, {a: null, b: 1}, 0, null).documents;
716-
assertEqual(res.length, 0);
718+
assertEqual(res.length, 1);
719+
assertEqual(res[0]._id, id7);
717720
}
718721

719722
};

0 commit comments

Comments
 (0)
0