Fix tests and improve server error debugging

Author: ColdHeat

ctfcli/core/challenge.py

@@ -292,7 +292,8 @@ def _get_initial_challenge_payload(self, ignore: Tuple[str] = ()) -> Dict:
             challenge_payload["connection_info"] = challenge.get("connection_info", None)
 
         if "logic" not in ignore:
-            challenge_payload["logic"] = challenge.get("logic", "any")
+            if challenge.get("logic"):
+                challenge_payload["logic"] = challenge.get("logic") or "any"
 
         if "extra" not in ignore:
             challenge_payload = {**challenge_payload, **challenge.get("extra", {})}
@@ -691,6 +692,8 @@ def sync(self, ignore: Tuple[str] = ()) -> None:
 
         # Update simple properties
         r = self.api.patch(f"/api/v1/challenges/{self.challenge_id}", json=challenge_payload)
+        if r.ok is False:
+            click.secho(f"Failed to sync challenge: ({r.status_code}) {r.text}", fg="red")
         r.raise_for_status()
 
         # Update flags
@@ -815,6 +818,8 @@ def create(self, ignore: Tuple[str] = ()) -> None:
                 challenge_payload[p] = ""
 
         r = self.api.post("/api/v1/challenges", json=challenge_payload)
+        if r.ok is False:
+            click.secho(f"Failed to create challenge: ({r.status_code}) {r.text}", fg="red")
         r.raise_for_status()
 
         self.challenge_id = r.json()["data"]["id"]