Skip to content

Conversation

@TravisEz13
Copy link
Member

@TravisEz13 TravisEz13 commented Dec 3, 2018

https://powershell.visualstudio.com/PowerShell/_build/results?buildId=7213&view=ms.vss-test-web.test-result-details

PR Summary

Fix an intermittent failure in macOS logging tests

  • make tests wait for the correct number of log entries
  • make tests not fail with an Index was outside the bounds of the array. error

PR Checklist

@TravisEz13 TravisEz13 requested a review from daxian-dbw December 3, 2018 18:55
@TravisEz13 TravisEz13 merged commit 142814c into PowerShell:master Dec 3, 2018
@TravisEz13 TravisEz13 deleted the fix_macos_logging_test branch December 3, 2018 22:29
@adityapatwardhan adityapatwardhan added the CL-Test Indicates that a PR should be marked as a test change in the Change Log label Dec 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-Test Indicates that a PR should be marked as a test change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants