Skip to content

Commit b7bda05

Browse files
author
Ruben Bridgewater
committed
Improve coverage; make tests ready for Redis 3.2
1 parent fa4f0e0 commit b7bda05

File tree

7 files changed

+23
-23
lines changed

7 files changed

+23
-23
lines changed

‎test/auth.spec.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -272,13 +272,13 @@ describe('client authentication', function () {
272272
var args = config.configureClient(parser, ip, {
273273
password: auth
274274
});
275-
client = redis.createClient.apply(redis.createClient, args);
275+
client = redis.createClient.apply(null, args);
276276
client.set('foo', 'bar');
277277
client.subscribe('somechannel', 'another channel', function (err, res) {
278278
client.once('ready', function () {
279279
assert.strictEqual(client.pub_sub_mode, 1);
280280
client.get('foo', function (err, res) {
281-
assert.strictEqual(err.message, 'ERR only (P)SUBSCRIBE / (P)UNSUBSCRIBE / QUIT allowed in this context');
281+
assert(/ERR only \(P\)SUBSCRIBE \/ \(P\)UNSUBSCRIBE/.test(err.message));
282282
done();
283283
});
284284
});
@@ -302,7 +302,7 @@ describe('client authentication', function () {
302302
var args = config.configureClient(parser, 'localhost', {
303303
noReadyCheck: true
304304
});
305-
client = redis.createClient.apply(redis.createClient, args);
305+
client = redis.createClient.apply(null, args);
306306
assert.strictEqual(client.selected_db, undefined);
307307
var end = helper.callFuncAfter(done, 8);
308308
client.on('monitor', function () {
@@ -317,9 +317,9 @@ describe('client authentication', function () {
317317
})
318318
.monitor()
319319
.set('foo', 'bar', helper.isString('OK'))
320-
.INFO(function (err, res) {
321-
assert.strictEqual(res.indexOf('# Server\r\nredis_version:'), 0);
322-
assert.deepEqual(client.serverInfo.db5, { avg_ttl: 0, expires: 0, keys: 1 });
320+
.INFO('stats', function (err, res) {
321+
assert.strictEqual(res.indexOf('# Stats\r\n'), 0);
322+
assert.strictEqual(client.serverInfo.sync_full, '0');
323323
})
324324
.get('foo', helper.isString('bar'))
325325
.subscribe(['foo', 'bar'])
@@ -328,8 +328,8 @@ describe('client authentication', function () {
328328
.psubscribe('*')
329329
.quit(helper.isString('OK')) // this might be interesting
330330
.exec(function (err, res) {
331-
res[4] = res[4].substr(0, 10);
332-
assert.deepEqual(res, ['OK', 'OK', 'OK', 'OK', '# Server\r\n', 'bar', 'bar', 'foo', '/foo', '*', 'OK']);
331+
res[4] = res[4].substr(0, 9);
332+
assert.deepEqual(res, ['OK', 'OK', 'OK', 'OK', '# Stats\r\n', 'bar', 'bar', 'foo', '/foo', '*', 'OK']);
333333
end();
334334
});
335335
});

‎test/commands/client.spec.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ describe("The 'client' method", function () {
121121
var client2;
122122

123123
beforeEach(function (done) {
124-
client2 = redis.createClient.apply(redis.createClient, args);
124+
client2 = redis.createClient.apply(null, args);
125125
client2.once('ready', function () {
126126
done();
127127
});
@@ -136,9 +136,9 @@ describe("The 'client' method", function () {
136136
// per chunk. So the execution order is only garanteed on each client
137137
var end = helper.callFuncAfter(done, 2);
138138

139-
client.client('setname', 'RUTH', helper.isString('OK'));
140-
client2.client('setname', 'RENEE', helper.isString('OK'));
141-
client2.client('setname', 'MARTIN', helper.isString('OK'));
139+
client.client('setname', 'RUTH');
140+
client2.client('setname', ['RENEE'], helper.isString('OK'));
141+
client2.client(['setname', 'MARTIN'], helper.isString('OK'));
142142
client2.client('getname', function (err, res) {
143143
assert.equal(res, 'MARTIN');
144144
end();

‎test/commands/hgetall.spec.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,23 +28,23 @@ describe("The 'hgetall' method", function () {
2828
assert.strictEqual('1', obj.mjr.toString());
2929
assert.strictEqual('23', obj.another.toString());
3030
assert.strictEqual('1234', obj.home.toString());
31-
return done(err);
31+
done(err);
3232
});
3333
});
3434

3535
it('handles fetching keys set using an object', function (done) {
36-
client.HMSET('msg_test', { message: 'hello' }, helper.isString('OK'));
36+
client.batch().HMSET('msg_test', { message: 'hello' }, undefined).exec();
3737
client.hgetall('msg_test', function (err, obj) {
3838
assert.strictEqual(1, Object.keys(obj).length);
3939
assert.strictEqual(obj.message, 'hello');
40-
return done(err);
40+
done(err);
4141
});
4242
});
4343

4444
it('handles fetching a messing key', function (done) {
4545
client.hgetall('missing', function (err, obj) {
4646
assert.strictEqual(null, obj);
47-
return done(err);
47+
done(err);
4848
});
4949
});
5050
});

‎test/commands/hmset.spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ describe("The 'hmset' method", function () {
3939
});
4040

4141
it('handles object-style syntax and the key being a number', function (done) {
42-
client.HMSET(231232, {'0123456789': 'abcdefghij', 'some manner of key': 'a type of value', 'otherTypes': 555}, helper.isString('OK'));
42+
client.HMSET(231232, {'0123456789': 'abcdefghij', 'some manner of key': 'a type of value', 'otherTypes': 555}, undefined);
4343
client.HGETALL(231232, function (err, obj) {
4444
assert.equal(obj['0123456789'], 'abcdefghij');
4545
assert.equal(obj['some manner of key'], 'a type of value');

‎test/multi.spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -704,7 +704,7 @@ describe("The 'multi' method", function () {
704704
});
705705
multi.get('foo', helper.isString('bar'));
706706
multi.exec(function (err, res) {
707-
res[3] = res[3].substr(0, 10);
707+
res[2] = res[2].substr(0, 10);
708708
assert.deepEqual(res, ['OK', 'OK', '# Server\r\n', 'bar']);
709709
done();
710710
});

‎test/pubsub.spec.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,8 @@ describe('publish/subscribe', function () {
1919
beforeEach(function (done) {
2020
var end = helper.callFuncAfter(done, 2);
2121

22-
pub = redis.createClient.apply(redis.createClient, args);
23-
sub = redis.createClient.apply(redis.createClient, args);
22+
pub = redis.createClient.apply(null, args);
23+
sub = redis.createClient.apply(null, args);
2424
pub.once('connect', function () {
2525
pub.flushdb(function () {
2626
end();
@@ -593,7 +593,7 @@ describe('publish/subscribe', function () {
593593
});
594594
// Get is forbidden
595595
sub.get('foo', function (err, res) {
596-
assert.strictEqual(err.message, 'ERR only (P)SUBSCRIBE / (P)UNSUBSCRIBE / QUIT allowed in this context');
596+
assert(/^ERR only \(P\)SUBSCRIBE \/ \(P\)UNSUBSCRIBE/.test(err.message));
597597
assert.strictEqual(err.command, 'GET');
598598
});
599599
// Quit is allowed
@@ -603,7 +603,7 @@ describe('publish/subscribe', function () {
603603
it('emit error if only pub sub commands are allowed without callback', function (done) {
604604
sub.subscribe('channel');
605605
sub.on('error', function (err) {
606-
assert.strictEqual(err.message, 'ERR only (P)SUBSCRIBE / (P)UNSUBSCRIBE / QUIT allowed in this context');
606+
assert(/^ERR only \(P\)SUBSCRIBE \/ \(P\)UNSUBSCRIBE/.test(err.message));
607607
assert.strictEqual(err.command, 'GET');
608608
done();
609609
});

‎test/return_buffers.spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -252,7 +252,7 @@ describe('return_buffers', function () {
252252
var subConnected;
253253

254254
pub = redis.createClient.apply(redis.createClient, basicArgs);
255-
sub = redis.createClient.apply(redis.createClient, args);
255+
sub = redis.createClient.apply(null, args);
256256
pub.once('connect', function () {
257257
pub.flushdb(function () {
258258
pubConnected = true;

0 commit comments

Comments
 (0)