azure-kusto-python icon indicating copy to clipboard operation
azure-kusto-python copied to clipboard

Uri Autocorrect

Open t-ronmoneta opened this issue 1 year ago • 2 comments

Added

  • New autocorrect url handling (via flag) to all ingest clients
  • Tests

Changed

  • Removed old autocorrect url handling
  • Removed Managed ingest from_dm\from_engine functions as they were no longer required

Fixed

t-ronmoneta avatar Feb 22 '24 09:02 t-ronmoneta

Test Results

    6 files  ± 0      6 suites  ±0   21m 7s :stopwatch: + 2m 7s   288 tests + 6    268 :white_check_mark: + 6   20 :zzz: ±0  0 :x: ±0  1 728 runs  +36  1 608 :white_check_mark: +36  120 :zzz: ±0  0 :x: ±0 

Results for commit 8e8d7878. ± Comparison against base commit 70ed0536.

This pull request removes 2 and adds 8 tests. Note that renamed tests count towards both.
azure-kusto-ingest.tests.test_kusto_ingest_client.TestQueuedIngestClient ‑ test_ingest_from_file_wrong_endpoint[ManagedStreamingIngestClient]
azure-kusto-ingest.tests.test_kusto_ingest_client.TestQueuedIngestClient ‑ test_ingest_from_file_wrong_endpoint[QueuedIngestClient]
azure-kusto-ingest.tests.test_kusto_ingest_client.TestQueuedIngestClient ‑ test_client_uri_from_ingestion_endpoint
azure-kusto-ingest.tests.test_kusto_ingest_client.TestQueuedIngestClient ‑ test_client_uri_from_query_endpoint
azure-kusto-ingest.tests.test_kusto_ingest_client.TestQueuedIngestClient ‑ test_from_ingestion_endpoint_reserved_hostname
azure-kusto-ingest.tests.test_kusto_streaming_ingest_client.TestKustoStreamingIngestClient ‑ test_client_uri_from_ingestion_endpoint
azure-kusto-ingest.tests.test_kusto_streaming_ingest_client.TestKustoStreamingIngestClient ‑ test_client_uri_from_query_endpoint
azure-kusto-ingest.tests.test_managed_streaming_ingest.TestManagedStreamingIngestClient ‑ test_client_uri_from_ingestion_endpoint
azure-kusto-ingest.tests.test_managed_streaming_ingest.TestManagedStreamingIngestClient ‑ test_client_uri_from_query_endpoint
azure-kusto-ingest.tests.test_managed_streaming_ingest.TestManagedStreamingIngestClient ‑ test_from_ingestion_endpoint_reserved_hostname

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

github-actions[bot] avatar Feb 22 '24 09:02 github-actions[bot]

Codecov Report

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

Project coverage is 88.83%. Comparing base (1f2c766) to head (510717a).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #524      +/-   ##
==========================================
- Coverage   88.90%   88.83%   -0.08%     
==========================================
  Files          44       44              
  Lines        3760     3744      -16     
==========================================
- Hits         3343     3326      -17     
- Misses        417      418       +1     

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

codecov[bot] avatar Feb 25 '24 10:02 codecov[bot]

Codecov Report

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

Project coverage is 88.14%. Comparing base (70ed053) to head (8e8d787).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #524      +/-   ##
==========================================
- Coverage   88.21%   88.14%   -0.08%     
==========================================
  Files          44       44              
  Lines        3760     3744      -16     
==========================================
- Hits         3317     3300      -17     
- Misses        443      444       +1     

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

codecov-commenter avatar Mar 25 '24 10:03 codecov-commenter