Changeset 3464

Timestamp:

Aug 8, 2006, 4:50:33 PM (18 years ago)

Author:

ole

Message:

Fixed up test_all.py which failed after removal of pymetis and parallel. Also added better error message to deal with such cases.

Remember *always* to run test suite before committing!

File:

• inundation/test_all.py (modified) (2 diffs)

inundation/test_all.py

@@ -16 +16 @@
 #List files that should be excluded from the testing process.
 #E.g. if they are known to fail and under development
-exclude_files = ['test_metis.py', 'test_version.py', 'test_parallel_sw.py',
+exclude_files = ['test_version.py', 
                  'test_advection.py', # removing this test for a bit
                  ]
@@ -76 +76 @@
     if globals().has_key('exclude_files'):
         for file in exclude_files:
-            files.remove(file)
-            print 'WARNING: File '+ file + ' excluded from testing'
+            print 'WARNING: File '+ file + ' to be excluded from testing'
+            try:    
+                files.remove(file)
+            except ValueError, e:
+                msg = 'File "%s" was not found in test suite.\n' %file
+                msg += 'Original error is "%s"\n' %e
+                msg += 'Perhaps it should be removed from exclude list?' 
+                raise Exception, msg
 
     filenameToModuleName = lambda f: os.path.splitext(f)[0]