diff options
author | Mats Wichmann <mats@linux.com> | 2018-08-09 15:38:41 (GMT) |
---|---|---|
committer | Mats Wichmann <mats@linux.com> | 2018-08-09 15:38:41 (GMT) |
commit | 522d235045a3e485c12f34ecb631c1ecfdd4c216 (patch) | |
tree | b7ca066fa792885da83ba95733b8713591c57741 | |
parent | 15ec7da378436e4d9ddb197a6bad43c7059b336c (diff) | |
download | SCons-522d235045a3e485c12f34ecb631c1ecfdd4c216.zip SCons-522d235045a3e485c12f34ecb631c1ecfdd4c216.tar.gz SCons-522d235045a3e485c12f34ecb631c1ecfdd4c216.tar.bz2 |
Fix the new missing-sconscript test for Windows
Missed that string matching on the invalid sconscript
path needs to be OS-agnostic - the CI builders don't
include a windows image, so this was not caught.
Call normpath on the path before pushing it into the
expected strings.
Signed-off-by: Mats Wichmann <mats@linux.com>
-rw-r--r-- | test/SConscript/must_exist.py | 27 |
1 files changed, 15 insertions, 12 deletions
diff --git a/test/SConscript/must_exist.py b/test/SConscript/must_exist.py index ac90cd1..3faf0ce 100644 --- a/test/SConscript/must_exist.py +++ b/test/SConscript/must_exist.py @@ -29,6 +29,7 @@ Test handling of must_exist flag and global setting requiring the file to exist in an SConscript call ''' +import os import TestSCons test = TestSCons.TestSCons() @@ -82,31 +83,33 @@ except SCons.Errors.UserError as e: # we should see two exceptions as "Fatal" and # and see four warnings, the first having the depr message +# need to build the path in the expected msg in an OS-agnostic way +missing = os.path.normpath('missing/SConscript') warn1 = """ scons: warning: Calling missing SConscript without error is deprecated. Transition by adding must_exist=0 to SConscript calls. -Missing SConscript 'missing/SConscript' -""" + test.python_file_line(SConstruct_path, 7) +Missing SConscript '{}' +""".format(missing) + test.python_file_line(SConstruct_path, 7) warn2 = """ -scons: warning: Ignoring missing SConscript 'missing/SConscript' -""" + test.python_file_line(SConstruct_path, 13) +scons: warning: Ignoring missing SConscript '{}' +""".format(missing) + test.python_file_line(SConstruct_path, 13) err1 = """ -scons: warning: Fatal: missing SConscript 'missing/SConscript' -""" + test.python_file_line(SConstruct_path, 22) +scons: warning: Fatal: missing SConscript '{}' +""".format(missing) + test.python_file_line(SConstruct_path, 22) warn3 = """ -scons: warning: Ignoring missing SConscript 'missing/SConscript' -""" + test.python_file_line(SConstruct_path, 25) +scons: warning: Ignoring missing SConscript '{}' +""".format(missing) + test.python_file_line(SConstruct_path, 25) err2 = """ -scons: warning: Fatal: missing SConscript 'missing/SConscript' -""" + test.python_file_line(SConstruct_path, 35) +scons: warning: Fatal: missing SConscript '{}' +""".format(missing) + test.python_file_line(SConstruct_path, 35) warn4 = """ -scons: warning: Ignoring missing SConscript 'missing/SConscript' -""" + test.python_file_line(SConstruct_path, 38) +scons: warning: Ignoring missing SConscript '{}' +""".format(missing) + test.python_file_line(SConstruct_path, 38) expect_stderr = warn1 + warn2 + err1 + warn3 + err2 + warn4 test.run(arguments = ".", stderr = expect_stderr) |