jemishgopani / wdio-json-reporter

A WebdriverIO plugin. Report results in json format. http://webdriver.io
13 stars 32 forks source link

testSuite.name should be original #2

Closed monolithed closed 8 years ago

monolithed commented 8 years ago
describe('TEST-1', () => { // `TEST-1` is an issue number in JIRA.
    it('Test 1', () => { });
});
{
    "suites": [
        {
            "name": "test_2" 
        }
    ]
}

https://github.com/fijijavis/wdio-json-reporter/blob/master/lib/reporter.js#L57

Could you fix it?

fijijavis commented 8 years ago

To make sure we are on the same page, the hyphen in the suite name is causing an issue?

monolithed commented 8 years ago

Yep, hyphen and lowercase are both. What is the original purpose of that?

fijijavis commented 8 years ago

Sorry for the delayed response. Forgot about this over the weekend.

That was a straight copy from the JUNIT version so I'm not sure what that was for. I'll fix it and get out a new version today.

monolithed commented 8 years ago

I got it, thanks!

fijijavis commented 8 years ago

v 0.0.3 is now available