Merge branch 'feature/DBOARD3-833-handle-inactive-bundle-config' into 'develop'

Feature/dboard3 833 handle inactive bundle config

See merge request !24

inventory_provider/juniper.py

@@ -232,10 +232,10 @@ def list_interfaces(netconf_config):
     def _ifc_info(e):
         # warning: this structure should match the default
         #          returned from routes.classifier.juniper_link_info
-        name = e.find('name')
-        assert name is not None, "expected interface 'name' child element"
+        _name = e.find('name')
+        assert _name is not None, "expected interface 'name' child element"
         ifc = {
-            'name': name.text,
+            'name': _name.text,
             'description': '',
             'bundle': [],
             'speed': ''  # set elsewhere but needs to be included to maintain default structure
@@ -244,12 +244,11 @@ def list_interfaces(netconf_config):
         if description is not None:
             ifc['description'] = description.text
 
-        for b in i.iterfind(".//bundle"):
-            ifc['bundle'].append(b.text)
-
+        ifc['bundle'] = e.xpath(
+            "./gigether-options[not(@inactive='inactive')]"
+            "/ieee-802.3ad[not(@inactive='inactive')]/bundle/text()")
         ifc['ipv4'] = e.xpath('./family/inet/address/name/text()')
         ifc['ipv6'] = e.xpath('./family/inet6/address/name/text()')
-
         return ifc
 
     def _inactive(interface_node):

test/test_juniper_data_global.py

 import ast
 import netifaces
 import ipaddress
+import os
 
+from lxml import etree
 import pytest
 
 from inventory_provider import juniper
@@ -89,3 +91,32 @@ def test_asn_to_int_functionality():
     with pytest.raises(ValueError):
         asn_to_int('66512.2')
         asn_to_int('Test')
+
+
+def test_DBOARD3_833():
+    """Test that the bundle inactive attribute is handled correctly.
+
+    The test data contains an interface: et-4/0/2 with the bundle
+    configuration (for ae6) marked as 'inactive'.  et-4/0/2 also
+    has a logical interface configured.
+
+    This test confirms that neither et-4/0/2 nor et-4/0/2.0 are
+    part of a bundle.
+    """
+
+    test_filename = os.path.join(
+        os.path.dirname(__file__),
+        'data',
+        'DBOARD3-833.xml')
+    with open(test_filename, 'r') as f:
+        netconf_config = etree.XML(f.read())
+
+    interfaces = {
+        i['name']: i for i in juniper.list_interfaces(netconf_config)}
+
+    assert not interfaces['et-4/0/2']['bundle']
+    assert not interfaces['et-4/0/2.0']['bundle']
+
+    # additional sanity check: parent bundle should be empty or exactly one
+    for ifc in interfaces.values():
+        assert len(ifc['bundle']) in (0, 1)