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

Update tests to use the SystemOutRule rule #103

Closed
oleg-nenashev opened this issue Jun 16, 2020 · 1 comment
Closed

Update tests to use the SystemOutRule rule #103

oleg-nenashev opened this issue Jun 16, 2020 · 1 comment
Labels
chore Project maintenance good first issue Good for newcomers

Comments

@oleg-nenashev
Copy link
Member

Currently several tests redirect the STDOUT to verify the output. In such case the tool's output does not get displayed in the execution log. Instead of that, it would be better to use a special JUnit Rule which can capture the output.

See the example in #102

    @Rule
    public final SystemOutRule systemOutRule = new SystemOutRule().enableLog();
    assertThat(systemOutRule.getLog(), not(containsString("uithemes")));
@oleg-nenashev oleg-nenashev added good first issue Good for newcomers chore Project maintenance labels Jun 16, 2020
@oleg-nenashev oleg-nenashev changed the title Update tests to use the Update tests to use the SystemOutRule rule Jun 16, 2020
@stefanbirkner
Copy link
Contributor

This should be fixed by #123.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Project maintenance good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants