diff options
| author |   Michal Cmarada <mcmarada@cisco.com> | 2019-04-08 12:27:32 +0200 |
|---|---|---|
| committer | Michal Cmarada <mcmarada@cisco.com> | 2019-04-09 08:09:21 +0200 |
| commit | aa25184f3f0925b789de52296053dd51e2bf8684 (patch) | |
| tree | f6291f8da0e2367c4714239b9f76bff46599ff35 /resources/libraries/robot/honeycomb/bridge_domain.robot | |
| parent | 5e6145a4260ffce1c302e94b9b241851f90838e1 (diff) | |
Fix l2 Honeycomb tests
Change-Id: Ic75a9279fffe3a5d11211f4e57fc635d7d33ee5f
Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
Diffstat (limited to 'resources/libraries/robot/honeycomb/bridge_domain.robot')
| -rw-r--r-- | resources/libraries/robot/honeycomb/bridge_domain.robot | 19 |
1 files changed, 10 insertions, 9 deletions
diff --git a/resources/libraries/robot/honeycomb/bridge_domain.robot b/resources/libraries/robot/honeycomb/bridge_domain.robot index e26c18d019..44dd450944 100644 --- a/resources/libraries/robot/honeycomb/bridge_domain.robot +++ b/resources/libraries/robot/honeycomb/bridge_domain.robot @@ -144,7 +144,7 @@ | | ... | ${if2_data['v3po:l2']['bridged-virtual-interface']} | | ... | ${settings['bvi']} -| VAT should show interfaces assigned to bridge domain +| PAPI should show interfaces assigned to bridge domain | | [Documentation] | Uses VAT to verify interface assignment to\ | | ... | bridge domain. | | ... @@ -159,7 +159,7 @@ | | ... | | ... | *Example:* | | ... -| | ... | \| VAT should show interfaces assigned to bridge domain \ +| | ... | \| PAPI should show interfaces assigned to bridge domain \ | | ... | \| ${nodes['DUT1']} \| ${4} \| GigabitEthernet0/8/0 \ | | ... | \| GigabitEthernet0/9/0 \| ${{split_horizon_group:2, bvi:False}} \| | | [Arguments] | ${node} | ${index} | ${interface1} | ${interface2} @@ -170,11 +170,12 @@ | | ... | ${node['interfaces']['${if1_link}']['vpp_sw_index']} | | ... | ${node['interfaces']['${if2_link}']['vpp_sw_index']} | | ${bd_data}= | VPP get bridge domain data | ${node} -| | ${bd_interfaces}= | Set Variable | ${bd_data[${index}]['sw_if']} +| | ${bd_interfaces}= | Set Variable | ${bd_data[${index}]['sw_if_details']} | | @{bd_interfaces}= | Create List | ${bd_interfaces[0]} | ${bd_interfaces[1]} | | :FOR | ${interface} | IN | @{bd_interfaces} -| | | Should contain | ${if_indices} | ${interface['sw_if_index']} -| | | Should be equal | ${interface['shg']} | ${settings['split_horizon_group']} +# interface[1] = sw_if_index, interface[2] = shg +| | | Should contain | ${if_indices} | ${interface[1]} +| | | Should be equal | ${interface[2]} | ${settings['split_horizon_group']} | Honeycomb removes all bridge domains | | [Documentation] | Uses Honeycomb API to remove all bridge domains from the \ @@ -205,7 +206,7 @@ | | Run keyword and expect error | *Not possible*Status code: 404* | | ... | Get all BDs oper data | ${node} -| VAT should show no bridge domains +| PAPI should show no bridge domains | | [Documentation] | Uses VAT to verify the removal of all bridge domains. | | ... | | ... | *Arguments:* @@ -213,10 +214,10 @@ | | ... | | ... | *Example:* | | ... -| | ... | \| VAT should show no bridge domains \| ${nodes['DUT1']} \| +| | ... | \| PAPI should show no bridge domains \| ${nodes['DUT1']} \| | | [Arguments] | ${node} -| | Run Keyword And Expect Error | ValueError: No JSON object could be decoded -| | ... | VPP get bridge domain data | ${node} +| | ${data}= | VPP get bridge domain data | ${node} +| | Should be empty | ${data} | Honeycomb adds interface to bridge domain | | [Documentation] | Uses Honeycomb API to assign interface to a bridge\ |