Merge pull request #4 from sharkcore/remove_listeners

remove listeners
sharkcore · Nov 30, 2017 · 68f1fcc · 68f1fcc
2 parents 5706348 + 8477cde
commit 68f1fcc
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/Makefile b/Makefile
@@ -1,7 +1,7 @@
 any: build
 
 .PHONY: test
-test:
+test: node_modules
 	yarn run lint
 	yarn test
 

diff --git a/src/index.js b/src/index.js
@@ -22,9 +22,21 @@ export default function tweenz(...tweens): Middleware {
 
         // Construct a Promise to be fulfilled when request has completed
         const requestDetails = new Promise(resolve => {
-            res.on('finish', () => {
+            function finish() {
                 resolve(context);
-            });
+                // eslint-disable-next-line no-use-before-define
+                removeListeners();
+            }
+
+            function removeListeners() {
+                res.removeListener('finish', finish);
+                res.removeListener('error', removeListeners);
+                res.removeListener('close', removeListeners);
+            }
+
+            res.once('finish', finish);
+            res.once('error', removeListeners);
+            res.once('close', removeListeners);
         });
 
         // Call each tween body