Fix inconsistencies on validation return

Author: simonkurtz-MSFT

shared/python/utils.py

@@ -630,7 +630,7 @@ def run(command: str, ok_message: str = '', error_message: str = '', print_outpu
 validate_http_verb      = lambda val: HTTP_VERB(val)
 validate_sku            = lambda val: APIM_SKU(val)
 
-def validate_infrastructure(infra: INFRASTRUCTURE, supported_infras: list[INFRASTRUCTURE]) -> bool:
+def validate_infrastructure(infra: INFRASTRUCTURE, supported_infras: list[INFRASTRUCTURE]) -> None:
     """
     Validate that the provided infrastructure is a supported infrastructure.
 

tests/python/test_utils.py

@@ -259,7 +259,7 @@ def test_extract_json_multiple_json_types():
 # ------------------------------
 
 def test_validate_infrastructure_supported():
-    # Should return True for supported infra
+    # Should return None for supported infra
     assert utils.validate_infrastructure(INFRASTRUCTURE.SIMPLE_APIM, [INFRASTRUCTURE.SIMPLE_APIM]) is None
 
 def test_validate_infrastructure_unsupported():