Skip to content

Commit 32de70f

Browse files
trentmPeterEinberger
authored andcommitted
test: fix redis TAV tests with redis 4.0.x (elastic#3004)
client.set('foo') fails on 4.1, but not on 4.0.
1 parent 2ce73bf commit 32de70f

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

test/instrumentation/modules/redis.test.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,9 @@ test('redis', function (t) {
134134
})
135135
})
136136

137-
test('redis client error', function (t) {
137+
// The `redis.set('foo')` we are using to trigger a client error case only
138+
// causes an error in redis >=4.1.0.
139+
test('redis client error', { skip: semver.lt(redisVersion, '4.1.0') }, function (t) {
138140
resetAgent(function (data) {
139141
t.equal(data.transactions.length, 1, 'got 1 transaction')
140142
t.equal(data.spans.length, 1, 'got 1 span')

0 commit comments

Comments
 (0)