Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[metricsadvisor] convert to test proxy #21974

Merged
merged 34 commits into from
Jan 8, 2022
Merged
Show file tree
Hide file tree
Changes from 33 commits
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
4bc3b1b
update sync base testcases and conftest to use test proxy
kristapratico Nov 23, 2021
71e023b
remove dep on unittest
kristapratico Nov 24, 2021
159c548
add sanitizers for test secrets
kristapratico Nov 24, 2021
f165bab
convert one test
kristapratico Nov 24, 2021
a43c845
fix weird spacing
kristapratico Nov 29, 2021
75e1213
refactor base testcase into preparer
kristapratico Nov 29, 2021
ebe5f85
convert test_alert_config.py + recordings
kristapratico Nov 29, 2021
6dcb02c
recordings
kristapratico Nov 29, 2021
7d688e2
updates for aad tests
kristapratico Nov 30, 2021
e0c01d5
test_credential and test_credential_entities, no recordings b/c need …
kristapratico Nov 30, 2021
9bd9c00
add rstrip / to client endpoint
kristapratico Nov 30, 2021
ef1f9af
convert data_feed_ingestion
kristapratico Nov 30, 2021
211ccea
convert data_feeds
kristapratico Dec 1, 2021
0790c9e
convert detection_config
kristapratico Dec 1, 2021
673f44e
convert hooks
kristapratico Dec 1, 2021
699c691
update ma client tests
kristapratico Dec 1, 2021
8a4ab5c
refactor to simplify base test class and secrets needed
kristapratico Dec 1, 2021
4cb2843
refactor, parametrize auth option
kristapratico Dec 2, 2021
0ccbfc1
refactor / parametrize async + add preparer
kristapratico Dec 2, 2021
2e80f35
convert async tests to proxy
kristapratico Dec 2, 2021
486dc54
add oauth sanitizer
kristapratico Dec 3, 2021
53dc084
Merge branch 'main' into ma-test-proxy
kristapratico Dec 13, 2021
8dca8fc
Merge branch 'main' into ma-test-proxy
kristapratico Dec 15, 2021
6c29756
add test proxy to ci.yaml
kristapratico Dec 15, 2021
c7121da
update testcases and have proxy auto start
kristapratico Dec 16, 2021
72907d1
rename test_id -> ids, fix clean up so we raise better errors, add mo…
kristapratico Dec 16, 2021
9cb1b32
rerecord and delete old recordings
kristapratico Dec 16, 2021
ab088a1
rename tests to fix max path length and try removing max parallel for…
kristapratico Dec 16, 2021
05b296e
ugh fix for py2.7
kristapratico Dec 16, 2021
6e9971b
fix envars for live tests
kristapratico Dec 16, 2021
223fc40
fixes for live tests
kristapratico Dec 16, 2021
69d8de8
adjust list data feeds tests to be less flaky
kristapratico Dec 16, 2021
1d3cb9c
merge main
kristapratico Jan 4, 2022
e8a1fbe
add rstrip before sanitizing
kristapratico Jan 7, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
Original file line number Diff line number Diff line change
Expand Up @@ -181,6 +181,7 @@ def __init__(self, endpoint, credential, **kwargs):
try:
if not endpoint.lower().startswith("http"):
endpoint = "https://" + endpoint
endpoint = endpoint.rstrip("/")
except AttributeError:
raise ValueError("Base URL must be a string.")

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,7 @@ def __init__(self, endpoint, credential, **kwargs):
try:
if not endpoint.lower().startswith("http"):
endpoint = "https://" + endpoint
endpoint = endpoint.rstrip("/")
except AttributeError:
raise ValueError("Base URL must be a string.")

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,7 @@ def __init__(self, endpoint, credential, **kwargs):
try:
if not endpoint.lower().startswith("http"):
endpoint = "https://" + endpoint
endpoint = endpoint.rstrip("/")
except AttributeError:
raise ValueError("Base URL must be a string.")

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,7 @@ def __init__(self, endpoint, credential, **kwargs):
try:
if not endpoint.lower().startswith("http"):
endpoint = "https://" + endpoint
endpoint = endpoint.rstrip("/")
except AttributeError:
raise ValueError("Base URL must be a string.")

Expand Down

This file was deleted.

Large diffs are not rendered by default.

This file was deleted.

Loading