Skip to content

Commit 4efdcad

Browse files
committed
Add tests for sessions
1 parent fecfcdf commit 4efdcad

File tree

2 files changed

+26
-3
lines changed

2 files changed

+26
-3
lines changed

policy/diamond/policy/session/session.rego

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,9 @@ write_to_beamline_visit if {
5858

5959
user_sessions contains user_session if {
6060
some session in data.diamond.data.sessions
61+
print(session)
6162
access_session(token.claims.fedid, session.proposal_number, session.visit_number)
63+
print(token.claims.fedid, session)
6264
user_session := sprintf(
6365
"{\"proposal\": %d, \"visit\": %d, \"beamline\": \"%s\"}",
6466
[session.proposal_number, session.visit_number, session.beamline],

policy/diamond/policy/session/session_test.rego

Lines changed: 24 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,11 @@ diamond_data := {
2525
"proposals": [2],
2626
"sessions": [13],
2727
},
28+
"edna": {
29+
"permissions": [],
30+
"proposals": [2],
31+
"sessions": [13, 14],
32+
},
2833
"oscar": {
2934
"permissions": [],
3035
"proposals": [],
@@ -47,15 +52,23 @@ diamond_data := {
4752
"proposal_number": 2,
4853
"visit_number": 1,
4954
},
55+
"14": {
56+
"beamline": "b07",
57+
"proposal_number": 2,
58+
"visit_number": 2,
59+
},
5060
},
5161
"proposals": {
5262
"1": {"sessions": {
5363
"1": 11,
5464
"2": 12,
5565
}},
56-
"2": {"sessions": {"1": 13}},
66+
"2": {"sessions": {
67+
"1": 13,
68+
"2": 14,
69+
}},
5770
},
58-
"beamlines": {"i03": {"sessions": [11]}, "b07": {"sessions": [12, 13]}},
71+
"beamlines": {"i03": {"sessions": [11]}, "b07": {"sessions": [12, 13, 14]}},
5972
"admin": {"b07_admin": ["b07"]},
6073
}
6174

@@ -207,16 +220,24 @@ test_user_session_tags if {
207220
"{\"proposal\": 1, \"visit\": 2, \"beamline\": \"b07\"}",
208221
"{\"proposal\": 1, \"visit\": 1, \"beamline\": \"i03\"}",
209222
"{\"proposal\": 2, \"visit\": 1, \"beamline\": \"b07\"}",
223+
"{\"proposal\": 2, \"visit\": 2, \"beamline\": \"b07\"}",
210224
} with data.diamond.data as diamond_data
211225
with data.diamond.policy.token.claims as {"fedid": "bob"}
212226
session.user_sessions == {
213227
"{\"proposal\": 1, \"visit\": 2, \"beamline\": \"b07\"}",
214228
"{\"proposal\": 1, \"visit\": 1, \"beamline\": \"i03\"}",
215229
"{\"proposal\": 2, \"visit\": 1, \"beamline\": \"b07\"}",
230+
"{\"proposal\": 2, \"visit\": 2, \"beamline\": \"b07\"}",
216231
} with data.diamond.data as diamond_data
217232
with data.diamond.policy.token.claims as {"fedid": "carol"}
218233
session.user_sessions == {
219-
"{\"proposal\": 2, \"visit\": 1, \"beamline\": \"b07\"}"
234+
"{\"proposal\": 2, \"visit\": 1, \"beamline\": \"b07\"}",
235+
"{\"proposal\": 2, \"visit\": 2, \"beamline\": \"b07\"}",
220236
} with data.diamond.data as diamond_data
221237
with data.diamond.policy.token.claims as {"fedid": "desmond"}
238+
session.user_sessions == {
239+
"{\"proposal\": 2, \"visit\": 1, \"beamline\": \"b07\"}",
240+
"{\"proposal\": 2, \"visit\": 2, \"beamline\": \"b07\"}",
241+
} with data.diamond.data as diamond_data
242+
with data.diamond.policy.token.claims as {"fedid": "edna"}
222243
}

0 commit comments

Comments
 (0)