fix: test

packages/client/src/devices/InputMediaDeviceManager.ts

@@ -74,11 +74,10 @@ export abstract class InputMediaDeviceManager<
    * Starts stream.
    */
   async enable() {
+    this.state.prevStatus = this.state.status;
     if (this.state.optimisticStatus === 'enabled') {
       return;
     }
-
-    this.state.prevStatus = this.state.status;
     this.state.setPendingStatus('enabled');
 
     await withCancellation(this.statusChangeConcurrencyTag, async (signal) => {
@@ -98,18 +97,18 @@ export abstract class InputMediaDeviceManager<
    * @param {boolean} [forceStop=false] when true, stops the tracks regardless of the state.disableMode
    */
   async disable(forceStop: boolean = false) {
-    if (!forceStop && this.state.optimisticStatus === 'disabled') {
-      return;
-    }
     this.state.prevStatus = this.state.status;
     if (
-      this.state.prevStatus === 'disabled' &&
+      this.state.prevStatus !== 'enabled' &&
       this.state.optimisticStatus === 'enabled'
     ) {
       // if previously enabled but its not done yet, set the prevStatus to enabled
       // this makes resume() work correctly
       this.state.prevStatus = 'enabled';
     }
+    if (!forceStop && this.state.optimisticStatus === 'disabled') {
+      return;
+    }
 
     this.state.setPendingStatus('disabled');
 
@@ -140,7 +139,7 @@ export abstract class InputMediaDeviceManager<
   async resume() {
     if (
       this.state.prevStatus === 'enabled' &&
-      this.state.status === 'disabled'
+      this.state.status !== 'enabled'
     ) {
       await this.enable();
     }

packages/client/src/devices/__tests__/InputMediaDeviceManager.test.ts

@@ -241,6 +241,29 @@ describe('InputMediaDeviceManager.test', () => {
     expect(manager.enable).toHaveBeenCalledOnce();
   });
 
+  it(`should resume if enable was cancelled due to disable call`, async () => {
+    vi.spyOn(manager, 'enable');
+
+    manager.enable();
+
+    expect(manager.enable).toHaveBeenCalledOnce();
+
+    // enable was not awaited so cancelled by disabled
+    await manager.disable();
+
+    manager.resume();
+
+    expect(manager.enable).toBeCalledTimes(2);
+
+    // this disable is not awaited, but will cancel the enable anyway
+    // so resume must work here too
+    manager.disable();
+
+    manager.resume();
+
+    expect(manager.enable).toBeCalledTimes(3);
+  });
+
   it('should provide default constraints to `getStream` method', () => {
     manager.setDefaultConstraints({
       echoCancellation: true,