Changed views default to match otel

Author: jeremydvoss

azure-monitor-opentelemetry/azure/monitor/opentelemetry/util/configurations.py

@@ -173,7 +173,7 @@ def _default_metric_readers(configurations):
 
 def _default_views(configurations):
     if VIEWS_ARG not in configurations:
-        configurations[VIEWS_ARG] = []
+        configurations[VIEWS_ARG] = ()
 
 
 # TODO: remove when sampler uses env var instead

azure-monitor-opentelemetry/tests/configuration/test_util.py

@@ -95,7 +95,7 @@ def test_get_configurations_defaults(self):
         self.assertEqual(configurations["tracing_export_interval_ms"], None)
         self.assertEqual(configurations["logging_export_interval_ms"], 5000)
         self.assertEqual(configurations["metric_readers"], [])
-        self.assertEqual(configurations["views"], [])
+        self.assertEqual(configurations["views"], ())
         self.assertEqual(configurations["instrumentation_config"], {})
 
     def test_get_configurations_validation(self):
@@ -140,7 +140,7 @@ def test_get_configurations_env_vars(self):
         self.assertEqual(configurations["tracing_export_interval_ms"], None)
         self.assertEqual(configurations["logging_export_interval_ms"], 10000)
         self.assertEqual(configurations["metric_readers"], [])
-        self.assertEqual(configurations["views"], [])
+        self.assertEqual(configurations["views"], ())
         self.assertEqual(
             configurations["instrumentation_config"],
             {"flask": {"excluded_urls": "http://localhost:8080/ignore"}},
@@ -178,7 +178,7 @@ def test_get_configurations_env_vars_validation(self):
         self.assertEqual(configurations["tracing_export_interval_ms"], None)
         self.assertEqual(configurations["logging_export_interval_ms"], 5000)
         self.assertEqual(configurations["metric_readers"], [])
-        self.assertEqual(configurations["views"], [])
+        self.assertEqual(configurations["views"], ())
         self.assertEqual(configurations["instrumentation_config"], {})
 
     @patch.dict(
@@ -205,7 +205,7 @@ def test_get_configurations_env_vars_false(self):
         self.assertEqual(configurations["tracing_export_interval_ms"], None)
         self.assertEqual(configurations["logging_export_interval_ms"], 5000)
         self.assertEqual(configurations["metric_readers"], [])
-        self.assertEqual(configurations["views"], [])
+        self.assertEqual(configurations["views"], ())
         self.assertEqual(configurations["instrumentation_config"], {})
 
     @patch.dict(
@@ -232,5 +232,5 @@ def test_get_configurations_env_vars_true(self):
         self.assertEqual(configurations["tracing_export_interval_ms"], None)
         self.assertEqual(configurations["logging_export_interval_ms"], 5000)
         self.assertEqual(configurations["metric_readers"], [])
-        self.assertEqual(configurations["views"], [])
+        self.assertEqual(configurations["views"], ())
         self.assertEqual(configurations["instrumentation_config"], {})