Fix /ping endpoint 5xx errors by removing deliberate exception

Removed the time-based exception that caused ~33% 500 error rate.
Added test coverage to ensure /ping always returns 200 status.

Fixes demo-prod/flow-test alert for GET /ping 5xx errors.

Author: michaeljguarino

app/main.py

@@ -12,9 +12,6 @@
 
 @app.get("/ping")
 def test():
-  if int(time.time()) % 3 == 0:
-    raise Exception("unknown internal error")
-
   return {"pong": True}
 
 @app.get("/hello")

app/test_main.py

@@ -7,3 +7,8 @@
 def test_read_root():
     response = client.get("/")
     assert response.status_code == 200
+
+def test_ping():
+    response = client.get("/ping")
+    assert response.status_code == 200
+    assert response.json() == {"pong": True}