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

Fix TasksCanAddRecursiveDirBuiltInMetadata() #6337

Merged
merged 4 commits into from
Apr 22, 2021

Conversation

KirillOsenkov
Copy link
Member

Don't use Program Files\Reference Assemblies as that might not exist for 64-bit.

Pass ITestOutputLogger to MockLogger to get the full log in case of failure.

Search for a log substring that will only be there in case of success. Even successful logs can contain [] when the file is at the root of the directory so RecursiveDir will be empty.

Fixes #6267

Don't use Program Files\Reference Assemblies as that might not exist for 64-bit.

Pass ITestOutputLogger to MockLogger to get the full log in case of failure.

Search for a log substring that will only be there in case of success. Even successful logs can contain [] when the file is at the root of the directory so RecursiveDir will be empty.
KirillOsenkov and others added 2 commits April 12, 2021 09:40
Co-authored-by: Forgind <Forgind@users.noreply.github.com>
@Forgind Forgind added the merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now. label Apr 15, 2021
@benvillalobos benvillalobos merged commit b2d404f into main Apr 22, 2021
@benvillalobos benvillalobos deleted the dev/kirillo/recursiveDirTest branch April 22, 2021 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flaky test: Microsoft.Build.UnitTests.BackEnd.TaskBuilder_Tests.TasksCanAddRecursiveDirBuiltInMetadata
5 participants