grafana / tempo-operator

Grafana Tempo Kubernetes operator
https://grafana.com/docs/tempo/latest/setup/operator/
GNU Affero General Public License v3.0
61 stars 30 forks source link

[Chore] Update tests for feature oauth proxy on single tenant #928

Closed IshwarKanse closed 5 months ago

IshwarKanse commented 5 months ago

Fix test failures caused due to oauth proxy on single tenant feature. https://github.com/grafana/tempo-operator/pull/910

codecov-commenter commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 76.41%. Comparing base (1a3d404) to head (43fc992).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #928 +/- ## ======================================= Coverage 76.41% 76.41% ======================================= Files 94 94 Lines 6071 6071 ======================================= Hits 4639 4639 Misses 1160 1160 Partials 272 272 ``` | [Flag](https://app.codecov.io/gh/grafana/tempo-operator/pull/928/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=grafana) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/grafana/tempo-operator/pull/928/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=grafana) | `76.41% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=grafana#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

IshwarKanse commented 5 months ago

@andreasgerstmayr Can you help to merge this. It fixes few more tests for the oauth proxy change.

andreasgerstmayr commented 5 months ago

@andreasgerstmayr Can you help to merge this. It fixes few more tests for the oauth proxy change.

Sure, merged!