Fixed --append bug in testing.py

bin/testing.py

@@ -12,13 +12,13 @@ It additionally installs all dependencies from a '../requirements.txt' via pip.
 Use command line argument '-w' to run on windows.
 '''
 
-PY = sys.argv[2] if (len(sys.argv) > 1 and sys.argv[1] == '-py') else 'python3.7' # use -py to use your own python command
-COVERAGE = "python3.7 -m coverage"
+PY = sys.argv[2] if (len(sys.argv) > 1 and sys.argv[1] == '-py') else 'python' # use -py to use your own python command
 ROOT = pathlib.Path(__file__).parent.parent.absolute()
 REPORTS = ROOT / 'reports'
 TESTS_FOLDER_NAME = os.path.normpath("/tests")
 
-print("\nSearching for tests at the path: "+ str(ROOT) +"\n")
+print("\nSearching for tests at the path: "+ str(ROOT))
+
 count = 0
 resultCodeList = []
 coverage_paths_set = set()
@@ -31,13 +31,15 @@ for (dirname, dirs, files) in os.walk(ROOT):
         and not("venv" in str(dirname)) \
         and not("Lib" in str(dirname)):
         try:
-            print(f"Executing tests in {dirname} \n")
+            print(f"Executing tests in {dirname}")
             os.chdir(os.path.normpath(dirname))
             # TODO do this during docker image setup
             exit_val = os.system(f"{PY} -m pip install -r ../requirements.txt") # install pip dependencies
-            #exit_val = os.system(f"{PY} -m unittest discover") # OLD execute the tests
-            exit_val = os.system(f"{COVERAGE} --append --omit=*/site-packages*,*/dist-packages* -m unittest discover") #TEST CODE COVERAGE
+            #resultCodeList.append(exit_val)
+            #exit_val = os.system(f"{PY} -m unittest discover") # execute the tests
+            exit_val = os.system(f"python3.7 -m coverage run --append --omit=*/site-packages*,*/dist-packages* -m unittest discover") #TEST CODE COVERAGE
             coverage_paths_set.add(os.path.normpath(dirname))
+            
             resultCodeList.append(exit_val) #once per folder  i.e if 3 tests are in a folder and crash, there will be just one exit val
         except Exception as e:
             print(e)
@@ -50,28 +52,31 @@ for (dirname, dirs, files) in os.walk(ROOT):
             for filename in os.listdir(cur_dir):
                 if filename_regular_expresion.match(filename): 
                     #gets here only if there is a test file which matches the regular expression in the app folder,
+                    
+                    #cur_dir = os.path(dirname).parent()
                     os.chdir(cur_dir)
-                    print(f"Executing coverage test in {cur_dir} \n")
-                    exit_val = os.system(f"{COVERAGE} run --append --omit=*/site-packages* -m unittest discover")
+                    print(f"Executing coverage test in {cur_dir}")
+                    exit_val = os.system(f"python3.7 -m coverage run --append --omit=*/site-packages* -m unittest discover")
                     coverage_paths_set.add(os.path.normpath(cur_dir))
 
         except Exception as e:
             print(e)
             continue
         
+
+
 #CHANGE FOLDER TO REPORTS, in order to combine the coverage
 try:
     if not os.path.exists(REPORTS):
         os.makedirs(REPORTS) 
 except:
     pass
+
+
 try: 
     os.chdir(REPORTS)
     target = REPORTS
     target = os.path.normpath( str(target) + f'/.coverage' )
-except:
-    pass
-try:    
     os.remove(target) #Try to Remove old coverage file, if exists
 except Exception as e:
     pass
@@ -85,6 +90,7 @@ for path in coverage_paths_set:
         target = REPORTS
         target = os.path.normpath( str(target) + f'/.coverage.{counter}' )
         counter += 1
+
         shutil.copyfile(original,target) #copy new generated coverage files
         os.remove(original)
     except Exception as e:
@@ -92,15 +98,15 @@ for path in coverage_paths_set:
         continue
 
 print("Generating Combined report")
-os.system(f"{COVERAGE} combine")
-os.system(f"{COVERAGE} xml")
-os.system(f"{COVERAGE} html")
+os.system("python3.7 -m coverage combine")
+os.system("python3.7 -m coverage xml")
+os.system("python3.7 -m coverage html") #if you want to generate the html as well
 
 firstError = -1
 i = 0
 while i < len(resultCodeList):
     if resultCodeList[i] != 0:
-        print("\nA test failed with code: "+ str(resultCodeList[i]))
+        # print("\nA test failed with code: "+ str(resultCodeList[i]))
         if (firstError < 0):
             firstError = i
     i += 1