Mirrors/lodash
1
0
Fork 0
mirror of https://github.com/whoisclebs/lodash.git synced 2026-02-01 15:57:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

Ensure _.sample chains correctly.

Browse Source
This commit is contained in:
John-David Dalton
2014-09-16 01:07:33 -07:00
parent 3ab19e81ac
commit f05f60f929
2 changed files with 8 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
lodash.js
View File

@@ -9587,11 +9587,12 @@
lodash.sample = sample;
lodash.prototype.sample = function(n, guard) {
if (!this.__chain__ && (n == null || guard)) {
n = guard ? null : n;
if (!this.__chain__ && n == null) {
return lodash.sample(this.value());
}
return this.tap(function(value) {
lodash.sample(value);
return this.thru(function(value) {
return lodash.sample(value, n);
});
};

8
test/test.js
View File

@@ -9496,7 +9496,7 @@
test('should return two random elements', 1, function() {
var actual = _.sample(array, 2);
ok(actual[0] !== actual[1] && _.contains(array, actual[0]) && _.contains(array, actual[1]));
ok(actual.length == 2 && actual[0] !== actual[1] && _.contains(array, actual[0]) && _.contains(array, actual[1]));
});
test('should contain elements of the collection', 1, function() {
@@ -9555,7 +9555,7 @@
ok(_.contains(array, actual));
actual = _.sample(object, 2);
ok(actual[0] !== actual[1] && _.contains(array, actual[0]) && _.contains(array, actual[1]));
ok(actual.length == 2 && actual[0] !== actual[1] && _.contains(array, actual[0]) && _.contains(array, actual[1]));
});
test('should work when used as a callback for `_.map`', 2, function() {
@@ -9575,7 +9575,7 @@
ok(wrapped instanceof _);
var actual = wrapped.value();
ok(actual[0] !== actual[1] && _.contains(array, actual[0]) && _.contains(array, actual[1]));
ok(actual.length == 2 && actual[0] !== actual[1] && _.contains(array, actual[0]) && _.contains(array, actual[1]));
}
else {
skipTest(2);
@@ -9602,7 +9602,7 @@
ok(_.contains(collection, actual));
actual = _.sample(collection, 2);
ok(actual[0] !== actual[1] && _.contains(collection, actual[0]) && _.contains(collection, actual[1]));
ok(actual.length == 2 && actual[0] !== actual[1] && _.contains(collection, actual[0]) && _.contains(collection, actual[1]));
});
});
}());