Skip to content

Commit 582eed0

Browse files
[vs]: Fix trimming test cases: avoid config consolidation (#3939)
Signed-off-by: Nazarii Hnydyn <[email protected]> Co-authored-by: Sudharsan Dhamal Gopalarathnam <[email protected]>
1 parent 34b9f36 commit 582eed0

File tree

1 file changed

+23
-3
lines changed

1 file changed

+23
-3
lines changed

tests/test_trimming.py

Lines changed: 23 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -431,6 +431,20 @@ def genericConfig(self, switchData):
431431
qualifiers=attr_dict
432432
)
433433

434+
sai_attr_dict = {
435+
"SAI_SWITCH_ATTR_PACKET_TRIM_SIZE": "100",
436+
"SAI_SWITCH_ATTR_PACKET_TRIM_DSCP_RESOLUTION_MODE": SAI_DSCP_MODE_DICT["from-tc"],
437+
"SAI_SWITCH_ATTR_PACKET_TRIM_TC_VALUE": "1",
438+
"SAI_SWITCH_ATTR_PACKET_TRIM_QUEUE_RESOLUTION_MODE": SAI_QUEUE_MODE_DICT["static"],
439+
"SAI_SWITCH_ATTR_PACKET_TRIM_QUEUE_INDEX": "1"
440+
}
441+
442+
trimlogger.info("Validate trimming global")
443+
self.dvs_switch.verify_switch(
444+
sai_switch_id=switchId,
445+
sai_qualifiers=sai_attr_dict
446+
)
447+
434448
# Symmetric DSCP mode
435449

436450
attr_dict = {
@@ -444,13 +458,10 @@ def genericConfig(self, switchData):
444458
qualifiers=attr_dict
445459
)
446460

447-
# Validation
448-
449461
sai_attr_dict = {
450462
"SAI_SWITCH_ATTR_PACKET_TRIM_SIZE": "100",
451463
"SAI_SWITCH_ATTR_PACKET_TRIM_DSCP_RESOLUTION_MODE": SAI_DSCP_MODE_DICT["dscp-value"],
452464
"SAI_SWITCH_ATTR_PACKET_TRIM_DSCP_VALUE": "10",
453-
"SAI_SWITCH_ATTR_PACKET_TRIM_TC_VALUE": "1",
454465
"SAI_SWITCH_ATTR_PACKET_TRIM_QUEUE_RESOLUTION_MODE": SAI_QUEUE_MODE_DICT["static"],
455466
"SAI_SWITCH_ATTR_PACKET_TRIM_QUEUE_INDEX": "1"
456467
}
@@ -463,6 +474,15 @@ def genericConfig(self, switchData):
463474

464475
yield
465476

477+
sai_attr_dict = {
478+
"SAI_SWITCH_ATTR_PACKET_TRIM_SIZE": "100",
479+
"SAI_SWITCH_ATTR_PACKET_TRIM_DSCP_RESOLUTION_MODE": SAI_DSCP_MODE_DICT["dscp-value"],
480+
"SAI_SWITCH_ATTR_PACKET_TRIM_DSCP_VALUE": "10",
481+
"SAI_SWITCH_ATTR_PACKET_TRIM_TC_VALUE": "1",
482+
"SAI_SWITCH_ATTR_PACKET_TRIM_QUEUE_RESOLUTION_MODE": SAI_QUEUE_MODE_DICT["static"],
483+
"SAI_SWITCH_ATTR_PACKET_TRIM_QUEUE_INDEX": "1"
484+
}
485+
466486
trimlogger.info("Validate trimming global")
467487
self.dvs_switch.verify_switch(
468488
sai_switch_id=switchId,

0 commit comments

Comments
 (0)