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

Reporting tests as skipped instead of ignored #63

Closed
benmccann opened this issue Nov 26, 2014 · 1 comment
Closed

Reporting tests as skipped instead of ignored #63

benmccann opened this issue Nov 26, 2014 · 1 comment

Comments

@benmccann
Copy link
Contributor

benmccann commented Nov 26, 2014

Shouldn't this be Status.Ignored instead of Status.Skipped in EventDispatcher.java? I'm not really sure when you'd report a test as skipped, but I'd imagine if you mark a test with @Ignore then it should be reported as ignored instead of skipped given that a Status exists for both of those

@Override
public void testIgnored(Description desc)
{
  postIfFirst(new InfoEvent(desc, Status.Skipped) {
    void logTo(RichLogger logger) {
      logger.info("Test "+ansiName+" ignored");
    }
  });
}
@eed3si9n
Copy link
Member

This was fixed in #97

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants