feat(cypress): add 2 more payout connectors and bank transfer support for payout

[pixincreate]

Type of Change

• Bugfix
• New feature
• Enhancement
• Refactoring
• Dependency updates
• Documentation
• CI/CD

Description

This PR modifies Payout tests. Notably, 2 more connectors has been added namely, Adyen Platform and Wise.
Adyen Platform and Wise payout connectors only support Bank Transfers unlike Adyen which supports cards as well. Hence, for connectors that does not support a payment method, we throw 501 by leveraging Common.js (except for Wise that has a different error message for 501 unlike other connectors -- this will optimized and refactored in the coming refactor).

Flow is as below:

• Bank Transfer
  • ACH (Not implemented)
  • BACS (Not implemented)
  • SEPA
    • Auto fulfill and Auto confirm
    • Auto fulfill and Manual confirm
    • Manual fulfill and Auto confirm
    • Manual fulfill and Manual confirm
• Saved Payouts
  • Card
    • Onboard customer prior to transaction
    • Save payment method after successful transaction
  • Bank Transfer
    • Onboard customer prior to transaction
    • Save payment method after successful transaction

Upcoming refactor will remove duplicate connector Create commands (we currently have 3 connector create commands in commands.js) by making it more generic and reusable. The same has to be done for confirm calls as well.

Additionally, configs for prettier has been added just to maintain consistency across all environments.
To run auto-formatting, execute the below command in terminal:

prettier . --write

Note

8c72275 removes KV support in Payouts. We'll be coming back to this later in future. Added here for reference

Additional Changes

• This PR modifies the API contract
• This PR modifies the database schema
• This PR modifies application configuration/environment variables

Motivation and Context

More tests

How did you test it?

Adyen	Adyen Platform	Wise

BACS, ACH not implemented yet.

Checklist

• I formatted the code cargo +nightly fmt --all
• I addressed lints thrown by cargo clippy
• I reviewed the submitted code
• I added unit tests for my changes where possible

[Sakilmostak]

This needs to be refactored such that metadata is handled while fetching credentials rather that hardcoding each value of metadata for MCA. Raise a seperate pr for the implementation

[pixincreate]

Yes, this was hardcoded and is specific to adyenplatform connector (mentioned in comments as well) as sourceBalanceAccount is being passed as a value to metadata. It is a secret value which cannot be made public (can be treated as key1)

For context, above is a temporary change and I'll be modifying get_value_by_key to accommodate below mentioned change:

• get_value_by_key() has been duplicated in multiple places, this can be unified
  • Can be 2 copies, one with merger (logic that merges common.js and connector.js files into one) and the other one being the legacy one. Will look into it
• format of creds will have to be modified first:

   // from
   {
   	"connector": {
   		"auth_type": "KeyType",
   		"api_key": "api_key",
   		"key1": "key1",
   		// other kv pairs
   	},
   	// another one
   }
   // to something like, will be decided later when picked up
   {
   	"connector": {
   		"auth_type": "KeyType",
   		"api_key": "api_key",
   		"key1": "key1",
   		// other kv pairs
   		"metadata": {
   			"key": "value",
   			// other kv pairs
   		}
   	},
   }

• Any key that lies outside of auth_info (auth_type, api_key, key1, key2, and api_secret) should be treated as metadata