From 2c616775b21ab4cd76ff170dfd5e818c9c3b79fb Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Tue, 02 Jul 2013 15:08:28 -0400
Subject: [PATCH] Fixed git servlet unit test
---
src/test/java/com/gitblit/tests/GitServletTest.java | 42 +++++++++++++++++++++++++-----------------
1 files changed, 25 insertions(+), 17 deletions(-)
diff --git a/src/test/java/com/gitblit/tests/GitServletTest.java b/src/test/java/com/gitblit/tests/GitServletTest.java
index 7607fbf..8513c83 100644
--- a/src/test/java/com/gitblit/tests/GitServletTest.java
+++ b/src/test/java/com/gitblit/tests/GitServletTest.java
@@ -60,10 +60,24 @@
String password = GitBlitSuite.password;
private static final AtomicBoolean started = new AtomicBoolean(false);
+
+ private static UserModel getUser() {
+ UserModel user = new UserModel("james");
+ user.password = "james";
+ return user;
+ }
+
+ private static void delete(UserModel user) {
+ if (GitBlit.self().getUserModel(user.username) != null) {
+ GitBlit.self().deleteUser(user.username);
+ }
+ }
@BeforeClass
public static void startGitblit() throws Exception {
started.set(GitBlitSuite.startGitblit());
+
+ delete(getUser());
}
@AfterClass
@@ -72,6 +86,8 @@
GitBlitSuite.stopGitblit();
deleteWorkingFolders();
}
+
+ delete(getUser());
}
public static void deleteWorkingFolders() throws Exception {
@@ -194,7 +210,8 @@
model.accessRestriction = AccessRestrictionType.NONE;
model.authorizationControl = AuthorizationControl.NAMED;
GitBlit.self().updateRepositoryModel(model.name, model, false);
- GitBlit.self().deleteUser(user.username);
+
+ delete(user);
}
@Test
@@ -347,8 +364,7 @@
@Test
public void testCommitterVerification() throws Exception {
- UserModel user = new UserModel("james");
- user.password = "james";
+ UserModel user = getUser();
// account only uses account name to verify
testCommitterVerification(user, user.username, null, true);
@@ -375,9 +391,7 @@
private void testCommitterVerification(UserModel user, String displayName, String emailAddress, boolean expectedSuccess) throws Exception {
- if (GitBlit.self().getUserModel(user.username) != null) {
- GitBlit.self().deleteUser(user.username);
- }
+ delete(user);
CredentialsProvider cp = new UsernamePasswordCredentialsProvider(user.username, user.password);
@@ -483,12 +497,8 @@
private void testRefChange(AccessPermission permission, Status expectedCreate, Status expectedDelete, Status expectedRewind) throws Exception {
- UserModel user = new UserModel("james");
- user.password = "james";
-
- if (GitBlit.self().getUserModel(user.username) != null) {
- GitBlit.self().deleteUser(user.username);
- }
+ UserModel user = getUser();
+ delete(user);
CredentialsProvider cp = new UsernamePasswordCredentialsProvider(user.username, user.password);
@@ -672,7 +682,7 @@
// close serving repository
GitBlitSuite.close(refChecks);
- GitBlit.self().deleteUser(user.username);
+ delete(user);
}
@Test
@@ -687,9 +697,7 @@
UserModel user = new UserModel("sampleuser");
user.password = user.username;
- if (GitBlit.self().getUserModel(user.username) != null) {
- GitBlit.self().deleteUser(user.username);
- }
+ delete(user);
user.canCreate = canCreate;
user.canAdmin = canAdmin;
@@ -780,7 +788,7 @@
}
GitBlitSuite.close(git);
- GitBlit.self().deleteUser(user.username);
+ delete(user);
}
@Test
--
Gitblit v1.9.1