RPCHistory tests resplved

WalletConnect · Jan 19, 2024 · 7d70977 · 7d70977
1 parent a41e93c
commit 7d70977
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/Sources/WalletConnectUtils/RPCHistory/RPCHistory.swift b/Sources/WalletConnectUtils/RPCHistory/RPCHistory.swift
@@ -106,7 +106,7 @@ public final class RPCHistory {
     }
 }
 
-private extension RPCHistory {
+extension RPCHistory {
 
     func removeOutdated() {
         let records = storage.getAll()

diff --git a/Tests/WalletConnectUtilsTests/RPCHistoryTests.swift b/Tests/WalletConnectUtilsTests/RPCHistoryTests.swift
@@ -35,10 +35,8 @@ final class RPCHistoryTests: XCTestCase {
         try sut.set(requestB, forTopic: String.randomTopic(), emmitedBy: .local)
         try sut.resolve(responseA)
         try sut.resolve(responseB)
-        let recordA = sut.get(recordId: requestA.id!)
-        let recordB = sut.get(recordId: requestB.id!)
-        XCTAssertEqual(recordA?.response, responseA)
-        XCTAssertEqual(recordB?.response, responseB)
+        XCTAssertNil(sut.get(recordId: requestA.id!))
+        XCTAssertNil(sut.get(recordId: requestB.id!))
     }
 
     func testDelete() throws {
@@ -95,7 +93,7 @@ final class RPCHistoryTests: XCTestCase {
     }
 
     func testResolveDuplicateResponse() throws {
-        let expectedError = RPCHistory.HistoryError.responseDuplicateNotAllowed
+        let expectedError = RPCHistory.HistoryError.requestMatchingResponseNotFound
 
         let request = RPCRequest.stub()
         let responseA = RPCResponse(matchingRequest: request, result: true)