refactor: ensure logRequestByItSelf is preserved in map functions and update test assertions for request listener calls

Author: dankinsoid

Sources/SwiftAPIClient/APIClientCaller.swift

@@ -65,11 +65,13 @@ public struct APIClientCaller<Response, Value, Result> {
 	/// try await client.call(.http, as: .decodable)
 	/// ```
 	public func map<T>(_ mapper: @escaping (Result) throws -> T) -> APIClientCaller<Response, Value, T> {
-		APIClientCaller<Response, Value, T> {
+		var result = APIClientCaller<Response, Value, T> {
 			try mapper(_call($0, $1, $2, $3))
 		} mockResult: {
 			try mapper(_mockResult($0))
 		}
+		result.logRequestByItSelf = logRequestByItSelf
+		return result
 	}
 
 	/// Maps the response to another type using the provided mapper.
@@ -81,13 +83,15 @@ public struct APIClientCaller<Response, Value, Result> {
 	/// try await client.call(.http, as: .decodable)
 	/// ```
 	public func mapResponse<T>(_ mapper: @escaping (Response) throws -> T) -> APIClientCaller<T, Value, Result> {
-		APIClientCaller<T, Value, Result> { id, request, configs, serialize in
+		var result = APIClientCaller<T, Value, Result> { id, request, configs, serialize in
 			try _call(id, request, configs) { response, validate in
 				try serialize(mapper(response), validate)
 			}
 		} mockResult: {
 			try _mockResult($0)
 		}
+		result.logRequestByItSelf = logRequestByItSelf
+		return result
 	}
 }
 

Tests/SwiftAPIClientTests/Modifiers/LoggingAndListenerTests.swift

@@ -350,8 +350,8 @@ final class LoggingAndListenerTests: XCTestCase {
 
 		XCTAssertEqual(result, mockData)
 
-		// Verify HTTP client listener calls (HTTP client may have multiple internal request stages)
-		XCTAssertGreaterThan(mockListener.requestStartedCalls.count, 0, "Expected at least 1 request started call")
+		// Verify HTTP client listener calls
+		XCTAssertEqual(mockListener.requestStartedCalls.count, 1, "Expected exactly 1 request started call")
 		XCTAssertEqual(mockListener.responseReceivedCalls.count, 1, "Expected 1 response received call")
 		XCTAssertEqual(mockListener.responseSerializedCalls.count, 1, "Expected 1 response serialized call")
 		XCTAssertEqual(mockListener.errorCalls.count, 0, "Expected no error calls")