chore: Remove commented code, add docstrings, remove redundant except…

…ion class

src/schemathesis/cli/options.py

@@ -36,5 +36,5 @@ def convert(  # type: ignore
         try:
             int(value)
             return super().convert(value, param, ctx)
-        except (ValueError, UnicodeError):
+        except ValueError:
             self.fail("%s is not a valid integer or None" % value, param, ctx)

src/schemathesis/hooks.py

@@ -184,42 +184,42 @@ def unregister_all(self) -> None:
 
 @all_scopes
 def before_generate_path_parameters(strategy: st.SearchStrategy, context: HookContext) -> st.SearchStrategy:
-    pass
+    """Called on a strategy that generates values for ``path_parameters``."""
 
 
 @all_scopes
 def before_generate_headers(strategy: st.SearchStrategy, context: HookContext) -> st.SearchStrategy:
-    pass
+    """Called on a strategy that generates values for ``headers``."""
 
 
 @all_scopes
 def before_generate_cookies(strategy: st.SearchStrategy, context: HookContext) -> st.SearchStrategy:
-    pass
+    """Called on a strategy that generates values for ``cookies``."""
 
 
 @all_scopes
 def before_generate_query(strategy: st.SearchStrategy, context: HookContext) -> st.SearchStrategy:
-    pass
+    """Called on a strategy that generates values for ``query``."""
 
 
 @all_scopes
 def before_generate_body(strategy: st.SearchStrategy, context: HookContext) -> st.SearchStrategy:
-    pass
+    """Called on a strategy that generates values for ``body``."""
 
 
 @all_scopes
 def before_generate_form_data(strategy: st.SearchStrategy, context: HookContext) -> st.SearchStrategy:
-    pass
+    """Called on a strategy that generates values for ``form_data``."""
 
 
 @all_scopes
 def before_process_path(context: HookContext, path: str, methods: Dict[str, Any]) -> None:
-    pass
+    """Called before API path is processed."""
 
 
 @HookDispatcher.register_spec([HookScope.GLOBAL])
 def before_load_schema(context: HookContext, raw_schema: Dict[str, Any]) -> None:
-    pass
+    """Called before schema instance is created."""
 
 
 GLOBAL_HOOK_DISPATCHER = HookDispatcher(scope=HookScope.GLOBAL)

test/cli/test_commands.py

@@ -806,7 +806,6 @@ def assert_threaded_executor_interruption(lines, expected, optional_interrupt=Fa
     if not optional_interrupt:
         assert "!! KeyboardInterrupt !!" in lines[11]
     if "F" in lines[10]:
-        # assert "=== FAILURES ===" in lines[12]
         if "!! KeyboardInterrupt !!" not in lines[11]:
             assert "=== FAILURES ===" in lines[12]
             position = 23