[Test] Add baseUrl environment tests to all service test files
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 55/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- python
- Área
- api, testing-qa
Línea de trabajo
Empieza con RecurringTest.py y compara sus tres pruebas de baseUrl con los archivos de pruebas de servicio indicados. Añade una cobertura equivalente para el entorno de pruebas y el entorno en vivo usando el patrón de URL de cada servicio, aplicando el caso sin prefijo solo a PAL y a los servicios basados en checkout. Ejecuta las pruebas de los servicios afectados y confirma que cada archivo indicado tenga la cobertura adecuada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Context
RecurringTest.py has three tests that validate the baseUrl resolves correctly for both environments:
def test_base_url_test_environment(self):
url = self.adyen.client._determine_api_url("test", self.baseUrl)
self.assertEqual(url, self.baseUrl)
self.assertTrue(url.startswith("https://pal-test.adyen.com/pal/servlet/Recurring/"))
def test_base_url_live_environment(self):
self.adyen.client.live_endpoint_prefix = "1797a841fbb37ca7-AdyenDemo"
recurring_version = self.baseUrl.split("/")[-1]
url = self.adyen.client._determine_api_url("live", self.baseUrl)
self.assertEqual(
url,
f"https://1797a841fbb37ca7-AdyenDemo-pal-live.adyenpayments.com"
f"/pal/servlet/Recurring/{recurring_version}",
)
self.adyen.client.live_endpoint_prefix = None
def test_base_url_live_environment_no_prefix_raises(self):
self.adyen.client.live_endpoint_prefix = None
self.assertRaises(
AdyenEndpointInvalidFormat,
self.adyen.client._determine_api_url,
"live",
self.baseUrl,
)
These tests are missing from all other service test files:
BalancePlatformTest.pyBinLookupTest.pyCapitalTest.pyCheckoutTest.pyDataProtectionTest.pyDisputesTest.pyLegalEntityManagementTest.pyManagementTest.pyModificationTest.py(payments)PaymentTest.pyPosMobileTest.pySessionAuthenticationTest.pyStoredValueTest.pyThirdPartyPayoutTest.pyTransfersTest.py
Proposed Change
Add the equivalent of test_base_url_test_environment, test_base_url_live_environment, and test_base_url_live_environment_no_prefix_raises to each service test file, adapting the expected URLs to match the service's URL pattern (PAL, checkout, balancePlatform, management, etc.).
Note: test_base_url_live_environment_no_prefix_raises only applies to services that require a live_endpoint_prefix (i.e. PAL and checkout-based services).
Benefit
Ensures that URL resolution for both test and live environments is explicitly verified for every service, catching regressions in _determine_api_url early.
- Lenguaje dominante
- Python
- Estrellas
- 68
- Forks
- 51
- Merge medio
- 2 h 3 min
- PR fusionados (30 d)
- 2
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Adyen/adyen-python-api-library
-
[Bug] api_*_version override has no effect on PAL-based service URLsPosiblemente ocupada @cobanfurkanx la tomó hace 10 días. AbiertoBug do not stale
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
[Feature]: Foreign Exchange APIAbiertoFeature
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Adyen/adyen-python-api-library#487 · 1 comentario · 1 reacción ·
-
chore: consolidate package metadata into pyproject.toml (PEP 621)Posiblemente ocupada @erdemtuna la tomó hace 192 días. Abiertodo not stale
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
-
[Bug]: Unknown properties return NonePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertodo not stale
Dificultad 2/5 1-3 horas Aptitud para principiantes 48/100
Adyen/adyen-python-api-library#397 · 2 comentarios ·
-
do not stale Feature
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
Adyen/adyen-python-api-library#395 · 1 reacción ·
Todos los issues de Adyen/adyen-python-api-library
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
noontide-co/mainbranch#1211 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
[1.x Bug] run-operation --sql reports success but rolls back writes on transaction-capable adaptersAbiertotriage
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
dbt-labs/dbt#16784 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
bug settings ui
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
DewaldOosthuizen/py-tray-command-launcher#162 ·
Los mantenedores suelen responder en 1 día
-
run_all sleeps after the final job despite documenting delays between jobsPosiblemente ocupada @Gambit-Checkmate la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100