kastiglione updated this revision to Diff 296217.
kastiglione added a comment.
"missing a matcher" -> "missing a matcher argument"
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88792/new/
https://reviews.llvm.org/D88792
Files:
lldb/packages/Python/lldbsuite/test/lldbtest.py
lldb/test/API/assert_messages_test/TestAssertMessages.py
lldb/test/API/commands/frame/diagnose/bad-reference/TestBadReference.py
lldb/test/API/commands/frame/diagnose/complicated-expression/TestComplicatedExpression.py
lldb/test/API/commands/frame/diagnose/dereference-argument/TestDiagnoseDereferenceArgument.py
lldb/test/API/commands/frame/diagnose/dereference-this/TestDiagnoseDereferenceThis.py
lldb/test/API/commands/frame/diagnose/inheritance/TestDiagnoseInheritance.py
lldb/test/API/commands/frame/diagnose/local-variable/TestLocalVariable.py
lldb/test/API/commands/frame/diagnose/virtual-method-call/TestDiagnoseDereferenceVirtualMethodCall.py
lldb/test/API/commands/settings/TestSettings.py
lldb/test/API/driver/batch_mode/TestBatchMode.py
lldb/test/API/functionalities/breakpoint/breakpoint_by_line_and_column/TestBreakpointByLineAndColumn.py
lldb/test/API/functionalities/data-formatter/data-formatter-objc/cmtime/TestDataFormatterCMTime.py
lldb/test/API/lang/cpp/constructors/TestCppConstructors.py
lldb/test/API/macosx/macCatalyst/TestMacCatalyst.py
lldb/test/API/types/TestRecursiveTypes.py
Index: lldb/test/API/types/TestRecursiveTypes.py
===================================================================
--- lldb/test/API/types/TestRecursiveTypes.py
+++ lldb/test/API/types/TestRecursiveTypes.py
@@ -50,5 +50,5 @@
self.runCmd("run", RUN_SUCCEEDED)
- self.expect("print tpi", RUN_SUCCEEDED)
- self.expect("print *tpi", RUN_SUCCEEDED)
+ self.expect("print tpi")
+ self.expect("print *tpi")
Index: lldb/test/API/macosx/macCatalyst/TestMacCatalyst.py
===================================================================
--- lldb/test/API/macosx/macCatalyst/TestMacCatalyst.py
+++ lldb/test/API/macosx/macCatalyst/TestMacCatalyst.py
@@ -24,8 +24,8 @@
self.expect("image list -t -b",
patterns=[self.getArchitecture() +
r'.*-apple-ios.*-macabi a\.out'])
- self.expect("fr v s", "Hello macCatalyst")
- self.expect("p s", "Hello macCatalyst")
+ self.expect("fr v s", substrs=["Hello macCatalyst"])
+ self.expect("p s", substrs=["Hello macCatalyst"])
self.check_debugserver(log)
def check_debugserver(self, log):
Index: lldb/test/API/lang/cpp/constructors/TestCppConstructors.py
===================================================================
--- lldb/test/API/lang/cpp/constructors/TestCppConstructors.py
+++ lldb/test/API/lang/cpp/constructors/TestCppConstructors.py
@@ -19,7 +19,7 @@
self.expect_expr("ClassWithDeletedDefaultCtor(7).value", result_type="int", result_value="7")
# FIXME: It seems we try to call the non-existent default constructor here which is wrong.
- self.expect("expr ClassWithDefaultedCtor().foo()", error=True, substrs="Couldn't lookup symbols:")
+ self.expect("expr ClassWithDefaultedCtor().foo()", error=True, substrs=["Couldn't lookup symbols:"])
# FIXME: Calling deleted constructors should fail before linking.
self.expect("expr ClassWithDeletedCtor(1).value", error=True, substrs=["Couldn't lookup symbols:"])
Index: lldb/test/API/functionalities/data-formatter/data-formatter-objc/cmtime/TestDataFormatterCMTime.py
===================================================================
--- lldb/test/API/functionalities/data-formatter/data-formatter-objc/cmtime/TestDataFormatterCMTime.py
+++ lldb/test/API/functionalities/data-formatter/data-formatter-objc/cmtime/TestDataFormatterCMTime.py
@@ -48,6 +48,6 @@
self.expect(
'frame variable t4',
substrs=['10 seconds', 'value = 10', 'timescale = 1', 'epoch = 0'])
- self.expect('frame variable t5', '-oo')
- self.expect('frame variable t6', '+oo')
- self.expect('frame variable t7', 'indefinite')
+ self.expect('frame variable t5', substrs=['+oo'])
+ self.expect('frame variable t6', substrs=['-oo'])
+ self.expect('frame variable t7', substrs=['indefinite'])
Index: lldb/test/API/functionalities/breakpoint/breakpoint_by_line_and_column/TestBreakpointByLineAndColumn.py
===================================================================
--- lldb/test/API/functionalities/breakpoint/breakpoint_by_line_and_column/TestBreakpointByLineAndColumn.py
+++ lldb/test/API/functionalities/breakpoint/breakpoint_by_line_and_column/TestBreakpointByLineAndColumn.py
@@ -21,7 +21,7 @@
main_c = lldb.SBFileSpec("main.c")
_, _, _, breakpoint = lldbutil.run_to_line_breakpoint(self,
main_c, 11, 50)
- self.expect("fr v did_call", substrs='1')
+ self.expect("fr v did_call", substrs=['1'])
in_then = False
for i in range(breakpoint.GetNumLocations()):
b_loc = breakpoint.GetLocationAtIndex(i).GetAddress().GetLineEntry()
@@ -35,7 +35,7 @@
self.build()
main_c = lldb.SBFileSpec("main.c")
_, _, _, breakpoint = lldbutil.run_to_line_breakpoint(self, main_c, 11)
- self.expect("fr v did_call", substrs='0')
+ self.expect("fr v did_call", substrs=['0'])
in_condition = False
for i in range(breakpoint.GetNumLocations()):
b_loc = breakpoint.GetLocationAtIndex(i).GetAddress().GetLineEntry()
Index: lldb/test/API/driver/batch_mode/TestBatchMode.py
===================================================================
--- lldb/test/API/driver/batch_mode/TestBatchMode.py
+++ lldb/test/API/driver/batch_mode/TestBatchMode.py
@@ -44,7 +44,7 @@
child.expect_exact('(char *) touch_me_not')
# Then we should have a live prompt:
self.expect_prompt()
- self.expect("frame variable touch_me_not", substrs='(char *) touch_me_not')
+ self.expect("frame variable touch_me_not", substrs=['(char *) touch_me_not'])
@expectedFlakeyFreeBSD("llvm.org/pr25172 fails rarely on the buildbot")
def test_batch_mode_run_exit(self):
Index: lldb/test/API/commands/settings/TestSettings.py
===================================================================
--- lldb/test/API/commands/settings/TestSettings.py
+++ lldb/test/API/commands/settings/TestSettings.py
@@ -461,15 +461,15 @@
# if they are provided
self.runCmd("settings set thread-format 'abc def' ")
self.expect("settings show thread-format",
- 'thread-format (format-string) = "abc def"')
+ startstr='thread-format (format-string) = "abc def"')
self.runCmd('settings set thread-format "abc def" ')
self.expect("settings show thread-format",
- 'thread-format (format-string) = "abc def"')
+ startstr='thread-format (format-string) = "abc def"')
# Make sure when no quotes are provided that we maintain any trailing
# spaces
self.runCmd('settings set thread-format abc def ')
self.expect("settings show thread-format",
- 'thread-format (format-string) = "abc def "')
+ startstr='thread-format (format-string) = "abc def "')
self.runCmd('settings clear thread-format')
def test_settings_with_trailing_whitespace(self):
Index: lldb/test/API/commands/frame/diagnose/virtual-method-call/TestDiagnoseDereferenceVirtualMethodCall.py
===================================================================
--- lldb/test/API/commands/frame/diagnose/virtual-method-call/TestDiagnoseDereferenceVirtualMethodCall.py
+++ lldb/test/API/commands/frame/diagnose/virtual-method-call/TestDiagnoseDereferenceVirtualMethodCall.py
@@ -22,4 +22,5 @@
self.runCmd("run", RUN_SUCCEEDED)
self.expect("thread list", "Thread should be stopped",
substrs=['stopped'])
- self.expect("frame diagnose", "Crash diagnosis was accurate", "foo")
+ self.expect("frame diagnose", "Crash diagnosis was accurate",
+ substrs=["foo"])
Index: lldb/test/API/commands/frame/diagnose/local-variable/TestLocalVariable.py
===================================================================
--- lldb/test/API/commands/frame/diagnose/local-variable/TestLocalVariable.py
+++ lldb/test/API/commands/frame/diagnose/local-variable/TestLocalVariable.py
@@ -22,4 +22,5 @@
self.runCmd("run", RUN_SUCCEEDED)
self.expect("thread list", "Thread should be stopped",
substrs=['stopped'])
- self.expect("frame diagnose", "Crash diagnosis was accurate", "myInt")
+ self.expect("frame diagnose", "Crash diagnosis was accurate",
+ substrs=["myInt"])
Index: lldb/test/API/commands/frame/diagnose/inheritance/TestDiagnoseInheritance.py
===================================================================
--- lldb/test/API/commands/frame/diagnose/inheritance/TestDiagnoseInheritance.py
+++ lldb/test/API/commands/frame/diagnose/inheritance/TestDiagnoseInheritance.py
@@ -22,4 +22,5 @@
self.runCmd("run", RUN_SUCCEEDED)
self.expect("thread list", "Thread should be stopped",
substrs=['stopped'])
- self.expect("frame diagnose", "Crash diagnosis was accurate", "d")
+ self.expect("frame diagnose", "Crash diagnosis was accurate",
+ substrs=["d"])
Index: lldb/test/API/commands/frame/diagnose/dereference-this/TestDiagnoseDereferenceThis.py
===================================================================
--- lldb/test/API/commands/frame/diagnose/dereference-this/TestDiagnoseDereferenceThis.py
+++ lldb/test/API/commands/frame/diagnose/dereference-this/TestDiagnoseDereferenceThis.py
@@ -25,4 +25,4 @@
self.expect(
"frame diagnose",
"Crash diagnosis was accurate",
- "this->a")
+ substrs=["this->a"])
Index: lldb/test/API/commands/frame/diagnose/dereference-argument/TestDiagnoseDereferenceArgument.py
===================================================================
--- lldb/test/API/commands/frame/diagnose/dereference-argument/TestDiagnoseDereferenceArgument.py
+++ lldb/test/API/commands/frame/diagnose/dereference-argument/TestDiagnoseDereferenceArgument.py
@@ -25,4 +25,4 @@
self.expect(
"frame diagnose",
"Crash diagnosis was accurate",
- "f->b->d")
+ substrs=["f->b->d"])
Index: lldb/test/API/commands/frame/diagnose/complicated-expression/TestComplicatedExpression.py
===================================================================
--- lldb/test/API/commands/frame/diagnose/complicated-expression/TestComplicatedExpression.py
+++ lldb/test/API/commands/frame/diagnose/complicated-expression/TestComplicatedExpression.py
@@ -25,4 +25,4 @@
self.expect(
"frame diagnose",
"Crash diagnosis was accurate",
- "f->b->d")
+ substrs=["f->b->d"])
Index: lldb/test/API/commands/frame/diagnose/bad-reference/TestBadReference.py
===================================================================
--- lldb/test/API/commands/frame/diagnose/bad-reference/TestBadReference.py
+++ lldb/test/API/commands/frame/diagnose/bad-reference/TestBadReference.py
@@ -22,4 +22,5 @@
self.runCmd("run", RUN_SUCCEEDED)
self.expect("thread list", "Thread should be stopped",
substrs=['stopped'])
- self.expect("frame diagnose", "Crash diagnosis was accurate", "f->b")
+ self.expect("frame diagnose", "Crash diagnosis was accurate",
+ substrs=["f->b"])
Index: lldb/test/API/assert_messages_test/TestAssertMessages.py
===================================================================
--- lldb/test/API/assert_messages_test/TestAssertMessages.py
+++ lldb/test/API/assert_messages_test/TestAssertMessages.py
@@ -113,3 +113,20 @@
Expecting start string: "cat" (was not found)
Reason for check goes here!""")
+
+ # Verify expect() preconditions.
+ # Both `patterns` and `substrs` cannot be of type string.
+ self.assert_expect_fails_with("any command",
+ dict(patterns="some substring"),
+ "patterns must be a collection of strings")
+ self.assert_expect_fails_with("any command",
+ dict(substrs="some substring"),
+ "substrs must be a collection of strings")
+ # Prevent `self.expect("cmd", "substr")`
+ self.assert_expect_fails_with("any command",
+ dict(msg="some substring"),
+ "expect() missing a matcher argument")
+ # Prevent `self.expect("cmd", "msg", "substr")`
+ self.assert_expect_fails_with("any command",
+ dict(msg="a message", patterns="some substring"),
+ "must be a collection of strings")
Index: lldb/packages/Python/lldbsuite/test/lldbtest.py
===================================================================
--- lldb/packages/Python/lldbsuite/test/lldbtest.py
+++ lldb/packages/Python/lldbsuite/test/lldbtest.py
@@ -2424,6 +2424,22 @@
set to False, the 'str' is treated as a string to be matched/not-matched
against the golden input.
"""
+ # Catch cases where `expect` has been miscalled. Specifically, prevent
+ # this easy to make mistake:
+ # self.expect("lldb command", "some substr")
+ # The `msg` parameter is used only when a failed match occurs. A failed
+ # match can only occur when one of `patterns`, `startstr`, `endstr`, or
+ # `substrs` has been given. Thus, if a `msg` is given, it's an error to
+ # not also provide one of the matcher parameters.
+ if msg and not (patterns or startstr or endstr or substrs or error):
+ assert False, "expect() missing a matcher argument"
+
+ # Check `patterns` and `substrs` are not accidentally given as strings.
+ assert not isinstance(patterns, six.string_types), \
+ "patterns must be a collection of strings"
+ assert not isinstance(substrs, six.string_types), \
+ "substrs must be a collection of strings"
+
trace = (True if traceAlways else trace)
if exe:
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
