Alan-Jowett / CoPilot-For-Consensus / 20423422233 / 26
78%
main: 78%

DEFAULT BRANCH: main
Ran
Files 2
Run time 0s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 90.0%. Remained the same
20423422233.26

push

github

web-flow 
Add Azure Monitor–Backed MetricsAdapter to copilot_metrics (#524)

* Initial plan

* Add Azure Monitor metrics adapter with OpenTelemetry integration

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix linting issues in Azure Monitor adapter (10/10 pylint score)

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix type annotations and test skip conditions per code review

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Changes before error encountered

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Address code review feedback: add test fixture, improve assertions, rename test

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix CI failure: remove undefined monkeypatch reference in test

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix test fixture usage and skip condition logic

- Use mock_azure_exporter fixture consistently across all Azure Monitor tests
- Fix inverted skip condition in test_requires_azure_monitor_packages
- Remove all inline mocking code duplication

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix Azure Monitor implementation issues per code review

- Store provider instance to avoid global state conflicts
- Fix gauge callback closure to capture key by value
- Initialize cache dictionaries before try block for consistent state
- Use instance-specific provider in shutdown instead of global
- Update test comment for accuracy
- Add mock_azure_exporter fixture to factory test

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Add get_gauge_value method and use f-string logging for consistency

- Add get_gauge_value() method to support test introspection like NoOp collector
- Convert all logging to f-strings for consistency with Prometheus adapter
- Maintains consistency across metrics ... (continued)

279 of 310 relevant lines covered (90.0%)

0.9 hits per line

Source Files on job reporting - 20423422233.26