[java] [dotnet] Add move to location method to Actions

[pujagani]

Thanks for contributing to Selenium!
A PR well described will help maintainers to quickly review and merge it

Before submitting your PR, please check our contributing guidelines.
Avoid large PRs, help reviewers by making them as simple and short as possible.

Description

Add move to location method to Actions class

Motivation and Context

Explained in #10724. The change implement proposal 2.

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

Patch and project coverage have no change.

Comparison is base (4468622) 57.11% compared to head (2755531) 57.11%.

❗ Current head 2755531 differs from pull request most recent head 8514aa9. Consider uploading reports for the commit 8514aa9 to get more accurate results

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files

@@           Coverage Diff           @@
##            trunk   #11509   +/-   ##
=======================================
  Coverage   57.11%   57.11%           
=======================================
  Files          86       86           
  Lines        5365     5365           
  Branches      193      193           
=======================================
  Hits         3064     3064           
  Misses       2108     2108           
  Partials      193      193           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[sonarqubecloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication

[titusfortner]

force pushing a rebased update; the new Java formatting updates made it easier to do it this way. Evaluating now.

[titusfortner]

@diemol which of these does it make more sense to support? It feels like we should be picking the "right" one and have people use it rather than supporting two options? Point isn't being used anywhere else, but everything else is offsets not something that represents coordinates on the viewport, so it might be more correct...

    Action moveAndClick = getBuilder(driver).moveToLocation(70, 60).click().build();

and

    Action moveAndClick = getBuilder(driver).moveToLocation(new Point(70, 60)).click().build();

Also, my default for multiple constructors or methods that do the same thing with different parameters is that one should call the other rather than be a re-implementation. So one way or the other I think this PR needs to be tweaked slightly.

.NET looks good; don't have .NET environment working on new laptop yet, though.

[titusfortner]

Hmm .NET test is failing in CI... Passing in my fresh .NET dev environment; probably a timing issue; trying the equivalent test from Ruby.

[diemol]

This one might be "easier"?

Action moveAndClick = getBuilder(driver).moveToLocation(70, 60).click().build();

[titusfortner]

Java looks good, but .NET + Windows + Firefox isn't happy and I'll have to load up a VM to troubleshoot it.