Skip to content

Commit 67a04cd

Browse files
committed
Return session unformatted
1 parent ce394ee commit 67a04cd

File tree

2 files changed

+15
-18
lines changed

2 files changed

+15
-18
lines changed

policy/diamond/policy/session/session.rego

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,5 @@ write_to_beamline_visit if {
5959
user_sessions contains user_session if {
6060
some session in data.diamond.data.sessions
6161
access_session(token.claims.fedid, session.proposal_number, session.visit_number)
62-
user_session := sprintf(
63-
`{"proposal": %d, "visit": %d, "beamline": "%s"}`,
64-
[session.proposal_number, session.visit_number, session.beamline],
65-
)
62+
user_session := sprintf("%s", [session])
6663
}

policy/diamond/policy/session/session_test.rego

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -212,32 +212,32 @@ test_user_session_tags if {
212212
session.user_sessions == set() with data.diamond.data as diamond_data
213213
with data.diamond.policy.token.claims as {"fedid": "oscar"}
214214
session.user_sessions == {
215-
`{"proposal": 1, "visit": 2, "beamline": "b07"}`,
216-
`{"proposal": 1, "visit": 1, "beamline": "i03"}`,
215+
"{\"beamline\": \"b07\", \"proposal_number\": 1, \"visit_number\": 2}",
216+
"{\"beamline\": \"i03\", \"proposal_number\": 1, \"visit_number\": 1}",
217217
} with data.diamond.data as diamond_data
218218
with data.diamond.policy.token.claims as {"fedid": "alice"}
219219
session.user_sessions == {
220-
`{"proposal": 1, "visit": 2, "beamline": "b07"}`,
221-
`{"proposal": 1, "visit": 1, "beamline": "i03"}`,
222-
`{"proposal": 2, "visit": 1, "beamline": "b07"}`,
223-
`{"proposal": 2, "visit": 2, "beamline": "b07"}`,
220+
"{\"beamline\": \"b07\", \"proposal_number\": 1, \"visit_number\": 2}",
221+
"{\"beamline\": \"i03\", \"proposal_number\": 1, \"visit_number\": 1}",
222+
"{\"beamline\": \"b07\", \"proposal_number\": 2, \"visit_number\": 1}",
223+
"{\"beamline\": \"b07\", \"proposal_number\": 2, \"visit_number\": 2}",
224224
} with data.diamond.data as diamond_data
225225
with data.diamond.policy.token.claims as {"fedid": "bob"}
226226
session.user_sessions == {
227-
`{"proposal": 1, "visit": 2, "beamline": "b07"}`,
228-
`{"proposal": 1, "visit": 1, "beamline": "i03"}`,
229-
`{"proposal": 2, "visit": 1, "beamline": "b07"}`,
230-
`{"proposal": 2, "visit": 2, "beamline": "b07"}`,
227+
"{\"beamline\": \"b07\", \"proposal_number\": 1, \"visit_number\": 2}",
228+
"{\"beamline\": \"i03\", \"proposal_number\": 1, \"visit_number\": 1}",
229+
"{\"beamline\": \"b07\", \"proposal_number\": 2, \"visit_number\": 1}",
230+
"{\"beamline\": \"b07\", \"proposal_number\": 2, \"visit_number\": 2}",
231231
} with data.diamond.data as diamond_data
232232
with data.diamond.policy.token.claims as {"fedid": "carol"}
233233
session.user_sessions == {
234-
`{"proposal": 2, "visit": 1, "beamline": "b07"}`,
235-
`{"proposal": 2, "visit": 2, "beamline": "b07"}`,
234+
"{\"beamline\": \"b07\", \"proposal_number\": 2, \"visit_number\": 1}",
235+
"{\"beamline\": \"b07\", \"proposal_number\": 2, \"visit_number\": 2}",
236236
} with data.diamond.data as diamond_data
237237
with data.diamond.policy.token.claims as {"fedid": "desmond"}
238238
session.user_sessions == {
239-
`{"proposal": 2, "visit": 1, "beamline": "b07"}`,
240-
`{"proposal": 2, "visit": 2, "beamline": "b07"}`,
239+
"{\"beamline\": \"b07\", \"proposal_number\": 2, \"visit_number\": 1}",
240+
"{\"beamline\": \"b07\", \"proposal_number\": 2, \"visit_number\": 2}",
241241
} with data.diamond.data as diamond_data
242242
with data.diamond.policy.token.claims as {"fedid": "edna"}
243243
}

0 commit comments

Comments
 (0)