diff options
author | selias <samelias@cisco.com> | 2016-05-13 13:33:45 +0200 |
---|---|---|
committer | selias <samelias@cisco.com> | 2016-05-20 13:52:40 +0200 |
commit | 1e7afbe3474d8ae8c5239ae3deee49963d6b1d93 (patch) | |
tree | 934ca404b3570a6299ed10eecdad05fb6cca6ff5 /resources | |
parent | 95bf11e637855dc91e490f526406fa7538a21ac6 (diff) |
Add Honeycomb TAP interface test
JIRA: CSIT-75
- add test suite for TAP interface management through Honeycomb
- add keywords required for TAP interface tests
- move Honeycomb suite setup and teardown to new __init__ file
- cleanup unused imports in Honeycomb test case files
- modify "...should be empty" keywords to allow more reuse
Change-Id: Ie3dc9069678b60c359f69c5a216e452e856a0107
Signed-off-by: selias <samelias@cisco.com>
Diffstat (limited to 'resources')
-rw-r--r-- | resources/libraries/python/InterfaceUtil.py | 28 | ||||
-rw-r--r-- | resources/libraries/robot/honeycomb/tap.robot | 145 | ||||
-rw-r--r-- | resources/libraries/robot/honeycomb/vxlan.robot | 3 | ||||
-rw-r--r-- | resources/templates/vat/tap_dump.vat | 1 |
4 files changed, 174 insertions, 3 deletions
diff --git a/resources/libraries/python/InterfaceUtil.py b/resources/libraries/python/InterfaceUtil.py index 1c1ada8f6c..838ba3bb69 100644 --- a/resources/libraries/python/InterfaceUtil.py +++ b/resources/libraries/python/InterfaceUtil.py @@ -490,7 +490,7 @@ class InterfaceUtil(object): if output["retval"] == 0: return output["sw_if_index"] else: - raise RuntimeError('Unable to create VXLAN interface on node {}' + raise RuntimeError('Unable to create VXLAN interface on node {0}' .format(node)) @staticmethod @@ -504,7 +504,7 @@ class InterfaceUtil(object): :type interface: int or str :return: Dictionary containing data for the given VxLAN interface or if interface=None, the list of dictionaries with all VxLAN interfaces. - :rtype dict or list + :rtype: dict or list """ param = "sw_if_index" if interface is None: @@ -544,6 +544,30 @@ class InterfaceUtil(object): return response[0] @staticmethod + def tap_dump(node, name=None): + """Get all TAP interface data from the given node, or data about + a specific TAP interface. + + :param node: VPP node to get data from. + :param name: Optional name of a specific TAP interface. + :type node: dict + :type name: str + :return: Dictionary of information about a specific TAP interface, or + a List of dictionaries containing all TAP data for the given node. + :rtype: dict or list + """ + with VatTerminal(node) as vat: + response = vat.vat_terminal_exec_cmd_from_template( + "tap_dump.vat") + if name is None: + return response[0] + else: + for item in response[0]: + if name == item['dev_name']: + return item + return {} + + @staticmethod def create_subinterface(node, interface, sub_id, outer_vlan_id, inner_vlan_id, type_subif): """Create sub-interface on node. diff --git a/resources/libraries/robot/honeycomb/tap.robot b/resources/libraries/robot/honeycomb/tap.robot new file mode 100644 index 0000000000..f503b60090 --- /dev/null +++ b/resources/libraries/robot/honeycomb/tap.robot @@ -0,0 +1,145 @@ +# Copyright (c) 2016 Cisco and/or its affiliates. +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at: +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. + +*** Settings *** +| Library | resources.libraries.python.InterfaceUtil +| ... | WITH NAME | interfaceCLI +| Library | resources.libraries.python.honeycomb.HcAPIKwInterfaces.InterfaceKeywords +| ... | WITH NAME | InterfaceAPI +| Documentation | Keywords used to manipulate TAP interfaces. + +*** Keywords *** +| Honeycomb creates TAP interface +| | [Documentation] | Uses Honeycomb API to configure a new TAP interface. +| | ... +| | ... | *Arguments:* +| | ... | - node - information about a DUT node. Type: dictionary +| | ... | - interface - name of an interface on the specified node. Type: string +| | ... | - settings - Configuration data for TAP. Type: dictionary +| | ... +| | ... | *Example:* +| | ... | \| Honeycomb creates TAP interface \ +| | ... | \| ${nodes['DUT1']} \| tap_int1 \| ${{'tap-name':'tap1',\ +| | ... | 'mac':'08:00:27:60:26:ab', 'device-instance':3}} \| +| | ... +| | [Arguments] | ${node} | ${interface} | ${settings} +| | interfaceAPI.Create TAP interface | ${node} | ${interface} +| | ... | &{settings} + +| Honeycomb configures TAP interface +| | [Documentation] | Uses Honeycomb API to configure an existing TAP interface. +| | ... +| | ... | *Arguments:* +| | ... | - node - information about a DUT node. Type: dictionary +| | ... | - interface - name of an interface on the specified node. Type: string +| | ... | - settings - Configuration data for TAP. Type: dictionary +| | ... +| | ... | *Example:* +| | ... | \| Honeycomb configures TAP interface \ +| | ... | \| ${nodes['DUT1']} \| tap_int1 \| ${{'tap-name':'tap1',\ +| | ... | 'mac':'08:00:27:60:26:ab', 'device-instance':3}} \| +| | ... +| | [Arguments] | ${node} | ${interface} | ${settings} +| | interfaceAPI.Configure interface TAP | ${node} | ${interface} +| | ... | &{settings} + +| Honeycomb removes TAP interface +| | [Documentation] | Uses Honeycomb API to remove a TAP interface. +| | ... +| | ... | *Arguments:* +| | ... | - node - information about a DUT node. Type: dictionary +| | ... | - interface - name of an interface on the specified node. Type: string +| | ... +| | ... | *Example:* +| | ... +| | ... | \| Honeycomb removes TAP interface \ +| | ... | \| ${nodes['DUT1']} \| tap_int1 \| +| | [Arguments] | ${node} | ${interface} +| | interfaceAPI.Delete interface | ${node} | ${interface} + +| TAP configuration from Honeycomb should be +| | [Documentation] | Retrieves interface TAP configuration through Honeycomb\ +| | ... | and compares with settings supplied in argument. +| | ... +| | ... | *Arguments:* +| | ... | - node - information about a DUT node. Type: dictionary +| | ... | - interface - name of an interface on the specified node. Type: string +| | ... | - settings - Configuration data for TAP. Type: dictionary +| | ... +| | ... | *Example:* +| | ... +| | ... | \| TAP configuration from Honeycomb should be \ +| | ... | \| ${nodes['DUT1']} \| tap_int1 \| ${{'tap-name':'tap1',\ +| | ... | 'mac':'08:00:27:60:26:ab', 'device-instance':3}} \| +| | ... +| | [Arguments] | ${node} | ${interface} | ${settings} +| | ${api_data}= | interfaceAPI.Get interface oper data | ${node} | ${interface} +| | ${api_tap}= | Set Variable | ${api_data['v3po:tap']} +| | Should be equal | ${api_tap['tap-name']} | ${settings['tap-name']} +| | ${api_mac}= | Set Variable | ${api_data['phys-address']} +| | Should be equal | ${api_mac} | ${settings['mac']} + +| TAP configuration from VAT should be +| | [Documentation] | Retrieves interface TAP configuration through VAT and\ +| | ... | compares with settings supplied in argument. +| | ... +| | ... | *Arguments:* +| | ... | - node - information about a DUT node. Type: dictionary +| | ... | - settings - Configuration data for TAP. Type: dictionary +| | ... +| | ... | *Example:* +| | ... +| | ... | \| TAP configuration from Honeycomb should be \ +| | ... | \| ${nodes['DUT1']} \| ${{'tap-name':'tap1',\ +| | ... | 'mac':'08:00:27:60:26:ab', 'device-instance':3}} \| +| | ... +| | [Arguments] | ${node} | ${interface} | ${settings} +| | ${vat_data}= | TAP Dump | ${node} | ${interface} +| | Should be equal | ${vat_data['dev_name']} | ${settings['tap-name']} +| | ${vat_data}= | VPP Get interface data | ${node} +# other settings not accessible through VAT commands + +| TAP configuration from Honeycomb should be empty +| | [Documentation] | Attempts to retrieve interface TAP configuration\ +| | ... | through Honeycomb and expects to recieve an empty dictionary. +| | ... +| | ... | *Arguments:* +| | ... | - node - information about a DUT node. Type: dictionary +| | ... | - interface - name of an interface on the specified node. Type: string +| | ... +| | ... | *Example:* +| | ... +| | ... | \| TAP configuration from Honeycomb should be empty\ +| | ... | \| ${nodes['DUT1']} \| tap_int1 \| +| | ... +| | [Arguments] | ${node} | ${interface} +| | ${api_data}= | interfaceAPI.Get interface oper data | ${node} | ${interface} +| | Run keyword and expect error | *KeyError: 'v3po:tap' | Set Variable +| | ... | ${api_data['v3po:tap']} + +| TAP configuration from VAT should be empty +| | [Documentation] | Attempts to retrieve interface TAP configuration\ +| | ... | through VAT and expects a "no data" error. +| | ... +| | ... | *Arguments:* +| | ... | - node - information about a DUT node. Type: dictionary +| | ... | - interface - name of an interface on the specified node. Type: string +| | ... +| | ... | *Example:* +| | ... +| | ... | \| TAP configuration from VAT should be empty\ +| | ... | \| ${nodes['DUT1']} \| tap_int1 \| +| | ... +| | [Arguments] | ${node} | ${interface} +| | Run Keyword And Expect Error | ValueError: No JSON object could be decoded +| | ... | TAP Dump | ${node} | ${interface} diff --git a/resources/libraries/robot/honeycomb/vxlan.robot b/resources/libraries/robot/honeycomb/vxlan.robot index 83a6e120b7..a57c965fa3 100644 --- a/resources/libraries/robot/honeycomb/vxlan.robot +++ b/resources/libraries/robot/honeycomb/vxlan.robot @@ -112,7 +112,8 @@ | | ... | | [Arguments] | ${node} | ${interface} | | ${api_data}= | interfaceAPI.Get interface oper data | ${node} | ${interface} -| | Should be empty | ${api_data} +| | Run keyword and expect error | *KeyError: 'v3po:vxlan' | Set Variable +| | ... | ${api_data['v3po:vxlan']} | VxLAN configuration from VAT should be empty | | [Documentation] | Attempts to retrieve interface VxLAN configuration\ diff --git a/resources/templates/vat/tap_dump.vat b/resources/templates/vat/tap_dump.vat new file mode 100644 index 0000000000..8f51f6e23c --- /dev/null +++ b/resources/templates/vat/tap_dump.vat @@ -0,0 +1 @@ +sw_interface_tap_dump
\ No newline at end of file |