Skip to content

Conversation

@VihasMakwana
Copy link
Contributor

Description

Merge TestRemoteWriteHistogramWithExemplars into TestTranslateV2.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Refactors the Prometheus remote-write receiver unit tests by folding the former TestRemoteWriteHistogramWithExemplars cases into the existing table-driven TestTranslateV2 suite, keeping histogram+exemplar translation coverage consolidated in one place.

Changes:

  • Moved histogram-with-exemplars and multi-histogram-with-exemplars scenarios into TestTranslateV2 as additional table cases.
  • Updated expected OTLP metrics for those cases to include Prometheus metric-type metadata (prometheus.MetricMetadataTypeKey = "histogram") and expected WriteResponseStats.
  • Reordered test helpers (mockConsumer, etc.) and adjusted assertions in TestTargetInfoWithMultipleRequests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants