From 48b7ff7a351f585f2f9d61efbeac5bd261bf5255 Mon Sep 17 00:00:00 2001 From: Maarten de Vries Date: Fri, 21 Dec 2018 01:24:48 +0100 Subject: [PATCH] Fix test_metrics.py compatibility prometheus_client 0.5 prometheus_client 0.5 has a named-tuple Sample type with more member than the old plain tuple had. This commit makes sure the unit test detects this and changes the way it reads the sample. Signed-off-by: Maarten de Vries --- changelog.d/4317.bugfix | 1 + tests/test_metrics.py | 24 +++++++++++++++++++++++- 2 files changed, 24 insertions(+), 1 deletion(-) create mode 100644 changelog.d/4317.bugfix diff --git a/changelog.d/4317.bugfix b/changelog.d/4317.bugfix new file mode 100644 index 0000000000..61bad5f2da --- /dev/null +++ b/changelog.d/4317.bugfix @@ -0,0 +1 @@ +Fix test_metric.py compatibility with prometheus_client 0.5. Contributed by Maarten de Vries . diff --git a/tests/test_metrics.py b/tests/test_metrics.py index 17897711a1..0ff6d0e283 100644 --- a/tests/test_metrics.py +++ b/tests/test_metrics.py @@ -19,6 +19,28 @@ from synapse.metrics import InFlightGauge from tests import unittest +def get_sample_labels_value(sample): + """ Extract the labels and values of a sample. + + prometheus_client 0.5 changed the sample type to a named tuple with more + members than the plain tuple had in 0.4 and earlier. This function can + extract the labels and value from the sample for both sample types. + + Args: + sample: The sample to get the labels and value from. + Returns: + A tuple of (labels, value) from the sample. + """ + + # If the sample has a labels and value attribute, use those. + if hasattr(sample, "labels") and hasattr(sample, "value"): + return sample.labels, sample.value + # Otherwise fall back to treating it as a plain 3 tuple. + else: + _, labels, value = sample + return labels, value + + class TestMauLimit(unittest.TestCase): def test_basic(self): gauge = InFlightGauge( @@ -75,7 +97,7 @@ class TestMauLimit(unittest.TestCase): for r in gauge.collect(): results[r.name] = { tuple(labels[x] for x in gauge.labels): value - for _, labels, value in r.samples + for labels, value in map(get_sample_labels_value, r.samples) } return results