Skip to content

Commit 627eb78

Browse files
committed
run XPEDNDING with consumers test on redis 6.2 and up, fix redis 6 tests
1 parent 7702220 commit 627eb78

File tree

2 files changed

+9
-11
lines changed

2 files changed

+9
-11
lines changed

.github/workflows/tests.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,8 @@ jobs:
1616
strategy:
1717
fail-fast: false
1818
matrix:
19-
node-version: [12, 14, 16]
20-
redis-version: [5, 6.0, 6.2]
19+
node-version: ['12', '14', '16']
20+
redis-version: ['5', '6.0', '6.2']
2121
steps:
2222
- uses: actions/checkout@v2.3.4
2323
with:

packages/client/lib/commands/XPENDING.spec.ts

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,9 @@ describe('XPENDING', () => {
1414

1515
describe('client.xPending', () => {
1616
testUtils.testWithClient('simple', async client => {
17-
await Promise.all([
18-
client.xGroupCreate('key', 'group', '$', {
19-
MKSTREAM: true
20-
}),
21-
client.xGroupCreateConsumer('key', 'group', 'consumer')
22-
]);
17+
await client.xGroupCreate('key', 'group', '$', {
18+
MKSTREAM: true
19+
});
2320

2421
assert.deepEqual(
2522
await client.xPending('key', 'group'),
@@ -57,8 +54,9 @@ describe('XPENDING', () => {
5754
}]
5855
}
5956
);
60-
}, GLOBAL.SERVERS.OPEN);
57+
}, {
58+
...GLOBAL.SERVERS.OPEN,
59+
minimumDockerVersion: [6, 2]
60+
});
6161
});
62-
63-
6462
});

0 commit comments

Comments
 (0)