Skip to content

Commit ce3f8da

Browse files
committed
Add test to check reviews are returned with the submission
1 parent 403fe9e commit ce3f8da

File tree

1 file changed

+21
-4
lines changed

1 file changed

+21
-4
lines changed

src/tests/authorization/test_authorization.py

Lines changed: 21 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -138,14 +138,30 @@ def test_a_submitted_asset_is_pending_for_review(client, publication):
138138
assert queue.status_code == HTTPStatus.OK, queue.json()
139139
assert len(queue.json()) == 1, "A submitted asset should be pending until a review is done."
140140

141-
142141
def test_get_submission_by_id(client, publication):
143142
register_asset(publication, owner=ALICE, status=EntryStatus.PUBLISHED)
144143

145144
with logged_in_user(REVIEWER):
146-
queue = client.get("/submissions/v1/1", headers={"Authorization": "Fake token"})
147-
assert queue.status_code == HTTPStatus.OK, queue.json()
148-
assert queue.json()["identifier"] == 1
145+
submission = client.get("/submissions/v1/1", headers={"Authorization": "Fake token"})
146+
assert submission.status_code == HTTPStatus.OK, submission.json()
147+
148+
submission_dict = submission.json()
149+
submission_date = submission_dict.pop("request_date")
150+
review_date = submission_dict["reviews"][0].pop("decision_date")
151+
assert submission_date < review_date
152+
assert submission_dict == {
153+
"identifier": 1,
154+
"aiod_entry_identifier": 1,
155+
"reviews": [
156+
{
157+
"identifier": 1,
158+
"decision": "accepted",
159+
"comment": "foo",
160+
"submission_identifier": 1,
161+
}
162+
],
163+
}
164+
149165

150166

151167
def test_unknown_submission_raises_404(client):
@@ -271,6 +287,7 @@ def register_asset(asset: AIoDConcept, /, *, owner: KeycloakUser, status: EntryS
271287
review = Review(
272288
decision=Decision.ACCEPTED,
273289
reviewer_identifier=REVIEWER._subject_identifier,
290+
comment="foo",
274291
)
275292
review.submission = submission
276293
session.add(review)

0 commit comments

Comments
 (0)