Skip to content

Commit e179864

Browse files
author
Michael Brewer
committed
fix(metrics): AttributeError raises when setting metadata_set
1 parent de7a6ef commit e179864

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

aws_lambda_powertools/metrics/base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ def __init__(
8888
self.service = resolve_env_var_choice(choice=service, env=os.getenv(constants.SERVICE_NAME_ENV))
8989
self._metric_units = [unit.value for unit in MetricUnit]
9090
self._metric_unit_options = list(MetricUnit.__members__)
91-
self.metadata_set = self.metadata_set if metadata_set is not None else {}
91+
self.metadata_set = metadata_set if metadata_set is not None else {}
9292

9393
def add_metric(self, name: str, unit: Union[MetricUnit, str], value: float):
9494
"""Adds given metric

tests/functional/test_metrics.py

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -739,3 +739,13 @@ def test_serialize_metric_set_metric_definition_multiple_values(
739739
assert "Timestamp" in metric_definition_output["_aws"]
740740
remove_timestamp(metrics=[metric_definition_output, expected_metric_definition])
741741
assert metric_definition_output == expected_metric_definition
742+
743+
744+
def test_metric_manage_metadata_set():
745+
expected_dict = {"setting": "On"}
746+
747+
try:
748+
metric = MetricManager(metadata_set=expected_dict)
749+
assert metric.metadata_set == expected_dict
750+
except AttributeError:
751+
pytest.fail("AttributeError should not be raised")

0 commit comments

Comments
 (0)