Method Signal Handler Test (#2630)

tests/test_app.py

@@ -348,12 +348,14 @@ def test_app_registry_retrieval_from_multiple():
 def test_get_app_does_not_exist():
     with pytest.raises(
         SanicException,
-        match="Sanic app name 'does-not-exist' not found.\n"
+        match=(
+            "Sanic app name 'does-not-exist' not found.\n"
             "App instantiation must occur outside "
             "if __name__ == '__main__' "
             "block or by using an AppLoader.\nSee "
             "https://sanic.dev/en/guide/deployment/app-loader.html"
-        " for more details.",
+            " for more details."
+        ),
     ):
         Sanic.get_app("does-not-exist")
 

tests/test_signals.py

@@ -7,7 +7,7 @@ import pytest
 
 from sanic_routing.exceptions import NotFound
 
-from sanic import Blueprint
+from sanic import Blueprint, Sanic, empty
 from sanic.exceptions import InvalidSignal, SanicException
 
 
@@ -20,6 +20,31 @@ def test_add_signal(app):
     assert len(app.signal_router.routes) == 1
 
 
+def test_add_signal_method_handler(app):
+    counter = 0
+
+    class TestSanic(Sanic):
+        def __init__(self, *args, **kwargs):
+            super().__init__(*args, **kwargs)
+            self.add_signal(
+                self.after_routing_signal_handler, "http.routing.after"
+            )
+
+        def after_routing_signal_handler(self, *args, **kwargs):
+            nonlocal counter
+            counter += 1
+
+    app = TestSanic("Test")
+    assert len(app.signal_router.routes) == 1
+
+    @app.route("/")
+    async def handler(_):
+        return empty()
+
+    app.test_client.get("/")
+    assert counter == 1
+
+
 def test_add_signal_decorator(app):
     @app.signal("foo.bar.baz")
     def sync_signal(*_):