Skip to content

Commit 675bc81

Browse files
Add set speed tests for handle_port_config_change
Signed-off-by: Pavan Naregundi <[email protected]>
1 parent 16aaa0f commit 675bc81

File tree

1 file changed

+21
-1
lines changed

1 file changed

+21
-1
lines changed

sonic-xcvrd/tests/test_xcvrd.py

Lines changed: 21 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1792,7 +1792,8 @@ def handle_port_change_event(self, port_event):
17921792
def test_handle_port_config_change(self, mock_select, mock_sub_table):
17931793
mock_selectable = MagicMock()
17941794
mock_selectable.pop = MagicMock(
1795-
side_effect=[('Ethernet0', swsscommon.SET_COMMAND, (('index', '1'), )), (None, None, None)])
1795+
side_effect=[('Ethernet0', swsscommon.SET_COMMAND, (('index', '1'), ('speed', '200000'))),
1796+
(None, None, None)])
17961797
mock_select.return_value = (swsscommon.Select.OBJECT, mock_selectable)
17971798
mock_sub_table.return_value = mock_selectable
17981799

@@ -1809,6 +1810,25 @@ def test_handle_port_config_change(self, mock_select, mock_sub_table):
18091810
assert port_mapping.get_physical_to_logical(1) == ['Ethernet0']
18101811
assert port_mapping.get_logical_to_physical('Ethernet0') == [1]
18111812

1813+
mock_selectable.pop = MagicMock(
1814+
side_effect=[('Ethernet0', swsscommon.SET_COMMAND, (('index', '1'), ('speed', '100000'))),
1815+
(None, None, None)])
1816+
handle_port_config_change(sel, asic_context, stop_event, port_mapping,
1817+
logger, port_mapping.handle_port_change_event)
1818+
assert port_mapping.logical_port_list.count('Ethernet0')
1819+
assert port_mapping.get_asic_id_for_logical_port('Ethernet0') == 0
1820+
assert port_mapping.get_physical_to_logical(1) == ['Ethernet0']
1821+
assert port_mapping.get_logical_to_physical('Ethernet0') == [1]
1822+
expected_cache = {
1823+
('Ethernet0'): {
1824+
'port_name': 'Ethernet0',
1825+
'index': '1',
1826+
'asic_id': 0,
1827+
'speed': '100000',
1828+
}
1829+
}
1830+
assert port_mapping.port_event_cache == expected_cache
1831+
18121832
mock_selectable.pop = MagicMock(
18131833
side_effect=[('Ethernet0', swsscommon.DEL_COMMAND, (('index', '1'), )), (None, None, None)])
18141834
handle_port_config_change(sel, asic_context, stop_event, port_mapping,

0 commit comments

Comments
 (0)