Skip to content

Commit 536c615

Browse files
arpit-nexthopclaude
arpit-nexthop
and
claude
committed
Fix CMIS task_worker tests for new should_wait_for_dp_settle call
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 69c0d8c commit 536c615

1 file changed

Lines changed: 50 additions & 0 deletions

File tree

‎sonic-xcvrd/tests/test_xcvrd.py‎

Lines changed: 50 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4280,6 +4280,16 @@ def test_CmisManagerTask_task_worker(self, mock_chassis, mock_get_status_sw_tbl)
42804280
'DP7State': 'DataPathDeactivated',
42814281
'DP8State': 'DataPathDeactivated'
42824282
},
4283+
{
4284+
'DP1State': 'DataPathDeactivated',
4285+
'DP2State': 'DataPathDeactivated',
4286+
'DP3State': 'DataPathDeactivated',
4287+
'DP4State': 'DataPathDeactivated',
4288+
'DP5State': 'DataPathDeactivated',
4289+
'DP6State': 'DataPathDeactivated',
4290+
'DP7State': 'DataPathDeactivated',
4291+
'DP8State': 'DataPathDeactivated'
4292+
},
42834293
{
42844294
'DP1State': 'DataPathInitialized',
42854295
'DP2State': 'DataPathInitialized',
@@ -4510,6 +4520,16 @@ def test_CmisManagerTask_task_worker_fastboot(self, mock_chassis, mock_get_statu
45104520
'ConfigStatusLane8': 'ConfigSuccess'
45114521
})
45124522
mock_xcvr_api.get_datapath_state = MagicMock(side_effect=[
4523+
{
4524+
'DP1State': 'DataPathDeactivated',
4525+
'DP2State': 'DataPathDeactivated',
4526+
'DP3State': 'DataPathDeactivated',
4527+
'DP4State': 'DataPathDeactivated',
4528+
'DP5State': 'DataPathDeactivated',
4529+
'DP6State': 'DataPathDeactivated',
4530+
'DP7State': 'DataPathDeactivated',
4531+
'DP8State': 'DataPathDeactivated'
4532+
},
45134533
{
45144534
'DP1State': 'DataPathDeactivated',
45154535
'DP2State': 'DataPathDeactivated',
@@ -4650,6 +4670,16 @@ def test_CmisManagerTask_task_worker_host_tx_ready_false_to_true(self, mock_chas
46504670
'ConfigStatusLane8': 'ConfigSuccess'
46514671
})
46524672
mock_xcvr_api.get_datapath_state = MagicMock(side_effect=[
4673+
{
4674+
'DP1State': 'DataPathDeactivated',
4675+
'DP2State': 'DataPathDeactivated',
4676+
'DP3State': 'DataPathDeactivated',
4677+
'DP4State': 'DataPathDeactivated',
4678+
'DP5State': 'DataPathDeactivated',
4679+
'DP6State': 'DataPathDeactivated',
4680+
'DP7State': 'DataPathDeactivated',
4681+
'DP8State': 'DataPathDeactivated'
4682+
},
46534683
{
46544684
'DP1State': 'DataPathDeactivated',
46554685
'DP2State': 'DataPathDeactivated',
@@ -4670,6 +4700,16 @@ def test_CmisManagerTask_task_worker_host_tx_ready_false_to_true(self, mock_chas
46704700
'DP7State': 'DataPathInitialized',
46714701
'DP8State': 'DataPathInitialized'
46724702
},
4703+
{
4704+
'DP1State': 'DataPathInitialized',
4705+
'DP2State': 'DataPathInitialized',
4706+
'DP3State': 'DataPathInitialized',
4707+
'DP4State': 'DataPathInitialized',
4708+
'DP5State': 'DataPathInitialized',
4709+
'DP6State': 'DataPathInitialized',
4710+
'DP7State': 'DataPathInitialized',
4711+
'DP8State': 'DataPathInitialized'
4712+
},
46734713
{
46744714
'DP1State': 'DataPathActivated',
46754715
'DP2State': 'DataPathActivated',
@@ -4720,6 +4760,16 @@ def test_CmisManagerTask_task_worker_host_tx_ready_false_to_true(self, mock_chas
47204760
'DP7State': 'DataPathInitialized',
47214761
'DP8State': 'DataPathInitialized'
47224762
},
4763+
{
4764+
'DP1State': 'DataPathInitialized',
4765+
'DP2State': 'DataPathInitialized',
4766+
'DP3State': 'DataPathInitialized',
4767+
'DP4State': 'DataPathInitialized',
4768+
'DP5State': 'DataPathInitialized',
4769+
'DP6State': 'DataPathInitialized',
4770+
'DP7State': 'DataPathInitialized',
4771+
'DP8State': 'DataPathInitialized'
4772+
},
47234773
])
47244774
mock_sfp = MagicMock()
47254775
mock_sfp.get_presence = MagicMock(return_value=True)

0 commit comments

Comments
 (0)