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

Output not cleared before build when running tests #1179

Closed
nbraem opened this issue Apr 16, 2020 · 2 comments
Closed

Output not cleared before build when running tests #1179

nbraem opened this issue Apr 16, 2020 · 2 comments
Assignees
Labels
bug a bug in the product Feature: ctest fixed (release pending) fixed in pre-release and is pending official release
Projects
Milestone

Comments

@nbraem
Copy link

nbraem commented Apr 16, 2020

Brief Issue Summary

When running tests, Build is run first, but output is not cleared, although Clear Output Before Build is turned on.

Expected:

  1. Turn on Clear Output Before Build option.
  2. Run CMake: Run Tests
  3. Run CMake: Run Tests again

I expect that output is cleared every time that tests are run, because it firsts builds.

Apparent Behavior:

Output is not cleared, although it's building and Clear Output Before Build is turned on

Other Notes/Information

Clearing output is especially useful when running tests, because you typically run tests repeatedly and want to inspect the output of of failing tests, which can be large. So you typically search in the output, but don't want to find matches in the previous output.

If you don't think clearing output before test is appropriate if the option Clear Output Before Build is turned on, I suggest addding an option Clear Output Before Run Tests

@andreeis andreeis added this to the On Deck milestone Apr 16, 2020
@andreeis andreeis added bug a bug in the product Feature: build and removed Feature: build labels Apr 22, 2020
@Stephenmccaul
Copy link

Not sure if this is worth a separate issue but the output is also not cleared when "Launch debugger for selected target" is clicked..

I think it safe to say that if "Clear Output Before Build" is true the output should be cleared before any cmake, build or test invocation.

@andreeis
Copy link
Contributor

@Stephenmccaul, no let's not open a new issue for this, it's an easy fix all around. Thank you for pointing out the additional scenario.

@gcampbell-msft gcampbell-msft added this to Triage in 1.16 via automation Jul 6, 2023
@gcampbell-msft gcampbell-msft modified the milestones: On Deck, 1.16 Jul 6, 2023
@gcampbell-msft gcampbell-msft added this to Triage in 1.17 via automation Oct 4, 2023
@gcampbell-msft gcampbell-msft removed this from Triage in 1.16 Oct 4, 2023
@gcampbell-msft gcampbell-msft modified the milestones: 1.16, 1.17 Oct 4, 2023
@gcampbell-msft gcampbell-msft modified the milestones: 1.17, On Deck Nov 29, 2023
@gcampbell-msft gcampbell-msft modified the milestones: On Deck, 1.17 Dec 12, 2023
@gcampbell-msft gcampbell-msft self-assigned this Dec 18, 2023
@gcampbell-msft gcampbell-msft added the fixed (release pending) fixed in pre-release and is pending official release label Jan 3, 2024
1.17 automation moved this from Triage to Done Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug a bug in the product Feature: ctest fixed (release pending) fixed in pre-release and is pending official release
Projects
Status: Completed
1.17
Done
Development

No branches or pull requests

5 participants