service: add service wide configuration

In the case where we need to update several entity sets,
we probably don't want to specify HTTP method for each of them.

This patch does nothing but enhances user convenience.

Author: filak-sap

CHANGELOG.md

@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Added
 - Specify PATCH or PUT method for EntityUpdateRequest - Barton Ip
+- Add a Service wide configuration (e.g. http.update\_method) - Jakub Filak
 - <, <=, >, >= operators on GetEntitySetFilter - Barton Ip
 
 ### Fixed

pyodata/v2/service.py

@@ -1157,7 +1157,7 @@ def create_entity_handler(response):
         return EntityCreateRequest(self._service.url, self._service.connection, create_entity_handler, self._entity_set,
                                    self.last_segment)
 
-    def update_entity(self, key=None, method="PATCH", **kwargs):
+    def update_entity(self, key=None, method=None, **kwargs):
         """Updates an existing entity in the given entity-set."""
 
         def update_entity_handler(response):
@@ -1174,6 +1174,9 @@ def update_entity_handler(response):
 
         self._logger.info('Updating entity %s for key %s and args %s', self._entity_set.entity_type.name, key, kwargs)
 
+        if method is None:
+            method = self._service.config['http']['update_method']
+
         return EntityModifyRequest(self._service.url, self._service.connection, update_entity_handler, self._entity_set,
                                    entity_key, method=method)
 
@@ -1309,6 +1312,8 @@ def __init__(self, url, schema, connection):
         self._entity_container = EntityContainer(self)
         self._function_container = FunctionContainer(self)
 
+        self._config = {'http': {'update_method': 'PATCH'}}
+
     @property
     def schema(self):
         """Parsed metadata"""
@@ -1339,6 +1344,12 @@ def functions(self):
 
         return self._function_container
 
+    @property
+    def config(self):
+        """Service specific configuration"""
+
+        return self._config
+
     def http_get(self, path, connection=None):
         """HTTP GET response for the passed path in the service"""
 

tests/test_service_v2.py

@@ -703,6 +703,22 @@ def test_update_entity_with_no_method_specified(service):
     assert query.get_method() == "PATCH"
 
 
+def test_update_entity_with_service_config_set_to_put(service):
+    """Make sure the method update_entity handles correctly when no method is specified"""
+
+    # pylint: disable=redefined-outer-name
+
+
+    key = EntityKey(
+        service.schema.entity_type('TemperatureMeasurement'),
+        Sensor='sensor1',
+        Date=datetime.datetime(2017, 12, 24, 18, 0))
+
+    service.config['http']['update_method'] = "PUT"
+    query = service.entity_sets.TemperatureMeasurements.update_entity(key)
+    assert query.get_method() == "PUT"
+
+
 def test_update_entity_with_wrong_method_specified(service):
     """Make sure the method update_entity raises ValueError when wrong method is specified"""