Removing PHPUnit::execute Coverage Tags

[Fenikkusu]

Contribution Type: bug fix Link to Bug: https://github.com/Block8/PHPCI/issues/1250

This pull request affects the following areas:

In raising this pull request, I confirm the following (please check boxes):

• [*] I have read and understood the contributing guidelines?
• [*] I have checked that another pull request for this purpose does not exist.
• [*] I have considered, and confirmed that this submission will be valuable to others.
• [*] I have created or updated the relevant documentation for this change on the PHPCI Wiki.
• [*] Do the PHPCI tests pass?

Detailed description of change:

Removed the @covers PHPUnit::execute declaration - no such thing exists in the code and code coverage testing will not finish otherwise.

[Fenikkusu]

Note: While regular unit tests pass, I'm coming across a odd issue when running the code coverage testing. For what ever reason, BuildLoggerTest::testLogFailure_AddsExceptionContext will not complete if running all tests. If running just the single test, the test completes fine. I'm not sure what is going on here. The only thing I've noticed is an error in PHPStorm indicating an additional attempted connection failed.