Fix TestWebKitAPI.WebKit.AppHighlightsInImageOverlays to work with changing menu states.
Created attachment 438370 [details] Patch
Comment on attachment 438370 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=438370&action=review > Tools/TestWebKitAPI/Tests/WebKitCocoa/WKContentViewEditingActions.mm:101 > + EXPECT_EQ([contentView targetForAction:createHighlightForCurrentQuickNoteWithRangeSelector withSender:nil], nil); Maybe `EXPECT_NULL` to match the ones above? But this works too.
Created attachment 438383 [details] Patch
Created attachment 438384 [details] Patch for landing
Comment on attachment 438383 [details] Patch Not new to this patch, in fact this patch makes things better, but: I think a lot of our tests are using the EXPECT family backwards. It’s expected value first, then test expression. This affects the output when tests fail, and is worth getting right.
(In reply to Darin Adler from comment #5) > Comment on attachment 438383 [details] > Patch > > Not new to this patch, in fact this patch makes things better, but: I think > a lot of our tests are using the EXPECT family backwards. It’s expected > value first, then test expression. This affects the output when tests fail, > and is worth getting right. Good catch! Filed: <https://bugs.webkit.org/show_bug.cgi?id=230368>.
Committed r282601 (241762@main): <https://commits.webkit.org/241762@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 438384 [details].
<rdar://problem/83213601>