[Bf-blender-cvs] [433ecb8] master: Revert "New cycles tests require buildbot to checkout tests of have test succeed if we test 0 files."

Sergey Sharybin noreply at git.blender.org
Fri Jan 23 16:56:41 CET 2015


Commit: 433ecb88bdf2fbc51cfdc1758300324587790ba9
Author: Sergey Sharybin
Date:   Fri Jan 23 20:48:28 2015 +0500
Branches: master
https://developer.blender.org/rB433ecb88bdf2fbc51cfdc1758300324587790ba9

Revert "New cycles tests require buildbot to checkout tests of have test succeed if we test 0 files."

This reverts commit d390e24c49ba8543b56948402622563cae6e860f.

Forcing tests to success is really bad idea. It'll only lead to cases when
you see PASSED and will think everything is OK.

Long story short: never force tests to pass!

===================================================================

M	tests/python/cycles_render_tests.py

===================================================================

diff --git a/tests/python/cycles_render_tests.py b/tests/python/cycles_render_tests.py
index af0f089..815811f 100755
--- a/tests/python/cycles_render_tests.py
+++ b/tests/python/cycles_render_tests.py
@@ -140,9 +140,6 @@ def main():
     ROOT = args.testdir[0]
     IDIFF = args.idiff[0]
 
-    if no os.path.exists(ROOT):
-        sys.exit(0)
-
     TEMP = tempfile.mkdtemp()
     TEMP_FILE_MASK = os.path.join(TEMP, "test")
     TEMP_FILE = TEMP_FILE_MASK + "0001.png"




More information about the Bf-blender-cvs mailing list