bugfix: indent not used in MetricsData to_json (open-telemetry#3197)

Alex Boten · web-flow · commit a9a96aad8b93 · 2023-02-24T11:11:05.000-08:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,7 +8,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## Unreleased
 - PeriodicExportingMetricReader will continue if collection times out 
   ([#3100](https://github.com/open-telemetry/opentelemetry-python/pull/3100))
-
+- Fix formatting of ConsoleMetricExporter.
+  ([#3197](https://github.com/open-telemetry/opentelemetry-python/pull/3197))
 
 ## Version 1.16.0/0.37b0 (2023-02-17)
 - Change ``__all__`` to be statically defined.
diff --git a/opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/point.py b/opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/point.py
@@ -211,5 +211,6 @@ def to_json(self, indent=4) -> str:
                     loads(resource_metrics.to_json(indent=indent))
                     for resource_metrics in self.resource_metrics
                 ]
-            }
+            },
+            indent=indent,
         )
diff --git a/opentelemetry-sdk/tests/metrics/integration_test/test_console_exporter.py b/opentelemetry-sdk/tests/metrics/integration_test/test_console_exporter.py
@@ -49,7 +49,7 @@ def test_console_exporter(self):
         provider.shutdown()
 
         output.seek(0)
-        result_0 = loads(output.readlines()[0])
+        result_0 = loads("".join(output.readlines()))
 
         self.assertGreater(len(result_0), 0)
 

Original file line number	Diff line number	Diff line change
`@@ -211,5 +211,6 @@ def to_json(self, indent=4) -> str:`
`211`	`211`	`loads(resource_metrics.to_json(indent=indent))`
`212`	`212`	`for resource_metrics in self.resource_metrics`
`213`	`213`	`]`
`214`		`- }`
	`214`	`+ },`
	`215`	`+ indent=indent,`
`215`	`216`	`)`