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.

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

Add support for "--failed" in ExUnit #7373

Closed

Commits on Mar 3, 2018

  1. Deal with :file tags that were incorrectly overriden by a setup b…

    …lock
    
    Given a `setup` block like:
    
        setup do
          {:ok, file: :foo}
        end
    
    ...ExUnit raises an exception, but it also overrides the test's
    `:file` tag, which could cause problems for the manifest since it
    looks up the existence of the file on the file system, and would
    get an error if it was not a string.
    myronmarston committed Mar 3, 2018
    Configuration menu
    Copy the full SHA
    2a73d4f View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    77adfb9 View commit details
    Browse the repository at this point in the history

Commits on Mar 4, 2018

  1. Add support for mix test --failed

    This uses the ExUnit manifest to filter to only tests that failed
    the last time they ran. As an optimization, we filter out files
    that have no failures so we load the minimum set of files necessary.
    myronmarston committed Mar 4, 2018
    Configuration menu
    Copy the full SHA
    0ba4b2c View commit details
    Browse the repository at this point in the history

Commits on Mar 5, 2018

  1. Optimize ExUnit manifest to store only failures

    Since that is the only information we care about, we do not
    have to store every test with its status. This allows us to
    vastly simplify the manifest, as we no longer have to use an
    `:entry` record. Reading and writing the manifest should be
    faster now, since the manifest tends toward being empty.
    
    Since we are only storing failures, I have renamed the manifest
    (and the `:manifest_file` option and file path) to indicate that
    it deals only with failures.
    myronmarston committed Mar 5, 2018
    Configuration menu
    Copy the full SHA
    a28bb62 View commit details
    Browse the repository at this point in the history
  2. Address code review feedback

    - Rename manifest fail to indicate that mix owns the file
    - Leave a TODO
    - Document new ExUnit config options
    - Fix ENV var typo
    myronmarston committed Mar 5, 2018
    Configuration menu
    Copy the full SHA
    c50937a View commit details
    Browse the repository at this point in the history