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

[Linux] Convert tests and repair jobs to use backing root #1411

Merged
merged 3 commits into from
Aug 19, 2019

Conversation

chrisd8088
Copy link
Contributor

@chrisd8088 chrisd8088 commented Aug 5, 2019

Convert all relevant uses of RepoRoot and WorkingDirectoryRoot in the GVFS provider and functional tests to use the backing root, to ensure these tests will pass on systems where they are distinct (e.g., Linux).

For example. repair jobs run while the GVFS enlistment is not mounted, which on Linux requires us to use the backing root in order to access the .git contents; similarly, the MountTests are adjusted to ensure they don't expect file contents within an unmounted repository.

/cc @jrbriggs, @kivikakk.

chrisd8088 and others added 3 commits August 4, 2019 22:22
Repair jobs run while the GVFS enlistment is not mounted,
which on Linux requires us to use the backing root in order
to access the .git contents.
@chrisd8088 chrisd8088 added the WIP label Aug 5, 2019
@chrisd8088 chrisd8088 changed the title [WIP] Convert tests and repair jobs to use backing root [Linux] Convert tests and repair jobs to use backing root Aug 5, 2019
@chrisd8088 chrisd8088 marked this pull request as ready for review August 5, 2019 13:03
@chrisd8088
Copy link
Contributor Author

Hi @derrickstolee -- do you want to look this one over as well, or are you comfortable with my merging it into master now that @wilbaker signed off? Let me know either way, and many thanks!

@wilbaker
Copy link
Member

Hi @derrickstolee -- do you want to look this one over as well, or are you comfortable with my merging it into master now that @wilbaker signed off? Let me know either way, and many thanks!

@chrisd8088 feel free to merge this PR in now, thanks!

cc: @derrickstolee

@chrisd8088 chrisd8088 merged commit e073946 into microsoft:master Aug 19, 2019
@chrisd8088 chrisd8088 deleted the backing-root-fixes branch August 19, 2019 18:11
@alameenshah alameenshah added this to the M159 milestone Sep 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants