TranslateToPrometheusHistogramNormal test fails with "Assertion failed: cannot increment end list iterator"

[malkia]

This is in debug only, where Microsoft STL does more checks than usual.

Steps to repro:

bazel test -c dbg //exporters/prometheus:prometheus_exporter_utils_test --test_output=streamed

gives

Executing tests from //exporters/prometheus:prometheus_exporter_utils_test
-----------------------------------------------------------------------------
Running main() from gmock_main.cc
[==========] Running 4 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 4 tests from PrometheusExporterUtils
[ RUN      ] PrometheusExporterUtils.TranslateToPrometheusEmptyInputReturnsEmptyCollection
[       OK ] PrometheusExporterUtils.TranslateToPrometheusEmptyInputReturnsEmptyCollection (0 ms)
[ RUN      ] PrometheusExporterUtils.TranslateToPrometheusIntegerCounter
[       OK ] PrometheusExporterUtils.TranslateToPrometheusIntegerCounter (0 ms)
[ RUN      ] PrometheusExporterUtils.TranslateToPrometheusIntegerLastValue
[       OK ] PrometheusExporterUtils.TranslateToPrometheusIntegerLastValue (0 ms)
[ RUN      ] PrometheusExporterUtils.TranslateToPrometheusHistogramNormal
C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\VC\Tools\MSVC\14.29.30133\include\list(162) : Assertion failed: cannot increment end list iterator

I've fixed in my own repo - malkia@974ec51

[esigo]

@malkia thanks for reaching out to us. Your change looks valid. Would you like to raise a PR?
Thanks

[malkia]

@malkia thanks for reaching out to us. Your change looks valid. Would you like to raise a PR? Thanks

I wish, but my branch has many other changes (a "dll" branch), when I have time I can try setup a new one with just this (and few other fixes in the future). Though not sure if CLA/etc. needs to be filled. Thanks!