[pv]add integration test for original untappable web view link behind context menu bug

[hellohuanlin]

This PR adds an integration test that reproduces the bug in #175099.

I have verified that this test would fail if remove the hacky workaround.

The next todo item would be to add test for admob banners in a scrollabe list.

List which issues are fixed by this PR. You must list at least one issue. An issue is not required if the PR fixes something trivial like a typo.

Related to #175099
And solve part of #156753

If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.

Pre-launch Checklist

• I read the Contributor Guide and followed the process outlined there for submitting PRs.
• I read the Tree Hygiene wiki page, which explains my responsibilities.
• I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
• I signed the CLA.
• I listed at least one issue that this PR fixes in the description above.
• I updated/added relevant documentation (doc comments with ///).
• I added new tests to check the change I am making, or this PR is test-exempt.
• I followed the breaking change policy and added Data Driven Fixes where supported.
• All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.

[gemini-code-assist]

Code Review

This pull request adds an integration test to reproduce a bug where a web view link becomes untappable after a context menu is shown and dismissed. The changes include new native iOS test code, the corresponding Dart UI for the test page, and a native platform view implementation for a WKWebView to set up the bug conditions. My feedback focuses on improving the maintainability and consistency of the new test code by using constants for identifiers and timeouts.

[gemini-code-assist]

To improve readability and maintainability, consider defining the string literals used for accessibility identifiers as static const variables. This avoids 'magic strings' and makes the test easier to read and update. Also, using the kStandardTimeOut constant for all timeouts ensures consistency.

- (void)testPlatformViewWebViewLinkTappableForContextMenuScenario {
  static NSString *const kEntranceButtonIdentifier = @"web view behind context menu test";
  static NSString *const kPlatformViewIdentifier = @"platform_view[0]";
  static NSString *const kShowMenuButtonIdentifier = @"Show menu";
  static NSString *const kMenuItemIdentifier = @"menu button 1";
  static NSString *const kTargetLinkIdentifier = @"Target Link";
  static NSString *const kSuccessTextIdentifier = @"Navigation Successful";

  XCUIElement *entranceButton = self.app.buttons[kEntranceButtonIdentifier];
  XCTAssertTrue([entranceButton waitForExistenceWithTimeout:kStandardTimeOut]);
  [entranceButton tap];

  XCUIElement *platformView = self.app.webViews[kPlatformViewIdentifier];
  XCTAssertTrue([platformView waitForExistenceWithTimeout:kStandardTimeOut]);

  // expand the context menu.
  XCUIElement *showMenuButton = self.app.buttons[kShowMenuButtonIdentifier];
  XCTAssertTrue([showMenuButton waitForExistenceWithTimeout:kStandardTimeOut]);
  [showMenuButton tap];

  // tap to dismiss the context menu.
  XCUIElement *menuItem = self.app.buttons[kMenuItemIdentifier];
  XCTAssertTrue([menuItem waitForExistenceWithTimeout:kStandardTimeOut]);
  XCUICoordinate *center = [self.app coordinateWithNormalizedOffset:CGVectorMake(0.5, 0.5)];
  [center tap];
  XCTAssertTrue([menuItem waitForNonExistenceWithTimeout:kStandardTimeOut]);

  // Verify that the web view link is still tappable.
  XCUIElement *link = self.app.links[kTargetLinkIdentifier];
  XCTAssertTrue([link waitForExistenceWithTimeout:kStandardTimeOut]);
  [link tap];
  XCUIElement *successText = self.app.staticTexts[kSuccessTextIdentifier];
  XCTAssertTrue([successText waitForExistenceWithTimeout:kStandardTimeOut]);
}

References

1. The repository style guide (line 47) requires following the Google Objective-C Style Guide. This guide recommends using static const for constants, which would improve the readability and maintainability of this test by replacing magic strings with named constants. ^(link)

[hellohuanlin]

this makes it even harder to read.

[hellohuanlin]

@vashworth friendly reminder :)

[vashworth]

Looks great!

[auto-submit]

autosubmit label was removed for flutter/flutter/182111, because The base commit of the PR is older than 7 days and can not be merged. Please merge the latest changes from the main into this branch and resubmit the PR.