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



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to