Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Report no results as error #144

Merged
merged 12 commits into from
Oct 4, 2020
Merged

Report no results as error #144

merged 12 commits into from
Oct 4, 2020

Commits on Oct 4, 2020

  1. Overrides suiteNameTemplate once for entire report 

    because the template configuration applies to the entire report
    @SamTheisens
    SamTheisens committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    d678eee View commit details
    Browse the repository at this point in the history

  2. Clones options at beginning of suite loop 

    to prevent potential setting overrides from spilling over to other
suites.
    @SamTheisens
    SamTheisens committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    648ac6d View commit details
    Browse the repository at this point in the history

  3. Reports test suites with no results as error 

     - adds `reportNoResultsAsError` configuration parameter. Default is
   false
 - when top level test result (suite) has no children, a stub suite is
   added with a single test with `error` status. The exception is
   displayed at test level.
 - any potential suite or title templates are overridden in order to
   guarantee that the file path is displayed. Since besides the
   `testExecError`, this is the only information we have.
    @SamTheisens
    SamTheisens committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    6ca58b7 View commit details
    Browse the repository at this point in the history

  4. Removes unused variable

    @SamTheisens
    SamTheisens committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    c2deeee View commit details
    Browse the repository at this point in the history

  5. Adds reportNoResultsAsError config setting to README.md

    @SamTheisens
    SamTheisens committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    5b6e527 View commit details
    Browse the repository at this point in the history

  6. Ensures suites that fail to load altogether get reported

    @SamTheisens
    SamTheisens committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    4f5d754 View commit details
    Browse the repository at this point in the history

  7. Ensures class name is set when suite fails to run 

    and the file path is used as suite title.
    @SamTheisens
    SamTheisens committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    f4c9f26 View commit details
    Browse the repository at this point in the history

  8. Renames error reporting flag to reportTestSuiteErrors 

    and explains in the README that file path will be used as suite name in
these cases
    @SamTheisens
    SamTheisens committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    733240d View commit details
    Browse the repository at this point in the history

  9. Assigns jsonResults to global variable 

    so that the validity of the resulting xml can be verified
    @SamTheisens
    SamTheisens committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    66e8789 View commit details
    Browse the repository at this point in the history

  10. Adds example of empty suite Jest output 

    to ensure that it is handled as suite error
    @SamTheisens
    SamTheisens committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    6931e54 View commit details
    Browse the repository at this point in the history

  11. Makes sure suite failure message is always reported 

    because `suite.testExecError` is a varying complex type and the information is
also reflected in `suite.failureMessage` anyway.
    @SamTheisens
    SamTheisens committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    7af55a7 View commit details
    Browse the repository at this point in the history

  12. Aligns env variable name with reporter option

    @SamTheisens
    SamTheisens committed Oct 4, 2020
    Configuration menu
    Copy the full SHA
    2d1d722 View commit details
    Browse the repository at this point in the history