Update SCMClient unit tests to check against run_process_exec().

Review Request #12580 — Created Sept. 5, 2022 and submitted

Information

RBTools
release-4.x

Reviewers

We're working to move away from execute() to run_process(), and as
part of that we've made execute() a wrapper around run_process().

To help with the porting effort, all SCMClient unit tests have been
updated to test against run_process_exec() (the hook point for
testing) instead of execute().

Unit tests pass.

Summary ID
Update SCMClient unit tests to check against run_process_exec().
We're working to move away from `execute()` to `run_process()`, and as part of that we've made `execute()` a wrapper around `run_process()`. To help with the porting effort, all SCMClient unit tests have been updated to test against `run_process_exec()` (the hook point for testing) instead of `execute()`.
fee770ba34105a3c1b40f33e6276c4aa5d46e9be
Description From Last Updated

This will need to be updated for the recent changes to the clearcase tests.

daviddavid

Aren't we supposed to spy on run_process_exec? Here and the rest of this file.

daviddavid
david
  1. 
      
  2. This will need to be updated for the recent changes to the clearcase tests.

  3. 
      
chipx86
david
  1. 
      
  2. 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.

  3. 
      
chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-4.x (b7f12fe)
Loading...