summaryrefslogtreecommitdiffstats
path: root/test
Commit message (Collapse)AuthorAgeFilesLines
* [PY 3.8] test fixes for file closings, rawstringsMats Wichmann2019-04-25333-1252/+1355
| | | | | | | On a linux host (missing some things that may be on the Travis CI setup), Py3.8a3 now shows 19 fails, 1048 pass, with 84 Warning: messages. Signed-off-by: Mats Wichmann <mats@linux.com>
* [PY 3.8] modernize common test fixturesMats Wichmann2019-04-255-49/+83
| | | | | | | Apply changes to quiet open-file warnings. Add docstring to a few. Switch os.system usage to subprocess.call. Add if-main logic. Signed-off-by: Mats Wichmann <mats@linux.com>
* Merge branch 'master' into tool-addWilliam Deegan2019-04-2310-36/+45
|\
| * Merge pull request #3339 from mwichmann/ci-multiWilliam Deegan2019-04-237-27/+30
| |\ | | | | | | [WIP] Run all CI builds with -j 2
| | * [PR #3339] Updates for that failedMats Wichmann2019-04-017-27/+30
| | | | | | | | | | | | | | | | | | | | | | | | Since the build with everything-CI-j2 triggered a couple of test fails, include the patches for open-file warnings in those test areas in the hopes it will help. Signed-off-by: Mats Wichmann <mats@linux.com>
| * | Merge branch 'master' into java-versWilliam Deegan2019-04-221-6/+6
| |\ \
| | * | Issue #3350 - Refactor EnvironmentError to SConsEnvironmentError to avoid ↵William Deegan2019-04-181-6/+6
| | | | | | | | | | | | | | | | overriding python's native EnvironmentError
| * | | Move the jdk-version-glob test to unit testsMats Wichmann2019-04-171-94/+0
| | | | | | | | | | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>
| * | | Add a test for JDK glob matchingMats Wichmann2019-04-161-0/+94
| | | | | | | | | | | | | | | | Signed-off-by: Mats Wichmann <mats@linux.com>
| * | | [PR #3352] fix sider complaints: bare exceptMats Wichmann2019-04-142-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sider in the CI system doesn't like the change which added a try/except block for a version without a dot. This was copied from a different file, so to fix the complaint, four instances are actually updated to "except ValueError" - I am assuming that this is the only thing that could go wrong, a version string cannot be converted with int(foo), which would throw ValueError. Signed-off-by: Mats Wichmann <mats@linux.com>
| * | | Fix problems with jdk detectionMats Wichmann2019-04-141-0/+6
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The java tool common routine finds a jdk by doing a filesystem glob. This had a problem on windows in the case a specific version is requested, because the format of name of the jdk directory has changed with JDK 9 - there is a dash between jdk and the version string. The glob which does not attempt to match a version was general enough not to trip on this, but with a version to match it would never match jdk-9 or higher. The test harness then asks the found javac what version it is, and the parsing of that did not work as expected once version numbers became double-digit, as the regex was for a single digit followed by a dot. The outcome is for 11.0.2 we get back '11' instead of '11.0'. Change the regex to match any number of digits followed by dot. The Repository/RMIC.py change is to align with an earlier change to Java/RMIC.py, but the may not be needed after the change to regex just described. Clean up some of the Java tool routines for consistency (there was no functional change outside of JavaCommon.py) Docstrings added or updated in several places. Signed-off-by: Mats Wichmann <mats@linux.com>
* | | [PR #3337] clean up lex and yacc toolsMats Wichmann2019-04-131-1/+1
|/ / | | | | | | | | | | | | Remove now unneeded code to save/restore the path, since the routine now does not modify the path. Signed-off-by: Mats Wichmann <mats@linux.com>
* | Tweaks to packaging tests: avoid default tool listMats Wichmann2019-04-069-12/+12
|/ | | | Signed-off-by: Mats Wichmann <mats@linux.com>
* [PR #333] close files to avoid scons-time racesMats Wichmann2019-03-291-1/+2
| | | | | | | | | | | | | | With runtest now honoring the -j 2 option given to it in CI setup on Windows, there were some problems where scons-time tests could try to remove a test directory while some files in it were still open (these locations were complained about by Python 3.8 also). Switch test framework to using mkdtemp also, and to not use tempfile.template (usage of that and mktemp are long deprecated) Signed-off-by: Mats Wichmann <mats@linux.com>
* [PR #3330] try to eliminate race in scons-timeMats Wichmann2019-03-291-1/+0
| | | | | | | | | | runtest change to run requested number of jobs exposed a concurrency problem on Appveyor CI (Windows) builds - which seems to be due to scons-time using mktemp to create a temporary directory name, and then later creating the directory. In scons-time the timing window for this is a bit longer than usual. Try using mkdtemp instead. Signed-off-by: Mats Wichmann <mats@linux.com>
* Add test to check if installed sdk version is supportedWilliam Deegan2019-03-181-0/+66
|
* Merge pull request #3318 from mwichmann/fedora-rpcWilliam Deegan2019-03-121-0/+8
|\ | | | | rpcgen test accomodate distros using tirpc
| * rpcgen test accomodate distros using tirpcMats Wichmann2019-03-031-0/+8
| | | | | | | | | | | | | | | | | | | | | | Fedora since 28 has removed SunRPC support from glibc and switched to using tirpc. Adjust a test to accomodate the new usage - adds an include path and if configure check passes, add libtirpc to link. This is a test-only change. Signed-off-by: Mats Wichmann <mats@linux.com>
* | Fix issue #2799 - Get mingw tool to respect SHCCCOMSTR, SHLINKCOMSTR and ↵William Deegan2019-03-115-0/+76
| | | | | | | | LDMODULECOMSTR
* | Merge pull request #3316 from mwichmann/pypy-syntaxWilliam Deegan2019-03-072-2/+2
|\ \ | | | | | | [PYPY] adjust two expect strings for PyPy msgs
| * | [PYPY support] adjust two expect strings for PyPy msgsMats Wichmann2019-03-032-2/+2
| |/ | | | | | | | | | | | | | | To help support pypy, two test expect-strings are adjusted to accept either the error message from CPython ("Syntax error: invalid syntax") and from PyPy ("Syntax error: Unknown character"). Signed-off-by: Mats Wichmann <mats@linux.com>
* | Fix logic in test to only use --wincompat if win_flex. Improve test skip ↵William Deegan2019-03-032-3/+4
| | | | | | | | messaging
* | fix typo in skip_testWilliam Deegan2019-03-031-2/+2
| |
* | Add test to cover this issue with MD5-timestamp decider breaking ↵William Deegan2019-03-035-0/+122
|/ | | | dependencies only on windows when one generated souce implicitly depends on another generated source.
* Add support for usign chocolatey install winflexbison package with tools ↵William Deegan2019-03-032-6/+4
| | | | named win_bison
* Merge pull request #3290 from dmoody256/make_lex_tool_cross_platformWilliam Deegan2019-03-038-20/+202
|\ | | | | Add windows default paths to lex tool
| * add win_flex as option for windows, add choco default path, and add flag for ↵Daniel2019-02-207-30/+200
| | | | | | | | nounistd on windows. also more testing
| * add check for tar before trying to run testDaniel2019-02-161-0/+4
| |
| * update lex tool to find paths on windowsDaniel2019-02-161-5/+13
| |
* | Merge pull request #3320 from mwichmann/test-file-closingsWilliam Deegan2019-03-034-38/+37
|\ \ | | | | | | [PYPY] [PY 3.8] more context manager use in tests
| * | For PR #3320 re-unindent expect strings in with blockMats Wichmann2019-03-021-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | Previous change created a with: block but accidentally indented two literal strings used as the expected stderr, causing them to no longer match; these are restored to original state. Signed-off-by: Mats Wichmann <mats@linux.com>
| * | [PYPY] [PY 3.8] more context manager use in testsMats Wichmann2019-03-024-41/+40
| | | | | | | | | | | | | | | | | | | | | | | | Similar to #3319, add additional use of context managers inside written tests. Fixes som PyPy3 fails and quiets matching warnings from Python 3.8. Signed-off-by: Mats Wichmann <mats@linux.com>
* | | [PYPY3] [PY 3.8] CacheDir tests use context managersMats Wichmann2019-03-0213-58/+70
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Python 3.8 complains about unclosed files on nearly all of the CacheDir tests. PyPy3 fails 8 of the 20 tests for the same reason, though since it's based on an earlier version of Python (3.5 at the moment) it does not display the resourcewarning messages. Update sequences of open().write() which are the primary problem area to use context managers for automatic closing, and for consistency, the rest of the open/read/write stuff as well. With this change, all these tests pass in PyPy3. Python 3.8 does not, as it is spewing other warning messages which also end up in the stderr, which is fatal to this set of tests, but it has quieted the warnings from the CacheDir tests themselves. Signed-off-by: Mats Wichmann <mats@linux.com>
* | Merge pull request #3314 from mwichmann/runtest-no-ttWilliam Deegan2019-03-0116-30/+50
|\ \ | | | | | | for #3304: drop use of -tt if py3
| * | [WIP] for #3304: drop use of -tt if py3Mats Wichmann2019-03-0116-30/+50
| |/ | | | | | | | | | | | | | | | | | | As noted in issue #3304, the Python flag to error on inconsistent tabs/spaces has been dropped for Python 3 interpreters; while CPython still accepts it, silently ignoring it, PyPy3 errors out. This change adds the flag throughout the testsuite run only if using a Python2 interpreter. Signed-off-by: Mats Wichmann <mats@linux.com>
* | Merge pull request #3313 from ↵William Deegan2019-02-282-0/+14
|\ \ | |/ |/| | | | | bdbaddog/fix_gh_3303_force_config_alters_env_decider Fix issue #3303 - --config=force overwriting environment passed to Configure()
| * Add test for GH Issue #3303William Deegan2019-02-272-0/+14
| |
* | Merge pull request #3270 from dmoody256/mingw_link_issueWilliam Deegan2019-02-151-0/+102
|\ \ | | | | | | Mingw link issue
| * \ Merge remote-tracking branch 'origin/master' into mingw_link_issueDaniel2019-02-086-64/+139
| |\ \
| * | | switch to 32 bit mingw in appveyor and add mingw env testDaniel2019-02-081-0/+102
| | | |
* | | | Clean up some tests: use context managersMats Wichmann2019-02-1414-32/+57
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | Plenty of complaints coming from Python 3.8alpha on unclosed files. Targeted those areas which intersect with PyPy failures - this changeset reduces the PyPy fails by 17 on the local test environment. So this affects both Issue #3299 and the PyPy support project. Signed-off-by: Mats Wichmann <mats@linux.com>
* | | Fix is/is not syntaxMats Wichmann2019-02-111-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | In a few places, "is" and "is not" are used to compare with a string or integer literal. Python 3.8 flags these with a SyntaxWarning. Changed to == and != Signed-off-by: Mats Wichmann <mats@linux.com>
* | | Add textfile tool to defaultsMats Wichmann2019-02-081-1/+3
| |/ |/| | | | | | | | | | | | | | | PR #3242 added the Textfile and Substfile builders to the default builder list (for issue #3147), but didn't finish the job: the textfile tool needs to be added to the default list of tools as well. This time with a testcase that fails if the tool is not added. Minor doc tweak. Signed-off-by: Mats Wichmann <mats@linux.com>
* | Merge pull request #3286 from dmoody256/make_m4_test_crossplatformWilliam Deegan2019-02-061-26/+12
|\ \ | | | | | | make M4 test cross platform
| * | update m4 test to use mingw on windows and not pass if m4 doesn't existDaniel Moody2019-02-041-26/+12
| | |
* | | add sconstest.skip for test fixturesDaniel2019-02-051-0/+0
| | |
* | | fix for case where nothing is return from vswhereDaniel2019-02-053-0/+73
|/ /
* | Fix new test binary/text read issue for py3.5+. Changed mode to 'r' as just ↵William Deegan2019-02-021-42/+39
| | | | | | | | plain part of a line
* | Add test to check for configure calling configured decider not handling ↵William Deegan2019-02-021-0/+19
|/ | | | DeciderNeedsNode exception which can be thrown by MD5-Timestamp decider logic
* Add test for TEMPFILESUFFIXMats Wichmann2019-01-162-3/+130
| | | | Signed-off-by: Mats Wichmann <mats@linux.com>