use fullTitle of a suite in reporter

[sherlock1982]

I have some nested describes in mocha and in junit reporter they look ugly because reporter takes name of inner describe only.
Here's suggested patch to take fullTitle of a suite so in report it will look nicer.
We can make it optional if you want no to break anything.

[christian-bromann]

they look ugly

Can you provide a screenshot of how the logs look before and after?

[sherlock1982]

I have two specs now:

describe('PBX', () => {
    describe('Validations', () => {
        it('should validate something important', () => {

        })
    })

    describe('Editor', () => {
        it('should edit something important', () => {

        })
    })
})

describe('Settings', () => {
    describe('Validations', () => {
        it('should validate something important', () => {

        })
    })

    describe('Editor', () => {
        it('should edit something important', () => {

        })
    })
})

This is before:

This is after:

[christian-bromann]

I believe this is only available in Mocha, how do we deal with this if someone uses a different framework?

[sherlock1982]

Yes, I hadn't thought about it. At least 'parent' attribute is available in Jasmine and Mocha so I can join fullTitle directly in this function. Let me extend it.
Also seems that parent suite always starts before. So I can suggest by the time child suite will start I will already have a name for parent.

[christian-bromann]

@sherlock1982 any updates?

[christian-bromann]

ping @sherlock1982

[dbashford]

FWIW, I've just got this copied into our codebase as we need it. Can hopefully get this work incorporated at some point.

[arstanton]

Any progress with this? As it is now, it's encouraged only using one describe block (because only the inner most block is visible), which makes it harder to include beforeEach(), afterEach(), etc..., hooks.