[Lldb-commits] [PATCH] D68293: [android/process list] support showing process arguments

[Pavel Labath via Phabricator via lldb-commits]

labath added inline comments.


================
Comment at: lldb/packages/Python/lldbsuite/test/functionalities/gdb_remote_client/TestPlatformClient.py:63-66
+            self.assertRaises(
+                Exception,
+                lambda : self.expect("platform process list -xv", substrs=["/system/bin/ls"])
+            )
----------------
wallace wrote:
> labath wrote:
> > Is this a negative check? We usually use `expect(..., error=True)` for that
> It's a negative test, but error=True checks for the success of the command itself, it doesn't check the substrs.
> I took at look at the base classes for testing and it seems I could add something like not_contain_substrs, but the code seems very fragile
Ok, how about `matching=False` then? Or maybe you could just put the entirety of the output to expect into the `patterns` argument, as it should not contain any values which vary from run to run.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68293