Disable GitTagActionTest on Windows

Unclear why the test is failing on ci.jenkins.io when it regularly
succeeds in my Windows environments.  Test is not valuable enough to
justify a long research project to identify why it behaves differently
on ci.jenkins.io than on other Windows computers.

Test runs on Linux.  Test coverage is measured on Linux.

Author: MarkEWaite

src/test/java/hudson/plugins/git/GitTagActionTest.java

@@ -1,6 +1,7 @@
 package hudson.plugins.git;
 
 import java.io.File;
+import java.io.StringWriter;
 import java.time.LocalDateTime;
 import java.time.format.DateTimeFormatter;
 import java.util.ArrayList;
@@ -19,6 +20,7 @@
 import hudson.model.FreeStyleProject;
 import hudson.model.Run;
 import hudson.model.TaskListener;
+import hudson.plugins.git.Branch;
 import hudson.plugins.git.extensions.GitSCMExtension;
 import hudson.plugins.git.extensions.impl.LocalBranch;
 
@@ -30,6 +32,7 @@
 
 import static org.hamcrest.Matchers.*;
 import static org.junit.Assert.*;
+import static org.junit.Assume.*;
 
 import com.gargoylesoftware.htmlunit.html.HtmlForm;
 import com.gargoylesoftware.htmlunit.html.HtmlPage;
@@ -41,7 +44,13 @@
 import org.jvnet.hudson.test.JenkinsRule;
 
 /**
- * Test git tag action.
+ * Test git tag action. Low value test that was created as part of
+ * another investigation.
+ *
+ * Unreliable on ci.jenkins.io Windows agents. Results are not worth
+ * sacrificing other things in order to investigate.  Runs reliably on
+ * Unix-like operating systems.  Runs reliably on Mark Waite's windows
+ * computers.
  *
  * @author Mark Waite
  */
@@ -71,13 +80,19 @@ public GitTagActionTest() {
     private static final DateTimeFormatter FORMAT = DateTimeFormatter.ofPattern("-yyyy-MM-dd-H-m-ss.SS");
     private static final String TAG_PREFIX = "test-tag-";
     private static final String TAG_SUFFIX = LocalDateTime.now().format(FORMAT);
+    private static final String INITIAL_COMMIT_MESSAGE = "init" + TAG_SUFFIX + "-" + random.nextInt(10000);
+    private static final String ADDED_COMMIT_MESSAGE_BASE = "added" + TAG_SUFFIX;
+    private static String sampleRepoHead = null;
 
     @BeforeClass
     public static void deleteMatchingTags() throws Exception {
+        if (isWindows()) { // Test is unreliable on Windows, too low value to investigate further
+            return;
+        }
         /* Remove tags from working repository that start with TAG_PREFIX and don't contain TAG_SUFFIX */
         GitClient gitClient = Git.with(TaskListener.NULL, new EnvVars())
                 .in(new File("."))
-                .using(random.nextBoolean() ? "git" : "jgit") // Use random implementation, both should work
+                .using(chooseGitImplementation()) // Use random implementation, both should work
                 .getClient();
         for (GitObject tag : gitClient.getTags()) {
             if (tag.getName().startsWith(TAG_PREFIX) && !tag.getName().contains(TAG_SUFFIX)) {
@@ -88,10 +103,13 @@ public static void deleteMatchingTags() throws Exception {
 
     @BeforeClass
     public static void createThreeGitTagActions() throws Exception {
+        if (isWindows()) { // Test is unreliable on Windows, too low value to investigate further
+            return;
+        }
         sampleRepo.init();
-        sampleRepo.write("file", "init");
-        sampleRepo.git("commit", "--all", "--message=init");
-        String head = sampleRepo.head();
+        sampleRepo.write("file", INITIAL_COMMIT_MESSAGE);
+        sampleRepo.git("commit", "--all", "--message=" + INITIAL_COMMIT_MESSAGE);
+        sampleRepoHead = sampleRepo.head();
         List<UserRemoteConfig> remotes = new ArrayList<>();
         String refSpec = "+refs/heads/master:refs/remotes/origin/master";
         remotes.add(new UserRemoteConfig(sampleRepo.fileUrl(), "origin", refSpec, ""));
@@ -100,7 +118,7 @@ public static void createThreeGitTagActions() throws Exception {
                 Collections.singletonList(new BranchSpec("origin/master")),
                 false, Collections.<SubmoduleConfig>emptyList(),
                 null,
-                random.nextBoolean() ? "git" : "jgit", // Both git implementations should work, choose randomly
+                chooseGitImplementation(), // Both git implementations should work, choose randomly
                 Collections.<GitSCMExtension>emptyList());
         scm.getExtensions().add(new LocalBranch("master"));
         p = r.createFreeStyleProject();
@@ -136,6 +154,8 @@ private static String getTagComment(String message) {
         return getTagName(message) + "-comment";
     }
 
+    private static int messageCounter = 1;
+
     /**
      * Return a GitTagAction which uses 'message' in the tag name, tag value, and tag comment.
      * If 'message' is null, the GitTagAction is returned but tag creation is not scheduled.
@@ -146,8 +166,9 @@ private static String getTagComment(String message) {
      */
     private static GitTagAction createTagAction(String message) throws Exception {
         /* Run with a tag action defined */
+        String commitMessage = message == null ? ADDED_COMMIT_MESSAGE_BASE + "-" + messageCounter++ : message;
         sampleRepo.write("file", message);
-        sampleRepo.git("commit", "--all", "--message=" + (message == null ? random.nextInt() : message));
+        sampleRepo.git("commit", "--all", "--message=" + commitMessage);
         List<Branch> masterBranchList = new ArrayList<>();
         ObjectId tagObjectId = ObjectId.fromString(sampleRepo.head());
         masterBranchList.add(new Branch("master", tagObjectId));
@@ -162,12 +183,38 @@ private static GitTagAction createTagAction(String message) throws Exception {
             /* Assumes workspace does not move after first run */
             workspaceGitClient = Git.with(TaskListener.NULL, new EnvVars())
                     .in(workspace)
-                    .using(random.nextBoolean() ? "git" : "jgit") // Use random implementation, both should work
+                    .using(chooseGitImplementation()) // Use random implementation, both should work
                     .getClient();
         }
         /* Fail if the workspace moved */
         assertThat(workspace, is(workspaceGitClient.getWorkTree()));
 
+        /* Fail if initial commit and subsequent commit not detected in workspace */
+        StringWriter stringWriter = new StringWriter();
+        workspaceGitClient.changelog(sampleRepoHead + "^", "HEAD", stringWriter);
+        assertThat(stringWriter.toString(), containsString(INITIAL_COMMIT_MESSAGE));
+        assertThat(stringWriter.toString(), containsString(commitMessage));
+
+        /* Fail if master branch is not defined in the workspace */
+        assertThat(workspaceGitClient.getRemoteUrl("origin"), is(sampleRepo.fileUrl().replace("file:/", "file:///")));
+        Set<Branch> branches = workspaceGitClient.getBranches();
+        if (branches.isEmpty()) {
+            /* Should not be required since the LocalBranch extension was enabled */
+            workspaceGitClient.branch("master");
+            branches = workspaceGitClient.getBranches();
+            assertThat(branches, is(not(empty())));
+        }
+        boolean foundMasterBranch = false;
+        String lastBranchName = null;
+        for (Branch branch : branches) {
+            lastBranchName = branch.getName();
+            assertThat(lastBranchName, endsWith("master"));
+            if (lastBranchName.equals("master")) {
+                foundMasterBranch = true;
+            }
+        }
+        assertTrue("master branch not found, last branch name was " + lastBranchName, foundMasterBranch);
+
         /* Create the GitTagAction */
         GitTagAction tagAction = new GitTagAction(tagRun, workspace, tagRevision);
 
@@ -200,12 +247,13 @@ private static void waitForTagCreation(GitTagAction tagAction, String message) t
             backoffDelay = backoffDelay * 2;
             Thread.sleep(backoffDelay); // Allow some time for tag creation
         }
-        assertThat(tagAction.getLastTagName(), is(getTagValue(message)));
         assertThat(tagAction.getLastTagException(), is(nullValue()));
+        assertThat(tagAction.getLastTagName(), is(getTagValue(message)));
     }
 
     @Test
     public void testDoPost() throws Exception {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         JenkinsRule.WebClient browser = r.createWebClient();
 
         // Don't need all cases until at least one case works fully
@@ -241,44 +289,52 @@ public void testDoPost() throws Exception {
 
     @Test
     public void testGetDescriptor() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         Descriptor<GitTagAction> descriptor = noTagAction.getDescriptor();
         assertThat(descriptor.getDisplayName(), is("Tag"));
     }
 
     // @Test
     public void testIsTagged() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         assertTrue(tagTwoAction.isTagged());
     }
 
     @Test
     public void testIsNotTagged() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         assertFalse(noTagAction.isTagged());
     }
 
     @Test
     public void testGetDisplayNameNoTagAction() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         assertThat(noTagAction.getDisplayName(), is("No Tags"));
     }
 
     // Not working yet
     // @Test
     public void testGetDisplayNameOneTagAction() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         assertThat(tagOneAction.getDisplayName(), is("One Tag"));
     }
 
     // Not working yet
     // @Test
     public void testGetDisplayNameTwoTagAction() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         assertThat(tagTwoAction.getDisplayName(), is("Multiple Tags"));
     }
 
     @Test
     public void testGetIconFileName() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         assertThat(noTagAction.getIconFileName(), is("save.gif"));
     }
 
     @Test
     public void testGetTagsNoTagAction() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         Collection<List<String>> valueList = noTagAction.getTags().values();
         for (List<String> value : valueList) {
             assertThat(value, is(empty()));
@@ -287,6 +343,7 @@ public void testGetTagsNoTagAction() {
 
     @Test
     public void testGetTagsOneTagAction() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         Collection<List<String>> valueList = tagOneAction.getTags().values();
         for (List<String> value : valueList) {
             assertThat(value, is(empty()));
@@ -295,6 +352,7 @@ public void testGetTagsOneTagAction() {
 
     @Test
     public void testGetTagsTwoTagAction() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         Collection<List<String>> valueList = tagTwoAction.getTags().values();
         for (List<String> value : valueList) {
             assertThat(value, is(empty()));
@@ -303,17 +361,32 @@ public void testGetTagsTwoTagAction() {
 
     @Test
     public void testGetTagInfo() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         assertThat(noTagAction.getTagInfo(), is(empty()));
     }
 
     @Test
     public void testGetTooltipNoTagAction() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         assertThat(noTagAction.getTooltip(), is(nullValue()));
     }
 
     @Test
     public void testGetPermission() {
+        assumeTrue(!isWindows()); // Test is unreliable on Windows, too low value to investigate further
         assertThat(noTagAction.getPermission(), is(GitSCM.TAG));
         assertThat(tagOneAction.getPermission(), is(GitSCM.TAG));
     }
+
+    private static String chooseGitImplementation() {
+        return random.nextBoolean() ? "git" : "jgit";
+    }
+
+    /**
+     * inline ${@link hudson.Functions#isWindows()} to prevent a transient
+     * remote classloader issue
+     */
+    private static boolean isWindows() {
+        return File.pathSeparatorChar == ';';
+    }
 }