Adding test

Author: adborbas

Modules/Tests/NetworkingTests/Remote/BookingsRemoteTests.swift

@@ -137,7 +137,8 @@ struct BookingsRemoteTests {
             from: sampleSiteID,
             bookingID: bookingID,
             attendanceStatus: .noShow,
-            bookingStatus: nil
+            bookingStatus: nil,
+            note: nil
         )
 
         // Then
@@ -157,7 +158,8 @@ struct BookingsRemoteTests {
             from: sampleSiteID,
             bookingID: bookingID,
             attendanceStatus: .noShow,
-            bookingStatus: nil
+            bookingStatus: nil,
+            note: nil
         )
 
         // Then
@@ -179,7 +181,8 @@ struct BookingsRemoteTests {
             from: sampleSiteID,
             bookingID: bookingID,
             attendanceStatus: nil,
-            bookingStatus: .confirmed
+            bookingStatus: .confirmed,
+            note: nil
         )
 
         // Then
@@ -201,7 +204,8 @@ struct BookingsRemoteTests {
             from: sampleSiteID,
             bookingID: bookingID,
             attendanceStatus: .booked,
-            bookingStatus: .paid
+            bookingStatus: .paid,
+            note: nil
         )
 
         // Then
@@ -254,4 +258,28 @@ struct BookingsRemoteTests {
         #expect((parameters["page"] as? String) == "3")
         #expect((parameters["per_page"] as? String) == "100")
     }
+
+    @Test func test_updateBookingNote_sends_correct_parameters_for_booking_note() async throws {
+        // Given
+        let remote = BookingsRemote(network: network)
+        let bookingID: Int64 = 206
+        network.simulateResponse(requestUrlSuffix: "bookings/\(bookingID)", filename: "booking-no-create-update-dates")
+
+        // When
+        _ = try await remote.updateBooking(
+            from: sampleSiteID,
+            bookingID: bookingID,
+            attendanceStatus: nil,
+            bookingStatus: nil,
+            note: "hello"
+        )
+
+        // Then
+        let request = try #require(network.requestsForResponseData.first as? JetpackRequest)
+        let parameters = request.parameters
+
+        #expect(parameters["attendance_status"] == nil)
+        #expect(parameters["status"] == nil)
+        #expect((parameters["note"] as? String) == "hello")
+    }
 }

Modules/Tests/YosemiteTests/Mocks/MockBookingsRemote.swift

@@ -9,6 +9,7 @@ final class MockBookingsRemote: BookingsRemoteProtocol {
     private var fetchResourceResult: Result<BookingResource?, Error>?
     private var updateBookingResult: Result<Booking?, Error>?
     private var fetchResourcesResult: Result<[BookingResource], Error>?
+    private var updateBookingNote: Result<Booking?, Error>?
 
     func whenLoadingAllBookings(thenReturn result: Result<[Booking], Error>) {
         loadAllBookingsResult = result
@@ -59,7 +60,8 @@ final class MockBookingsRemote: BookingsRemoteProtocol {
     func updateBooking(from siteID: Int64,
                        bookingID: Int64,
                        attendanceStatus: BookingAttendanceStatus?,
-                       bookingStatus: BookingStatus?) async throws -> Booking? {
+                       bookingStatus: BookingStatus?,
+                       note: String?) async throws -> Booking? {
         guard let result = updateBookingResult else {
             throw NetworkError.timeout()
         }