release-20251211

[Kariamos]

This pull request introduces several new features, updates, and improvements across the codebase. The most significant changes include the addition of new API routes for customer agreements and survey imports, enhancements to user filtering logic for campaign notifications, and updates to dependencies. Below are the most important changes grouped by theme:

New Features:

• Added a new GET API route src/routes/customers/customer/agreements/_get/index.ts to retrieve customer agreements, including logic for permission checks and calculation of remaining tokens.
• Added a new POST API route src/routes/campaigns/campaignId/tasks/usecase/survey/jotform/_post/index.ts to import Jotform surveys into usecase surveys, with corresponding tests. [1] [2]
• Added comprehensive tests for customer agreements API in src/routes/customers/customer/agreements/_get/index.spec.ts.

User Filtering and Notification Logic:

• Enhanced user selection logic in src/routes/dossiers/_post/index.ts to exclude users with pending customer account invitations when notifying campaign watchers.
• Updated dossier notification tests in src/routes/dossiers/_post/notifyEveryone.spec.ts to cover scenarios with pending invitations and multiple users, ensuring correct watcher notifications. [1] [2] [3]

Dependency Updates:

• Updated @appquality/tryber-database dependency in package.json to version ^0.46.4.
• Added yaml-sort as a new development dependency in package.json.

Bug Review Versioning:

• Changed AI bug review retrieval logic in src/routes/campaigns/campaignId/bugs/bugId/aiReview/_get/index.ts to use version "v2" instead of "v1", and updated related tests. [1] [2] [3]

[github-actions]

Tests difference:

New Tests

< Campaign Plan - Should return whether the campaign has a plan or not
< GET /customers/customerId/agreements - Should answer 200 if logged as user with full access on campaigns
< GET /customers/customerId/agreements - Should answer 200 if logged as user with olps to some owned campaigns
< GET /customers/customerId/agreements - Should answer 403 if customer does not exists
< GET /customers/customerId/agreements - Should answer 403 if logged as user with olps to only other campaigns
< GET /customers/customerId/agreements - Should answer 403 if logged in without permissions
< GET /customers/customerId/agreements - Should answer 403 if not logged in
< GET /customers/customerId/agreements - Should answer with a list of all agreements of the customer if has full access
< GET /dossiers/campaignId/agreements - Should answer 200 if logged as user with full access on the campaign
< GET /dossiers/campaignId/agreements - Should answer 400 if campaign does not exists
< GET /dossiers/campaignId/agreements - Should answer 403 if logged as user without permissions on the campaign
< GET /dossiers/campaignId/agreements - Should answer 403 if logged in without permissions
< GET /dossiers/campaignId/agreements - Should answer 403 if not logged in
< GET /dossiers/campaignId/agreements - Should answer with the campaigns agreement
< GET /dossiers/campaignId/costs - Should answer 200 if logged as user with full access on the campaign
< GET /dossiers/campaignId/costs - Should answer 400 if campaign does not exists
< GET /dossiers/campaignId/costs - Should answer 403 if logged as user without permissions on the campaign
< GET /dossiers/campaignId/costs - Should answer 403 if logged in without permissions
< GET /dossiers/campaignId/costs - Should answer 403 if not logged in
< GET /dossiers/campaignId/costs - Should answer with the campaigns costs
< GET /dossiers/campaignId/costs - Should filterBy the costs by multiple work_type_id
< GET /dossiers/campaignId/costs - Should filterBy the costs by work_type_id
< GET /dossiers/campaignId/costs - Should ignore the filter if it the filter is invalid
< GET /dossiers/campaignId/costs - Should ignore the filter if it the type is invalid
< 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
< POST /campaigns/{campaignId}/tasks/{usecase}/survey/jotform - should answer 400 if user is logged with permission but campaign does not exists
< POST /campaigns/{campaignId}/tasks/{usecase}/survey/jotform - should answer 403 if user is logged in without manage_preselection_forms 
< POST /campaigns/{campaignId}/tasks/{usecase}/survey/jotform - should answer 403 if user is not logged in 
< POST /campaigns/{campaignId}/tasks/{usecase}/survey/jotform - should return 200 if user is logged in with manage_preselection_forms permission
< Route GET /dossiers/:campaignId/humanResources - Should answer 200 if admin
< Route GET /dossiers/:campaignId/humanResources - Should answer 200 if has access to campaign
< Route GET /dossiers/:campaignId/humanResources - Should answer 400 if tester
< Route GET /dossiers/:campaignId/humanResources - Should answer 403 if not logged in
< Route GET /dossiers/rates - Should answer 200 if user is admin
< Route GET /dossiers/rates - Should answer 401 if not admin
< Route GET /dossiers/rates - Should answer 403 if not logged in
< Route GET /dossiers/rates - Should return 200 if user has full campaign olps
< Route GET /dossiers/rates - Should return 200 if user has only some campaign olps
< Route GET /dossiers/rates - Should return array of items
< Route GET /dossiers/rates - Should return items with id, name and rate
< 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
< Route PUT/dossiers/:campaignId/humanResources - Should answer 200 if admin
< Route PUT/dossiers/:campaignId/humanResources - Should answer 200 if has access to campaign
< Route PUT/dossiers/:campaignId/humanResources - Should answer 400 if invalid body
< Route PUT/dossiers/:campaignId/humanResources - Should answer 400 if profile does not exist
< Route PUT/dossiers/:campaignId/humanResources - Should answer 400 if tester
< Route PUT/dossiers/:campaignId/humanResources - Should answer 400 if work rate does not exist
< Route PUT/dossiers/:campaignId/humanResources - Should answer 403 if no access to campaign
< Route PUT/dossiers/:campaignId/humanResources - Should answer 403 if not logged in
< Valid body - Should update the tokens usage and return 200
< With basic data - Should remove the old records and update the human resources
< With basic data - Should return 200 if body is empty and simply delete all human resources
< With basic data - Should return human resources for campaign 1