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

Eliminate Pester duplicate failures in Problems view. #941

Merged
merged 1 commit into from
Jul 6, 2017

Conversation

rkeithhill
Copy link
Contributor

When tasks is set to 2.0.0, the tasks run in the terminal and in this case, Pester outputs a bunch of progress text to the terminal. This results in duplicate failures being found by the problem matcher.

Work around this problem for now by eliminating the Pester progress output. For more details see - microsoft/vscode#22065 (comment)

When tasks is set to 2.0.0, the tasks run in the terminal and in this case, Pester outputs a bunch of progress text to the terminal.  This results in duplicate failures being found by the problem matcher.

Work around this problem for now by eliminating the Pester progress output.  For more details see - microsoft/vscode#22065 (comment)
@rkeithhill
Copy link
Contributor Author

Note: we still have a problem if the terminal window is too narrow. PowerShell will wrap the line that contains the line number and file path. The current $pester problem matcher expects this info to be on a single line. I'll look into updating the problem matcher but that might not be so easy. Workaround for now is to make your terminal window wider (or use tasks 0.1.0).

@daviwil
Copy link
Contributor

daviwil commented Jul 6, 2017

Cool, thanks!

@daviwil daviwil added this to the July 2017 milestone Jul 6, 2017
@daviwil daviwil merged commit 92bf3ce into master Jul 6, 2017
@rkeithhill rkeithhill deleted the rkeihthill/fix-pester-task-problem-dupes branch July 6, 2017 17:37
@daviwil daviwil modified the milestones: July 2017, 1.5.0 Oct 26, 2017
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

Successfully merging this pull request may close these issues.

3 participants