From b57d3e7e34d98042533369790d9f51462b04f9c0 Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Wed, 11 Dec 2013 11:01:37 -0500 Subject: [PATCH] Fix NPE in getting a user's fork if repository caching is disabled --- src/test/java/com/gitblit/tests/UserServiceTest.java | 7 +------ 1 files changed, 1 insertions(+), 6 deletions(-) diff --git a/src/test/java/com/gitblit/tests/UserServiceTest.java b/src/test/java/com/gitblit/tests/UserServiceTest.java index 613e46d..cdb0a33 100644 --- a/src/test/java/com/gitblit/tests/UserServiceTest.java +++ b/src/test/java/com/gitblit/tests/UserServiceTest.java @@ -85,14 +85,9 @@ assertTrue(newUser.hasRepositoryPermission("repo2")); assertTrue(newUser.hasRepositoryPermission("sub/repo3")); - // confirm authentication of test user - UserModel testUser = service.authenticate("test", "testPassword".toCharArray()); - assertEquals("test", testUser.username); - assertEquals("testPassword", testUser.password); - // delete a repository role and confirm role removal from test user service.deleteRepositoryRole("repo2"); - testUser = service.getUserModel("test"); + UserModel testUser = service.getUserModel("test"); assertEquals(2, testUser.permissions.size()); // delete garbage user and confirm user count -- Gitblit v1.9.1