fix: misue of NoReturn

httprunner/ext/uploader/__init__.py

@@ -44,7 +44,7 @@ For compatibility, you can also write upload test script in old way:
 
 import os
 import sys
-from typing import Text, NoReturn
+from typing import Text
 
 from httprunner.models import TStep, FunctionsMapping
 from httprunner.parser import parse_variables_mapping
@@ -75,7 +75,7 @@ def ensure_upload_ready():
     sys.exit(1)
 
 
-def prepare_upload_step(step: TStep, functions: FunctionsMapping) -> "NoReturn":
+def prepare_upload_step(step: TStep, functions: FunctionsMapping):
     """ preprocess for upload test
         replace `upload` info with MultipartEncoder
 

httprunner/make.py

@@ -2,7 +2,7 @@ import os
 import string
 import subprocess
 import sys
-from typing import Dict, List, NoReturn, Set, Text, Tuple
+from typing import Dict, List, Set, Text, Tuple
 
 import jinja2
 from loguru import logger
@@ -133,7 +133,7 @@ def ensure_file_abs_path_valid(file_abs_path: Text) -> Text:
     return new_file_path
 
 
-def __ensure_testcase_module(path: Text) -> NoReturn:
+def __ensure_testcase_module(path: Text):
     """ ensure pytest files are in python module, generate __init__.py on demand
     """
     init_file = os.path.join(os.path.dirname(path), "__init__.py")
@@ -158,7 +158,7 @@ def convert_testcase_path(testcase_abs_path: Text) -> Tuple[Text, Text]:
     return testcase_python_abs_path, name_in_title_case
 
 
-def format_pytest_with_black(*python_paths: Text) -> NoReturn:
+def format_pytest_with_black(*python_paths: Text):
     logger.info("format pytest cases with black ...")
     try:
         if is_support_multiprocessing() or len(python_paths) <= 1:
@@ -436,7 +436,7 @@ def make_testcase(testcase: Dict, dir_path: Text = None) -> Text:
     return testcase_python_abs_path
 
 
-def make_testsuite(testsuite: Dict) -> NoReturn:
+def make_testsuite(testsuite: Dict):
     """convert valid testsuite dict to pytest folder with testcases"""
     # validate testsuite format
     load_testsuite(testsuite)
@@ -493,7 +493,7 @@ def make_testsuite(testsuite: Dict) -> NoReturn:
         pytest_files_run_set.add(testcase_pytest_path)
 
 
-def __make(tests_path: Text) -> NoReturn:
+def __make(tests_path: Text):
     """ make testcase(s) with testcase/testsuite/folder absolute path
         generated pytest file path will be cached in pytest_files_made_cache_mapping
 

httprunner/response.py

@@ -1,4 +1,4 @@
-from typing import Dict, Text, Any, NoReturn
+from typing import Dict, Text, Any
 
 import jmespath
 import requests
@@ -193,7 +193,7 @@ class ResponseObject(object):
         validators: Validators,
         variables_mapping: VariablesMapping = None,
         functions_mapping: FunctionsMapping = None,
-    ) -> NoReturn:
+    ):
 
         variables_mapping = variables_mapping or {}
         functions_mapping = functions_mapping or {}

httprunner/runner.py

@@ -2,7 +2,7 @@ import os
 import time
 import uuid
 from datetime import datetime
-from typing import List, Dict, Text, NoReturn
+from typing import List, Dict, Text
 
 try:
     import allure
@@ -55,7 +55,7 @@ class HttpRunner(object):
     # log
     __log_path: Text = ""
 
-    def __init_tests__(self) -> NoReturn:
+    def __init_tests__(self):
         self.__config = self.config.perform()
         self.__teststeps = []
         for step in self.teststeps:
@@ -90,7 +90,7 @@ class HttpRunner(object):
 
     def __call_hooks(
         self, hooks: Hooks, step_variables: VariablesMapping, hook_msg: Text,
-    ) -> NoReturn:
+    ):
         """ call hook actions.
 
         Args:
@@ -304,7 +304,7 @@ class HttpRunner(object):
         logger.info(f"run step end: {step.name} <<<<<<\n")
         return step_data.export_vars
 
-    def __parse_config(self, config: TConfig) -> NoReturn:
+    def __parse_config(self, config: TConfig):
         config.variables.update(self.__session_variables)
         config.variables = parse_variables_mapping(
             config.variables, self.__project_meta.functions