RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2025-02-16 07:17:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

mgr/dashboard: fix linting in unittest

Browse Source 
Signed-off-by: Laura Flores <lflores@redhat.com>
This commit is contained in:
Laura Flores 2021-09-22 21:36:33 +00:00
parent f7d6642f5e
commit 9158cbe9da
1 changed files with 37 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

53
src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/telemetry/telemetry.component.spec.ts
View File

@ -170,33 +170,54 @@ describe('TelemetryComponent', () => {
expect(component).toBeTruthy();
});
it ('should only replace the ranges and values of a JSON object', () => {
var report = component.replacerTest({'ranges': [[null, -1], [0, 511], [512, 1023]],
'values': [[0,0,0], [0,0,0], [0,0,0]],
'other': [[0,0,0], [0,0,0], [0,0,0]]});
it('should only replace the ranges and values of a JSON object', () => {
let report = component.replacerTest({
ranges: [
[null, -1],
[0, 511],
[512, 1023]
],
values: [
[0, 0, 0],
[0, 0, 0],
[0, 0, 0]
],
other: [
[0, 0, 0],
[0, 0, 0],
[0, 0, 0]
]
});
report = JSON.parse(report);
// Ensure that the outer arrays have remained untouched by replacer
expect(Array.isArray(report['ranges']) && Array.isArray(report['values']) && Array.isArray(report['other'])).toBeTruthy()
expect(
Array.isArray(report['ranges']) &&
Array.isArray(report['values']) &&
Array.isArray(report['other'])
).toBeTruthy();
if (Array.isArray(report['ranges']) && Array.isArray(report['values']) && Array.isArray(report['other'])) {
var idx;
if (
Array.isArray(report['ranges']) &&
Array.isArray(report['values']) &&
Array.isArray(report['other'])
) {
let idx;
// Check that each range in 'ranges' was replaced by a string
for (idx=0; idx<report['ranges'].length; idx++) {
expect(typeof report['ranges'][idx] === 'string').toBeTruthy();
for (idx = 0; idx < report['ranges'].length; idx++) {
expect(typeof report['ranges'][idx] === 'string').toBeTruthy();
}
// Check that each value in 'values' was replaced by a string
for (idx=0; idx<report['values'].length; idx++) {
expect(typeof report['values'][idx] === 'string').toBeTruthy();
// Check that each value in 'values' was replaced by a string
for (idx = 0; idx < report['values'].length; idx++) {
expect(typeof report['values'][idx] === 'string').toBeTruthy();
}
// Check that each value in 'other' has remained untouched, as it is not a value or range
for (idx=0; idx<report['other'].length; idx++) {
expect(Array.isArray(report['other'][idx])).toBeTruthy();
// Check that each value in 'other' has remained untouched, as it is not a value or range
for (idx = 0; idx < report['other'].length; idx++) {
expect(Array.isArray(report['other'][idx])).toBeTruthy();
}
}
});