Skip to content

Commit 1cc0084

Browse files
committed
update cache
1 parent 8b30677 commit 1cc0084

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

test/functional/NewUserTest.php

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ private function approveGroup($uid)
9494
// does not remove user from PI groups
9595
private function ensureUserDoesNotExist()
9696
{
97-
global $USER, $SQL, $LDAP;
97+
global $USER, $SQL, $LDAP, $REDIS;
9898
$SQL->deleteRequestsByUser($USER->uid);
9999
if ($USER->exists()) {
100100
$org = $USER->getOrgGroup();
@@ -116,6 +116,7 @@ private function ensureUserDoesNotExist()
116116
$all_users_group->write();
117117
assert(!in_array($USER->uid, $all_users_group->getAttribute("memberuid")));
118118
}
119+
$REDIS->removeCacheArray("sorted_users", "", $USER->uid);
119120
}
120121

121122
private function ensureOrgGroupDoesNotExist()
@@ -126,24 +127,28 @@ private function ensureOrgGroupDoesNotExist()
126127
$org_group->delete();
127128
assert(!$org_group->exists());
128129
}
130+
$REDIS->removeCacheArray("sorted_orgs", "", $SSO["org"]);
129131
}
130132

131133
private function ensureUserNotInPIGroup(UnityGroup $pi_group)
132134
{
133-
global $USER;
135+
global $USER, $REDIS;
134136
if ($pi_group->userExists($USER)) {
135137
$pi_group->removeUser($USER);
136138
assert(!$pi_group->userExists($USER));
137139
}
140+
$REDIS->removeCacheArray($pi_group->gid, "members", $USER->uid);
138141
}
139142

140143
private function ensurePIGroupDoesNotExist()
141144
{
142-
global $USER, $LDAP;
145+
global $USER, $LDAP, $REDIS;
146+
$gid = $USER->getPIGroup()->gid;
143147
if ($USER->getPIGroup()->exists()) {
144-
$LDAP->getPIGroupEntry($USER->getPIGroup()->gid)->delete();
148+
$LDAP->getPIGroupEntry($gid)->delete();
145149
assert(!$USER->getPIGroup()->exists());
146150
}
151+
$REDIS->removeCacheArray("sorted_groups", "", $gid);
147152
}
148153

149154
public function testCreateUserByJoinGoupByPI()

0 commit comments

Comments
 (0)