Merge pull request #979 from paldepind/master

Path should return entire object for empty path
ramda · Mar 30, 2015 · 8f38c9b · 8f38c9b
2 parents b7cfe5c + 80e6156
commit 8f38c9b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/internal/_path.js b/src/internal/_path.js
@@ -14,7 +14,7 @@
  *      _path(['a', 'b'], {a: {b: 2}}); //=> 2
  */
 module.exports = function _path(paths, obj) {
-  if (obj == null || paths.length === 0) {
+  if (obj == null) {
     return;
   } else {
     var val = obj;

diff --git a/test/path.js b/test/path.js
@@ -22,7 +22,7 @@ describe('path', function() {
       j: ['J']
     };
     assert.strictEqual(R.path(['a', 'b', 'c'], obj), 100);
-    assert.strictEqual(R.path([], obj), undefined);
+    assert.strictEqual(R.path([], obj), obj);
     assert.strictEqual(R.path(['a', 'e', 'f', '1'], obj), 101);
     assert.strictEqual(R.path(['j', '0'], obj), 'J');
     assert.strictEqual(R.path(['j', '1'], obj), undefined);

diff --git a/test/pathEq.js b/test/pathEq.js
@@ -29,7 +29,7 @@ describe('pathEq', function() {
 
   it('accepts empty path', function() {
     assert.strictEqual(R.pathEq([], 42, {a: 1, b: 2}), false);
-    assert.strictEqual(R.pathEq([], undefined, {a: 1, b: 2}), true);
+    assert.strictEqual(R.pathEq([], obj, obj), true);
   });
 
 });