Skip to content

Commit cbe9437

Browse files
committed
#RI-4145 - fix api unit tests
1 parent 99a2ad8 commit cbe9437

File tree

3 files changed

+10
-2
lines changed

3 files changed

+10
-2
lines changed

redisinsight/api/src/__mocks__/databases.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { Database } from 'src/modules/database/models/database';
22
import { mockCaCertificate, mockClientCertificate } from 'src/__mocks__/certificates';
33
import { SentinelMaster } from 'src/modules/redis-sentinel/models/sentinel-master';
4-
import { ConnectionType, DatabaseEntity } from 'src/modules/database/entities/database.entity';
4+
import { Compressor, ConnectionType, DatabaseEntity } from 'src/modules/database/entities/database.entity';
55
import { EncryptionStrategy } from 'src/modules/encryption/models';
66
import { mockIORedisClient } from 'src/__mocks__/redis';
77
import { mockSentinelMasterDto } from 'src/__mocks__/redis-sentinel';
@@ -34,6 +34,7 @@ export const mockDatabase = Object.assign(new Database(), {
3434
connectionType: ConnectionType.STANDALONE,
3535
timeout: 30_000,
3636
new: false,
37+
compressor: Compressor.NONE,
3738
});
3839

3940
export const mockDatabaseEntity = Object.assign(new DatabaseEntity(), {

redisinsight/api/src/modules/database/database.analytics.spec.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,7 @@ describe('DatabaseAnalytics', () => {
104104
numberOfModules: 0,
105105
timeout: mockDatabaseWithTlsAuth.timeout / 1_000, // milliseconds to seconds
106106
databaseIndex: 0,
107+
useDecompression: mockDatabaseWithTlsAuth.compressor,
107108
...DEFAULT_REDIS_MODULES_SUMMARY,
108109
},
109110
);
@@ -134,6 +135,7 @@ describe('DatabaseAnalytics', () => {
134135
numberOfModules: 0,
135136
timeout: mockDatabaseWithTlsAuth.timeout / 1_000, // milliseconds to seconds
136137
databaseIndex: 0,
138+
useDecompression: mockDatabaseWithTlsAuth.compressor,
137139
...DEFAULT_REDIS_MODULES_SUMMARY,
138140
},
139141
);
@@ -166,6 +168,7 @@ describe('DatabaseAnalytics', () => {
166168
numberOfModules: 2,
167169
timeout: mockDatabaseWithTlsAuth.timeout / 1_000, // milliseconds to seconds
168170
databaseIndex: 0,
171+
useDecompression: mockDatabaseWithTlsAuth.compressor,
169172
...DEFAULT_REDIS_MODULES_SUMMARY,
170173
RediSearch: {
171174
loaded: true,
@@ -204,6 +207,7 @@ describe('DatabaseAnalytics', () => {
204207
numberOfModules: 2,
205208
timeout: mockDatabaseWithTlsAuth.timeout / 1_000, // milliseconds to seconds
206209
databaseIndex: 2,
210+
useDecompression: mockDatabaseWithTlsAuth.compressor,
207211
...DEFAULT_REDIS_MODULES_SUMMARY,
208212
RediSearch: {
209213
loaded: true,
@@ -240,6 +244,7 @@ describe('DatabaseAnalytics', () => {
240244
useSNI: 'enabled',
241245
useSSH: 'disabled',
242246
timeout: mockDatabaseWithTlsAuth.timeout / 1_000, // milliseconds to seconds
247+
useDecompression: mockDatabaseWithTlsAuth.compressor,
243248
previousValues: {
244249
connectionType: prev.connectionType,
245250
provider: prev.provider,
@@ -275,6 +280,7 @@ describe('DatabaseAnalytics', () => {
275280
useSNI: 'enabled',
276281
useSSH: 'disabled',
277282
timeout: mockDatabaseWithTlsAuth.timeout / 1_000, // milliseconds to seconds
283+
useDecompression: mockDatabaseWithTlsAuth.compressor,
278284
previousValues: {
279285
connectionType: prev.connectionType,
280286
provider: prev.provider,

redisinsight/api/src/modules/database/database.analytics.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ export class DatabaseAnalytics extends TelemetryBaseService {
6767
numberOfModules: instance.modules?.length || 0,
6868
timeout: instance.timeout / 1_000, // milliseconds to seconds
6969
databaseIndex: instance.db || 0,
70-
useCompressor: instance.compressor || null,
70+
useDecompression: instance.compressor || null,
7171
...modulesSummary,
7272
},
7373
);
@@ -101,6 +101,7 @@ export class DatabaseAnalytics extends TelemetryBaseService {
101101
useSNI: cur?.tlsServername ? 'enabled' : 'disabled',
102102
useSSH: cur?.ssh ? 'enabled' : 'disabled',
103103
timeout: cur?.timeout / 1_000, // milliseconds to seconds
104+
useDecompression: cur?.compressor || null,
104105
previousValues: {
105106
connectionType: prev.connectionType,
106107
provider: prev.provider,

0 commit comments

Comments
 (0)