refactor(test): remove unnecessary test

Signed-off-by: Shingo OKAWA <[email protected]>

Author: ognis1205

scripts/grpc_gen_post_processor.py

@@ -1,5 +1,5 @@
-"""
-Fix absolute imports in *_pb2_grpc.py files.
+"""Fix absolute imports in *_pb2_grpc.py files.
+
 Example:
 import a2a_pb2 as a2a__pb2
 from . import a2a_pb2 as a2a__pb2
@@ -16,40 +16,40 @@ def process_generated_code(src_folder: str = 'src/a2a/grpc'):
     dir_path = Path(src_folder)
     print(dir_path)
     if not dir_path.is_dir():
-      print('Source folder not found')
-      sys.exit(1)
+        print('Source folder not found')
+        sys.exit(1)
 
     grpc_pattern = '**/*_pb2_grpc.py'
     files = dir_path.glob(grpc_pattern)
 
     for file in files:
-      print(f'Processing {file}')
-      try:
-        with file.open('r', encoding='utf-8') as f:
-          src_content = f.read()
-
-          # Change import a2a_pb2 as a2a__pb2
-          import_pattern = r'^import (\w+_pb2) as (\w+__pb2)$'
-          # to from . import a2a_pb2 as a2a__pb2
-          replacement_pattern = r'from . import \1 as \2'
-
-          fixed_src_content = re.sub(
-            import_pattern,
-            replacement_pattern,
-            src_content,
-            flags=re.MULTILINE,
-          )
-
-          if fixed_src_content != src_content:
-            with file.open('w', encoding='utf-8') as f:
-              f.write(fixed_src_content)
-              print('Imports fixed')
-          else:
-              print('No changes needed')
-
-      except Exception as e:
-        print(f'Error processing file {file}: {e}')
-        sys.exit(1)
+        print(f'Processing {file}')
+        try:
+            with file.open('r', encoding='utf-8') as f:
+                src_content = f.read()
+
+                # Change import a2a_pb2 as a2a__pb2
+                import_pattern = r'^import (\w+_pb2) as (\w+__pb2)$'
+                # to from . import a2a_pb2 as a2a__pb2
+                replacement_pattern = r'from . import \1 as \2'
+
+                fixed_src_content = re.sub(
+                    import_pattern,
+                    replacement_pattern,
+                    src_content,
+                    flags=re.MULTILINE,
+                )
+
+                if fixed_src_content != src_content:
+                    with file.open('w', encoding='utf-8') as f:
+                        f.write(fixed_src_content)
+                        print('Imports fixed')
+                else:
+                    print('No changes needed')
+
+        except Exception as e:
+            print(f'Error processing file {file}: {e}')
+            sys.exit(1)
 
 
 if __name__ == '__main__':

tests/server/apps/jsonrpc/test_jsonrpc_app.py

@@ -10,13 +10,8 @@
     StarletteBaseUser = MagicMock()  # type: ignore
 
 from a2a.server.apps.jsonrpc.jsonrpc_app import (
-    JSONRPCApplication,  # Still needed for JSONRPCApplication default constructor arg
     StarletteUserProxy,
 )
-from a2a.server.request_handlers.request_handler import (
-    RequestHandler,  # For mock spec
-)
-from a2a.types import AgentCard  # For mock spec
 
 
 # --- StarletteUserProxy Tests ---
@@ -55,33 +50,5 @@ def test_starlette_user_proxy_user_name_raises_attribute_error(self):
             _ = proxy.user_name
 
 
-# --- JSONRPCApplication Tests (Selected) ---
-
-
-class TestJSONRPCApplicationSetup:  # Renamed to avoid conflict
-    def test_jsonrpc_app_build_method_abstract_raises_typeerror(
-        self,
-    ):  # Renamed test
-        mock_handler = MagicMock(spec=RequestHandler)
-        # Mock agent_card with essential attributes accessed in JSONRPCApplication.__init__
-        mock_agent_card = MagicMock(spec=AgentCard)
-        # Ensure 'url' attribute exists on the mock_agent_card, as it's accessed in __init__
-        mock_agent_card.url = 'http://mockurl.com'
-        # Ensure 'supportsAuthenticatedExtendedCard' attribute exists
-        mock_agent_card.supportsAuthenticatedExtendedCard = False
-
-        class AbstractTester(JSONRPCApplication):
-            # No 'build' method implemented
-            pass
-
-        # Instantiating an ABC subclass that doesn't implement all abstract methods raises TypeError
-        with pytest.raises(
-            TypeError,
-            match="Can't instantiate abstract class AbstractTester with abstract method build",
-        ):
-            # Using positional arguments for the abstract class constructor
-            AbstractTester(mock_handler, mock_agent_card)
-
-
 if __name__ == '__main__':
     pytest.main([__file__])