diff options
author | Ilya Mashchenko <ilya@netdata.cloud> | 2022-02-22 11:44:14 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-02-22 11:44:14 +0300 |
commit | e4ed564832e0ee4c67b4ec518796168cf0743200 (patch) | |
tree | 0a8ab99d32aa3b8c64151dbd22ad628cb5c94885 /tests | |
parent | 20dfab81927abb4f99c99416e9a2c551d650f1b9 (diff) |
remove deprecated node.d modules (#12047)
Diffstat (limited to 'tests')
-rw-r--r-- | tests/Makefile.am | 10 | ||||
-rw-r--r-- | tests/README.md | 148 | ||||
-rw-r--r-- | tests/node.d/fronius.chart.spec.js | 162 | ||||
-rw-r--r-- | tests/node.d/fronius.parse.spec.js | 333 | ||||
-rw-r--r-- | tests/node.d/fronius.process.spec.js | 75 | ||||
-rw-r--r-- | tests/node.d/fronius.validation.spec.js | 155 | ||||
-rw-r--r-- | tests/web/easypiechart.chart.spec.js | 39 | ||||
-rw-r--r-- | tests/web/easypiechart.percentage.spec.js | 142 | ||||
-rw-r--r-- | tests/web/fixtures/easypiechart.chart.fixture1.html | 6 | ||||
-rw-r--r-- | tests/web/karma.conf.js | 110 | ||||
-rw-r--r-- | tests/web/lib/jasmine-jquery.js | 841 |
11 files changed, 0 insertions, 2021 deletions
diff --git a/tests/Makefile.am b/tests/Makefile.am index 29fe7ee60f..eb8b5e26e8 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -15,16 +15,6 @@ include $(top_srcdir)/build/subst.inc SUFFIXES = .in dist_noinst_DATA = \ - README.md \ - web/lib/jasmine-jquery.js \ - web/easypiechart.chart.spec.js \ - web/easypiechart.percentage.spec.js \ - web/karma.conf.js \ - web/fixtures/easypiechart.chart.fixture1.html \ - node.d/fronius.chart.spec.js \ - node.d/fronius.parse.spec.js \ - node.d/fronius.process.spec.js \ - node.d/fronius.validation.spec.js \ health_mgmtapi/health-cmdapi-test.sh.in \ acls/acl.sh.in \ urls/request.sh.in \ diff --git a/tests/README.md b/tests/README.md deleted file mode 100644 index 5f51e2dfdf..0000000000 --- a/tests/README.md +++ /dev/null @@ -1,148 +0,0 @@ -<!-- -title: "Testing" -custom_edit_url: https://github.com/netdata/netdata/edit/master/tests/README.md ---> - -# Testing - -This readme is a manual on how to get started with unit testing on javascript and nodejs - -Original author: BrainDoctor (github), July 2017 - -## Installation - -Tested on Linux Mint 18.2 Sara (Ubuntu/debian derivative) - -Make sure you are the user who is developer (permissions, except sudo ofc) - -```sh -sudo apt-get install nodejs npm chromium-browser - -cd /path/to/your/netdata -npm install -``` - -That should install the necessary node modules. - -Other browsers work too (Chrome, Firefox). However, only the Chromium Browser 59 has been tested for headless unit testing. - -### Versions - -The commands above leave me with the following versions (July 2017): - -- nodejs: v4.2.6 -- npm: 3.5.2 -- chromium-browser: 59.0.3071.109 -- WebStorm (optional): 2017.1.4 - -## Configuration - -### NPM - -The dependencies are installed in `netdata/package.json`. If you install a new NPM module, it gets added here. Future developers just need to execute `npm install` and every dep gets added automatically. - -### Karma - -Karma configuration is in `tests/web/karma.conf.js`. Documentation is provided via comments. - -### WebStorm - -If you use the JetBrains WebStorm IDE, you can integrate the karma runtime. - -#### for Karma (Client side testing) - -Headless Chromium: - -1. Run > Edit Configurations -2. "+" > Karma -3. - Name: Karma Headless Chromium - - Configuration file: /path/to/your/netdata/tests/web/karma.conf.js - - Browsers to start: ChromiumHeadless - - Node interpreter: /usr/bin/nodejs (MUST be absolute, NVM works too) - - Karma package: /path/to/your/netdata/node_modules/karma - -GUI Chromium is similar: - -1. Run > Edit Configurations -2. "+" > Karma -3. - Name: Karma Chromium - - Configuration file: /path/to/your/netdata/tests/web/karma.conf.js - - Browsers to start: Chromium - - Node interpreter: /usr/bin/nodejs (MUST be absolute, NVM works too) - - Karma package: /path/to/your/netdata/node_modules/karma - -You may add other browsers too (comma separated). With the "Browsers to start" field you can override any settings in karma.conf.js. - -Also it is recommended to install WebStorm IDE Extension/Addon to Chrome/Chromium for awesome debugging. - -#### for node.d plugins (nodejs) - -1. Run > Edit Configurations -2. "+" > Node.js -3. - Name: Node.d plugins - - Node interpreter: /usr/bin/nodejs (MUST be absolute, NVM works too) - - JavaScript file: node_modules/jasmine-node/bin/jasmine-node - - Application parameters: --captureExceptions tests/node.d - -## Running - -### In WebStorm - -#### Karma - -Just run the configured run configurations and they produce nice test trees: - -![karma_run_2](https://user-images.githubusercontent.com/12159026/28277789-559149f6-6b1b-11e7-9cc7-a81d81d12c35.png) - -#### node.js - -Debugging is awesome too! -![node_debug](https://user-images.githubusercontent.com/12159026/28277879-8beee5ee-6b1b-11e7-9356-3156956f2282.png) - -### From CLI - -#### Karma - -```sh -cd /path/to/your/netdata - -nodejs ./node_modules/karma/bin/karma start tests/web/karma.conf.js --single-run=true --browsers=ChromiumHeadless -``` - -will start the karma server, start chromium in headless mode and exit. - -If a test fails, it produces even a stack trace: -![karma_run_1](https://user-images.githubusercontent.com/12159026/28277754-3682bebe-6b1b-11e7-8b7e-66b23d87177d.png) - -#### Node.d plugins - -```sh -cd /path/to/your/netdata - -nodejs node_modules/jasmine-node/bin/jasmine-node --captureExceptions tests/node.d -``` - -will run the tests in `tests/node.d` and produce a stacktrace too on error: -![node_run](https://user-images.githubusercontent.com/12159026/28277812-65bb69b0-6b1b-11e7-8500-bcdbb3436574.png) - -### Coverage - -#### Karma - -A nice HTML is produced from Karma which shows which code paths were executed. It is located somewhere in `/path/to/your/netdata/coverage/` - -![coverage_2](https://user-images.githubusercontent.com/12159026/28277719-142146c4-6b1b-11e7-9992-3e88dee2efd2.png) -and -![coverage_1](https://user-images.githubusercontent.com/12159026/28277687-fa93e360-6b1a-11e7-995f-cbb4c5d012a7.png) - -#### Node.d - -Apparently, jasmine-node can produce a junit report with the `--junitreport` flag. But that output was not very useful. Maybe it's configurable? - -### CI - -The karma and node.d runners can be integrated in Travis (AFAIK), but that is outside my ability. - -Note: Karma is for browser-testing. On a build server, no GUI or browser might by available, unless browsers support headless mode. - - diff --git a/tests/node.d/fronius.chart.spec.js b/tests/node.d/fronius.chart.spec.js deleted file mode 100644 index 92e88d2eeb..0000000000 --- a/tests/node.d/fronius.chart.spec.js +++ /dev/null @@ -1,162 +0,0 @@ -"use strict"; -// SPDX-License-Identifier: GPL-3.0-or-later - -var netdata = require("../../node.d/node_modules/netdata"); -// remember: subject will be a singleton! -var subject = require("../../node.d/fronius.node"); - -var service = netdata.service({ - name: "chart", - module: this -}); - -describe("fronius chart creation", function () { - - var chartPrefix = "fronius_chart."; - - beforeAll(function () { - // change this to enable debug log - netdata.options.DEBUG = false; - }); - - afterAll(function () { - deleteProperties(subject.charts) - }); - - it("should return a basic chart dimension", function () { - var result = subject.createBasicDimension("id", "name", 2); - - expect(result.divisor).toBe(2); - expect(result.id).toBe("id"); - expect(result.algorithm).toEqual(netdata.chartAlgorithms.absolute); - expect(result.multiplier).toBe(1); - }); - - it("should return the power chart definition", function () { - var suffix = "power"; - var result = subject.getSitePowerChart(service, suffix); - - expect(result.id).toBe(chartPrefix + suffix); - expect(result.units).toBe("W"); - expect(result.type).toBe(netdata.chartTypes.area); - expect(result.family).toBe("power"); - expect(result.context).toBe("fronius.power"); - expect(result.dimensions[subject.powerGridId].name).toBe("grid"); - expect(result.dimensions[subject.powerPvId].name).toBe("photovoltaics"); - expect(result.dimensions[subject.powerAccuId].name).toBe("accumulator"); - expect(Object.keys(result.dimensions).length).toBe(3); - }); - - it("should return the consumption chart definition", function () { - var suffix = "Load"; - var result = subject.getSiteConsumptionChart(service, suffix); - - expect(result.id).toBe(chartPrefix + suffix); - expect(result.units).toBe("W"); - expect(result.type).toBe(netdata.chartTypes.area); - expect(result.family).toBe("consumption"); - expect(result.context).toBe("fronius.consumption"); - expect(Object.keys(result.dimensions).length).toBe(1); - expect(result.dimensions[subject.consumptionLoadId].name).toBe("load"); - }); - - it("should return the autonomy chart definition", function () { - var suffix = "Autonomy"; - var result = subject.getSiteAutonomyChart(service, suffix); - - expect(result.id).toBe(chartPrefix + suffix); - expect(result.units).toBe("%"); - expect(result.type).toBe(netdata.chartTypes.area); - expect(result.family).toBe("autonomy"); - expect(result.context).toBe("fronius.autonomy"); - expect(Object.keys(result.dimensions).length).toBe(3); - expect(result.dimensions[subject.autonomyId].name).toBe("autonomy"); - expect(result.dimensions[subject.consumptionSelfId].name).toBe("self_consumption"); - }); - - it("should return the energy today chart definition", function () { - var suffix = "Energy today"; - var result = subject.getSiteEnergyTodayChart(service, suffix); - - expect(result.id).toBe(chartPrefix + suffix); - expect(result.units).toBe("kWh"); - expect(result.type).toBe(netdata.chartTypes.area); - expect(result.family).toBe("energy"); - expect(result.context).toBe("fronius.energy.today"); - expect(Object.keys(result.dimensions).length).toBe(1); - expect(result.dimensions[subject.energyTodayId].name).toBe("today"); - }); - - it("should return the energy year chart definition", function () { - var suffix = "Energy year"; - var result = subject.getSiteEnergyYearChart(service, suffix); - - expect(result.id).toBe(chartPrefix + suffix); - expect(result.units).toBe("kWh"); - expect(result.type).toBe(netdata.chartTypes.area); - expect(result.family).toBe("energy"); - expect(result.context).toBe("fronius.energy.year"); - expect(Object.keys(result.dimensions).length).toBe(1); - expect(result.dimensions[subject.energyYearId].name).toBe("year"); - }); - - it("should return the inverter chart definition with a single numerical inverter", function () { - var inverters = { - "1": {} - }; - var suffix = "numerical"; - var result = subject.getInverterPowerChart(service, suffix, inverters); - - expect(result.id).toBe(chartPrefix + suffix); - expect(result.units).toBe("W"); - expect(result.type).toBe(netdata.chartTypes.stacked); - expect(result.family).toBe("inverters"); - expect(result.context).toBe("fronius.inverter.output"); - expect(Object.keys(result.dimensions).length).toBe(1); - expect(result.dimensions["1"].name).toBe("inverter_1"); - }); - - it("should return the inverter chart definition with a single alphabetical inverter", function () { - var key = "Cellar"; - var inverters = { - "Cellar": {} - }; - var suffix = "alphabetical"; - var result = subject.getInverterPowerChart(service, suffix, inverters); - - expect(result.id).toBe(chartPrefix + suffix); - expect(result.units).toBe("W"); - expect(result.type).toBe(netdata.chartTypes.stacked); - expect(result.family).toBe("inverters"); - expect(result.context).toBe("fronius.inverter.output"); - expect(Object.keys(result.dimensions).length).toBe(1); - expect(result.dimensions[key].name).toBe(key); - }); - - it("should return the inverter chart definition with multiple alphanumerical inverter", function () { - var alpha = "Cellar"; - var numerical = 1; - var inverters = { - "Cellar": {}, - "1": {} - }; - var suffix = "alphanumerical"; - var result = subject.getInverterPowerChart(service, suffix, inverters); - - expect(result.id).toBe(chartPrefix + suffix); - expect(result.units).toBe("W"); - expect(result.type).toBe(netdata.chartTypes.stacked); - expect(result.family).toBe("inverters"); - expect(result.context).toBe("fronius.inverter.output"); - expect(Object.keys(result.dimensions).length).toBe(2); - expect(result.dimensions[alpha].name).toBe(alpha); - expect(result.dimensions[numerical].name).toBe("inverter_" + numerical); - }); - - it("should return the same chart definition on second call for lazy loading", function () { - var first = subject.getSitePowerChart(service, "id"); - var second = subject.getSitePowerChart(service, "id"); - - expect(first).toBe(second); - }); -}); diff --git a/tests/node.d/fronius.parse.spec.js b/tests/node.d/fronius.parse.spec.js deleted file mode 100644 index 01fb38666e..0000000000 --- a/tests/node.d/fronius.parse.spec.js +++ /dev/null @@ -1,333 +0,0 @@ -"use strict"; -// SPDX-License-Identifier: GPL-3.0-or-later - -var netdata = require("../../node.d/node_modules/netdata"); -// remember: subject will be a singleton! -var subject = require("../../node.d/fronius.node"); - -var service = netdata.service({ - name: "parse", - module: this -}); - -var root = { - "Body": { - "Data": { - "Site": {}, - "Inverters": {} - } - } -}; - -describe("fronius parsing for power chart", function () { - - var site = root.Body.Data.Site; - - afterEach(function () { - deleteProperties(site); - }); - - it("should return 3000 for P_Grid when rounded", function () { - site.P_Grid = 2999.501; - var result = subject.parsePowerChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.powerGridId); - expect(result.value).toBe(3000); - }); - - it("should return -3000 for P_Grid", function () { - site.P_Grid = -3000; - var result = subject.parsePowerChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.powerGridId); - expect(result.value).toBe(-3000); - }); - - it("should return 0 for P_Grid if it is null", function () { - site.P_Grid = null; - var result = subject.parsePowerChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.powerGridId); - expect(result.value).toBe(0); - }); - - it("should return 0 for P_Grid if it is zero", function () { - site.P_Grid = 0; - var result = subject.parsePowerChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.powerGridId); - expect(result.value).toBe(0); - }); - - it("should return -100 for P_Akku", function () { - // it is unclear whether negative values are possible for p_akku (couldn't test, nor any API docs found). - site.P_Akku = -100; - var result = subject.parsePowerChart(service, site).dimensions[2]; - - expect(result.name).toBe(subject.powerAccuId); - expect(result.value).toBe(-100); - }); - - it("should return 0 for P_Akku if it is null", function () { - site.P_Akku = null; - var result = subject.parsePowerChart(service, site).dimensions[2]; - - expect(result.name).toBe(subject.powerAccuId); - expect(result.value).toBe(0); - }); - - it("should return 0 for P_Akku if it is zero", function () { - site.P_Akku = 0; - var result = subject.parsePowerChart(service, site).dimensions[2]; - - expect(result.name).toBe(subject.powerAccuId); - expect(result.value).toBe(0); - }); - - it("should return 100 for P_PV", function () { - site.P_PV = 100; - var result = subject.parsePowerChart(service, site).dimensions[1]; - - expect(result.name).toBe(subject.powerPvId); - expect(result.value).toBe(100); - }); - - it("should return 0 for P_PV if it is zero", function () { - site.P_PV = 0; - var result = subject.parsePowerChart(service, site).dimensions[1]; - - expect(result.name).toBe(subject.powerPvId); - expect(result.value).toBe(0); - }); - - it("should return 0 for P_PV if it is null", function () { - site.P_PV = null; - var result = subject.parsePowerChart(service, site).dimensions[1]; - - expect(result.name).toBe(subject.powerPvId); - expect(result.value).toBe(0); - }); - - it("should return 0 for P_PV if it is negative", function () { - // solar panels shouldn't consume anything, only produce. - site.P_PV = -1; - var result = subject.parsePowerChart(service, site).dimensions[1]; - - expect(result.name).toBe(subject.powerPvId); - expect(result.value).toBe(0); - }); - -}); - -describe("fronius parsing for consumption", function () { - - var site = root.Body.Data.Site; - - afterEach(function () { - deleteProperties(site); - }); - - it("should return 1000 for P_Load when rounded", function () { - site.P_Load = 1000.499; - var result = subject.parseConsumptionChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.consumptionLoadId); - expect(result.value).toBe(1000); - }); - - it("should return absolute value for P_Load when negative", function () { - /* - with firmware 3.7.4 it is sometimes possible that negative values are returned for P_Load, - which makes absolutely no sense. There is always a device that consumes some electricity around the clock. - Best we can do is to make it a positive value, since 0 also doesn't make much sense. - This "workaround" seems to work, as there couldn't be any strange peaks observed during long-time testing. - */ - site.P_Load = -50; - var result = subject.parseConsumptionChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.consumptionLoadId); - expect(result.value).toBe(50); - }); - - it("should return 0 for P_Load if it is null", function () { - site.P_Load = null; - var result = subject.parseConsumptionChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.consumptionLoadId); - expect(result.value).toBe(0); - }); - - it("should return 0 for P_Load if it is zero", function () { - site.P_Load = 0; - var result = subject.parseConsumptionChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.consumptionLoadId); - expect(result.value).toBe(0); - }); - -}); - -describe("fronius parsing for autonomy", function () { - - var site = root.Body.Data.Site; - - afterEach(function () { - deleteProperties(site); - }); - - it("should return 100 for rel_Autonomy", function () { - site.rel_Autonomy = 100; - var result = subject.parseAutonomyChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.autonomyId); - expect(result.value).toBe(100); - }); - - it("should return 0 for rel_Autonomy if it is zero", function () { - site.rel_Autonomy = 0; - var result = subject.parseAutonomyChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.autonomyId); - expect(result.value).toBe(0); - }); - - it("should return 0 for rel_Autonomy if it is null", function () { - site.rel_Autonomy = null; - var result = subject.parseAutonomyChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.autonomyId); - expect(result.value).toBe(0); - }); - - it("should return 20 for rel_Autonomy if it is 20", function () { - site.rel_Autonomy = 20.1; - var result = subject.parseAutonomyChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.autonomyId); - expect(result.value).toBe(20); - }); - - it("should return 20 for rel_SelfConsumption if it is 19.5", function () { - site.rel_SelfConsumption = 19.5; - var result = subject.parseAutonomyChart(service, site).dimensions[1]; - - expect(result.name).toBe(subject.consumptionSelfId); - expect(result.value).toBe(20); - }); - - it("should return 100 for rel_SelfConsumption if it is null", function () { - /* - During testing it could be observed that the API is delivering null if the solar panels - do not produce enough energy to supply the local load. But in this case it should be 100, since all - the produced energy is directly consumed. - */ - site.rel_SelfConsumption = null; - var result = subject.parseAutonomyChart(service, site).dimensions[1]; - - expect(result.name).toBe(subject.consumptionSelfId); - expect(result.value).toBe(100); - }); - - it("should return 0 for rel_SelfConsumption if it is zero", function () { - site.rel_SelfConsumption = 0; - var result = subject.parseAutonomyChart(service, site).dimensions[1]; - - expect(result.name).toBe(subject.consumptionSelfId); - expect(result.value).toBe(0); - }); - - it("should return 0 for solarConsumption if PV is null", function () { - site.P_PV = null; - site.P_Load = -1000; - var result = subject.parseAutonomyChart(service, site).dimensions[2]; - - expect(result.name).toBe(subject.solarConsumptionId); - expect(result.value).toBe(0); - }); - - it("should return 100 for solarConsumption if Load is higher than solar power", function () { - site.P_PV = 500; - site.P_Load = -1500; - var result = subject.parseAutonomyChart(service, site).dimensions[2]; - - expect(result.name).toBe(subject.solarConsumptionId); - expect(result.value).toBe(100); - }); - - it("should return 50 for solarConsumption if Load is half than solar power", function () { - site.P_PV = 3000; - site.P_Load = -1500; - var result = subject.parseAutonomyChart(service, site).dimensions[2]; - - expect(result.name).toBe(subject.solarConsumptionId); - expect(result.value).toBe(50); - }); -}); - -describe("fronius parsing for energy", function () { - - var site = root.Body.Data.Site; - - afterEach(function () { - deleteProperties(site); - }); - - it("should return 10000 for E_Day", function () { - site.E_Day = 10000; - var result = subject.parseEnergyTodayChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.energyTodayId); - expect(result.value).toBe(10000); - }); - - it("should return 0 for E_Day if it is negative", function () { - /* - The solar panels can't produce negative energy, really. It would be a fault of the API. - */ - site.E_Day = -0.4; - var result = subject.parseEnergyTodayChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.energyTodayId); - expect(result.value).toBe(0); - }); - - it("should return 100'000 for E_Year", function () { - site.E_Year = 100000.4; - var result = subject.parseEnergyYearChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.energyYearId); - expect(result.value).toBe(100000); - }); - - it("should return 0 for E_Year if it is negative", function () { - /* - A return value of 0 only makes sense in the silvester night anyway, when the counter is being reset. - A negative value is a fault from the API though. It wouldn't make sense. - */ - site.E_Year = -1; - var result = subject.parseEnergyYearChart(service, site).dimensions[0]; - - expect(result.name).toBe(subject.energyYearId); - expect(result.value).toBe(0); - }); -}); - -describe("fronius parsing for inverters", function () { - - var inverters = root.Body.Data.Inverters; - - afterEach(function () { - deleteProperties(inverters); - }); - - it("should return 1000 for P for inverter with name", function () { - inverters["cellar"] = { - P: 1000 - }; - var result = subject.parseInverterChart(service, inverters).dimensions[0]; - - expect(result.name).toBe("cellar"); - expect(result.value).toBe(1000); - }); - -}); diff --git a/tests/node.d/fronius.process.spec.js b/tests/node.d/fronius.process.spec.js deleted file mode 100644 index 141fa8a6aa..0000000000 --- a/tests/node.d/fronius.process.spec.js +++ /dev/null @@ -1,75 +0,0 @@ -"use strict"; -// SPDX-License-Identifier: GPL-3.0-or-later - -var netdata = require("../../node.d/node_modules/netdata"); -// remember: subject will be a singleton! -var subject = require("../../node.d/fronius.node"); - -var service = netdata.service({ - name: "process", - module: this -}); - -var exampleResponse = { - "Body": { - "Data": { - "Site": { - "Mode": "meter", - "P_Grid": -3430.729923, - "P_Load": -910.270077, - "P_Akku": null, - "P_PV": 4341, - "rel_SelfConsumption": 20.969133, - "rel_Autonomy": 100, - "E_Day": 57230, - "E_Year": 6425915.5, - "E_Total": 15388710, - "Meter_Location": "grid" - }, - "Inverters": { - "1": { - "DT": 123, - "P": 4341, - "E_Day": 57230, - "E_Year": 6425915.5, - "E_Total": 15388710 - } - } - } - } -}; - -describe("fronius main processing", function () { - - beforeAll(function () { - // change this to enable debug log - netdata.options.DEBUG = false; - }); - - beforeEach(function () { - deleteProperties(subject.charts); - }); - - it("should send parsed values to netdata", function () { - netdata.send = jasmine.createSpy("send"); - - subject.processResponse(service, exampleResponse); - - expect(netdata.send.calls.count()).toBe(6); - - // check if some parsed values were sent. - var powerChart = netdata.send.calls.argsFor(5)[0]; - - expect(powerChart).toContain("SET p_grid = -3431"); - expect(powerChart).toContain("SET p_pv = 4341"); - - var inverterChart = netdata.send.calls.argsFor(0)[0]; - - expect(inverterChart).toContain("SET 1 = 4341"); - - var autonomyChart = netdata.send.calls.argsFor(3)[0]; - expect(autonomyChart).toContain("SET rel_selfconsumption = 21"); - }); - - -}); diff --git a/tests/node.d/fronius.validation.spec.js b/tests/node.d/fronius.validation.spec.js deleted file mode 100644 index b7938d52f0..0000000000 --- a/tests/node.d/fronius.validation.spec.js +++ /dev/null @@ -1,155 +0,0 @@ -"use strict"; -// SPDX-License-Identifier: GPL-3.0-or-later - -var netdata = require("../../node.d/node_modules/netdata"); -// remember: subject will be a singleton! -var subject = require("../../node.d/fronius.node"); - -var service = netdata.service({ - name: "validation", - module: this -}); - -describe("fronius response validation", function () { - - it("should do nothing if response is null", function () { - netdata.send = jasmine.createSpy("send"); - - subject.processResponse(service, null); - var result = netdata.send.calls.count(); - - expect(result).toBe(0); - }); - - it("should return null if response is null", function () { - var result = subject.convertToJson(null); - - expect(result).toBeNull(); - }); - - it("should return null and log error if response cannot be parsed", function () { - netdata.error = jasmine.createSpy("error"); - - // trailing commas are enough to create syntax exceptions - var result = subject.convertToJson("{name,}"); - - expect(result).toBeNull(); - expect(netdata.error.calls.count()).toBe(1); - }); - - it("should return true if response is valid", function () { - var result = subject.isResponseValid({ - "Body": { - "Data": { - "Site": { - "Mode": "meter" - }, - "Inverters": { - "1": {} - } - } - } - }); - - expect(result).toBeTruthy(); - }); - - it("should return false if response is missing data", function () { - var result = subject.isResponseValid({ - "Body": {} - }); - - expect(result).toBeFalsy(); - }); - - it("should return false if response is missing inverter", function () { - var result = subject.isResponseValid({ - "Body": { - "Data": { - "Site": {} - } - } - }); - - expect(result).toBeFalsy(); - }); - - it("should return false if response is missing inverter", function () { - var result = subject.isResponseValid({ - "Body": { - "Data": { - "Inverters": {} - } - } - }); - - expect(result).toBeFalsy(); - }); - -}); - -describe("fronius configuration validation", function () { - - it("should return 0 if there are no servers configured", function () { - var result = subject.configure({}); - - expect(result).toBe(0); - }); - - it("should return 0 if the servers array is empty", function () { - var result = subject.configure({ - "servers": [] - }); - - expect(result).toBe(0); - }); - - it("should return 0 if there is one server configured incorrectly", function () { - var result = subject.configure({ - "servers": [{}] - }); - - expect(result).toBe(0); - }); - - it("should return 1 if there is one server configured", function () { - subject.serviceExecute = jasmine.createSpy("serviceExecute"); - var name = "solar1"; - var result = subject.configure({ - "servers": [{ - "name": name, - "api_path": "/api/", - "hostname": "solar1.local" - }] - }); - - expect(result).toBe(1); - expect(subject.serviceExecute).toHaveBeenCalledWith(name, "solar1.local/api/", 5); - }); - - it("should return 2 if there are two servers configured", function () { - subject.serviceExecute = jasmine.createSpy("serviceExecute"); - var name1 = "solar 1"; - var name2 = "solar 2"; - var result = subject.configure({ - "servers": [ - { - "name": name1, - "api_path": "/", - "hostname": "solar1.local" - }, - { - "name": name2, - |