Skip to content

Commit bf36292

Browse files
fix unit tests
1 parent dc8ef5a commit bf36292

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

redisinsight/api/src/modules/ai/query/ai-query.service.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ describe('AiQueryService', () => {
5757
clientSocket.disconnect();
5858
serverSocket.disconnect();
5959
wsServer.close();
60-
httpServer?.stop();
60+
httpServer?.stop?.();
6161
});
6262

6363
beforeEach(async () => {

redisinsight/api/src/modules/autodiscovery/autodiscovery.service.spec.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ import {
1313
import { SettingsService } from 'src/modules/settings/settings.service';
1414
import { AutodiscoveryService } from 'src/modules/autodiscovery/autodiscovery.service';
1515
import { DatabaseService } from 'src/modules/database/database.service';
16-
import { mocked } from 'ts-jest/utils';
1716
import config, { Config } from 'src/utils/config';
1817
import { RedisClientFactory } from 'src/modules/redis/redis.client.factory';
1918
import { ConstantsProvider } from 'src/modules/constants/providers/constants.provider';
@@ -84,7 +83,7 @@ describe('AutodiscoveryService', () => {
8483
databaseService = module.get(DatabaseService);
8584
redisClientFactory = module.get(RedisClientFactory);
8685

87-
mocked(utils.convertRedisInfoReplyToObject).mockReturnValue({
86+
(utils.convertRedisInfoReplyToObject as jest.Mock).mockReturnValue({
8887
server: {
8988
redis_mode: 'standalone',
9089
},
@@ -155,7 +154,7 @@ describe('AutodiscoveryService', () => {
155154
let addRedisDatabaseSpy;
156155

157156
beforeEach(async () => {
158-
mocked(getAvailableEndpoints).mockResolvedValue([]);
157+
(getAvailableEndpoints as jest.Mock).mockResolvedValue([]);
159158
addRedisDatabaseSpy = jest.spyOn(service as any, 'addRedisDatabase');
160159
addRedisDatabaseSpy.mockResolvedValue(null);
161160
});
@@ -167,7 +166,8 @@ describe('AutodiscoveryService', () => {
167166
});
168167

169168
it('should should call addRedisDatabase 2 times', async () => {
170-
mocked(getAvailableEndpoints).mockResolvedValueOnce([mockAutodiscoveryEndpoint, mockAutodiscoveryEndpoint]);
169+
(getAvailableEndpoints as jest.Mock)
170+
.mockResolvedValueOnce([mockAutodiscoveryEndpoint, mockAutodiscoveryEndpoint]);
171171
await service['discoverDatabases'](mockSessionMetadata);
172172

173173
expect(addRedisDatabaseSpy).toHaveBeenCalledTimes(2);
@@ -193,7 +193,7 @@ describe('AutodiscoveryService', () => {
193193
});
194194

195195
it('should not create database if redis_mode is not standalone', async () => {
196-
mocked(utils.convertRedisInfoReplyToObject).mockReturnValueOnce({
196+
(utils.convertRedisInfoReplyToObject as jest.Mock).mockReturnValueOnce({
197197
server: {
198198
redis_mode: 'cluster',
199199
},

0 commit comments

Comments
 (0)