(PC-31625) Add PriceCategory.idAtProvider in public API

[tcoudray-pass]

But de la pull request

Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-31625

In this PR:

• update of offer core api create_price_category and edit_price_category methods
• update of POST event, POST price categories and PATCH price category public API endpoints
• addition of GET price categories endpoint

Vérifications

• J'ai écrit les tests nécessaires
• J'ai mis à jour le fichier des plans de tests du portail pro si nécessaire
• J'ai mis à jour la liste des routes et des titres de pages du portail pro si j'en ai rajouté/modifié ou supprimé une.
• J'ai relu attentivement les migrations, en particulier pour éviter les locks, et je préviens les équipes Shérif et Data
• J'ai ajouté des screenshots pour d'éventuels changements graphiques

[github-actions]

mypy cop report: 436 (master) ↗ 437 (your branch)

[jeremieb-pass]

A voir aussi si on ne veut pas rajouter des cas de tests pour l'api où on s'assure qu'on ne peut pas aller modifier une catégorie de prix à laquelle on n'a pas accès avec la clé d'API.

[jeremieb-pass]

Simple suggestion, c'est du détail (l'avantage est d'avoir en quelques lignes tous les ids dupliqués)
⚠️ écrit à la main, sans vérifier en détail que tout fonctionne, c'est juste pour donner l'idée

Suggested change

	# unique idAtProvider
	if price_category.id_at_provider:
	if price_category.id_at_provider in unique_id_at_provider_list:
	raise ValueError(
	f"Price category `idAtProvider` must be unique. Duplicated value : {price_category.id_at_provider}"
	)
	unique_id_at_provider_list.add(price_category.id_at_provider)
	from collections import Counter
	ids = Counter([pc.id_at_provider for pc in price_categories if pc.id_at_provider])
	duplicates = [id_at_provider for id_at_provider, count in ids.items() if count > 1]
	if duplicates:
	raise ValueError("Price category `idAtProvider` must be unique. Duplicate values: {duplicates}")

[tcoudray-pass]

Je suis d'accord avec toi. Est-ce que ça te va si je fais ça dans une petite refacto à côté (en appliquant la même logique pour les couples (label, prix)) ?

[tcoudray-pass]

A voir aussi si on ne veut pas rajouter des cas de tests pour l'api où on s'assure qu'on ne peut pas aller modifier une catégorie de prix à laquelle on n'a pas accès avec la clé d'API.

@jeremieb-pass : En fait ce check est fait à partir de l'offre. Pour pouvoir accéder à la price category, il faut avoir accès à l'offre. C'est ce que check les test 404 en début de class.