check gigether-options active/inactive for bundle info

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

@@ -110,6 +110,14 @@ def test_DBOARD3_833():
     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)}
+    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)
+
+