Fix /ping endpoint intermittent 500 errors

Remove conditional exception that caused /ping to fail when time.time() % 3 == 0.
Health check endpoints should be deterministic and always succeed.
Add test coverage for /ping endpoint.

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}