dorny / test-reporter

Displays test results from popular testing frameworks directly in GitHub
MIT License
804 stars 201 forks source link

Using mochawesome #147

Open yonatanaftali opened 2 years ago

yonatanaftali commented 2 years ago

Hey,

Can I use my mochawesome.json file with this action? I tried using it with the mocha setting, but it doesn't work. I'm getting the error mocha.passes is not iterable for the following configuration:

      - name: Report E2E Results
        if: ${{ !cancelled() }}
        uses: dorny/test-reporter@v1
        with:
          name: E2E Tests Results
          path: test/e2e/report/mochawesome.json
          reporter: mocha-json
IanMoroney commented 2 years ago

@yonatanaftali, As I wasn't sure whether this project was abandoned or not, i've forked it and released some enhancements for it already, so you may wish to try that reporter instead. It includes the ability to report on mochawesome!

https://github.com/phoenix-actions/test-reporting/ https://github.com/marketplace/actions/test-reporting

Waterstraal commented 2 years ago

@IanMoroney That's nice, but it would be better if you could make a PR to this action imho.

IanMoroney commented 2 years ago

@IanMoroney That's nice, but it would be better if you could make a PR to this action imho.

@Waterstraal , at the time (bear in mind this was in Feb of this year), the project hadn't been committed to since June of last year, and i'd waited weeks for a reply on my own issue I raised to be looked at. I noticed none of the issues were being responded to, and I had a real need to get my issue fixed, so I decided to fork the project and release my own variant with new features and a lot of the outstanding issues resolved.

I would love to have been able to do a PR, but at the time it was near impossible for a project that appeared to be dead.

flobernd commented 2 years ago

@IanMoroney It seems like the project is active again. Maybe some of your improvements could be merged upstream now.