Skip to content

Add additional diagnostics to GitStatusAfterRenameFileIntoRepo functional test#1006

Merged
wilbaker merged 1 commit into
microsoft:masterfrom
wilbaker:test_reliability_GitStatusAfterRenameFileIntoRepo
Apr 4, 2019
Merged

Add additional diagnostics to GitStatusAfterRenameFileIntoRepo functional test#1006
wilbaker merged 1 commit into
microsoft:masterfrom
wilbaker:test_reliability_GitStatusAfterRenameFileIntoRepo

Conversation

@wilbaker
Copy link
Copy Markdown
Member

@wilbaker wilbaker commented Apr 3, 2019

GitStatusAfterRenameFileIntoRepo was occasionally failing (see #890 for details).

From the failure report in #890, it was unclear if the failure was due to the move/rename failing, or the rename notification not being delivered to VFS4G.

Changes in this PR:

  • ShouldNotExistOnDisk was being called with the wrong path, this has been fixed
  • Added validation that the renamed file does exist at the location we expect it to before calling git status.

@wilbaker wilbaker added the type: test-reliability Issues that contribute to test failures label Apr 3, 2019
@wilbaker wilbaker added this to the M151 milestone Apr 3, 2019
@wilbaker
Copy link
Copy Markdown
Member Author

wilbaker commented Apr 3, 2019

/azp run Windows - Full Functional Tests

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Member

@jrbriggs jrbriggs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: test-reliability Issues that contribute to test failures

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants