Inform developers about non-working "rightClick" on Selenium Server 3.x

[aik099]

Closes #381

[codecov]

Codecov Report

Attention: Patch coverage is 93.75000% with 1 line in your changes missing coverage. Please review.

Project coverage is 86.21%. Comparing base (bedccba) to head (3145d7e).
Report is 3 commits behind head on master.

Files with missing lines	Patch %	Lines
src/Selenium2Driver.php	93.75%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master     #407      +/-   ##
============================================
+ Coverage     85.45%   86.21%   +0.75%     
- Complexity      182      185       +3     
============================================
  Files             1        1              
  Lines           502      515      +13     
============================================
+ Hits            429      444      +15     
+ Misses           73       71       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[uuf6429]

@aik099 I have some fixes built on top of your PR changes. How can I send the changes to you?

Never mind, I made a patch... here you go: tests-and-fixes.patch

• SeleniumSupportTest.php - the 2 (new) tests mentioned earlier
• Selenium2Driver.php - mainly, fixes for start() logic in particular:
  • throw DriverException instead of \Exception
  • do not catch DriverException immediately while trying to start - it's pointless
  • handle status() response structure in case of Selenium 4
• Selenium2Config.php - many improvements:
  • refactored $testCase+$test usage
  • improved selenium version handling + added utility methods
  • utility methods are public, to be used in the test

[aik099]

@aik099 I have some fixes built on top of your PR changes. How can I send the changes to you?

Never mind, I made a patch... here you go: tests-and-fixes.patch

• SeleniumSupportTest.php - the 2 (new) tests mentioned earlier

• Selenium2Driver.php - mainly, fixes for start() logic in particular:

  • throw DriverException instead of \Exception
  • do not catch DriverException immediately while trying to start - it's pointless
  • handle status() response structure in case of Selenium 4

• Selenium2Config.php - many improvements:

  • refactored $testCase+$test usage
  • improved selenium version handling + added utility methods
  • utility methods are public, to be used in the test

@uuf6429 , I've tried to apply provided Patch in the PhpStorm and it had huge conflicts in several files. Are you sure, that you've created it based on last commit in this PR?

Anyway I've applied it, did some minor adjustments, pushed a few commits based on that.

Please see I haven't missed anything.

[aik099]

@uuf6429 , ready for review.