Commits

Anonymous committed a5d1582

Better test handling of File lines in stack traces. (Chad Austin)

Comments (0)

Files changed (9)

     fortran_lib = gccFortranLibs()
 
 
+
+file_expr = r"""File "[^"]*", line \d+, in .+
+"""
+
+# re.escape escapes too much.
+def re_escape(str):
+    for c in ['.', '[', ']', '(', ')', '*', '+', '?']:  # Not an exhaustive list.
+        str = string.replace(str, c, '\\' + c)
+    return str
+
+
+
 class TestSCons(TestCommon):
     """Class for testing SCons.
 
 test.write('f3.in', "f3.in 3 \n")
 
 test.run(arguments = 'f1.out',
-         stdout = test.wrap_stdout('.* build.py f3.out f3.in\n.* build.py f1.out f1.in\n'))
+         match = TestCmd.match_exact,
+         stdout = test.wrap_stdout("""\
+%s build.py f3.out f3.in
+%s build.py f1.out f1.in
+""" % (python, python)))
 
 test.up_to_date(arguments = 'f1.out')
 

test/ParseConfig.py

 import os
 import sys
 
+import TestCmd
 import TestSCons
 
 test = TestSCons.TestSCons()
 """ )
 
 test.write(['qt', 'bin', 'myuic.py'], """
+import sys
 import os.path
-import re
-import sys
 import string
 output_arg = 0
 impl_arg = 0
         sourceFile = arg
 if impl:
     output.write( '#include "' + impl + '"\\n' )
-    includes = re.findall('<include.*?>(.*?)</include>', source.read())
-    for incFile in includes:
-        # this is valid for ui.h files, at least
-        if os.path.exists(incFile):
-            output.write('#include "' + incFile + '"\\n')
+    if string.find(source.read(), '// ui.h') != -1:
+        output.write(
+           '#include "' +
+           os.path.basename(os.path.splitext(sourceFile)[0]) + '.ui.h"\\n')
 else:
     output.write( '#include "my_qobject.h"\\n' + source.read() + " Q_OBJECT \\n" )
 output.close()
 test.not_up_to_date(chdir='work2', options = '-n', arguments = cpp)
 test.not_up_to_date(chdir='work2', options = '-n', arguments = h)
 test.run(chdir='work2', arguments = aaa_dll)
+test.write(['work2', 'aaa.ui.h'], r"""
+/* test dependency to .ui.h */
+""")
 test.write(['work2', 'aaa.ui'], r"""
 void aaa(void)
-//<include>aaa.ui.h</include>
-""")
-test.run(chdir='work2', arguments = aaa_dll) # test that non-existant ui.h files are ignored (as uic does)
-test.write(['work2', 'aaa.ui.h'], r"""
-/* test dependency to .ui.h */
+// ui.h
 """)
 test.run(chdir='work2', arguments = aaa_dll)
 test.write(['work2', 'aaa.ui.h'], r"""
 /* changed */
 """)
 test.not_up_to_date(chdir='work2', options = '-n', arguments = obj)
-test.not_up_to_date(chdir='work2', options = '-n', arguments = cpp)
-test.not_up_to_date(chdir='work2', options = '-n', arguments = h)
-test.not_up_to_date(chdir='work2', options = '-n', arguments = moc)
+test.up_to_date(chdir='work2', options = '-n', arguments = cpp)
+test.up_to_date(chdir='work2', options = '-n', arguments = h)
+test.up_to_date(chdir='work2', options = '-n', arguments = moc)
 # clean up
 test.run(chdir='work2', arguments = '-c ' + aaa_dll)
 
                os.path.exists(test.workpath('work2', moc)) or
                os.path.exists(test.workpath('work2', cpp)) or
                os.path.exists(test.workpath('work2', h)))
-cppContents = test.read(test.workpath('work2', 'build', cpp))
-test.fail_test(string.find(cppContents, '#include "aaa.ui.h"') == -1)
-
 test.run(chdir='work2',
          arguments = "build_dir=1 chdir=1 " +
                      test.workpath('work2', 'build', aaa_dll) )
                     ['work2', cpp],
                     ['work2', h])
 
+
 ##############################################################################
 # 3. create a moc file from a cpp file
 
                   CXXFILESUFFIX=".cpp")
 
 conf = env.Configure()
-if not conf.CheckLib(env.subst("$QT_LIB"), autoadd=0):
+if not conf.CheckLib(env.subst("$QT_LIB")):
     conf.env['QT_LIB'] = 'qt-mt'
-    if not conf.CheckLib(env.subst("$QT_LIB"), autoadd=0):
+    if not conf.CheckLib(env.subst("$QT_LIB")):
          Exit(0)
 env = conf.Finish()
-BuildDir('bld', '.')
-env.Program('bld/test_realqt', ['bld/mocFromCpp.cpp',
-                                'bld/mocFromH.cpp',
-                                'bld/anUiFile.ui',
-                                'bld/main.cpp'])
+
+env.Program('test_realqt', ['mocFromCpp.cpp',
+                            'mocFromH.cpp',
+                            'anUiFile.ui',
+                            'main.cpp'])
 """)
 
     test.write( ['work7', 'mocFromCpp.h'],"""
         <string>MyWidget</string>
     </property>
 </widget>
-<includes>
-    <include location="local" impldecl="in implementation">anUiFile.ui.h</include>
-</includes>
-<slots>
-    <slot>testSlot()</slot>
-</slots>
 <layoutdefaults spacing="6" margin="11"/>
 </UI>
 """)
-    test.write( ['work7', 'anUiFile.ui.h'], r"""
-#include <stdio.h>
-#if QT_VERSION >= 0x030100
-void MyWidget::testSlot()
-{
-    printf("Hello World\n");
-}
-#endif
-""")
 
-    test.write( ['work7', 'main.cpp'], r"""
-#include <qapp.h>
+    test.write( ['work7', 'main.cpp'], """
 #include "mocFromCpp.h"
 #include "mocFromH.h"
 #include "anUiFile.h"
-#include <stdio.h>
-    
-int main(int argc, char **argv) {
-  QApplication app(argc, argv);
+int main() {
   mocFromCpp();
   mocFromH();
-  MyWidget mywidget;
-#if QT_VERSION >= 0x030100
-  mywidget.testSlot();
-#else
-  printf("Hello World\n");
-#endif
-  return 0;
+  MyWidget mywidget();
 }
 """)
 
-    test.run(chdir='work7', arguments="bld/test_realqt" + _exe)
-    test.run(program=test.workpath("work7", "bld", "test_realqt"),
-             stdout="Hello World\n")
+    test.run(chdir='work7', arguments="test_realqt" + _exe)
 
     QTDIR=os.environ['QTDIR']
     del os.environ['QTDIR']
     PATH=os.environ['PATH']
     os.environ['PATH']='.'
 
-    test.run(chdir='work7', stderr=None, arguments="-c bld/test_realqt" + _exe)
+    test.run(chdir='work7', stderr=None, arguments="-c test_realqt" + _exe)
     test.fail_test(not test.match_re_dotall(test.stderr(), r""".*
 scons: warning: Could not detect qt, using empty QTDIR
 """ + TestSCons.file_expr))
     os.environ['PATH'] = PATH
 
     test.run(chdir='work7', stderr=None,
-             arguments="PATH=%s%sbin bld%stest_realqt%s"%(QTDIR,os.sep,os.sep,_exe))
+             arguments="PATH=%s%sbin test_realqt%s"%(QTDIR,os.sep,_exe))
     test.fail_test(not test.match_re(test.stderr(), (r"""
 scons: warning: Could not detect qt, using moc executable as a hint \(QTDIR=%s\)
 """ + TestSCons.file_expr) % re.escape(QTDIR)))

test/SConstruct.py

          stdout = "",
          stderr = r"""
 scons: \*\*\* No SConstruct file found.
-File "[^"]+", line \d+, in \S+
-""")
+""" + TestSCons.file_expr)
 
 test.match_func = TestCmd.match_exact
 
 test.run(arguments='file5.out', 
          stderr=TestSCons.re_escape("""
 scons: warning: Two different environments were specified for target file5.out,
-	but they appear to have the same action: build(target, source, env)
+	but they appear to have the same action: build(["file5.out"], ["file5b.in"])
 """) + TestSCons.file_expr)
 
 test.must_match('file5.out', "file5a.in\nfile5b.in\n")

test/option--warn.py

 test.run(arguments='file1.out', 
          stderr=r"""
 scons: warning: Two different environments were specified for target file1.out,
-	but they appear to have the same action: build\(target, source, env\)
+	but they appear to have the same action: build\(\["file1.out"\], \["file1b.in"\]\)
 """ + TestSCons.file_expr)
 
 test.must_match('file1.out', "file1a.in\nfile1b.in\n")
 test.run(arguments='--warn=duplicate-environment file1.out', 
          stderr=r"""
 scons: warning: Two different environments were specified for target file1.out,
-	but they appear to have the same action: build\(target, source, env\)
+	but they appear to have the same action: build\(\["file1.out"\], \["file1b.in"\]\)
 """ + TestSCons.file_expr)
 
 test.run(arguments='--warn=no-duplicate-environment file1.out')
          stderr = None)
 test.fail_test(not test.match_re(test.stderr(), """
 scons: warning: Ignoring missing SConscript 'no_such_file'
-File "[^"]+", line \d+, in .*
-"""))
+""" + TestSCons.file_expr))
 
 test.pass_test()

test/overrides.py

 test.run(arguments='goodbye.not_exe', stderr=None)
 test.fail_test(not test.match_re(test.stderr(), r"""
 scons: warning: Did you mean to use `(target|source)' instead of `(targets|sources)'\?
-File "SConstruct", line \d+, in .+
-
+""" + TestSCons.file_expr + r"""
 scons: warning: Did you mean to use `(target|source)' instead of `(targets|sources)'\?
-File "SConstruct", line \d+, in .+
-"""))
+""" + TestSCons.file_expr))
 
 assert test.read('goodbye.not_obj') == 'this is no object file!'
 assert test.read('goodbye.not_exe') == 'this is not a program!'