Raise exceptions for API errors on network connect/disconnect

Signed-off-by: Aanand Prasad <[email protected]>

Author: aanand

docker/api/network.py

@@ -57,11 +57,13 @@ def connect_container_to_network(self, container, net_id,
             },
         }
         url = self._url("/networks/{0}/connect", net_id)
-        self._post_json(url, data=data)
+        res = self._post_json(url, data=data)
+        self._raise_for_status(res)
 
     @check_resource
     @minimum_version('1.21')
     def disconnect_container_from_network(self, container, net_id):
         data = {"container": container}
         url = self._url("/networks/{0}/disconnect", net_id)
-        self._post_json(url, data=data)
+        res = self._post_json(url, data=data)
+        self._raise_for_status(res)

tests/integration/network_test.py

@@ -117,10 +117,16 @@ def test_connect_and_disconnect_container(self):
             list(network_data['Containers'].keys()),
             [container['Id']])
 
+        with pytest.raises(docker.errors.APIError):
+            self.client.connect_container_to_network(container, net_id)
+
         self.client.disconnect_container_from_network(container, net_id)
         network_data = self.client.inspect_network(net_id)
         self.assertFalse(network_data.get('Containers'))
 
+        with pytest.raises(docker.errors.APIError):
+            self.client.disconnect_container_from_network(container, net_id)
+
     @requires_api_version('1.22')
     def test_connect_with_aliases(self):
         net_name, net_id = self.create_network()