From db91a34ccb5135f733c7431fdba49669131e97da Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Sat, 19 Nov 2011 07:33:59 -0500 Subject: [PATCH] Reorganizing the recent activity feature --- tests/com/gitblit/tests/JGitUtilsTest.java | 73 +++++++++++++++++++++++------------- 1 files changed, 47 insertions(+), 26 deletions(-) diff --git a/tests/com/gitblit/tests/JGitUtilsTest.java b/tests/com/gitblit/tests/JGitUtilsTest.java index 277166c..7cd79ad 100644 --- a/tests/com/gitblit/tests/JGitUtilsTest.java +++ b/tests/com/gitblit/tests/JGitUtilsTest.java @@ -17,6 +17,7 @@ import java.io.File; import java.io.FileOutputStream; +import java.text.SimpleDateFormat; import java.util.Arrays; import java.util.Date; import java.util.List; @@ -33,6 +34,7 @@ import org.eclipse.jgit.lib.RepositoryCache.FileKey; import org.eclipse.jgit.revwalk.RevCommit; import org.eclipse.jgit.util.FS; +import org.eclipse.jgit.util.FileUtils; import com.gitblit.GitBlit; import com.gitblit.Keys; @@ -41,7 +43,6 @@ 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 { @@ -85,27 +86,28 @@ } public void testLastCommit() throws Exception { - assertTrue(JGitUtils.getLastChange(null).equals(new Date(0))); + assertTrue(JGitUtils.getLastChange(null, null).equals(new Date(0))); Repository repository = GitBlitSuite.getHelloworldRepository(); assertTrue(JGitUtils.getCommit(repository, null) != null); - Date date = JGitUtils.getLastChange(repository); + Date date = JGitUtils.getLastChange(repository, null); repository.close(); assertTrue("Could not get last repository change date!", date != null); } public void testCreateRepository() throws Exception { String[] repositories = { "NewTestRepository.git", "NewTestRepository" }; - for (String repositoryName : repositories) { + for (String repositoryName : repositories) { Repository repository = JGitUtils.createRepository(GitBlitSuite.REPOSITORIES, repositoryName); - File folder = FileKey.resolve(new File(GitBlitSuite.REPOSITORIES, repositoryName), FS.DETECTED); + File folder = FileKey.resolve(new File(GitBlitSuite.REPOSITORIES, repositoryName), + FS.DETECTED); assertTrue(repository != null); 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.getLastChange(repository, null).getTime() == folder.lastModified()); assertTrue(JGitUtils.getCommit(repository, null) == null); repository.close(); assertTrue(GitBlit.self().deleteRepository(repositoryName)); @@ -121,10 +123,12 @@ 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.toString().equals("refs/tags/spearce-gpg-pub")); + assertTrue(ref.getObjectId().getName() + .equals("8bbde7aacf771a9afb6992434f1ae413e010c6d8")); assertTrue(ref.getAuthorIdent().getEmailAddress().equals("spearce@spearce.org")); assertTrue(ref.getShortMessage().startsWith("GPG key")); - assertTrue(ref.getFullMessage().startsWith("GPG key")); + assertTrue(ref.getFullMessage().startsWith("GPG key")); assertTrue(ref.getReferencedObjectType() == Constants.OBJ_BLOB); } else if (ref.displayName.equals("refs/tags/v0.12.1")) { assertTrue(ref.isAnnotatedTag()); @@ -135,6 +139,7 @@ public void testBranches() throws Exception { 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)); @@ -157,6 +162,7 @@ public void testTags() throws Exception { 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("d28091fb2977077471138fe97da1440e0e8ae0da")) { assertTrue("Not an annotated tag!", model.isAnnotatedTag()); @@ -168,13 +174,14 @@ + model.getName().hashCode()); } repository.close(); - + repository = GitBlitSuite.getBluezGnomeRepository(); for (RefModel model : JGitUtils.getTags(repository, true, -1)) { if (model.getObjectId().getName().equals("728643ec0c438c77e182898c2f2967dbfdc231c8")) { assertFalse(model.isAnnotatedTag()); assertTrue(model.getAuthorIdent().getEmailAddress().equals("marcel@holtmann.org")); - assertTrue(model.getFullMessage().equals("Update changelog and bump version number\n")); + assertTrue(model.getFullMessage().equals( + "Update changelog and bump version number\n")); } } repository.close(); @@ -189,6 +196,13 @@ assertTrue(list.size() > 0); assertTrue(list.get(0).notesRef.getReferencedObjectId().getName() .equals("183474d554e6f68478a02d9d7888b67a9338cdff")); + } + + public void testCreateOrphanedBranch() throws Exception { + Repository repository = JGitUtils.createRepository(GitBlitSuite.REPOSITORIES, "orphantest"); + assertTrue(JGitUtils.createOrphanBranch(repository, + "x" + Long.toHexString(System.currentTimeMillis()).toUpperCase())); + FileUtils.delete(repository.getDirectory(), FileUtils.RECURSIVE); } public void testStringContent() throws Exception { @@ -272,55 +286,61 @@ } 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(com.gitblit.Constants.SearchType.forName("commit").equals(com.gitblit.Constants.SearchType.COMMIT)); + assertTrue(com.gitblit.Constants.SearchType.forName("committer").equals(com.gitblit.Constants.SearchType.COMMITTER)); + assertTrue(com.gitblit.Constants.SearchType.forName("author").equals(com.gitblit.Constants.SearchType.AUTHOR)); + assertTrue(com.gitblit.Constants.SearchType.forName("unknown").equals(com.gitblit.Constants.SearchType.COMMIT)); - assertTrue(SearchType.COMMIT.toString().equals("commit")); - assertTrue(SearchType.COMMITTER.toString().equals("committer")); - assertTrue(SearchType.AUTHOR.toString().equals("author")); + assertTrue(com.gitblit.Constants.SearchType.COMMIT.toString().equals("commit")); + assertTrue(com.gitblit.Constants.SearchType.COMMITTER.toString().equals("committer")); + assertTrue(com.gitblit.Constants.SearchType.AUTHOR.toString().equals("author")); } public void testSearchRevlogs() throws Exception { - List<RevCommit> results = JGitUtils.searchRevlogs(null, null, "java", SearchType.COMMIT, 0, + assertTrue(JGitUtils.searchRevlogs(null, null, "java", com.gitblit.Constants.SearchType.COMMIT, 0, 0).size() == 0); + List<RevCommit> results = JGitUtils.searchRevlogs(null, null, "java", com.gitblit.Constants.SearchType.COMMIT, 0, 3); assertTrue(results.size() == 0); // test commit message search Repository repository = GitBlitSuite.getHelloworldRepository(); - results = JGitUtils.searchRevlogs(repository, null, "java", SearchType.COMMIT, 0, 3); + results = JGitUtils.searchRevlogs(repository, null, "java", com.gitblit.Constants.SearchType.COMMIT, 0, 3); assertTrue(results.size() == 3); // test author search - results = JGitUtils.searchRevlogs(repository, null, "timothy", SearchType.AUTHOR, 0, -1); + results = JGitUtils.searchRevlogs(repository, null, "timothy", com.gitblit.Constants.SearchType.AUTHOR, 0, -1); assertTrue(results.size() == 1); // test committer search - results = JGitUtils.searchRevlogs(repository, null, "mike", SearchType.COMMITTER, 0, 10); + results = JGitUtils.searchRevlogs(repository, null, "mike", com.gitblit.Constants.SearchType.COMMITTER, 0, 10); assertTrue(results.size() == 10); // test paging and offset - RevCommit commit = JGitUtils.searchRevlogs(repository, null, "mike", SearchType.COMMITTER, + RevCommit commit = JGitUtils.searchRevlogs(repository, null, "mike", com.gitblit.Constants.SearchType.COMMITTER, 9, 1).get(0); assertTrue(results.get(9).equals(commit)); @@ -349,4 +369,5 @@ assertTrue(zipFileB.length() > 0); zipFileB.delete(); } + } \ No newline at end of file -- Gitblit v1.9.1