Skip to content

Commit cbfdfa8

Browse files
committed
libyang3: high frequency telemetry test fix
Its not entirely clear what this test was attempting to validate. Its possible they wanted to ensure the lists weren't empty, but the yang model itself does not have that restriction. It would be out of scope for this change to modify an invalid yang model. The test was passing a string to a list which is invalid, so just modified the test to validate both object_names and object_counters will error out if a string is provided.
1 parent 896fdad commit cbfdfa8

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/sonic-yang-models/tests/yang_model_tests/tests_config/high_frequency_telemetry.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -256,7 +256,7 @@
256256
"profile_name": "invalid_dummy",
257257
"group_name": "BUFFER_POOL",
258258
"object_names": "",
259-
"object_counters": ""
259+
"object_counters": []
260260
}
261261
]
262262
}
@@ -279,7 +279,7 @@
279279
"profile_name": "invalid_dummy",
280280
"group_name": "PORT",
281281
"object_names": [],
282-
"object_counters": []
282+
"object_counters": ""
283283
}
284284
]
285285
}

0 commit comments

Comments
 (0)