Merge pull request #478 from dodona-edu/fix/b

Fix bug for stdout/stderrr on timeouts

tested/judge/planning.py

@@ -46,7 +46,8 @@ class PlannedExecutionUnit:
     index: int
 
     def get_stdin(self, resources: Path) -> str:
-        return "\n".join(c.context.get_stdin(resources) or "" for c in self.contexts)
+        potential = [c.context.get_stdin(resources) for c in self.contexts]
+        return "\n".join(p for p in potential if p)
 
     def has_main_testcase(self) -> bool:
         return self.contexts[0].context.has_main_testcase()

tested/judge/utils.py

@@ -60,8 +60,8 @@ def run_command(
         )
     except subprocess.TimeoutExpired as e:
         return BaseExecutionResult(
-            stdout=str(e.stdout or ""),
-            stderr=str(e.stderr or ""),
+            stdout=e.stdout.decode("utf-8", "backslashreplace") if e.stdout else "",
+            stderr=e.stderr.decode("utf-8", "backslashreplace") if e.stderr else "",
             exit=0,
             timeout=True,
             memory=False,