From fbe265fa787e4be9cb63c6bae2ef30b9400d9afc Mon Sep 17 00:00:00 2001
From: Simon Harrer <simon.harrer@gmail.com>
Date: Thu, 18 Jul 2013 10:11:04 -0400
Subject: [PATCH] Fixes findbugs warning - dereferencing null in exception case
---
src/test/java/com/gitblit/tests/Issue0259Test.java | 11 ++++++++++-
1 files changed, 10 insertions(+), 1 deletions(-)
diff --git a/src/test/java/com/gitblit/tests/Issue0259Test.java b/src/test/java/com/gitblit/tests/Issue0259Test.java
index 174f7cc..7267971 100644
--- a/src/test/java/com/gitblit/tests/Issue0259Test.java
+++ b/src/test/java/com/gitblit/tests/Issue0259Test.java
@@ -132,6 +132,11 @@
a.teams.add(t2);
}
+ // simulate a repository rename
+ a.setRepositoryPermission("projects/renamed.git", null);
+ t1.setRepositoryPermission("projects/renamed.git", null);
+ t2.setRepositoryPermission("projects/renamed.git", null);
+
assertEquals(AccessPermission.CLONE, a.getRepositoryPermission(test).permission);
assertEquals(AccessPermission.DELETE, a.getRepositoryPermission(projects_test).permission);
@@ -185,6 +190,10 @@
}
UserModel a = new UserModel("a");
a.teams.add(t1);
+
+ // simulate a repository rename
+ a.setRepositoryPermission("projects/renamed.git", null);
+ t1.setRepositoryPermission("projects/renamed.git", null);
assertEquals(AccessPermission.CLONE, a.getRepositoryPermission(test).permission);
assertTrue(a.canClone(test));
@@ -199,6 +208,6 @@
// R permission is found first
assertEquals(AccessPermission.CLONE, a.getRepositoryPermission(projects_test).permission);
assertFalse(a.canDeleteRef(projects_test));
- }
+ }
}
}
--
Gitblit v1.9.1