Skip to content

Remove --nocapture from nosetests on .travis.yml? #5337

Closed

Description

@mdboom

As mentioned in #5324 (comment), we are passing the -s or --nocapture flag to nose when we run the tests on Travis. This causes the output of any tests to be printed immediately rather than with its failure block at the end of the run. On the one hand, this can be nice if the test run is failing catastrophically (e.g. a segfault) because we know what printed right before the process died. On the other hand, with the "running inside of gdb" functionality we have now, we will at least get a nice traceback when that happens.

I'm inclined to turn off -s, but I wanted to confirm there weren't good current reasons for keeping it on.

It seems to have been added by @pelson in b8bc21f

Activity

  1. tacaswell commented on Oct 28, 2015

    @tacaswell
    Member

    👍 It makes even less sense now that we print dots instead of the test names (which we do so that travis will render our test logs).

  2. pelson commented on Oct 30, 2015

    @pelson
    Member

    Yep, I'm ok with this. 👍

  3. added a commit that references this issue on Oct 30, 2015
    142cfdc
  4. self-assigned this
    on Oct 30, 2015
  5. added a commit that references this issue on Nov 2, 2015
    3302ea6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions