Return HTTP 400 when attempting to post an event with an unregistered schema

Author: afshin

jupyter_server/services/events/handlers.py

@@ -121,6 +121,9 @@ async def post(self):
             self.finish()
         except web.HTTPError:
             raise
+        except KeyError as e:
+            message = f"Unregistered event schema: ${str(e)}"
+            raise web.HTTPError(400, message) from e
         except Exception as e:
             raise web.HTTPError(500, str(e)) from e
 

tests/services/events/test_api.py

@@ -117,16 +117,26 @@ async def test_post_event(jp_fetch, event_logger_sink, payload):
 }
 """
 
+payload_7 = """\
+{
+    "schema_id": "http://event.mock.jupyter.org/UNREGISTERED-SCHEMA",
+    "version": 1,
+    "data": {
+        "event_message": "Hello, world!"
+    }
+}
+"""
+
 
-@pytest.mark.parametrize("payload", [payload_3, payload_4, payload_5, payload_6])
+@pytest.mark.parametrize("payload", [payload_3, payload_4, payload_5, payload_6, payload_7])
 async def test_post_event_400(jp_fetch, event_logger, payload):
     with pytest.raises(tornado.httpclient.HTTPClientError) as e:
         await jp_fetch("api", "events", method="POST", body=payload)
 
     assert expected_http_error(e, 400)
 
 
-payload_7 = """\
+payload_8 = """\
 {
     "schema_id": "http://event.mock.jupyter.org/message",
     "version": 1,
@@ -136,7 +146,7 @@ async def test_post_event_400(jp_fetch, event_logger, payload):
 }
 """
 
-payload_8 = """\
+payload_9 = """\
 {
     "schema_id": "http://event.mock.jupyter.org/message",
     "version": 2,
@@ -147,7 +157,7 @@ async def test_post_event_400(jp_fetch, event_logger, payload):
 """
 
 
-@pytest.mark.parametrize("payload", [payload_7, payload_8])
+@pytest.mark.parametrize("payload", [payload_8, payload_9])
 async def test_post_event_500(jp_fetch, event_logger, payload):
     with pytest.raises(tornado.httpclient.HTTPClientError) as e:
         await jp_fetch("api", "events", method="POST", body=payload)