Some arguments are double escaped in the reporting script when using SSH

[desrosj]

Describe the bug
When an SSH connection is configured for the runner, the $junit_location is passed through escapeshellarg() twice. This results in a the following error:

rsync: [sender] change_dir "/home4/USERNAME/'wp-test-runner-80'/tests/phpunit/build/logs" failed: No such file or directory (2)

The actual command that's run is rsync -r -e "ssh " 'USERNAME@HOST:'\''wp-test-runner-80'\''/tests/phpunit/build/logs/*' '/tmp/wp-test-runner'. Note the excessive single quotations in the path.

To Reproduce
Steps to reproduce the behavior:

1. Configure the runner to run using an SSH connection
2. Run the prepare.php and test.php scripts.
3. Confirm that the tests/phpunit/build/logs directory is present and contains a junit.xml and env.json file.
4. Run the report.php script to see the error.

Expected behavior
The entire path should be enclosed in quotes once.

Screenshots

[desrosj]

It looks like this also happens in the test.php script. The output there is: ssh -o StrictHostKeyChecking=no 'USERNAME@HOST' 'cd '\''wp-test-runner-80'\'' && /opt/cpanel/ea-php80/root/usr/bin/php ./vendor/phpunit/phpunit/phpunit --dont-report-useless-tests'