Fix closing of async token providers

AsafMah commented 10 months ago

Fixed

Async token providers are properly closed (#417)
- This PR makes using sync clients with async token providers an error (since they can't be properly closed). Is this considered breaking or bug fix?

github-actions[bot] commented 10 months ago

Test Results

      6 files ±0       6 suites ±0 20m 10s :stopwatch: - 1m 26s   294 tests ±0   273 :heavy_check_mark: ±0   21 :zzz: ±0 0 :x: ±0 1 764 runs ±0 1 638 :heavy_check_mark: ±0 126 :zzz: ±0 0 :x: ±0

Results for commit 73287348. ± Comparison against base commit 7cc90234.

:recycle: This comment has been updated with latest results.

codecov[bot] commented 10 months ago

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (7cc9023) 89.01% compared to head (7328734) 88.96%.

Files	Patch %	Lines
...re-kusto-data/azure/kusto/data/_token_providers.py	76.66%	7 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #513 +/- ## ========================================== - Coverage 89.01% 88.96% -0.06% ========================================== Files 44 44 Lines 3670 3696 +26 ========================================== + Hits 3267 3288 +21 - Misses 403 408 +5 ```

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

Azure / azure-kusto-python