assigments/parallel-firewall: Add resilience to compilation failures

Make checker look out for the binary to run because if the compiation
failed, there won't be any.

Also, use `$(MAKE)` instead of plain `make` from inside the
`tests/Makefile` as that is the intended way to do it.

Also 2, removing `-i` from the makefile invocation make the inner
makefile fail after the first error, and adding `-` makes it not stop
the outter makefile rule (i.e. also run the checker)

Signed-off-by: Anton-Fabian Patras <[email protected]>

content/assignments/parallel-firewall/tests/Makefile

@@ -6,10 +6,10 @@ UTILS_PATH = $(realpath ../utils)
 all: src
 
 src:
-	make -C $(SRC_PATH) UTILS_PATH=$(UTILS_PATH)
+	$(MAKE) -C $(SRC_PATH) UTILS_PATH=$(UTILS_PATH)
 
 check: clean
-	make -i SRC_PATH=$(SRC_PATH)
+	-$(MAKE) SRC_PATH=$(SRC_PATH)
 	SRC_PATH=$(SRC_PATH) python3 checker.py
 
 lint:
@@ -20,7 +20,7 @@ lint:
 	# -cd $(SRC_PATH)/.. && pylint tests/*.py
 
 clean:
-	make -C $(SRC_PATH) clean
+	$(MAKE) -C $(SRC_PATH) clean
 	-rm -f *~
 	-rm -rf __pycache__
 	-rm -rf out

content/assignments/parallel-firewall/tests/checker.py

@@ -175,6 +175,10 @@ def run_once_and_check_output(
     except FileNotFoundError:
         pass
 
+    if not os.path.exists(binary):
+        print("No program to run, maybe compilation failed?")
+        return False
+
     with subprocess.Popen(
         [binary, in_file_path, out_file_path, f"{threads}"],
         stdout=subprocess.PIPE,