Add new function to fetch resources in bookings remote

Author: itsmeichigo

Modules/Sources/Networking/Remote/BookingsRemote.swift

@@ -25,6 +25,10 @@ public protocol BookingsRemoteProtocol {
 
     func fetchResource(resourceID: Int64,
                        siteID: Int64) async throws -> BookingResource?
+
+    func fetchResources(for siteID: Int64,
+                        pageNumber: Int,
+                        pageSize: Int) async throws -> [BookingResource]
 }
 
 /// Booking: Remote Endpoints
@@ -133,6 +137,37 @@ public final class BookingsRemote: Remote, BookingsRemoteProtocol {
 
         return try await enqueue(request, mapper: mapper)
     }
+
+    /// Retrieves all of the `BookingResources` available.
+    ///
+    /// - Parameters:
+    ///     - siteID: Site for which we'll fetch remote booking resources.
+    ///     - pageNumber: Number of page that should be retrieved.
+    ///     - pageSize: Number of resources to be retrieved per page.
+    ///
+    public func fetchResources(
+        for siteID: Int64,
+        pageNumber: Int = Default.pageNumber,
+        pageSize: Int = Default.pageSize
+    ) async throws -> [BookingResource] {
+        let parameters = [
+            ParameterKey.page: String(pageNumber),
+            ParameterKey.perPage: String(pageSize)
+        ]
+
+        let path = Path.resources
+        let request = JetpackRequest(
+            wooApiVersion: .wcBookings,
+            method: .get,
+            siteID: siteID,
+            path: path,
+            parameters: parameters,
+            availableAsRESTRequest: true
+        )
+        let mapper = ListMapper<BookingResource>(siteID: siteID)
+
+        return try await enqueue(request, mapper: mapper)
+    }
 }
 
 // MARK: - Constants

Modules/Tests/NetworkingTests/Remote/BookingsRemoteTests.swift

@@ -123,4 +123,47 @@ struct BookingsRemoteTests {
             _ = try await remote.fetchResource(resourceID: 22, siteID: sampleSiteID)
         }
     }
+
+    @Test func test_fetchResources_properly_returns_parsed_resources() async throws {
+        // Given
+        let remote = BookingsRemote(network: network)
+        network.simulateResponse(requestUrlSuffix: "resources/team-members", filename: "booking-resource-list")
+
+        // When
+        let resources = try await remote.fetchResources(for: sampleSiteID)
+
+        // Then
+        #expect(resources.count == 2)
+        let firstResource = try #require(resources.first)
+        #expect(firstResource.resourceID == 22)
+        #expect(firstResource.name == "Joel (Sample resource)")
+        #expect(firstResource.quantity == 1)
+        #expect(firstResource.siteID == sampleSiteID)
+    }
+
+    @Test func test_fetchResources_properly_relays_networking_errors() async {
+        // Given
+        let remote = BookingsRemote(network: network)
+
+        // Then
+        await #expect(throws: NetworkError.notFound()) {
+            _ = try await remote.fetchResources(for: sampleSiteID)
+        }
+    }
+
+    @Test func test_fetchResources_sends_correct_parameters() async throws {
+        // Given
+        let remote = BookingsRemote(network: network)
+        network.simulateResponse(requestUrlSuffix: "resources/team-members", filename: "booking-resource-list")
+
+        // When
+        _ = try await remote.fetchResources(for: sampleSiteID, pageNumber: 3, pageSize: 100)
+
+        // Then
+        let request = try #require(network.requestsForResponseData.first as? JetpackRequest)
+        let parameters = request.parameters
+
+        #expect((parameters["page"] as? String) == "3")
+        #expect((parameters["per_page"] as? String) == "100")
+    }
 }

Modules/Tests/NetworkingTests/Responses/booking-resource-list.json

@@ -0,0 +1,26 @@
+[
+    {
+        "id": 22,
+        "name": "Joel (Sample resource)",
+        "qty": 1,
+        "role": "",
+        "email": "",
+        "phone_number": "",
+        "image_id": 0,
+        "image_url": "",
+        "description": "",
+        "note": ""
+    },
+    {
+        "id": 23,
+        "name": "Sarah (Sample resource)",
+        "qty": 2,
+        "role": "Manager",
+        "email": "[email protected]",
+        "phone_number": "+1234567890",
+        "image_id": 45,
+        "image_url": "https://example.com/image.jpg",
+        "description": "Sample resource description",
+        "note": "Sample note"
+    }
+]

Modules/Tests/YosemiteTests/Mocks/MockBookingsRemote.swift

@@ -7,6 +7,7 @@ final class MockBookingsRemote: BookingsRemoteProtocol {
     private var loadAllBookingsResult: Result<[Booking], Error>?
     private var loadBookingResult: Result<Booking?, Error>?
     private var fetchResourceResult: Result<BookingResource?, Error>?
+    private var fetchResourcesResult: Result<[BookingResource], Error>?
 
     func whenLoadingAllBookings(thenReturn result: Result<[Booking], Error>) {
         loadAllBookingsResult = result
@@ -20,6 +21,10 @@ final class MockBookingsRemote: BookingsRemoteProtocol {
         fetchResourceResult = result
     }
 
+    func whenFetchingResources(thenReturn result: Result<[BookingResource], Error>) {
+        fetchResourcesResult = result
+    }
+
     func loadAllBookings(for siteID: Int64,
                          pageNumber: Int,
                          pageSize: Int,
@@ -50,4 +55,11 @@ final class MockBookingsRemote: BookingsRemoteProtocol {
     func updateBooking(from siteID: Int64, bookingID: Int64, attendanceStatus: Networking.BookingAttendanceStatus) async throws -> Networking.Booking? {
         return nil
     }
+
+    func fetchResources(for siteID: Int64, pageNumber: Int, pageSize: Int) async throws -> [Networking.BookingResource] {
+        guard let result = fetchResourcesResult else {
+            throw NetworkError.timeout()
+        }
+        return try result.get()
+    }
 }