Improve failure handling in ContactLookupFeature

Examples/xx-messenger/Sources/ContactLookupFeature/ContactLookupFeature.swift

@@ -57,6 +57,7 @@ public let contactLookupReducer = Reducer<ContactLookupState, ContactLookupActio
   switch action {
   case .lookupTapped:
     state.isLookingUp = true
+    state.failure = nil
     return Effect.result { [state] in
       do {
         let contact = try env.messenger.lookupContact(id: state.id)
@@ -71,11 +72,12 @@ public let contactLookupReducer = Reducer<ContactLookupState, ContactLookupActio
 
   case .didLookup(_):
     state.isLookingUp = false
+    state.failure = nil
     return .none
 
   case .didFail(let error):
-    state.failure = error.localizedDescription
     state.isLookingUp = false
+    state.failure = error.localizedDescription
     return .none
   }
 }

Examples/xx-messenger/Tests/ContactLookupFeatureTests/ContactLookupFeatureTests.swift

@@ -23,12 +23,14 @@ final class ContactLookupFeatureTests: XCTestCase {
 
     store.send(.lookupTapped) {
       $0.isLookingUp = true
+      $0.failure = nil
     }
 
     XCTAssertEqual(didLookupId, [id])
 
     store.receive(.didLookup(lookedUpContact)) {
       $0.isLookingUp = false
+      $0.failure = nil
     }
   }
 
@@ -47,11 +49,12 @@ final class ContactLookupFeatureTests: XCTestCase {
 
     store.send(.lookupTapped) {
       $0.isLookingUp = true
+      $0.failure = nil
     }
 
     store.receive(.didFail(failure)) {
-      $0.failure = failure.localizedDescription
       $0.isLookingUp = false
+      $0.failure = failure.localizedDescription
     }
   }
 }