[core] Config global object uses deepcopy once

Author: Emanuele Palazzetti

ddtrace/configuration.py

@@ -1,3 +1,6 @@
+from copy import deepcopy
+
+
 class ConfigException(Exception):
     """Configuration exception when an integration that is not available
     is called in the `Config` object.
@@ -33,4 +36,4 @@ def _add(self, integration, settings):
             since it contains integration defaults.
         """
 
-        self._config[integration] = settings.copy()
+        self._config[integration] = deepcopy(settings)

tests/test_configuration.py

@@ -22,13 +22,19 @@ def test_registration(self):
     def test_settings_copy(self):
         # ensure that once an integration is registered, a copy
         # of the settings is stored to avoid side-effects
+        experimental = {
+            'request_enqueuing': True,
+        }
         settings = {
             'distributed_tracing': True,
+            'experimental': experimental,
         }
         self.config._add('requests', settings)
 
         settings['distributed_tracing'] = False
+        experimental['request_enqueuing'] = False
         ok_(self.config.requests['distributed_tracing'] is True)
+        ok_(self.config.requests['experimental']['request_enqueuing'] is True)
 
     def test_missing_integration(self):
         # ensure a meaningful exception is raised when an integration
@@ -38,7 +44,6 @@ def test_missing_integration(self):
             self.config.new_integration['some_key']
 
         ok_(isinstance(e.exception, ConfigException))
-        eq_(e.exception.message, 'Integration "new_integration" is not registered in this configuration')
 
     def test_global_configuration(self):
         # ensure a global configuration is available in the `ddtrace` module