Stricten tests by always ending redis with .end(true) if possible

README.md

@@ -246,7 +246,7 @@ something like this `Error: Ready check failed: ERR operation not permitted`.
 Forcibly close the connection to the Redis server. Note that this does not wait until all replies have been parsed.
 If you want to exit cleanly, call `client.quit()` to send the `QUIT` command after you have handled all replies.
 
-If flush is set to true, all commands will be rejected instead of ignored after using `.end`.
+If flush is set to true, all still running commands will be rejected instead of ignored after using `.end`.
 
 This example closes the connection to the Redis server before the replies have been read. You probably don't
 want to do this:
@@ -263,8 +263,7 @@ client.get("foo_rand000000000000", function (err, reply) {
 });
 ```
 
-`client.end()` is useful for timeout cases where something is stuck or taking too long and you want
-to start over.
+`client.end()` without the flush parameter should not be used in production!
 
 ## client.unref()
 

test/auth.spec.js

@@ -24,7 +24,9 @@ describe("client authentication", function () {
                 client = null;
             });
             afterEach(function () {
-                client.end();
+                // Explicitly ignore still running commands
+                // The ready command could still be running
+                client.end(false);
             });
 
             it("allows auth to be provided with 'auth' method", function (done) {

test/commands/blpop.spec.js

@@ -61,8 +61,8 @@ describe("The 'blpop' method", function () {
             });
 
             afterEach(function () {
-                client.end();
-                bclient.end();
+                client.end(true);
+                bclient.end(true);
             });
         });
     });

test/commands/client.spec.js

@@ -28,8 +28,8 @@ describe("The 'client' method", function () {
             });
 
             afterEach(function () {
-                client.end();
-                client2.end();
+                client.end(true);
+                client2.end(true);
             });
 
             describe('list', function () {

test/commands/dbsize.spec.js

@@ -53,7 +53,7 @@ describe("The 'dbsize' method", function () {
                 });
 
                 afterEach(function () {
-                    client.end();
+                    client.end(true);
                 });
 
                 it("returns a zero db size", function (done) {

test/commands/del.spec.js

@@ -50,7 +50,7 @@ describe("The 'del' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/eval.spec.js

@@ -22,7 +22,7 @@ describe("The 'eval' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
 
             it('converts a float to an integer when evaluated', function (done) {

test/commands/exits.spec.js

@@ -33,7 +33,7 @@ describe("The 'exits' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/expire.spec.js

@@ -35,7 +35,7 @@ describe("The 'expire' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/flushdb.spec.js

@@ -50,7 +50,7 @@ describe("The 'flushdb' method", function () {
                 });
 
                 afterEach(function () {
-                    client.end();
+                    client.end(true);
                 });
 
                 describe("when there is data in Redis", function () {

test/commands/geoadd.spec.js

@@ -28,7 +28,7 @@ describe("The 'geoadd' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/get.spec.js

@@ -56,7 +56,7 @@ describe("The 'get' method", function () {
                 });
 
                 afterEach(function () {
-                    client.end();
+                    client.end(true);
                 });
 
                 describe("when the key exists in Redis", function () {

test/commands/getset.spec.js

@@ -52,7 +52,7 @@ describe("The 'getset' method", function () {
                 });
 
                 afterEach(function () {
-                    client.end();
+                    client.end(true);
                 });
 
                 describe("when the key exists in Redis", function () {

test/commands/hgetall.spec.js

@@ -77,7 +77,7 @@ describe("The 'hgetall' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/hincrby.spec.js

@@ -33,7 +33,7 @@ describe("The 'hincrby' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/hlen.spec.js

@@ -31,7 +31,7 @@ describe("The 'hlen' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/hmget.spec.js

@@ -64,7 +64,7 @@ describe("The 'hmget' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/hmset.spec.js

@@ -110,7 +110,7 @@ describe("The 'hmset' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/hset.spec.js

@@ -56,7 +56,7 @@ describe("The 'hset' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/incr.spec.js

@@ -29,7 +29,7 @@ describe("The 'incr' method", function () {
                 });
 
                 afterEach(function () {
-                    client.end();
+                    client.end(true);
                 });
 
                 it("reports an error", function (done) {
@@ -64,7 +64,7 @@ describe("The 'incr' method", function () {
                 });
 
                 after(function () {
-                    client.end();
+                    client.end(true);
                 });
 
                 it("changes the last digit from 2 to 3", function (done) {

test/commands/keys.spec.js

@@ -63,7 +63,7 @@ describe("The 'keys' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/mget.spec.js

@@ -63,7 +63,7 @@ describe("The 'mget' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/mset.spec.js

@@ -52,7 +52,7 @@ describe("The 'mset' method", function () {
                 });
 
                 afterEach(function () {
-                    client.end();
+                    client.end(true);
                 });
 
                 describe("and a callback is specified", function () {

test/commands/msetnx.spec.js

@@ -31,7 +31,7 @@ describe("The 'msetnx' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/randomkey.test.js

@@ -28,7 +28,7 @@ describe("The 'randomkey' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/rename.spec.js

@@ -31,7 +31,7 @@ describe("The 'rename' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/renamenx.spec.js

@@ -34,7 +34,7 @@ describe("The 'renamenx' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/rpush.spec.js

@@ -29,7 +29,7 @@ describe("The 'rpush' command", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/sadd.spec.js

@@ -55,7 +55,7 @@ describe("The 'sadd' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/scard.spec.js

@@ -24,7 +24,7 @@ describe("The 'scard' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/script.spec.js

@@ -23,7 +23,7 @@ describe("The 'script' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
 
             it("loads script with client.script('load')", function (done) {

test/commands/sdiff.spec.js

@@ -40,7 +40,7 @@ describe("The 'sdiff' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/sdiffstore.spec.js

@@ -40,7 +40,7 @@ describe("The 'sdiffstore' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/select.spec.js

@@ -41,7 +41,7 @@ describe("The 'select' method", function () {
                 });
 
                 afterEach(function () {
-                    client.end();
+                    client.end(true);
                 });
 
                 it("changes the database and calls the callback", function (done) {

test/commands/set.spec.js

@@ -50,7 +50,7 @@ describe("The 'set' method", function () {
                 });
 
                 afterEach(function () {
-                    client.end();
+                    client.end(true);
                 });
 
                 describe("and a callback is specified", function () {

test/commands/setex.spec.js

@@ -34,7 +34,7 @@ describe("The 'setex' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/setnx.spec.js

@@ -30,7 +30,7 @@ describe("The 'setnx' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/sinter.spec.js

@@ -56,7 +56,7 @@ describe("The 'sinter' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/sinterstore.spec.js

@@ -41,7 +41,7 @@ describe("The 'sinterstore' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/sismember.spec.js

@@ -28,7 +28,7 @@ describe("The 'sismember' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });

test/commands/slowlog.spec.js

@@ -34,7 +34,7 @@ describe("The 'slowlog' method", function () {
             });
 
             afterEach(function () {
-                client.end();
+                client.end(true);
             });
         });
     });