Remove dependency on Coveralls

[pbrisbin]

We rely on classes from satooshi/php-coveralls for converting Clover XML to the proper JSON payload for our API endpoint. It would be good to just (re)write the logic we do rely on and remove that dependency.

Example Input:

<?xml version="1.0" encoding="UTF-8"?>
<coverage generated="1365848893">
  <project timestamp="1365848893">
    <package name="Hoge">
      <file name="/home/patrick/Code/codeclimate/php-test-reporter/src/CodeClimate/Bundle/TestReporterBundle/ApiClient.php">
        <class name="TestFile" namespace="Hoge">
          <metrics methods="1" coveredmethods="0" conditionals="0" coveredconditionals="0" statements="1" coveredstatements="0" elements="2" coveredelements="0"/>
        </class>
        <line num="6" type="method" name="__construct" crap="1" count="0"/>
        <line num="8" type="stmt" count="0"/>
      </file>
    </package>
  </project>
</coverage>

The API endpoint for reporting coverage is shared across all test reporters for all languages, so this format must be maintained.

Example Output:

{
  "partial": false,
  "run_at": 1365848893,
  "repo_token": "...",
  "environment": {
    "pwd": "\/home\/patrick\/Code\/codeclimate\/php-test-reporter",
    "package_version": "0.1.0-dev"
  },
  "git":{
    "head": "7e3f3495218f92ad5ce847248e306f279b11a9f9",
    "branch": "pb-bundled-cert",
    "committed_at": 1404230153
  },
  "ci_service": "travis",
  "source_files": [{
    "name": "src\/CodeClimate\/Bundle\/TestReporterBundle\/ApiClient.php",
    "blob_id": "272fdc7fbbcd2d99f99db2f4fa59747ed2f35b05",
    "coverage":"[null, 0, null, 1, null, 3, null, 0, ...]"
  },{ 
    // ...
  }]
}

Values in JSON not from XML:

• Environment information (CWD, token, CI service, etc)
• Git information (head, branch, blob ids, etc)

[elazar]

This would seem prudent particularly because php-coveralls recently relaxed their Symfony dependency versions and those Symfony dependencies recently updated their minimum PHP version to 5.5.9, which exceeds this repository's current requirement of 5.3:

• symfony/console@9503491
• symfony/yaml@2daf2f6
• symfony/config@d3e7b57
• symfony/stopwatch@e24eab1

Related: I've filed an issue with php-coveralls regarding this.

[Ocramius]

@elazar that doesn't enforce usage of 5.5, no? 2.* versions of the symfony components should work fine.

[elazar]

@Ocramius They should, but are not what's being used at this point unless the project using them explicitly sets differing versions. Here's an example of what I mean. /cc @chrismou

[elazar]

Adding these to this repository's composer.json file should mitigate the issue described in my original comment (by providing the version upper bound that php-coveralls has removed) until the dependency on php-coveralls can be removed:

{
    "require": {
        "symfony/console": "~2",
        "symfony/yaml": "~2",
        "symfony/config": "~2",
        "symfony/stopwatch": "~2"
    }
}

[dblandin]

@pbrisbin Is this still a concern?

[dblandin]

Going to go ahead and close this issue for now.