fix(gocardless): Treat Gocardless webhooks events in separate jobs

[vincent-pochet]

This pull request focuses on improving the GoCardless webhook events management and refactoring the integration by introducing new services for handling webhooks and events using the standard call/Result pattern.

The main changes include replacing direct calls to GocardlessService with more specific services and updating the logic to treat each webhook events in isolation to avoid failling a batch if a single event is causing an error.

Refactoring and Service Extraction:

• Refactored Webhook Handling:

  • Replaced GocardlessService.handle_incoming_webhook with Gocardless::HandleIncomingWebhookService.call in WebhooksController. (app/controllers/webhooks_controller.rb)
  • Extracted webhook handling logic into a new service HandleIncomingWebhookService. (app/services/payment_providers/gocardless/handle_incoming_webhook_service.rb)

• Refactored Event Handling:

  • Replaced GocardlessService.handle_event with Gocardless::HandleEventService.call in HandleEventJob. (app/jobs/payment_providers/gocardless/handle_event_job.rb)
  • Extracted event handling logic into a new service HandleEventService. (app/services/payment_providers/gocardless/handle_event_service.rb)
  • Keep acception the legacy events_json input argument in HandleEventJob to avoid issues with enqueued/dead jobs for now