tinybrain: Parse two-digit numbers correctly (#235)

* Add a test * Fix numeral parsing
Terkwood · Apr 23, 2020 · 8e8bfc3 · 8e8bfc3
1 parent d756a05
commit 8e8bfc3
Show file tree

Hide file tree

Showing 3 changed files with 31 additions and 12 deletions.
diff --git a/tinybrain/Cargo.lock b/tinybrain/Cargo.lock
diff --git a/tinybrain/Cargo.toml b/tinybrain/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "tinybrain"
-version = "0.1.8"
+version = "0.1.9"
 authors = ["terkwood <38859656+Terkwood@users.noreply.github.com>"]
 edition = "2018"
 

diff --git a/tinybrain/src/katago/mod.rs b/tinybrain/src/katago/mod.rs
@@ -86,7 +86,7 @@ impl TryFrom<KataGoResponse> for MoveComputed {
             let left: char = ans[0];
             Some(AlphaNumCoord(
                 left,
-                alpha_num_or_pass[1..2].parse::<u16>().expect("alphanum "),
+                alpha_num_or_pass[1..].parse::<u16>().expect("alphanum"),
             ))
         };
 
@@ -137,6 +137,25 @@ mod tests {
         assert_eq!(actual, expected)
     }
 
+    #[test]
+    fn y_coord_not_truncated() {
+        let actual = MoveComputed::try_from(KataGoResponse {
+            id: Id(format!("{}_1_WHITE", Uuid::nil().to_string())),
+            turn_number: 1,
+            move_infos: vec![MoveInfo {
+                r#move: "D10".to_string(),
+                order: 0,
+            }],
+        })
+        .expect("fail");
+        let expected = MoveComputed {
+            game_id: GameId(Uuid::nil()),
+            alphanum_coord: Some(AlphaNumCoord('D', 10)),
+            player: Player::WHITE,
+        };
+        assert_eq!(actual, expected)
+    }
+
     #[test]
     fn move_computed_from_pass() {
         let actual = MoveComputed::try_from(KataGoResponse {