Skip to content

Commit 58f88b5

Browse files
authored
Merge pull request #1 from sjpotter/change-testing
Change testing
2 parents ea51f39 + b4b67f6 commit 58f88b5

File tree

7 files changed

+20
-25
lines changed

7 files changed

+20
-25
lines changed

packages/bloom/lib/test-utils.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,15 @@ import TestUtils from '@redis/test-utils';
22
import RedisBloomModules from '.';
33

44
export default new TestUtils({
5-
dockerImageName: 'redislabs/rebloom',
5+
dockerImageName: 'redis',
66
dockerImageVersionArgument: 'redisbloom-version',
7-
defaultDockerVersion: 'edge'
7+
defaultDockerVersion: '8.0-M01'
88
});
99

1010
export const GLOBAL = {
1111
SERVERS: {
1212
OPEN: {
13-
serverArguments: ['--loadmodule /usr/lib/redis/modules/redisbloom.so'],
13+
serverArguments: [],
1414
clientOptions: {
1515
modules: RedisBloomModules
1616
}

packages/client/lib/test-utils.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { setTimeout } from 'node:timers/promises';
55
const utils = new TestUtils({
66
dockerImageName: 'redis',
77
dockerImageVersionArgument: 'redis-version',
8-
defaultDockerVersion: '7.4'
8+
defaultDockerVersion: '8.0-M01'
99
});
1010

1111
export default utils;

packages/graph/lib/test-utils.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,15 @@ import TestUtils from '@redis/test-utils';
22
import RedisGraph from '.';
33

44
export default new TestUtils({
5-
dockerImageName: 'redislabs/redisgraph',
6-
dockerImageVersionArgument: 'redisgraph-version'
5+
dockerImageName: 'redis',
6+
dockerImageVersionArgument: 'redisgraph-version',
7+
defaultDockerVersion: '8.0-M01'
78
});
89

910
export const GLOBAL = {
1011
SERVERS: {
1112
OPEN: {
12-
serverArguments: ['--loadmodule /usr/lib/redis/modules/redisgraph.so'],
13+
serverArguments: [],
1314
clientOptions: {
1415
modules: {
1516
graph: RedisGraph

packages/json/lib/test-utils.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,15 @@ import TestUtils from '@redis/test-utils';
22
import RedisJSON from '.';
33

44
export default new TestUtils({
5-
dockerImageName: 'redislabs/rejson',
6-
dockerImageVersionArgument: 'rejson-version'
5+
dockerImageName: 'redis',
6+
dockerImageVersionArgument: 'redisgraph-version',
7+
defaultDockerVersion: '8.0-M01'
78
});
89

910
export const GLOBAL = {
1011
SERVERS: {
1112
OPEN: {
12-
serverArguments: ['--loadmodule /usr/lib/redis/modules/rejson.so'],
13+
serverArguments: [],
1314
clientOptions: {
1415
modules: {
1516
json: RedisJSON

packages/search/lib/test-utils.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,15 @@ import TestUtils from '@redis/test-utils';
22
import RediSearch from '.';
33

44
export default new TestUtils({
5-
dockerImageName: 'redislabs/redisearch',
5+
dockerImageName: 'redis',
66
dockerImageVersionArgument: 'redisearch-version',
7-
defaultDockerVersion: '2.4.9'
7+
defaultDockerVersion: '8.0-M01'
88
});
99

1010
export const GLOBAL = {
1111
SERVERS: {
1212
OPEN: {
13-
serverArguments: ['--loadmodule /usr/lib/redis/modules/redisearch.so'],
13+
serverArguments: [],
1414
clientOptions: {
1515
modules: {
1616
ft: RediSearch

packages/test-utils/lib/dockers.ts

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ import { once } from 'node:events';
33
import { createClient } from '@redis/client/index';
44
import { setTimeout } from 'node:timers/promises';
55
// import { ClusterSlotsReply } from '@redis/client/dist/lib/commands/CLUSTER_SLOTS';
6-
import * as path from 'node:path';
76
import { promisify } from 'node:util';
87
import { exec } from 'node:child_process';
98
const execAsync = promisify(exec);
@@ -46,17 +45,10 @@ export interface RedisServerDocker {
4645
dockerId: string;
4746
}
4847

49-
// extra ".." cause it'll be in `./dist`
50-
const DOCKER_FODLER_PATH = path.join(__dirname, '../../docker');
51-
5248
async function spawnRedisServerDocker({ image, version }: RedisServerDockerConfig, serverArguments: Array<string>): Promise<RedisServerDocker> {
5349
const port = (await portIterator.next()).value,
5450
{ stdout, stderr } = await execAsync(
55-
'docker run -d --network host $(' +
56-
`docker build ${DOCKER_FODLER_PATH} -q ` +
57-
`--build-arg IMAGE=${image}:${version} ` +
58-
`--build-arg REDIS_ARGUMENTS="--save '' --port ${port.toString()} ${serverArguments.join(' ')}"` +
59-
')'
51+
`docker run -d --network host ${image}:${version} --port ${port.toString()} ${serverArguments.join(' ')}`
6052
);
6153

6254
if (!stdout) {

packages/time-series/lib/test-utils.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,15 @@ import TestUtils from '@redis/test-utils';
22
import TimeSeries from '.';
33

44
export default new TestUtils({
5-
dockerImageName: 'redislabs/redistimeseries',
6-
dockerImageVersionArgument: 'timeseries-version'
5+
dockerImageName: 'redis',
6+
dockerImageVersionArgument: 'timeseries-version',
7+
defaultDockerVersion: '8.0-M01'
78
});
89

910
export const GLOBAL = {
1011
SERVERS: {
1112
OPEN: {
12-
serverArguments: ['--loadmodule /usr/lib/redis/modules/redistimeseries.so'],
13+
serverArguments: [],
1314
clientOptions: {
1415
modules: {
1516
ts: TimeSeries

0 commit comments

Comments
 (0)