[Lldb-commits] [PATCH] D68354: [platform process list] add a flag for showing the processes of all users

[walter erquinigo via Phabricator via lldb-commits]

wallace marked an inline comment as done.
wallace added inline comments.


================
Comment at: lldb/packages/Python/lldbsuite/test/functionalities/gdb_remote_client/TestPlatformClient.py:30
+            self.expect("platform process list -x",
+                        startstr="1 matching process was found", endstr="process" + os.linesep)
+            self.expect("platform process list",
----------------
labath wrote:
> Maybe also check that some of the values from the mock response made it through? (e.g., pid, process name)
I'm adding that in the other diffs, but i'll make stricter checks anyway


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68354/new/

https://reviews.llvm.org/D68354