Document assumption in pendingMocks()

nock · Oct 17, 2016 · 475c7a5 · 475c7a5
1 parent 143e42e
commit 475c7a5
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/lib/scope.js b/lib/scope.js
@@ -124,6 +124,8 @@ Scope.prototype.pendingMocks = function pendingMocks() {
   var pendingInterceptorKeys = Object.keys(this.keyedInterceptors).filter(function (key) {
     var interceptorList = self.keyedInterceptors[key];
     var pendingInterceptors = interceptorList.filter(function (interceptor) {
+      // TODO: This assumes that completed mocks are removed from the keyedInterceptors list
+      // (when persistence is off). We should change that (and this) in future.
       var persistedAndUsed = self._persist && interceptor.interceptionCounter > 0;
       return !persistedAndUsed && !interceptor.optional;
     });