summaryrefslogtreecommitdiffstats
path: root/SCons/Taskmaster/TaskmasterTests.py
Commit message (Collapse)AuthorAgeFilesLines
* Followon to PR #4348: more bool fixesMats Wichmann2023-06-161-11/+11
| | | | | | | | | | | | | | Manually fixed up some things related to bool, e.g. simple functions which just did "return 1" were interpreted by the tool as returning int, when bool was really the intent. Functions/methods named like "is*", "has*", "exists", "rexists" (and others) are now pretty consistently marked as returning bool. A couple of minor alignments of branched definitions, and a couple of docstring adjustments made. If Tools which had old heading style were touched, they got the new style. Signed-off-by: Mats Wichmann <mats@linux.com>
* Drop dead code from unit testsMats Wichmann2023-05-221-9/+0
| | | | | | | | | | Remove dead code: some mocked classes in unit tests had methods which have been removed from the Node class they're mocking, there's no need to shadow those any more as there are no callers. The methods are depends_on (base functionality removed in 2005 ) and is_pseudeo_derived (base functionality removed in 2006). Signed-off-by: Mats Wichmann <mats@linux.com>
* Add some cheap return and parameter annotationsMats Wichmann2023-05-011-45/+45
| | | | | | | | | | | | | | | | | Use: https://github.com/JelleZijlstra/autotyping to add "safe" return annotations. Where a parameter has a default value that is an obvious scalar type (bool, int, str, etc.) add those annotations as well. Also fixed two small bugs that popped up when sanity-checking with mypy. One in FortranCommon, where a return had been previously annotated to be a tuple of Action, which should be ActionBase - Action is the factory function, not the base class. The other was a typo in the error raised in _add_cppdefines - the message was formatted with the value of "define" which should have been "defines". Signed-off-by: Mats Wichmann <mats@linux.com>
* Added test for failing to unlink cached target filesWilliam Deegan2022-12-031-3/+56
|
* Fixed taskmaster trace tests. Previously there was an extra line at the end ↵William Deegan2022-10-261-46/+61
| | | | of the file. It's no longer there. Added TestCommon.detailed_diff() function which can be used to diff large text blobs expected vs actual
* [ci skip] Fix sider/PEP8 complaintsWilliam Deegan2022-10-231-2/+2
|
* Migrate Taskmaster tracing to use python loggingWilliam Deegan2022-10-221-3/+12
|
* move Taskmaster and Jobs to SCons.TaskmasterWilliam Deegan2022-10-121-0/+1257