[Ganesh2325]

Change list
Fixes #1926. An @OverrideWidget field, or a List of that field, may be declared as an interface implemented by the platform widget classes. Calls are proxied to the concrete Widget for the current content type. Fields declared as Widget classes, including abstract classes, behave as they do today.

The override class must implement the interface and extend Widget.
A content type with no override stays unresolved until that content is used, then throws InstantiationException.
Locators are still taken from the selected widget class, including class-level @AndroidFindBy and @iOSXCUITFindBy.
An interface that extends WebElement is left to the normal element decorator.

Types of changes
New feature (non-breaking change which adds functionality)

Details

public interface UsersWidget {
void doWork();
}
public class IosUsersWidget extends Widget implements UsersWidget {
protected IosUsersWidget(WebElement element) {
super(element);
}
@OverRide
public void doWork() {
// iOS behavior
}
}
@findby(id = "users")
@OverrideWidget(iOSXCUITAutomation = IosUsersWidget.class,
androidUIAutomator = AndroidUsersWidget.class,
html = HtmlUsersWidget.class)
UsersWidget widget;
Name a concrete class for every content type the field is used with. A partial override, such as only iOSXCUITAutomation, works on that content and throws InstantiationException on another.

./gradlew test checkstyleMain checkstyleTest javadoc

Local validation on a732abba: ./gradlew test checkstyleMain checkstyleTest javadoc passed. All 240 java-client tests and 12 selenium-bridge tests passed with no failures. Checkstyle and Javadoc also passed. These are local results, not upstream CI results.

The interface must be public. The library’s class loader must be able to load the interface, or the interface’s loader must be able to load the library. Unsupported loader setups and non-public interfaces throw IllegalArgumentException.

Rebased on #2480; widget interceptors created for interface fields use the configured default LocatorGroupStrategy.

[mykola-mokhnach]

@Ganesh2325 please resolve conflicts in the PR

[Ganesh2325]

@mykola-mokhnach Thanks, conflicts are resolved. I rebased onto the latest master. The conflicts were with #2480 (default LocatorGroupStrategy), and both changes are kept: interface-typed @OverrideWidget fields and list items now also receive the decorator's defaultLocatorGroupStrategy. ./gradlew test checkstyleMain checkstyleTest javadoc passes locally (230 java-client + 12 selenium-bridge tests). Could you please approve the workflows when you get a chance?

[mykola-mokhnach]

• Test coverage: OverriddenWidgetInterfaceTest doesn't exercise the paths where interface proxies differ from class proxies. These are list items after the element reference is cleared (see the inline comment on WidgetListInterceptor), toString/equals/hashCode on an interface proxy, a widget with nested @FindBy fields, a non-public interface, and an interface with default methods. Please add tests for the ones you intend to support.

[Ganesh2325]

Thanks! Added tests for cleared weak references, toString/equals/hashCode, nested @FindBy fields, and default methods—including one the widget does not override. Non-public interfaces are tested as unsupported and fail clearly. These tests pass on a732abba.

[valfirst]

@Ganesh2325 could you please resolve the conflicts?

[Ganesh2325]

Thanks! I merged the latest master and resolved the conflicts in 8a414c5. All tests pass locally.