From b2d7f418bf289e5fa6149ac1b5744cfb03b60dea Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Mon, 23 Sep 2013 10:55:19 -0400
Subject: [PATCH] Eliminate redundant read() and use accessor instead of direct cache access
---
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