Let me do that after that I'll ping you

Hi @christian-bromann
I have changed required things if we are agree on this change then I'll add test for wdio v7 as well

Can you please go through this generates xml output correctly when the addFileAttribute option is set I feel like we don't need that test.

I feel like we don't need that test.

How so?

I feel like we don't need that test.

How so?

it('generates xml output correctly when the addFileAttribute option is set', () => {
        reporter = new WDIOJunitReporter({ stdout: true, addFileAttribute: true })
        reporter.suites = featuresLog as any

        // verifies the content of the report but omits format by stripping all whitespace and new lines
        expect(reporter['_buildJunitXml'](mochaRunnerLog as any).replace(/\s/g, '').replace(/file:\/\//g, '').replace(/C:\//g, '')).toMatchSnapshot()
    })

Because that test we are running on mocha-runner mochaRunnerLog but the specified suites reporter.suites = featuresLog as any are the cucumber feature.

@christian-bromann

Hi @christian-bromann
Can you please help here with what can I do with that unit test case that I mentioned above?

but the specified suites reporter.suites = featuresLog as any are the cucumber feature.

Just replace featureLog with suiteLog and verify that the test still tests what it suppose to.

@christian-bromann Fixed junit reporter unit test please review it

Great job, thank you @jemishgopani