#396: Fix issue where ApplicationFee was set on the OrderRequest inst…

…ead of the OrderPaymentParameters

src/Mollie.Api/Models/Order/Request/OrderRequest.cs

@@ -2,6 +2,7 @@
 using System.Collections.Generic;
 using System.Linq;
 using Mollie.Api.JsonConverters;
+using Mollie.Api.Models.Order.Request.PaymentSpecificParameters;
 using Newtonsoft.Json;
 
 namespace Mollie.Api.Models.Order.Request {
@@ -101,7 +102,7 @@ public string? Method {
         /// <summary>
         /// Optional - Any payment specific properties can be passed here.
         /// </summary>
-        public PaymentSpecificParameters.PaymentSpecificParameters? Payment { get; set; }
+        public OrderPaymentParameters? Payment { get; set; }
 
         /// <summary>
         /// Provide any data you like, and we will save the data alongside the subscription. Whenever you fetch the subscription
@@ -122,12 +123,6 @@ public string? Method {
         /// </summary>
         public bool? ShopperCountryMustMatchBillingCountry { get; set; }
 
-        /// <summary>
-        /// Adding an application fee allows you to charge the merchant for the payment and transfer this to your own account.
-        /// </summary>
-        [JsonProperty("payment.applicationFee")]
-        public ApplicationFee? ApplicationFee { get; set; }
-
         /// <summary>
         ///	Oauth only - The payment profile's unique identifier, for example pfl_3RkSN1zuPE.
         /// </summary>

src/Mollie.Api/Models/Order/Request/PaymentSpecificParameters/ApplePaySpecificParameters.cs

@@ -1,5 +1,5 @@
 namespace Mollie.Api.Models.Order.Request.PaymentSpecificParameters {
-    public record ApplePaySpecificParameters : PaymentSpecificParameters {
+    public record ApplePaySpecificParameters : OrderPaymentParameters {
         /// <summary>
         /// Optional - The Apple Pay Payment Token object (encoded as JSON) that is part of the result of authorizing a payment
         /// request. The token contains the payment information needed to authorize the payment.

src/Mollie.Api/Models/Order/Request/PaymentSpecificParameters/BillieSpecificParameters.cs

@@ -1,5 +1,5 @@
 namespace Mollie.Api.Models.Order.Request.PaymentSpecificParameters {
-    public record BillieSpecificParameters : PaymentSpecificParameters {
+    public record BillieSpecificParameters : OrderPaymentParameters {
         /// <summary>
         /// Billie is a B2B payment method, thus it requires some extra information to identify the business
         /// that is creating the order. It is recommended to include these parameters as part of the create

src/Mollie.Api/Models/Order/Request/PaymentSpecificParameters/CreditCardSpecificParameters.cs

@@ -1,5 +1,5 @@
 namespace Mollie.Api.Models.Order.Request.PaymentSpecificParameters {
-    public record CreditCardSpecificParameters : PaymentSpecificParameters {
+    public record CreditCardSpecificParameters : OrderPaymentParameters {
         /// <summary>
         /// The card token you get from Mollie Components. The token contains the card information
         /// (such as card holder, card number and expiry date) needed to complete the payment.

src/Mollie.Api/Models/Order/Request/PaymentSpecificParameters/GiftcardSpecificParameters.cs

@@ -1,5 +1,5 @@
 namespace Mollie.Api.Models.Order.Request.PaymentSpecificParameters {
-    public record GiftcardSpecificParameters : PaymentSpecificParameters {
+    public record GiftcardSpecificParameters : OrderPaymentParameters {
         /// <summary>
         /// The gift card brand to use for the payment. These issuers are not dynamically available through the Issuers API, 
         /// but can be retrieved by using the issuers include in the Methods API. If you need a brand not in the list, contact

src/Mollie.Api/Models/Order/Request/PaymentSpecificParameters/IDealSpecificParameters.cs

@@ -1,5 +1,5 @@
 namespace Mollie.Api.Models.Order.Request.PaymentSpecificParameters {
-    public record IDealSpecificParameters : PaymentSpecificParameters {
+    public record IDealSpecificParameters : OrderPaymentParameters {
         /// <summary>
         /// Optional - iDEAL issuer id. The id could for example be ideal_INGBNL2A. The returned paymentUrl will then directly
         /// point to the ING web site.

src/Mollie.Api/Models/Order/Request/PaymentSpecificParameters/KbcSpecificParameters.cs

@@ -1,5 +1,5 @@
 namespace Mollie.Api.Models.Order.Request.PaymentSpecificParameters { 
-    public record KbcSpecificParameters : PaymentSpecificParameters {
+    public record KbcSpecificParameters : OrderPaymentParameters {
         /// <summary>
         /// The issuer to use for the KBC/CBC payment. These issuers are not dynamically available through the Issuers API, 
         /// but can be retrieved by using the issuers include in the Methods API. See the Mollie.Api.Models.Payment.Request.KbcIssuer 

src/Mollie.Api/Models/Order/Request/PaymentSpecificParameters/KlarnaSpecificParameters.cs

@@ -1,5 +1,5 @@
 namespace Mollie.Api.Models.Order.Request.PaymentSpecificParameters {
-    public record KlarnaSpecificParameters<T> : PaymentSpecificParameters where T : class {
+    public record KlarnaSpecificParameters<T> : OrderPaymentParameters where T : class {
         public T? ExtraMerchantData { get; set; }
     }
-}
+}

src/Mollie.Api/Models/Order/Request/PaymentSpecificParameters/PaymentSpecificParameters.cs → src/Mollie.Api/Models/Order/Request/PaymentSpecificParameters/OrderPaymentParameters.cs

@@ -1,10 +1,15 @@
 namespace Mollie.Api.Models.Order.Request.PaymentSpecificParameters {
-    public record PaymentSpecificParameters {
+    public record OrderPaymentParameters {
         public string? CustomerId { get; set; }
         /// <summary>
         /// See the Mollie.Api.Models.Payment.SequenceType class for a full list of known values.
         /// </summary>
         public string? SequenceType { get; set; }
         public string? WebhookUrl { get; set; }
+
+        /// <summary>
+        /// Adding an application fee allows you to charge the merchant for the payment and transfer this to your own account.
+        /// </summary>
+        public ApplicationFee? ApplicationFee { get; set; }
     }
 }

src/Mollie.Api/Models/Order/Request/PaymentSpecificParameters/PaySafeCardSpecificParameters.cs

@@ -1,5 +1,5 @@
 namespace Mollie.Api.Models.Order.Request.PaymentSpecificParameters {
-    public record PaySafeCardSpecificParameters : PaymentSpecificParameters {
+    public record PaySafeCardSpecificParameters : OrderPaymentParameters {
         /// <summary>
         /// Used for consumer identification. For example, you could use the consumer’s IP address.
         /// </summary>

src/Mollie.Api/Models/Order/Request/PaymentSpecificParameters/SepaDirectDebitSpecificParameters.cs

@@ -1,5 +1,5 @@
 namespace Mollie.Api.Models.Order.Request.PaymentSpecificParameters {
-    public record SepaDirectDebitSpecificParameters : PaymentSpecificParameters {
+    public record SepaDirectDebitSpecificParameters : OrderPaymentParameters {
         /// <summary>
         /// Optional - IBAN of the account holder.
         /// </summary>

tests/Mollie.Tests.Integration/Api/OrderTests.cs

@@ -87,6 +87,36 @@ public async Task CreateOrderAsync_OrderWithRequiredFields_OrderIsCreated() {
         orderResponseLine.Metadata.Should().Be(expectedMetadataString);
     }
 
+    [DefaultRetryFact]
+    public async Task CreateOrderAsync_OrderWithApplicationFee_OrderIsCreated() {
+        // If: we create a order request with only the required parameters
+        OrderRequest orderRequest = CreateOrder() with {
+            Payment = new OrderPaymentParameters {
+                ApplicationFee = new ApplicationFee {
+                    Amount = new Amount(Currency.EUR, 0.25m),
+                    Description = "Test"
+                }
+            }
+        };
+
+        // When: We send the order request to Mollie
+        OrderResponse result = await _orderClient.CreateOrderAsync(orderRequest);
+
+        // Then: Make sure we get a valid response
+        result.Should().NotBeNull();
+        result.Amount.Should().Be(orderRequest.Amount);
+        result.OrderNumber.Should().Be(orderRequest.OrderNumber);
+        result.Lines.Should().HaveCount(orderRequest.Lines.Count());
+        result.Links.Should().NotBeNull();
+        OrderLineRequest orderLineRequest = orderRequest.Lines.First();
+        OrderLineResponse orderResponseLine = result.Lines.First();
+        orderResponseLine.Type.Should().Be(orderLineRequest.Type);
+        orderResponseLine.Links.ImageUrl!.Href.Should().Be(orderLineRequest.ImageUrl);
+        orderResponseLine.Links.ProductUrl!.Href.Should().Be(orderLineRequest.ProductUrl);
+        var expectedMetadataString = result.Lines.First().Metadata;
+        orderResponseLine.Metadata.Should().Be(expectedMetadataString);
+    }
+
     [DefaultRetryFact]
     public async Task CreateOrderAsync_WithMultiplePaymentMethods_OrderIsCreated() {
         // When: we create a order request and specify multiple payment methods
@@ -179,7 +209,7 @@ public async Task CreateOrderAsync_WithSinglePaymentMethod_OrderIsCreated() {
     [DefaultRetryTheory]
     [MemberData(nameof(PaymentSpecificParameters))]
     public async Task CreateOrderAsync_WithPaymentSpecificParameters_OrderIsCreated(
-        PaymentSpecificParameters paymentSpecificParameters) {
+        OrderPaymentParameters paymentSpecificParameters) {
 
         // If: we create a order request with payment specific parameters
         OrderRequest orderRequest = CreateOrder();