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

tests: remove hiredis from testing suite

Browse Source 
as parser no longer supports hiredis
This commit is contained in:
Salakar
2020-02-06 21:26:53 +00:00
parent c78e55d874
commit fbdac9e0ad
2 changed files with 2 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
test/helper.js
View File

@@ -163,11 +163,6 @@ module.exports = {
}
var parsers = ['javascript'];
var protocols = ['IPv4'];
// The js parser works the same as the hiredis parser, just activate this if you want to be on the safe side
// try {
// require('hiredis');
// parsers.push('hiredis');
// } catch (e) {/* ignore eslint */}
if (process.platform !== 'win32') {
protocols.push('IPv6', '/tmp/redis.sock');
}

8
test/pubsub.spec.js
View File

@@ -515,8 +515,8 @@ describe('publish/subscribe', function () {
sub.stream.once('data', function () {
assert.strictEqual(sub.message_buffers, false);
assert.strictEqual(sub.shouldBuffer, false);
sub.on('pmessageBuffer', function (pattern, channel, message) {
if (parser !== 'javascript' && typeof pattern === 'string') {
sub.on('pmessageBuffer', function (pattern, channel) {
if (typeof pattern === 'string') {
pattern = new Buffer(pattern);
channel = new Buffer(channel);
}
@@ -524,10 +524,6 @@ describe('publish/subscribe', function () {
assert.strictEqual(channel.inspect(), new Buffer('/foo').inspect());
sub.quit(end);
});
if (parser === 'javascript') {
assert.notStrictEqual(sub.message_buffers, sub.buffers);
}
});
var batch = sub.batch();
batch.psubscribe('*');