Skip to content

Conversation

@Kariamos
Copy link
Contributor

@Kariamos Kariamos commented Dec 4, 2025

No description provided.

@Kariamos Kariamos added core Team Core activity feature labels Dec 4, 2025
@coveralls
Copy link
Collaborator

coveralls commented Dec 4, 2025

Coverage Status

coverage: 78.816% (+0.006%) from 78.81%
when pulling f4afa1e on UN-2159-put-dossier-id-agreements
into 6dd72bd on develop.

@d-beezee d-beezee force-pushed the UN-2159-put-dossier-id-agreements branch from f4afa1e to 27f8dfd Compare December 4, 2025 14:26
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

Tests difference:

New Tests

< Invalid body - Should return 400 if agreementId is missing
< Invalid body - Should return 400 if agreementId is not a positive integer
< Invalid body - Should return 400 if body is missing
< Invalid body - Should return 400 if both tokens and agreementId are not valid
< Invalid body - Should return 400 if tokens is missing
< Invalid body - Should return 400 if tokens is not a positive number
< Link between campaign and agreement already exists - Should update the link between campaign and agreement
< Link between campaign and agreement does not exists - Should create the link between campaign and agreement
< Route PUT /dossiers/:id/agreements - Should return 200 if has access to the campaign
< Route PUT /dossiers/:id/agreements - Should return 200 if has all campaign olps
< Route PUT /dossiers/:id/agreements - Should return 200 if logged in as admin
< Route PUT /dossiers/:id/agreements - Should return 400 if campaign does not exists
< Route PUT /dossiers/:id/agreements - Should return 403 if logged in as tester
< Route PUT /dossiers/:id/agreements - Should return 403 if not logged in
< Valid body - Should update the tokens usage and return 200

@d-beezee d-beezee merged commit 7a08b97 into develop Dec 4, 2025
4 checks passed
@d-beezee d-beezee deleted the UN-2159-put-dossier-id-agreements branch December 4, 2025 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Team Core activity feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants