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
@@ -72,4 +73,11 @@ final class MockBookingsRemote: BookingsRemoteProtocol {
         }
         return try result.get()
     }
+
+    func updateBooking(from siteID: Int64, bookingID: Int64, attendanceStatus: Networking.BookingAttendanceStatus?, bookingStatus: Networking.BookingStatus?, note: String?) async throws -> Networking.Booking? {
+        guard let result = updateBookingNote else {
+            throw NetworkError.timeout()
+        }
+        return try result.get()
+    }
 }