Merge pull request #4 from WillFrew/bugfix/multifetch-calls-itself

Multifetch can call itself
debitoor · Mar 10, 2015 · 6012370 · 6012370
2 parents 71f5a4f + 6b586e7
commit 6012370
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/source/index.js b/source/index.js
@@ -31,7 +31,7 @@ var getResources = function(request, ignore) {
 	var path = url.parse(request.url).pathname;
 
 	return Object.keys(query).reduce(function(acc, key) {
-		if(key !== path && ignore.indexOf(key) === -1) {
+		if(query[key] !== path && ignore.indexOf(key) === -1) {
 			acc[key] = query[key];
 		}
 

diff --git a/test/integration/multifetch.spec.js b/test/integration/multifetch.spec.js
@@ -311,10 +311,14 @@ describe('multifetch', function() {
 			});
 		});
 
-		it('should ignore multifetch resource', function() {
+		it('should be a successful response', function() {
 			chai.expect(body).to.have.property('_error', false);
 		});
 
+		it('should ignore multifetch resource', function() {
+			chai.expect(body).not.to.have.property('multifetch');
+		});
+
 		it('should contain user album', function() {
 			chai.expect(body)
 				.to.have.property('album')
@@ -348,7 +352,7 @@ describe('multifetch', function() {
 			});
 		});
 
-		it('should ignore multifetch resource', function() {
+		it('should be a successful response', function() {
 			chai.expect(body).to.have.property('_error', false);
 		});