IOS: Return empty dictionary if BGP not running

napalm/base/test/getters.py

@@ -211,7 +211,8 @@ def test_get_environment(self, test_case):
     def test_get_bgp_neighbors(self, test_case):
         """Test get_bgp_neighbors."""
         get_bgp_neighbors = self.device.get_bgp_neighbors()
-        assert "global" in get_bgp_neighbors.keys()
+        if len(get_bgp_neighbors) > 0:
+            assert "global" in get_bgp_neighbors.keys()
 
         for vrf, vrf_data in get_bgp_neighbors.items():
             assert isinstance(vrf_data["router_id"], str)

napalm/ios/ios.py

@@ -1497,9 +1497,9 @@ def get_bgp_neighbors(self):
         # get summary output from device
         cmd_bgp_all_sum = "show bgp all summary"
         summary_output = self._send_command(cmd_bgp_all_sum).strip()
-
-        if "Invalid input detected" in summary_output:
-            raise CommandErrorException("BGP is not running on this device")
+        bgp_not_running = ["Invalid input", "BGP not active"]
+        if any((s in summary_output for s in bgp_not_running)):
+            return {}
 
         # get neighbor output from device
         neighbor_output = ""

test/ios/mocked_data/test_get_bgp_neighbors/bgp_inactive/expected_result.json

@@ -0,0 +1 @@
+{}

test/ios/mocked_data/test_get_bgp_neighbors/bgp_inactive/show_bgp_all_summary.txt

@@ -0,0 +1 @@
+% BGP not active