Merge pull request #562 from wheresrhys/all-done

fix bug with detecting if done when mocking with *

src/lib/inspecting.js

@@ -42,7 +42,7 @@ FetchMock.filterCalls = function (nameOrMatcher, options) {
 		);
 		calls = calls.filter(({ identifier }) => identifier === nameOrMatcher);
 	} else {
-		matcher = normalizeUrl(nameOrMatcher);
+		matcher = nameOrMatcher === '*' ? '*' : normalizeUrl(nameOrMatcher);
 		if (this.routes.some(({ identifier }) => identifier === matcher)) {
 			debug(
 				`Filter provided, ${nameOrMatcher}, identifies a route. Returning only calls handled by that route`

test/specs/repeat.test.js

@@ -81,6 +81,18 @@ describe('repeat and done()', () => {
 		expect(fm.done('http://c.com/')).to.be.false;
 	});
 
+	it("can tell when done if using '*'", () => {
+		fm.mock('*', '200');
+		fm.fetchHandler('http://a.com');
+		expect(fm.done()).to.be.true;
+	});
+
+	it('can tell when done if using begin:', () => {
+		fm.mock('begin:http', '200');
+		fm.fetchHandler('http://a.com');
+		expect(fm.done()).to.be.true;
+	});
+
 	it("won't mock if route already matched enough times", async () => {
 		fm.mock('http://a.com/', 200, { repeat: 1 });