Skip to content

Commit 31a94fc

Browse files
Fix URL validation test failures by enabling proper mocking (#89)
* Initial plan * Fix URL validation during tests to avoid network calls Co-authored-by: alexeygrigorev <875246+alexeygrigorev@users.noreply.github.com> * Refine URL validation skip logic to be more targeted Co-authored-by: alexeygrigorev <875246+alexeygrigorev@users.noreply.github.com> * Fix URL validation mocking by changing default parameter handling Co-authored-by: alexeygrigorev <875246+alexeygrigorev@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: alexeygrigorev <875246+alexeygrigorev@users.noreply.github.com>
1 parent 34ed88e commit 31a94fc

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

courses/validators/custom_url_validators.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,11 @@ def get_error_message(status_code, url):
2323

2424

2525
def validate_url_200(
26-
url, get_method=requests.get, code=None, params=None
26+
url, get_method=None, code=None, params=None
2727
):
28+
if get_method is None:
29+
get_method = requests.get
30+
2831
try:
2932
response = get_method(url)
3033
status_code = response.status_code

0 commit comments

Comments
 (0)