Skip to content

auto-gen: SDK changes (547aba95) - #14

Draft
luismeli10 wants to merge 1 commit into
masterfrom
auto-gen/547aba95-generic
Draft

luismeli10 wants to merge 1 commit into
masterfrom
auto-gen/547aba95-generic

Conversation

@luismeli10

Copy link
Copy Markdown
Owner

Automated SDK code generation.

generation_id: 547aba95-18a9-4644-88f1-4748ed964d8a

Review required before merge.

@luismeli10

Copy link
Copy Markdown
Owner Author

Automated reviewer warnings

This pull request was created with the following advisory findings. Review them before merging:

  • planned targets were not modified: tests/MercadoPago/Client/Unit/Order/OrderClientUnitTest.php, tests/MercadoPago/Client/Integration/Order/OrderClientITTest.php, tests/MercadoPago/Resources/Mocks/Response/Order/order_confirm.json, tests/Mer
  • required symbol is not proven by loaded repository context: createEvent
  • required symbol is not proven by loaded repository context: getRefunds
  • src/MercadoPago/Client/Order/OrderClient.php: falta el método público confirm(...) para POST /v1/orders/{order_id}/confirm. El fragmento nuevo private const URL_CONFIRM = self::URL_WITH_ID . '/confirm'; queda sin uso.
  • src/MercadoPago/Client/Order/OrderClient.php: falta el método público createEvent(...) para POST /v1/orders/{order_id}/events. El fragmento nuevo private const URL_EVENTS = self::URL_WITH_ID . '/events'; queda sin uso.
  • src/MercadoPago/Client/Order/OrderClient.php: falta el método público getRefunds(...) para GET /v1/orders/{order_id}/refund; la ruta existente solo recibió evidencia de una constante previa y no una implementación GET.
  • examples/Order/Confirm.php: $client->confirm($order_id, $request, $request_options) referencia un método inexistente en el diff.
  • examples/Order/CreateEvent.php: $client->createEvent($order_id, $request) referencia un método inexistente en el diff.
  • examples/Order/GetRefunds.php: $client->getRefunds($order_id, $request_options) referencia un método inexistente en el diff.
  • src/MercadoPago/Client/Order/OrderClient.php: los imports agregados use MercadoPago\Net\MPResponse; y use MercadoPago\Resources\Order\Transactions; no son utilizados por ningún cambio mostrado y no constituyen evidencia suficiente de
  • tests/MercadoPago/Client/Unit/Order/OrderClientUnitTest.php: no hay cambios para cubrir confirmación, simulación de eventos ni consulta de reembolsos.
  • tests/MercadoPago/Client/Integration/Order/OrderClientITTest.php: no hay cambios para cubrir los tres endpoints nuevos/modificados.
  • tests/MercadoPago/Resources/Mocks/Response/Order/order_confirm.json: el mock planificado está ausente del diff.
  • tests/MercadoPago/Resources/Mocks/Response/Order/order_refund_list.json: el mock planificado está ausente del diff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant