FunctionalTests: Use temporary branch to fix all tests

Signed-off-by: Derrick Stolee <[email protected]>
microsoft · Oct 15, 2020 · 3d4436b · 3d4436b
1 parent b9c6445
commit 3d4436b
Show file tree

Hide file tree

Showing 14 changed files with 125 additions and 80 deletions.
diff --git a/GVFS/GVFS.FunctionalTests/Settings.cs b/GVFS/GVFS.FunctionalTests/Settings.cs
@@ -35,7 +35,13 @@ public static void Initialize()
                 CurrentDirectory = Path.GetFullPath(Path.GetDirectoryName(Environment.GetCommandLineArgs()[0]));
 
                 RepoToClone = @"https://gvfs.visualstudio.com/ci/_git/ForTests";
-                Commitish = @"FunctionalTests/20180214";
+
+                // HACK: This is only different from FunctionalTests/20180214
+                // in that it deletes the GVFlt_MoveFileTests/LongFileName folder,
+                // which is causing problems in all tests due to a ProjFS
+                // regression. Replace this with the expected default after
+                // ProjFS is fixed and deployed to our build machines.
+                Commitish = @"FunctionalTests/20201014";
 
                 if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
                 {

diff --git a/GVFS/GVFS.FunctionalTests/Tests/EnlistmentPerFixture/GVFSLockTests.cs b/GVFS/GVFS.FunctionalTests/Tests/EnlistmentPerFixture/GVFSLockTests.cs
@@ -46,7 +46,7 @@ public void GitCheckoutFailsOutsideLock()
                 this.fileSystem.MoveFile(Path.Combine(hooksBase, preCommand), Path.Combine(hooksBase, BackupPrefix + preCommand));
                 this.fileSystem.MoveFile(Path.Combine(hooksBase, postCommand), Path.Combine(hooksBase, BackupPrefix + postCommand));
 
-                ProcessResult result = GitHelpers.InvokeGitAgainstGVFSRepo(this.Enlistment.RepoRoot, "checkout FunctionalTests/20170510_minor");
+                ProcessResult result = GitHelpers.InvokeGitAgainstGVFSRepo(this.Enlistment.RepoRoot, "checkout FunctionalTests/20201014_minor");
                 result.Errors.ShouldContain("fatal: unable to write new index file");
 
                 // Ensure that branch didnt move, note however that work dir might not be clean

diff --git a/GVFS/GVFS.FunctionalTests/Tests/EnlistmentPerFixture/HealthTests.cs b/GVFS/GVFS.FunctionalTests/Tests/EnlistmentPerFixture/HealthTests.cs
@@ -25,7 +25,7 @@ public void AfterCloningRepoIsPerfectlyHealthy()
 
             this.ValidateHealthOutputValues(
                 directory: string.Empty,
-                totalFiles: 1211,
+                totalFiles: 1197,
                 totalFilePercent: 100,
                 fastFiles: 1,
                 fastFilePercent: 0,
@@ -53,7 +53,7 @@ public void PlaceholdersChangeHealthScores()
 
             this.ValidateHealthOutputValues(
                 directory: string.Empty,
-                totalFiles: 1211,
+                totalFiles: 1197,
                 totalFilePercent: 100,
                 fastFiles: 7,
                 fastFilePercent: 1,
@@ -78,7 +78,7 @@ public void ModifiedPathsChangeHealthScores()
 
             this.ValidateHealthOutputValues(
                 directory: string.Empty,
-                totalFiles: 1211,
+                totalFiles: 1197,
                 totalFilePercent: 100,
                 fastFiles: 8,
                 fastFilePercent: 1,
@@ -105,7 +105,7 @@ public void TurnPlaceholdersIntoModifiedPaths()
 
             this.ValidateHealthOutputValues(
                 directory: string.Empty,
-                totalFiles: 1211,
+                totalFiles: 1197,
                 totalFilePercent: 100,
                 fastFiles: 3,
                 fastFilePercent: 0,

diff --git a/GVFS/GVFS.FunctionalTests/Tests/EnlistmentPerFixture/PrefetchVerbTests.cs b/GVFS/GVFS.FunctionalTests/Tests/EnlistmentPerFixture/PrefetchVerbTests.cs
@@ -15,7 +15,7 @@ namespace GVFS.FunctionalTests.Tests.EnlistmentPerFixture
     public class PrefetchVerbTests : TestsWithEnlistmentPerFixture
     {
         private const string PrefetchCommitsAndTreesLock = "prefetch-commits-trees.lock";
-        private const string LsTreeTypeInPathBranchName = "FunctionalTests/20181105_LsTreeTypeInPath";
+        private const string LsTreeTypeInPathBranchName = "FunctionalTests/20201014_LsTreeTypeInPath";
 
         // on case-insensitive filesystems, test case-blind matching in
         // folder lists using "gvfs/" instead of "GVFS/"

diff --git a/GVFS/GVFS.FunctionalTests/Tests/EnlistmentPerFixture/SymbolicLinkTests.cs b/GVFS/GVFS.FunctionalTests/Tests/EnlistmentPerFixture/SymbolicLinkTests.cs
@@ -44,11 +44,11 @@ public SymbolicLinkTests()
         [TestCase, Order(1)]
         public void CheckoutBranchWithSymLinks()
         {
-            GitHelpers.InvokeGitAgainstGVFSRepo(this.Enlistment.RepoRoot, "checkout FunctionalTests/20180925_SymLinksPart1");
+            GitHelpers.InvokeGitAgainstGVFSRepo(this.Enlistment.RepoRoot, "checkout FunctionalTests/20201014_SymLinksPart1");
             GitHelpers.CheckGitCommandAgainstGVFSRepo(
                 this.Enlistment.RepoRoot,
                 "status",
-                "On branch FunctionalTests/20180925_SymLinksPart1",
+                "On branch FunctionalTests/20201014_SymLinksPart1",
                 "nothing to commit, working tree clean");
 
             string testFilePath = this.Enlistment.GetVirtualPathTo(Path.Combine(TestFolderName, TestFileName));
@@ -75,11 +75,11 @@ public void CheckoutBranchWithSymLinks()
         [TestCase, Order(2)]
         public void CheckoutBranchWhereSymLinksChangeContentsAndTransitionToFile()
         {
-            GitHelpers.InvokeGitAgainstGVFSRepo(this.Enlistment.RepoRoot, "checkout FunctionalTests/20180925_SymLinksPart2");
+            GitHelpers.InvokeGitAgainstGVFSRepo(this.Enlistment.RepoRoot, "checkout FunctionalTests/20201014_SymLinksPart2");
             GitHelpers.CheckGitCommandAgainstGVFSRepo(
                 this.Enlistment.RepoRoot,
                 "status",
-                "On branch FunctionalTests/20180925_SymLinksPart2",
+                "On branch FunctionalTests/20201014_SymLinksPart2",
                 "nothing to commit, working tree clean");
 
             // testFilePath and testFile2Path are unchanged from FunctionalTests/20180925_SymLinksPart2
@@ -114,11 +114,11 @@ public void CheckoutBranchWhereSymLinksChangeContentsAndTransitionToFile()
         [TestCase, Order(3)]
         public void CheckoutBranchWhereFilesTransitionToSymLinks()
         {
-            GitHelpers.InvokeGitAgainstGVFSRepo(this.Enlistment.RepoRoot, "checkout FunctionalTests/20180925_SymLinksPart3");
+            GitHelpers.InvokeGitAgainstGVFSRepo(this.Enlistment.RepoRoot, "checkout FunctionalTests/20201014_SymLinksPart3");
             GitHelpers.CheckGitCommandAgainstGVFSRepo(
                 this.Enlistment.RepoRoot,
                 "status",
-                "On branch FunctionalTests/20180925_SymLinksPart3",
+                "On branch FunctionalTests/20201014_SymLinksPart3",
                 "nothing to commit, working tree clean");
 
             // In this branch testFilePath has been changed to point to newGrandChildFilePath
@@ -155,11 +155,11 @@ public void CheckoutBranchWhereFilesTransitionToSymLinks()
         [TestCase, Order(4)]
         public void CheckoutBranchWhereSymLinkTransistionsToFolderAndFolderTransitionsToSymlink()
         {
-            GitHelpers.InvokeGitAgainstGVFSRepo(this.Enlistment.RepoRoot, "checkout FunctionalTests/20180925_SymLinksPart4");
+            GitHelpers.InvokeGitAgainstGVFSRepo(this.Enlistment.RepoRoot, "checkout FunctionalTests/20201014_SymLinksPart4");
             GitHelpers.CheckGitCommandAgainstGVFSRepo(
                 this.Enlistment.RepoRoot,
                 "status",
-                "On branch FunctionalTests/20180925_SymLinksPart4",
+                "On branch FunctionalTests/20201014_SymLinksPart4",
                 "nothing to commit, working tree clean");
 
             // In this branch ChildLinkName has been changed to a directory and ChildFolder2Name has been changed to a link to ChildFolderName
@@ -179,7 +179,7 @@ public void GitStatusReportsSymLinkChanges()
             GitHelpers.CheckGitCommandAgainstGVFSRepo(
                 this.Enlistment.RepoRoot,
                 "status",
-                "On branch FunctionalTests/20180925_SymLinksPart4",
+                "On branch FunctionalTests/20201014_SymLinksPart4",
                 "nothing to commit, working tree clean");
 
             string testFilePath = this.Enlistment.GetVirtualPathTo(Path.Combine(TestFolderName, TestFileName));
@@ -199,7 +199,7 @@ public void GitStatusReportsSymLinkChanges()
             GitHelpers.CheckGitCommandAgainstGVFSRepo(
                 this.Enlistment.RepoRoot,
                 "status",
-                "On branch FunctionalTests/20180925_SymLinksPart4",
+                "On branch FunctionalTests/20201014_SymLinksPart4",
                 $"modified:   {TestFolderName}/{TestFileName}");
         }
     }