darvin/tap-mochawesome-reporter

tap-mochawesome-reporter's Issues

Is this maintained?

Hi there,

This is a great idea and I really like the mochawesome reporter and I can't find any other html reporters for tap.

I am finding one issue where I think the syntax the plugin creates is not the same as what marge is expecting.

If you use the library directly you will run into an issue:

 'expected':diag.data ? (diag.data.expected || diag.data.expect )  : (diag.expected || diag.data.expect),
                                                                                                      
TypeError: Cannot read property 'expect' of undefined

@tap-mochawesome-reporter/index.js:75:10

I fixed this by npm linking it and updating the formatError function:

function formatError (diag){
    if (diag && diag.data) return {
        'name':"",
        'message':"",
        "showDiff": true,
        'actual':diag.data ? (diag.data.got || diag.data.actual)  : (diag.got  || diag.actual),
        'expected':diag.data ? (diag.data.expected || diag.data.expect )  : (diag.expected || diag.data.expect),
    }
    return {};
}

Beyond this marge 4.0.1 reports an error:

supplied to /suites: Nil
  Invalid value 2019 supplied to /copyrightYear: Nil

I assume the JSON syntax for MA changed since this was made. Hence the original question!

Would appreciate your input and if I can solve it I will submit a PR.

Thanks