1
0
mirror of https://github.com/redis/node-redis.git synced 2025-08-07 13:22:56 +03:00
Commit Graph

1474 Commits

Author SHA1 Message Date
Benjamin Coe
1e9613fa8f slight tweak to test command 2015-08-14 21:31:12 -07:00
Erin Spiceland
3aaef47756 Fix bug in mocha tests Redis shutdown which expected exit code to eq 0.
Move a miscategorized select test into the correct describe.
2015-08-14 21:31:10 -07:00
Benjamin Coe
5da083322d fixed up some tests, changed how redis spawns 2015-08-14 21:31:09 -07:00
Erin Spiceland
2b44245056 Add example test with grunt and mocha.
Add test for reconnect.

Run each test for both parsers and both IP versions.

Don't save a reference to this nodified assertion function.

Add DEBUG env var which enables extra debug logging in node_redis.

Remove Grunt, run Redis on 6378 for non-interference.

Remove the tests already ported to Mocha.

Port reconnect after pubsub test; add subscribed after reconnect test.

Reconnet after pubsub test confused me. I don't think it tested
anything, and it didn't pass for me after I ported it. I've disabled it
and added a note. In its place, I've added a test to make sure we are
still subscribed and can receive pubsub messages after a reconnect.

Move test suite config-like stuff into a library.

Move test suite createClient args generation into the config library.

WIP. Some select tests, most disabled and still WIP.
2015-08-14 21:30:12 -07:00
Erin Spiceland
6cae0b880f Merge pull request #785 from fintura/master
Add "use strict", semicolons, whitespace & code cleanup, remove util.print.
2015-08-06 11:11:13 -05:00
Erin Spiceland
b53ebb208a Merge pull request #788 from Starefossen/enable-docker-testing
Make it possible to test node_redis in a Docker environment
2015-07-25 16:30:10 -05:00
Hans Kristian Flaatten
b1ae21d221 Revert "Skip SOCKET test if Redis is not listening on a socket"
This reverts commit e646b4eb39.
2015-07-24 15:33:37 +02:00
Hans Kristian Flaatten
da535a6977 Normalize linked REDIS environment variables 2015-07-24 08:07:33 +02:00
Hans Kristian Flaatten
8b5ce1088a Allways pass HOST and PORT when creating a new client 2015-07-23 22:16:48 +02:00
Hans Kristian Flaatten
e646b4eb39 Skip SOCKET test if Redis is not listening on a socket 2015-07-23 22:16:07 +02:00
Hans Kristian Flaatten
d34dcc8b07 Check for REDIS HOST and PORT environment varibles 2015-07-23 22:15:55 +02:00
Almog Melamed
b77a48739a Update changelog.md: Binary not Binay 2015-07-23 09:35:42 +03:00
Ruben Bridgewater
01a377383b Use type safe comparison 2015-07-22 17:51:24 +02:00
Ruben Bridgewater
0908e9a46b Use console.log instead of deprecated util.print
util.print has been deprecated in node
2015-07-22 17:51:24 +02:00
Ruben Bridgewater
745fccebfb Add var statements to undeclared variables and remove unused variables 2015-07-22 17:51:24 +02:00
Ruben Bridgewater
6866ff9b1e Remove trailing whitespace 2015-07-22 17:51:08 +02:00
Ruben Bridgewater
a2bc597212 Add simicolons
This is just a style change
2015-07-22 17:51:07 +02:00
Ruben Bridgewater
1f9e536ca0 Add use strict statements
This is going to improve the performance minimal and improves the safety of the code
2015-07-22 17:50:37 +02:00
Erin Spiceland
e34debd6ee Merge pull request #786 from fintura/fix-login
Rewrite createClient. Fixes #651
2015-07-22 10:47:30 -05:00
Ruben Bridgewater
208b7874bf Rewrite createClient. Fixes #651 2015-07-22 17:15:12 +02:00
Erin Spiceland
e4da320a29 Merge pull request #783 from stockholmux/master
Clarify createClient in README as per #781 and other comments
2015-07-16 11:46:12 -05:00
Kyle
defec9c1e1 fixed duplicate and typo 2015-07-16 11:30:32 -05:00
Kyle
bc48c2b069 fix typo 2015-07-16 11:02:25 -05:00
Kyle
dc461f08d4 clarifies createClient in README as per #781 2015-07-16 11:01:29 -05:00
Raymond Myers
5eb98e4e04 Merge pull request #733 from raydog/multi-buffer-fix
Fix for multi/exec logic with detect_buffers enabled
2015-07-11 23:06:37 -07:00
Raymond Myers
19c5571229 Added a test case for detect_buffer behavior in a multi/exec 2015-07-11 22:47:25 -07:00
Raymond Myers
ded75c8ea2 Fixed detect_buffers keeping all multi/exec results as buffers 2015-07-11 22:47:25 -07:00
Raymond Myers
aea03d60be Fixed exec result arrays being stringed in detect_buffers mode 2015-07-11 22:47:25 -07:00
Raymond Myers
f384d1c774 Fixed the pub/sub logic accidentally stringing the first value of a multi/exec response 2015-07-11 22:47:25 -07:00
Raymond Myers
6f1acb0678 Fixed bug where buffer hgetall's in a multi would throw exceptions 2015-07-11 22:47:25 -07:00
Benjamin E. Coe
08f1a4de18 Merge pull request #777 from NodeRedis/test-tweaks
had a typo in the coverage badge
2015-07-11 19:48:26 -07:00
Erin Spiceland
93db3b10d5 Merge pull request #775 from jeremiahlee/master
Added example of a Redis SCAN command
2015-07-11 21:46:39 -05:00
Benjamin Coe
06d051ea1d had a bad URL to test coverage 2015-07-11 19:45:05 -07:00
Benjamin E. Coe
131f92b866 Merge pull request #776 from NodeRedis/test-tweaks
various fixes to tests, along with groundwork for adding travis and coveralls support
2015-07-11 19:41:21 -07:00
Benjamin Coe
96da40719d badges \o/ 2015-07-11 19:35:39 -07:00
Benjamin Coe
8ca45e3f1f loose socket perms 2015-07-11 19:35:38 -07:00
Benjamin Coe
7657fdbfb6 daemonize sure 2015-07-11 19:35:38 -07:00
Benjamin Coe
e17228a365 enable sudo 2015-07-11 19:35:38 -07:00
Benjamin Coe
f4e1e2ce6d getting redis configured 2015-07-11 19:35:38 -07:00
Benjamin Coe
116a27a120 update changelog, force travis build 2015-07-11 19:35:38 -07:00
Benjamin Coe
d30e80abbe making an attempt to improve the test suite 2015-07-11 19:35:38 -07:00
Erin Spiceland
b92a62d643 Merge pull request #658 from kuwabarahiroshi/features/add-url-support-to-createClient
added URL support to createClient
2015-07-11 21:32:45 -05:00
Jeremiah Lee Cohick
ff020d37f3 Clarified COUNT's behavior. scan() now will run until cursor reaches 0. 2015-07-10 18:04:38 -07:00
Raymond Myers
9b02ae61b8 Merge pull request #758 from bcoe/coverage
adds coverage reporting using nyc bin
2015-07-10 17:46:43 -07:00
Raymond Myers
e2e790bca0 Merge pull request #746 from kemitchell/spdx-license
Use a Standard-Compliant License Identifier
2015-07-10 17:03:40 -07:00
Jeremiah Lee Cohick
6f28c6acf5 Cursor should be a string 2015-07-10 16:25:52 -07:00
Jeremiah Lee Cohick
010a2acf33 Added example of a Redis SCAN command 2015-07-10 16:10:00 -07:00
Matt Ranney
eb03bb943c Update README and multi_bench 2015-07-10 17:25:17 -04:00
David Vas
6af443639e A little optimization in a function if the client is closing. 2015-06-11 16:19:47 +02:00
vitaliylag
11735099d6 Update index.js
It was almost bug: imagine args is array but callback is not defined - in this case all args (that is consists of one array) are packing to new array. That mean we get this: this.send_command(command, [[...]]). It doesn't make any sense.

After I fix it we get this: this.send_command(command, [...], undefined). It's really ok because if we call for example client.hget("test", "aaa") we actually do the same: this.send_command("hget", ["test", "aaa"], undefined). No different from this.send_command(command, [...], undefined).

By the way, «this.send_command(command, [[...]])» could be a bug. Try to call client.eval(["return 1", 0]) and you should get throw because "eval" required 2 param, but program thinks it's only one param: ["return 1", 0] (at the beginning it was [["return 1", 0]]). There's only one reason why you don't get throw - RedisClient.prototype.eval is overridden for some optimizations, lucky. But that doesn't mean everything is ok.
2015-05-28 14:50:46 +03:00