From e377dce8818ecd55bb599a8532376cf55e56381a Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Fri, 30 Nov 2012 16:20:24 -0500 Subject: [PATCH] Fixed regression on expiration date of CA and SSL certs --- tests/com/gitblit/tests/JGitUtilsTest.java | 283 +++++++++++++++++++++++++++++++++++++++---------------- 1 files changed, 199 insertions(+), 84 deletions(-) diff --git a/tests/com/gitblit/tests/JGitUtilsTest.java b/tests/com/gitblit/tests/JGitUtilsTest.java index 6646bf8..7e4d630 100644 --- a/tests/com/gitblit/tests/JGitUtilsTest.java +++ b/tests/com/gitblit/tests/JGitUtilsTest.java @@ -15,14 +15,19 @@ */ package com.gitblit.tests; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertNull; +import static org.junit.Assert.assertTrue; + import java.io.File; import java.io.FileOutputStream; +import java.text.SimpleDateFormat; import java.util.Arrays; import java.util.Date; import java.util.List; import java.util.Map; - -import junit.framework.TestCase; import org.eclipse.jgit.diff.DiffEntry.ChangeType; import org.eclipse.jgit.lib.Constants; @@ -30,8 +35,15 @@ import org.eclipse.jgit.lib.ObjectId; import org.eclipse.jgit.lib.PersonIdent; import org.eclipse.jgit.lib.Repository; +import org.eclipse.jgit.lib.RepositoryCache; +import org.eclipse.jgit.lib.RepositoryCache.FileKey; import org.eclipse.jgit.revwalk.RevCommit; +import org.eclipse.jgit.revwalk.RevTree; +import org.eclipse.jgit.util.FS; +import org.eclipse.jgit.util.FileUtils; +import org.junit.Test; +import com.gitblit.Constants.SearchType; import com.gitblit.GitBlit; import com.gitblit.Keys; import com.gitblit.models.GitNote; @@ -39,83 +51,103 @@ import com.gitblit.models.PathModel.PathChangeModel; import com.gitblit.models.RefModel; import com.gitblit.utils.JGitUtils; -import com.gitblit.utils.JGitUtils.SearchType; import com.gitblit.utils.StringUtils; -public class JGitUtilsTest extends TestCase { +public class JGitUtilsTest { + @Test public void testDisplayName() throws Exception { - assertTrue(JGitUtils.getDisplayName(new PersonIdent("Napoleon Bonaparte", "")).equals( - "Napoleon Bonaparte")); - assertTrue(JGitUtils.getDisplayName(new PersonIdent("", "someone@somewhere.com")).equals( - "<someone@somewhere.com>")); - assertTrue(JGitUtils.getDisplayName( - new PersonIdent("Napoleon Bonaparte", "someone@somewhere.com")).equals( - "Napoleon Bonaparte <someone@somewhere.com>")); + assertEquals("Napoleon Bonaparte", + JGitUtils.getDisplayName(new PersonIdent("Napoleon Bonaparte", ""))); + assertEquals("<someone@somewhere.com>", + JGitUtils.getDisplayName(new PersonIdent("", "someone@somewhere.com"))); + assertEquals("Napoleon Bonaparte <someone@somewhere.com>", + JGitUtils.getDisplayName(new PersonIdent("Napoleon Bonaparte", + "someone@somewhere.com"))); } + @Test public void testFindRepositories() { - List<String> list = JGitUtils.getRepositoryList(null, true, true); - assertTrue(list.size() == 0); - list.addAll(JGitUtils.getRepositoryList(new File("DoesNotExist"), true, true)); - assertTrue(list.size() == 0); - list.addAll(JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, true, true)); + List<String> list = JGitUtils.getRepositoryList(null, false, true, -1, null); + assertEquals(0, list.size()); + list.addAll(JGitUtils.getRepositoryList(new File("DoesNotExist"), true, true, -1, null)); + assertEquals(0, list.size()); + list.addAll(JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, false, true, -1, null)); assertTrue("No repositories found in " + GitBlitSuite.REPOSITORIES, list.size() > 0); } + @Test + public void testFindExclusions() { + List<String> list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, false, true, -1, null); + assertTrue("Missing jgit repository?!", list.contains("test/jgit.git")); + + list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, false, true, -1, Arrays.asList("test/jgit\\.git")); + assertFalse("Repository exclusion failed!", list.contains("test/jgit.git")); + + list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, false, true, -1, Arrays.asList("test/*")); + assertFalse("Repository exclusion failed!", list.contains("test/jgit.git")); + + list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, false, true, -1, Arrays.asList(".*jgit.*")); + assertFalse("Repository exclusion failed!", list.contains("test/jgit.git")); + assertFalse("Repository exclusion failed!", list.contains("working/jgit")); + assertFalse("Repository exclusion failed!", list.contains("working/jgit2")); + + } + + @Test public void testOpenRepository() throws Exception { Repository repository = GitBlitSuite.getHelloworldRepository(); repository.close(); - assertTrue("Could not find repository!", repository != null); + assertNotNull("Could not find repository!", repository); } + @Test public void testFirstCommit() throws Exception { - assertTrue(JGitUtils.getFirstChange(null, null).equals(new Date(0))); + assertEquals(new Date(0), JGitUtils.getFirstChange(null, null)); Repository repository = GitBlitSuite.getHelloworldRepository(); RevCommit commit = JGitUtils.getFirstCommit(repository, null); Date firstChange = JGitUtils.getFirstChange(repository, null); repository.close(); - assertTrue("Could not get first commit!", commit != null); - assertTrue("Incorrect first commit!", - commit.getName().equals("f554664a346629dc2b839f7292d06bad2db4aece")); + assertNotNull("Could not get first commit!", commit); + assertEquals("Incorrect first commit!", "f554664a346629dc2b839f7292d06bad2db4aece", + commit.getName()); assertTrue(firstChange.equals(new Date(commit.getCommitTime() * 1000L))); } + @Test public void testLastCommit() throws Exception { - assertTrue(JGitUtils.getLastChange(null).equals(new Date(0))); + assertEquals(new Date(0), JGitUtils.getLastChange(null)); Repository repository = GitBlitSuite.getHelloworldRepository(); assertTrue(JGitUtils.getCommit(repository, null) != null); Date date = JGitUtils.getLastChange(repository); repository.close(); - assertTrue("Could not get last repository change date!", date != null); + assertNotNull("Could not get last repository change date!", date); } + @Test public void testCreateRepository() throws Exception { String[] repositories = { "NewTestRepository.git", "NewTestRepository" }; for (String repositoryName : repositories) { - boolean isBare = repositoryName.endsWith(".git"); Repository repository = JGitUtils.createRepository(GitBlitSuite.REPOSITORIES, - repositoryName, isBare); - File folder; - if (isBare) { - folder = new File(GitBlitSuite.REPOSITORIES, repositoryName); - } else { - folder = new File(GitBlitSuite.REPOSITORIES, repositoryName + "/.git"); - } - assertTrue(repository != null); + repositoryName); + File folder = FileKey.resolve(new File(GitBlitSuite.REPOSITORIES, repositoryName), + FS.DETECTED); + assertNotNull(repository); assertFalse(JGitUtils.hasCommits(repository)); - assertTrue(JGitUtils.getFirstCommit(repository, null) == null); - assertTrue(JGitUtils.getFirstChange(repository, null).getTime() == folder - .lastModified()); - assertTrue(JGitUtils.getLastChange(repository).getTime() == folder.lastModified()); - assertTrue(JGitUtils.getCommit(repository, null) == null); + assertNull(JGitUtils.getFirstCommit(repository, null)); + assertEquals(folder.lastModified(), JGitUtils.getFirstChange(repository, null) + .getTime()); + assertEquals(folder.lastModified(), JGitUtils.getLastChange(repository).getTime()); + assertNull(JGitUtils.getCommit(repository, null)); repository.close(); - assertTrue(GitBlit.self().deleteRepository(repositoryName)); + RepositoryCache.close(repository); + FileUtils.delete(repository.getDirectory(), FileUtils.RECURSIVE); } } + @Test public void testRefs() throws Exception { Repository repository = GitBlitSuite.getJGitRepository(); Map<ObjectId, List<RefModel>> map = JGitUtils.getAllRefs(repository); @@ -125,11 +157,13 @@ List<RefModel> list = entry.getValue(); for (RefModel ref : list) { if (ref.displayName.equals("refs/tags/spearce-gpg-pub")) { - assertTrue(ref.getObjectId().getName().equals("8bbde7aacf771a9afb6992434f1ae413e010c6d8")); - assertTrue(ref.getAuthorIdent().getEmailAddress().equals("spearce@spearce.org")); + assertEquals("refs/tags/spearce-gpg-pub", ref.toString()); + assertEquals("8bbde7aacf771a9afb6992434f1ae413e010c6d8", ref.getObjectId() + .getName()); + assertEquals("spearce@spearce.org", ref.getAuthorIdent().getEmailAddress()); assertTrue(ref.getShortMessage().startsWith("GPG key")); - assertTrue(ref.getFullMessage().startsWith("GPG key")); - assertTrue(ref.getReferencedObjectType() == Constants.OBJ_BLOB); + assertTrue(ref.getFullMessage().startsWith("GPG key")); + assertEquals(Constants.OBJ_BLOB, ref.getReferencedObjectType()); } else if (ref.displayName.equals("refs/tags/v0.12.1")) { assertTrue(ref.isAnnotatedTag()); } @@ -137,8 +171,10 @@ } } + @Test public void testBranches() throws Exception { - Repository repository = GitBlitSuite.getTicgitRepository(); + Repository repository = GitBlitSuite.getJGitRepository(); + assertTrue(JGitUtils.getLocalBranches(repository, true, 0).size() == 0); for (RefModel model : JGitUtils.getLocalBranches(repository, true, -1)) { assertTrue(model.getName().startsWith(Constants.R_HEADS)); assertTrue(model.equals(model)); @@ -155,14 +191,16 @@ + model.getName().hashCode()); assertTrue(model.getShortMessage().equals(model.getShortMessage())); } - assertTrue(JGitUtils.getRemoteBranches(repository, true, 10).size() == 10); + assertTrue(JGitUtils.getRemoteBranches(repository, true, 8).size() == 8); repository.close(); } + @Test public void testTags() throws Exception { - Repository repository = GitBlitSuite.getTicgitRepository(); + Repository repository = GitBlitSuite.getJGitRepository(); + assertTrue(JGitUtils.getTags(repository, true, 5).size() == 5); for (RefModel model : JGitUtils.getTags(repository, true, -1)) { - if (model.getObjectId().getName().equals("283035e4848054ff1803cb0e690270787dc92399")) { + if (model.getObjectId().getName().equals("d28091fb2977077471138fe97da1440e0e8ae0da")) { assertTrue("Not an annotated tag!", model.isAnnotatedTag()); } assertTrue(model.getName().startsWith(Constants.R_TAGS)); @@ -172,18 +210,19 @@ + model.getName().hashCode()); } repository.close(); - - repository = GitBlitSuite.getBluezGnomeRepository(); + + repository = GitBlitSuite.getGitectiveRepository(); for (RefModel model : JGitUtils.getTags(repository, true, -1)) { - if (model.getObjectId().getName().equals("728643ec0c438c77e182898c2f2967dbfdc231c8")) { + if (model.getObjectId().getName().equals("035254295a9bba11f72b1f9d6791a6b957abee7b")) { assertFalse(model.isAnnotatedTag()); - assertTrue(model.getAuthorIdent().getEmailAddress().equals("marcel@holtmann.org")); - assertTrue(model.getFullMessage().equals("Update changelog and bump version number\n")); + assertTrue(model.getAuthorIdent().getEmailAddress().equals("kevinsawicki@gmail.com")); + assertEquals("Add scm and issue tracker elements to pom.xml\n", model.getFullMessage()); } } repository.close(); } + @Test public void testCommitNotes() throws Exception { Repository repository = GitBlitSuite.getJGitRepository(); RevCommit commit = JGitUtils.getCommit(repository, @@ -191,13 +230,63 @@ List<GitNote> list = JGitUtils.getNotesOnCommit(repository, commit); repository.close(); assertTrue(list.size() > 0); - assertTrue(list.get(0).notesRef.getReferencedObjectId().getName() - .equals("183474d554e6f68478a02d9d7888b67a9338cdff")); + assertEquals("183474d554e6f68478a02d9d7888b67a9338cdff", list.get(0).notesRef + .getReferencedObjectId().getName()); + } + + @Test + public void testRelinkHEAD() throws Exception { + Repository repository = GitBlitSuite.getJGitRepository(); + // confirm HEAD is master + String currentRef = JGitUtils.getHEADRef(repository); + assertEquals("refs/heads/master", currentRef); + List<String> availableHeads = JGitUtils.getAvailableHeadTargets(repository); + assertTrue(availableHeads.size() > 0); + + // set HEAD to stable-1.2 + JGitUtils.setHEADtoRef(repository, "refs/heads/stable-1.2"); + currentRef = JGitUtils.getHEADRef(repository); + assertEquals("refs/heads/stable-1.2", currentRef); + + // restore HEAD to master + JGitUtils.setHEADtoRef(repository, "refs/heads/master"); + currentRef = JGitUtils.getHEADRef(repository); + assertEquals("refs/heads/master", currentRef); + + repository.close(); } + @Test + public void testRelinkBranch() throws Exception { + Repository repository = GitBlitSuite.getJGitRepository(); + + // create/set the branch + JGitUtils.setBranchRef(repository, "refs/heads/reftest", "3b358ce514ec655d3ff67de1430994d8428cdb04"); + assertEquals(1, JGitUtils.getAllRefs(repository).get(ObjectId.fromString("3b358ce514ec655d3ff67de1430994d8428cdb04")).size()); + assertEquals(null, JGitUtils.getAllRefs(repository).get(ObjectId.fromString("755dfdb40948f5c1ec79e06bde3b0a78c352f27f"))); + + // reset the branch + JGitUtils.setBranchRef(repository, "refs/heads/reftest", "755dfdb40948f5c1ec79e06bde3b0a78c352f27f"); + assertEquals(null, JGitUtils.getAllRefs(repository).get(ObjectId.fromString("3b358ce514ec655d3ff67de1430994d8428cdb04"))); + assertEquals(1, JGitUtils.getAllRefs(repository).get(ObjectId.fromString("755dfdb40948f5c1ec79e06bde3b0a78c352f27f")).size()); + + // delete the branch + assertTrue(JGitUtils.deleteBranchRef(repository, "refs/heads/reftest")); + repository.close(); + } + + @Test + public void testCreateOrphanedBranch() throws Exception { + Repository repository = JGitUtils.createRepository(GitBlitSuite.REPOSITORIES, "orphantest"); + assertTrue(JGitUtils.createOrphanBranch(repository, + "x" + Long.toHexString(System.currentTimeMillis()).toUpperCase(), null)); + FileUtils.delete(repository.getDirectory(), FileUtils.RECURSIVE); + } + + @Test public void testStringContent() throws Exception { Repository repository = GitBlitSuite.getHelloworldRepository(); - String contentA = JGitUtils.getStringContent(repository, null, "java.java"); + String contentA = JGitUtils.getStringContent(repository, (RevTree) null, "java.java"); RevCommit commit = JGitUtils.getCommit(repository, Constants.HEAD); String contentB = JGitUtils.getStringContent(repository, commit.getTree(), "java.java"); String contentC = JGitUtils.getStringContent(repository, commit.getTree(), "missing.txt"); @@ -212,10 +301,11 @@ assertTrue("ContentA is null!", contentA != null && contentA.length() > 0); assertTrue("ContentB is null!", contentB != null && contentB.length() > 0); assertTrue(contentA.equals(contentB)); - assertTrue(contentC == null); + assertNull(contentC); assertTrue(contentA.equals(contentD)); } + @Test public void testFilesInCommit() throws Exception { Repository repository = GitBlitSuite.getHelloworldRepository(); RevCommit commit = JGitUtils.getCommit(repository, @@ -238,19 +328,21 @@ assertTrue("PathChangeModel equals itself failed!", path.equals(path)); assertFalse("PathChangeModel equals string failed!", path.equals("")); } - assertTrue(deletions.get(0).changeType.equals(ChangeType.DELETE)); - assertTrue(additions.get(0).changeType.equals(ChangeType.ADD)); + assertEquals(ChangeType.DELETE, deletions.get(0).changeType); + assertEquals(ChangeType.ADD, additions.get(0).changeType); assertTrue(latestChanges.size() > 0); } + @Test public void testFilesInPath() throws Exception { - assertTrue(JGitUtils.getFilesInPath(null, null, null).size() == 0); + assertEquals(0, JGitUtils.getFilesInPath(null, null, null).size()); Repository repository = GitBlitSuite.getHelloworldRepository(); List<PathModel> files = JGitUtils.getFilesInPath(repository, null, null); repository.close(); assertTrue(files.size() > 10); } + @Test public void testDocuments() throws Exception { Repository repository = GitBlitSuite.getTicgitRepository(); List<String> extensions = GitBlit.getStrings(Keys.web.markdownExtensions); @@ -264,73 +356,95 @@ assertTrue(allFiles.size() > markdownDocs.size()); } + @Test public void testFileModes() throws Exception { - assertTrue(JGitUtils.getPermissionsFromMode(FileMode.TREE.getBits()).equals("drwxr-xr-x")); - assertTrue(JGitUtils.getPermissionsFromMode(FileMode.REGULAR_FILE.getBits()).equals( - "-rw-r--r--")); - assertTrue(JGitUtils.getPermissionsFromMode(FileMode.EXECUTABLE_FILE.getBits()).equals( - "-rwxr-xr-x")); - assertTrue(JGitUtils.getPermissionsFromMode(FileMode.SYMLINK.getBits()).equals("symlink")); - assertTrue(JGitUtils.getPermissionsFromMode(FileMode.GITLINK.getBits()).equals("gitlink")); - assertTrue(JGitUtils.getPermissionsFromMode(FileMode.MISSING.getBits()).equals("missing")); + assertEquals("drwxr-xr-x", JGitUtils.getPermissionsFromMode(FileMode.TREE.getBits())); + assertEquals("-rw-r--r--", + JGitUtils.getPermissionsFromMode(FileMode.REGULAR_FILE.getBits())); + assertEquals("-rwxr-xr-x", + JGitUtils.getPermissionsFromMode(FileMode.EXECUTABLE_FILE.getBits())); + assertEquals("symlink", JGitUtils.getPermissionsFromMode(FileMode.SYMLINK.getBits())); + assertEquals("submodule", JGitUtils.getPermissionsFromMode(FileMode.GITLINK.getBits())); + assertEquals("missing", JGitUtils.getPermissionsFromMode(FileMode.MISSING.getBits())); } + @Test public void testRevlog() throws Exception { + assertTrue(JGitUtils.getRevLog(null, 0).size() == 0); List<RevCommit> commits = JGitUtils.getRevLog(null, 10); - assertTrue(commits.size() == 0); + assertEquals(0, commits.size()); Repository repository = GitBlitSuite.getHelloworldRepository(); // get most recent 10 commits commits = JGitUtils.getRevLog(repository, 10); - assertTrue(commits.size() == 10); + assertEquals(10, commits.size()); // test paging and offset by getting the 10th most recent commit RevCommit lastCommit = JGitUtils.getRevLog(repository, null, 9, 1).get(0); - assertTrue(commits.get(9).equals(lastCommit)); + assertEquals(lastCommit, commits.get(9)); // grab the two most recent commits to java.java commits = JGitUtils.getRevLog(repository, null, "java.java", 0, 2); - assertTrue(commits.size() == 2); + assertEquals(2, commits.size()); + + // grab the commits since 2008-07-15 + commits = JGitUtils.getRevLog(repository, null, + new SimpleDateFormat("yyyy-MM-dd").parse("2008-07-15")); + assertEquals(12, commits.size()); repository.close(); } - public void testSearchTypes() throws Exception { - assertTrue(SearchType.forName("commit").equals(SearchType.COMMIT)); - assertTrue(SearchType.forName("committer").equals(SearchType.COMMITTER)); - assertTrue(SearchType.forName("author").equals(SearchType.AUTHOR)); - assertTrue(SearchType.forName("unknown").equals(SearchType.COMMIT)); - - assertTrue(SearchType.COMMIT.toString().equals("commit")); - assertTrue(SearchType.COMMITTER.toString().equals("committer")); - assertTrue(SearchType.AUTHOR.toString().equals("author")); + @Test + public void testRevLogRange() throws Exception { + Repository repository = GitBlitSuite.getHelloworldRepository(); + List<RevCommit> commits = JGitUtils.getRevLog(repository, + "fbd14fa6d1a01d4aefa1fca725792683800fc67e", + "85a0e4087b8439c0aa6b1f4f9e08c26052ab7e87"); + repository.close(); + assertEquals(14, commits.size()); } + @Test + public void testSearchTypes() throws Exception { + assertEquals(SearchType.COMMIT, SearchType.forName("commit")); + assertEquals(SearchType.COMMITTER, SearchType.forName("committer")); + assertEquals(SearchType.AUTHOR, SearchType.forName("author")); + assertEquals(SearchType.COMMIT, SearchType.forName("unknown")); + + assertEquals("commit", SearchType.COMMIT.toString()); + assertEquals("committer", SearchType.COMMITTER.toString()); + assertEquals("author", SearchType.AUTHOR.toString()); + } + + @Test public void testSearchRevlogs() throws Exception { + assertEquals(0, JGitUtils.searchRevlogs(null, null, "java", SearchType.COMMIT, 0, 0).size()); List<RevCommit> results = JGitUtils.searchRevlogs(null, null, "java", SearchType.COMMIT, 0, 3); - assertTrue(results.size() == 0); + assertEquals(0, results.size()); // test commit message search Repository repository = GitBlitSuite.getHelloworldRepository(); results = JGitUtils.searchRevlogs(repository, null, "java", SearchType.COMMIT, 0, 3); - assertTrue(results.size() == 3); + assertEquals(3, results.size()); // test author search results = JGitUtils.searchRevlogs(repository, null, "timothy", SearchType.AUTHOR, 0, -1); - assertTrue(results.size() == 1); + assertEquals(1, results.size()); // test committer search results = JGitUtils.searchRevlogs(repository, null, "mike", SearchType.COMMITTER, 0, 10); - assertTrue(results.size() == 10); + assertEquals(10, results.size()); // test paging and offset RevCommit commit = JGitUtils.searchRevlogs(repository, null, "mike", SearchType.COMMITTER, 9, 1).get(0); - assertTrue(results.get(9).equals(commit)); + assertEquals(results.get(9), commit); repository.close(); } + @Test public void testZip() throws Exception { assertFalse(JGitUtils.zip(null, null, null, null)); Repository repository = GitBlitSuite.getHelloworldRepository(); @@ -353,4 +467,5 @@ assertTrue(zipFileB.length() > 0); zipFileB.delete(); } + } \ No newline at end of file -- Gitblit v1.9.1