search

rwjblue / ember-secure-builder

MIT License
0 stars 2 forks source link

Redundant fields #16

Closed twokul closed 11 years ago

twokul commented 11 years ago

See url and test_url. And build also. And tags.

{
    "url": "https://s3.amazonaws.com/rwjblue-ember-dev-test/ember/pull-request/3567/ember-tests.html",
    "name": "Ember PR #3567",
    "build": "8cef7aeb8fa80cc18e686f577bd8d6a4d94aa548",
    "tags": [
        "ember",
        "PR #3567"
    ],
    "results_path": "ember/pull-request/3567",
    "completed": false,
    "job_results": [
        {
            "browser": "firefox",
            "test_url": "https://s3.amazonaws.com/rwjblue-ember-dev-test/ember/pull-request/3567/ember-tests.html",
            "platform": "Windows 7",
            "version": 24,
            "result": {
                "failed": 0,
                "total": 7569,
                "failedTests": [],
                "passed": 7569,
                "runtime": 78138
            },
            "passed": true,
            "build": "8cef7aeb8fa80cc18e686f577bd8d6a4d94aa548",
            "tags": [
                "ember",
                "PR #3567"
            ]
        },
rwjblue commented 11 years ago

The data is structured this way due to how we were originally generating/parsing the results. I'll look into removing the duplicate fields from the job_results array items.

rwjblue commented 11 years ago

Fixed and deployed. All new jobs will not have those fields.