fix(interceptor): don't hang on the last response when using .times()

lib/interceptor.js

@@ -442,13 +442,13 @@ module.exports = class Interceptor {
   markConsumed() {
     this.interceptionCounter++
 
-    remove(this)
-
     if ((this.scope.shouldPersist() || this.counter > 0) && this.filePath) {
       this.body = fs.createReadStream(this.filePath)
       this.body.pause()
     }
 
+    remove(this)
+
     if (!this.scope.shouldPersist() && this.counter < 1) {
       this.scope.remove(this._key, this)
     }

tests/test_reply_with_file.js

@@ -39,6 +39,25 @@ describe('`replyWithFile()`', () => {
     scope.done()
   })
 
+  it('reply with file with repeated', async () => {
+    const scope = nock('http://example.test')
+      .get('/')
+      .times(2)
+      .replyWithFile(200, binaryFilePath, {
+        'content-encoding': 'gzip',
+      })
+
+    const response1 = await got('http://example.test/')
+    expect(response1.statusCode).to.equal(200)
+    expect(response1.body).to.have.lengthOf(20)
+
+    const response2 = await got('http://example.test/')
+    expect(response2.statusCode).to.equal(200)
+    expect(response2.body).to.have.lengthOf(20)
+
+    scope.done()
+  })
+
   describe('with no fs', () => {
     const { Scope } = proxyquire('../lib/scope', {
       './interceptor': proxyquire('../lib/interceptor', {