feat: support multiple HelloAsso configurations

[Rotheem]

Description

Please explain the changes you made here.

Checklist

• Created tests which fail without the change (if possible)
• All tests passing
• Extended the documentation, if necessary

[codecov]

Codecov Report

Attention: Patch coverage is 79.10448% with 14 lines in your changes missing coverage. Please review.

Project coverage is 85.21%. Comparing base (efd9863) to head (1ca5444).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
app/core/utils/config.py	53.33%	7 Missing ⚠️
app/dependencies.py	76.47%	4 Missing ⚠️
app/types/exceptions.py	71.42%	2 Missing ⚠️
app/core/payment/payment_tool.py	92.85%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #750      +/-   ##
==========================================
- Coverage   85.27%   85.21%   -0.07%     
==========================================
  Files         163      163              
  Lines       11514    11553      +39     
==========================================
+ Hits         9819     9845      +26     
- Misses       1695     1708      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull Request Overview

Adds support for multiple HelloAsso configurations by introducing a HelloAssoConfigName enum, a HELLOASSO_CONFIGURATIONS setting list, and updating the payment tool and dependencies to select the correct configuration by name.

• Introduce HelloAssoConfigName and HelloAssoConfig model
• Refactor Settings to parse multiple configurations and validate names
• Update get_payment_tool dependency to return a named tool instance
• Update PaymentTool to accept a single HelloAssoConfig and remove per-slug parameters
• Adjust tests and endpoints to use the new multi-config setup

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
tests/test_payment.py	Refactored tests to use HelloAssoConfigName and override fn
tests/conftest.py	Dependency overrides updated per config name
tests/commons.py	MockedPaymentTool and override signature updated
app/types/exceptions.py	Added DotenvInvalidHelloAssoConfigNameError
app/modules/raid/endpoints_raid.py	Switched to Depends(get_payment_tool(HelloAssoConfigName.RAID))
app/modules/cdr/endpoints_cdr.py	Switched to Depends(get_payment_tool(HelloAssoConfigName.CDR))
app/dependencies.py	Refactored get_payment_tool to support multiple configs
app/core/utils/config.py	Added HELLOASSO_CONFIGURATIONS and PARSED_HELLOASSO_CONFIGURATIONS
app/core/payment/types_payment.py	Defined HelloAssoConfigName enum and HelloAssoConfig model
app/core/payment/payment_tool.py	Updated constructor to take HelloAssoConfig
app/core/myeclpay/endpoints_myeclpay.py	Switched to Depends(get_payment_tool(HelloAssoConfigName.MYECLPAY))
.env.test	Replaced single slugs with an empty HELLOASSO_CONFIGURATIONS and set HELLOASSO_API_BASE

Comments suppressed due to low confidence (3)

app/core/payment/payment_tool.py:42

• Attribute _hello_asso_api_base is inconsistent with _helloasso_slug. Consider renaming to _helloasso_api_base for consistent naming.

_hello_asso_api_base: str

app/core/utils/config.py:340

• Add a test case for invalid entries in HELLOASSO_CONFIGURATIONS to verify that DotenvInvalidHelloAssoConfigNameError is raised as expected.

if name not in HelloAssoConfigName._member_names_:

app/core/utils/config.py:320

• The decorator @cached_property is used but not imported. Add from functools import cached_property to avoid a NameError at runtime.

@cached_property

[armanddidierjean]

Great PR. Thanks

[armanddidierjean]

Instead of an optional _redirection_uri, I would suggest having a list _allowed_redirection_uri. When we would initiate a checkout, we would ensure that the redirect_uri passed by the front is allowed