Skip to content

Commit 4a43a3c

Browse files
committed
[FIX] Fixed error display on libft and Added a disclaimer for linux users.
1 parent 81c4099 commit 4a43a3c

2 files changed

Lines changed: 6 additions & 1 deletion

File tree

PyChecker/testing_suite/maintest.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
import os
77
import subprocess
88
import shutil
9+
import platform
910

1011

1112
def run_libft(project_path: str, root_path: str):
@@ -64,5 +65,7 @@ def run_libft(project_path: str, root_path: str):
6465
# @todo: Count number of OK and FAILs and yellow tests to get score for maintest
6566
file.write(result + '\n')
6667
print(result)
68+
if platform.system() == "Linux":
69+
print("-- Disclaimer: Some of these test may fail where they woudn't on Darwin. (Because Linux ?)")
6770
os.remove("libft_main.c")
6871
os.remove("libft_main.out")

PyChecker/utils/norme.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,11 @@ def check(project_path: str, root_path: str):
4040
return 3
4141
else:
4242
file.write(result)
43+
error_count = 0
44+
warning_count = 0
4345
error_count = result.count('Error')
4446
warning_count = result.count('Warning')
45-
if error_count != 0 and warning_count != 0:
47+
if error_count != 0 or warning_count != 0:
4648
print("Found {} errors and {} warnings".format(error_count,
4749
warning_count))
4850
return 2

0 commit comments

Comments
 (0)