Adds one more test case for deadlock fix.

The cache for temporary service is not immediately released after the outermost eDO invocation. Instead, it is released when autorelease pool ends, such as the end of dispatch_sync/async. So adding a test case to imply that.

At the same time, fix an issue that will fail the new test case, and cause retain cycle of host service -> handler -> host service.

PiperOrigin-RevId: 361217991

Author: AlbertWang0116

Service/Sources/EDOHostService.m

@@ -394,8 +394,8 @@ - (void)startReceivingRequestsForChannel:(id<EDOChannel>)channel {
     EDOHostService *strongSelf = weakSelf;
     NSException *exception;
     // TODO(haowoo): Add the proper error handler.
-    NSAssert(error == nil, @"Failed to receive the data (%d) for %@.",
-             strongSelf.port.hostPort.port, error);
+    NSCAssert(error == nil, @"Failed to receive the data (%d) for %@.",
+              strongSelf.port.hostPort.port, error);
     if (data == nil) {
       // the client socket is closed.
       NSLog(@"The channel (%p) with port %d is closed", targetChannel,

Service/Tests/UnitTests/EDOServiceTest.m

@@ -255,6 +255,28 @@ - (void)testTemporaryServiceResolvesLocalObject {
   XCTAssertEqual(anObject, rountTripObject);
 }
 
+/** Verifies the temporary host keeps being reused until the end of an external autorelease pool. */
+- (void)testTemporaryServiceIsReleasedLazily {
+  dispatch_queue_t testQueue = dispatch_queue_create("com.google.edotest", DISPATCH_QUEUE_SERIAL);
+  EDOTestDummy *dummyOnBackground = self.rootObjectOnBackground;
+  XCTestExpectation *expectation =
+      [self expectationWithDescription:@"temporary service retain count check"];
+
+  dispatch_async(testQueue, ^{
+    __weak EDOHostService *temporaryHostService = nil;
+    @autoreleasepool {
+      temporaryHostService = [EDOHostService temporaryServiceForCurrentThread];
+      [dummyOnBackground voidWithBlock:^{
+      }];
+      XCTAssertNotNil(temporaryHostService);
+    }
+    XCTAssertNil(temporaryHostService);
+    [expectation fulfill];
+  });
+
+  [self waitForExpectations:@[ expectation ] timeout:1.0];
+}
+
 - (void)testClassMethodsAndInit {
   Class remoteClass = EDO_REMOTE_CLASS(EDOTestDummy, self.serviceOnBackground.port.hostPort.port);