google / mobly

E2E test framework for tests with complex environment requirements.
https://github.com/google/mobly
Apache License 2.0
621 stars 174 forks source link

Fix whitespace stripping in _collect_process_tree #891

Closed puradox closed 5 months ago

puradox commented 10 months ago

Fix UtilsTest.test_stop_standing_subproc_and_descendants failure for versions of ps that do not include a space after each newline.


This change is Reviewable

xpconanfan commented 5 months ago

Looked into the related code more.

Looks like the current logic for stopping all children of a pid is a bit cumbersome. We should clean it up soon.

So merging this in as is since it'll be cleaned up next.