[java] Add ability to decorate child classes of WebDriver

[valfirst]

Description

The changes enable decoration of custom WebDriver implementations.

Motivation and Context

Relates appium/java-client#1694

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• I have read the contributing document.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov-commenter]

Codecov Report

Merging #10737 (d77b653) into trunk (0a2a4a9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            trunk   #10737   +/-   ##
=======================================
  Coverage   50.42%   50.42%           
=======================================
  Files          84       84           
  Lines        5475     5475           
  Branches      278      278           
=======================================
  Hits         2761     2761           
  Misses       2436     2436           
  Partials      278      278           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a6b161a...d77b653. Read the comment docs.

[diemol]

Thanks for this PR, we will have a look in depth soon, since it is a sizeable change.

[pujagani]

I have gone through the PR in detail. Overall logic seems right on the looks of it and I understand the use-case the changes are trying to accomplish. I am expressing my concern about merging this and further releasing this. I tried importing the changes into an existing project. It is not a breaking change but existing users will see an inspection message that says "Raw use of parameterized class 'WebDriverDecorator". So to avoid that, they will need to make a small update to their existing code. I will check with the selenium-tlc members about how we handle introducing such changes. But I think this is a candidate for a major release.

[pujagani]

@valfirst Thank you for your contribution! Appreciate it.

[sonarqubecloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication

[diemol]

Thank you, @valfirst!

[kirannailwal1]

@diemol @pujagani
Can you please confirm on which version this issue will be resolved.
As we are still facing issue : java.lang.ClassCastException: net.bytebuddy.renamed.java.lang.Object$ByteBuddy$F6Ow9062 cannot be cast to io.appium.java_client.android.AndroidDriver

org.seleniumhq.selenium :4.3.0
java-client :8.1.1
appium version : 1.22.3
node version : v16.16.0

[diemol]

4.4.0, should be released first week of August.