Loop once only when picking point

mauriciofierrom · Jul 23, 2024 · 252ea32 · 252ea32
1 parent 9edf6c3
commit 252ea32
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/app/javascript/controllers/player/state.js b/app/javascript/controllers/player/state.js
@@ -78,7 +78,7 @@ export class EditingState extends PlayerState {
 export class PickingPointState extends PlayerState {
   async loop(from, to) {
     debug("Looping 3 times", { state: this.constructor.name, from, to })
-    await this.context.loopManager.loop(from, to, 3)
+    await this.context.loopManager.loop(from, to, 1)
     debug("Done point looping. Doing editing loop:", this.context.editState)
     this.context.state = this.context.editingState
     this.context.editState = { ...this.context.editState, setting: null }

diff --git a/test/javascript/controllers/player/state_test.js b/test/javascript/controllers/player/state_test.js
@@ -103,7 +103,7 @@ describe("PlayerState", () => {
         playerController.state = playerController.pickingPointState
         await playerController.loop(1, 5)
 
-        expect(mockLoop).toHaveBeenCalledWith(1, 5, 3)
+        expect(mockLoop).toHaveBeenCalledWith(1, 5, 1)
         expect(playerController.state).toBeInstanceOf(EditingState)
       })
     })

diff --git a/test/javascript/controllers/player_controller_test.js b/test/javascript/controllers/player_controller_test.js
@@ -120,7 +120,7 @@ describe("PlayerController", () => {
 
         playerController.updatePoints({detail: { start: 22, end: 77, setting: 22 }})
 
-        expect(mockLoop).toHaveBeenCalledWith(22, 25, 3)
+        expect(mockLoop).toHaveBeenCalledWith(22, 25, 1)
       })
     })
 
@@ -135,7 +135,7 @@ describe("PlayerController", () => {
 
         playerController.updatePoints({detail: { start: 22, end: 77, setting: 22 }})
 
-        expect(mockLoop).toHaveBeenCalledWith(22, 25, 3)
+        expect(mockLoop).toHaveBeenCalledWith(22, 25, 1)
       })
     })
   })