Track timeSettings state

Author: yishn

README.md

@@ -287,9 +287,14 @@ The state of an engine is represented by an object of the following structure:
 
 ```js
 {
-  komi: <number | null>,
-  boardsize: <[number, number] | null>,
-  history: <Command[] | null>
+  komi: <number>,
+  boardsize: <[number, number]>,
+  timeSettings: {
+    mainTime: <number>,
+    byoyomiTime: <number>,
+    byoyomiStones: <number>
+  },
+  history: <Command[]>
 }
 ```
 

src/ControllerStateTracker.js

@@ -12,6 +12,7 @@ const commandEquals = (cmd1, cmd2) =>
 const getDefaultState = () => ({
   komi: null,
   boardsize: null,
+  timeSettings: null,
   history: []
 })
 
@@ -63,6 +64,15 @@ class ControllerStateTracker {
           this.state.history = []
         } else if (command.name === 'komi' && command.args.length >= 1) {
           this.state.komi = +command.args[0]
+        } else if (
+          command.name === 'time_settings' &&
+          command.args.length >= 3
+        ) {
+          this.state.timeSettings = {
+            mainTime: +command.args[0],
+            byoyomiTime: +command.args[1],
+            byoyomiStones: +command.args[2]
+          }
         } else if (
           ['fixed_handicap', 'place_free_handicap'].includes(command.name)
         ) {
@@ -208,6 +218,28 @@ class ControllerStateTracker {
         throw new Error('Board size is not supported by engine')
     }
 
+    // Update timeSettings
+
+    if (
+      state.timeSettings != null &&
+      (this.state.timeSettings == null ||
+        state.timeSettings.mainTime !== this.state.timeSettings.mainTime ||
+        state.timeSettings.byoyomiTime !==
+          this.state.timeSettings.byoyomiTime ||
+        state.timeSettings.byoyomiStones !==
+          this.state.timeSettings.byoyomiStones)
+    ) {
+      let {error} = await controller.sendCommand({
+        name: 'time_settings',
+        args: [
+          `${state.timeSettings.mainTime}`,
+          `${state.timeSettings.byoyomiTime}`,
+          `${state.timeSettings.byoyomiStones}`
+        ]
+      })
+      if (error) throw new Error('Time settings are not supported by engine')
+    }
+
     // Update history
 
     if (state.history != null) {

tests/ControllerStateTracker.test.js

@@ -31,7 +31,7 @@ t.test('knowsCommand', async t => {
   t.equals(await stateTracker.knowsCommand('version'), true)
 })
 
-t.test('parallel syncing', async t => {
+t.test('handle parallel syncing sequentially', async t => {
   let {stateTracker} = t.context
 
   await Promise.all([
@@ -42,6 +42,7 @@ t.test('parallel syncing', async t => {
   t.strictDeepEquals(stateTracker.state, {
     komi: 8,
     boardsize: [18, 18],
+    timeSettings: null,
     history: []
   })
 })
@@ -109,6 +110,28 @@ t.test('sync rectangular boardsize state', async t => {
   t.equals(stateTracker.state.history, null)
 })
 
+t.test('sync time settings', async t => {
+  let {stateTracker} = t.context
+
+  t.equals(stateTracker.state.timeSettings, null)
+
+  let timeSettings = {mainTime: 30, byoyomiTime: 10, byoyomiStones: 20}
+  await stateTracker.sync({timeSettings})
+
+  t.strictDeepEquals(stateTracker.state.timeSettings, timeSettings)
+
+  await stateTracker.controller.sendCommand({
+    name: 'time_settings',
+    args: ['15', '1', '10']
+  })
+
+  t.strictDeepEquals(stateTracker.state.timeSettings, {
+    mainTime: 15,
+    byoyomiTime: 1,
+    byoyomiStones: 10
+  })
+})
+
 t.test('sync history state', async t => {
   t.test('sync playing commands', async t => {
     let {stateTracker} = t.context

tests/engines/testEngine.js

@@ -21,6 +21,7 @@ for (let commandName of [
   'boardsize',
   'rectangular_boardsize',
   'komi',
+  'time_settings',
   'set_free_handicap',
   'loadsgf'
 ]) {