[No QA] Update comment followup from 54110

src/libs/Network/SequentialQueue.ts

@@ -198,6 +198,12 @@ function unpause() {
     const numberOfPersistedRequests = PersistedRequests.getAll().length || 0;
     Log.info(`[SequentialQueue] Unpausing the queue and flushing ${numberOfPersistedRequests} requests`);
     isQueuePaused = false;
+
+    // A READ request will wait until all the WRITE requests are done, using the isReadyPromise promise.
+    // When the queue is paused and then unpaused, we call flush which by defaults recreates the isReadyPromise.
+    // After all the WRITE requests are done, the isReadyPromise is resolved, but since it's a new instance of promise,
+    // the pending READ request never received the resolved callback. That's why we don't want to recreate
+    // the promise when unpausing the queue.
     flush(false);
 }
 

tests/unit/APITest.ts

@@ -581,7 +581,7 @@ describe('APITests', () => {
             });
     });
 
-    test('Read request should not stuck when SequentialQueue is paused an resumed', async () => {
+    test('Read request should not stuck when SequentialQueue is paused and resumed', async () => {
         // Given 2 WRITE requests and 1 READ request where the first write request pauses the SequentialQueue
         const xhr = jest.spyOn(HttpUtils, 'xhr').mockResolvedValueOnce({previousUpdateID: 1});
         API.write('MockWriteCommandOne' as WriteCommand, {});