From ff940a3754be32ed9eedf381d0d2a8ca2aef1bd7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Zolt=C3=A1n=20Bedi?= Date: Thu, 13 Jun 2024 16:35:09 +0200 Subject: [PATCH] Use spy instead of new argument --- .../connectors/zabbix_api/zabbixAPIConnector.test.ts | 10 ++++++++-- .../zabbix/connectors/zabbix_api/zabbixAPIConnector.ts | 4 +--- 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/src/datasource/zabbix/connectors/zabbix_api/zabbixAPIConnector.test.ts b/src/datasource/zabbix/connectors/zabbix_api/zabbixAPIConnector.test.ts index 343df2b4..d2ff32ea 100644 --- a/src/datasource/zabbix/connectors/zabbix_api/zabbixAPIConnector.test.ts +++ b/src/datasource/zabbix/connectors/zabbix_api/zabbixAPIConnector.test.ts @@ -2,8 +2,13 @@ import { ZabbixAPIConnector } from './zabbixAPIConnector'; describe('Zabbix API connector', () => { describe('getProxies function', () => { + beforeAll(() => { + jest.spyOn(ZabbixAPIConnector.prototype, 'initVersion').mockResolvedValue(''); + }); + it('should send the name parameter to the request when version is 7 or greater for the getProxies', async () => { - const zabbixAPIConnector = new ZabbixAPIConnector(true, true, 123, '7.0.0'); + const zabbixAPIConnector = new ZabbixAPIConnector(true, true, 123); + zabbixAPIConnector.version = '7.0.0'; zabbixAPIConnector.request = jest.fn(); await zabbixAPIConnector.getProxies(); @@ -11,7 +16,8 @@ describe('Zabbix API connector', () => { }); it('should send the host parameter when version is less than 7.0.0', () => { - const zabbixAPIConnector = new ZabbixAPIConnector(true, true, 123, '6.0.0'); + const zabbixAPIConnector = new ZabbixAPIConnector(true, true, 123); + zabbixAPIConnector.version = '6.0.0'; zabbixAPIConnector.request = jest.fn(); zabbixAPIConnector.getProxies(); diff --git a/src/datasource/zabbix/connectors/zabbix_api/zabbixAPIConnector.ts b/src/datasource/zabbix/connectors/zabbix_api/zabbixAPIConnector.ts index 5ded01ba..845e08a4 100644 --- a/src/datasource/zabbix/connectors/zabbix_api/zabbixAPIConnector.ts +++ b/src/datasource/zabbix/connectors/zabbix_api/zabbixAPIConnector.ts @@ -28,7 +28,7 @@ export class ZabbixAPIConnector { getVersionPromise: Promise; datasourceId: number; - constructor(basicAuth: any, withCredentials: boolean, datasourceId: number, zabbixVersion?: string) { + constructor(basicAuth: any, withCredentials: boolean, datasourceId: number) { this.datasourceId = datasourceId; this.backendAPIUrl = `/api/datasources/${this.datasourceId}/resources/zabbix-api`; @@ -37,8 +37,6 @@ export class ZabbixAPIConnector { withCredentials: withCredentials, }; - this.version = zabbixVersion || null; - this.getTrend = this.getTrend_ZBXNEXT1193; //getTrend = getTrend_30;