Update SCMClient unit tests to check against run_process_exec().
Review Request #12580 — Created Sept. 5, 2022 and submitted
Information | |
---|---|
chipx86 | |
RBTools | |
release-4.x | |
Reviewers | |
rbtools | |
We're working to move away from
execute()
torun_process()
, and as
part of that we've madeexecute()
a wrapper aroundrun_process()
.To help with the porting effort, all SCMClient unit tests have been
updated to test againstrun_process_exec()
(the hook point for
testing) instead ofexecute()
.
Unit tests pass.
Change Summary:
Updated the ClearCase tests for the latest directory diffing changes.
Commits: |
|
|||||||||
---|---|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+2204 -1012) |
Checks run (2 succeeded)
flake8
passed.
JSHint
passed.
-
-
rbtools/clients/tests/test_git.py (Diff revision 2) Aren't we supposed to spy on
run_process_exec
? Here and the rest of this file.
Change Summary:
Fixed Git unit tests to spy on
run_process_exec()
.
Commits: |
|
|||||||||
---|---|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 3 (+2208 -1012) |
Checks run (2 succeeded)
flake8
passed.
JSHint
passed.