From 92a69ee74196894adc45c01fe045d3f622938390 Mon Sep 17 00:00:00 2001
From: Bruno BELANYI <bruno@belanyi.fr>
Date: Mon, 1 Apr 2024 21:35:48 +0100
Subject: [PATCH] Move FEN-related tests to its module

---
 src/board/chess_board/mod.rs | 72 --------------------------------
 src/fen.rs                   | 79 ++++++++++++++++++++++++++++++++++++
 2 files changed, 79 insertions(+), 72 deletions(-)

diff --git a/src/board/chess_board/mod.rs b/src/board/chess_board/mod.rs
index c5105ab..16e402d 100644
--- a/src/board/chess_board/mod.rs
+++ b/src/board/chess_board/mod.rs
@@ -673,78 +673,6 @@ mod test {
         );
     }
 
-    #[test]
-    fn fen_default_position() {
-        let default_position = ChessBoard::default();
-        assert_eq!(
-            ChessBoard::from_fen("rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1")
-                .unwrap(),
-            default_position
-        );
-    }
-
-    #[test]
-    fn fen_en_passant() {
-        // Start from default position
-        let mut position = ChessBoard::default();
-        position.do_move(
-            MoveBuilder {
-                piece: Piece::Pawn,
-                start: Square::E2,
-                destination: Square::E4,
-                capture: None,
-                promotion: None,
-                en_passant: false,
-                double_step: true,
-                castling: false,
-            }
-            .into(),
-        );
-        assert_eq!(
-            ChessBoard::from_fen("rnbqkbnr/pppppppp/8/8/4P3/8/PPPP1PPP/RNBQKBNR b KQkq e3 0 1")
-                .unwrap(),
-            position
-        );
-        // And now c5
-        position.do_move(
-            MoveBuilder {
-                piece: Piece::Pawn,
-                start: Square::C7,
-                destination: Square::C5,
-                capture: None,
-                promotion: None,
-                en_passant: false,
-                double_step: true,
-                castling: false,
-            }
-            .into(),
-        );
-        assert_eq!(
-            ChessBoard::from_fen("rnbqkbnr/pp1ppppp/8/2p5/4P3/8/PPPP1PPP/RNBQKBNR w KQkq c6 0 2")
-                .unwrap(),
-            position
-        );
-        // Finally, Nf3
-        position.do_move(
-            MoveBuilder {
-                piece: Piece::Knight,
-                start: Square::G1,
-                destination: Square::F3,
-                capture: None,
-                promotion: None,
-                en_passant: false,
-                double_step: false,
-                castling: false,
-            }
-            .into(),
-        );
-        assert_eq!(
-            ChessBoard::from_fen("rnbqkbnr/pp1ppppp/8/2p5/4P3/5N2/PPPP1PPP/RNBQKB1R b KQkq - 1 2 ")
-                .unwrap(),
-            position
-        );
-    }
-
     #[test]
     fn do_move() {
         // Start from default position
diff --git a/src/fen.rs b/src/fen.rs
index 8baf4af..3096c95 100644
--- a/src/fen.rs
+++ b/src/fen.rs
@@ -202,3 +202,82 @@ impl FromFen for ChessBoard {
         Ok(builder.try_into()?)
     }
 }
+
+#[cfg(test)]
+mod test {
+    use crate::board::MoveBuilder;
+
+    use super::*;
+
+    #[test]
+    fn default_position() {
+        let default_position = ChessBoard::default();
+        assert_eq!(
+            ChessBoard::from_fen("rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1")
+                .unwrap(),
+            default_position
+        );
+    }
+
+    #[test]
+    fn en_passant() {
+        // Start from default position
+        let mut position = ChessBoard::default();
+        position.do_move(
+            MoveBuilder {
+                piece: Piece::Pawn,
+                start: Square::E2,
+                destination: Square::E4,
+                capture: None,
+                promotion: None,
+                en_passant: false,
+                double_step: true,
+                castling: false,
+            }
+            .into(),
+        );
+        assert_eq!(
+            ChessBoard::from_fen("rnbqkbnr/pppppppp/8/8/4P3/8/PPPP1PPP/RNBQKBNR b KQkq e3 0 1")
+                .unwrap(),
+            position
+        );
+        // And now c5
+        position.do_move(
+            MoveBuilder {
+                piece: Piece::Pawn,
+                start: Square::C7,
+                destination: Square::C5,
+                capture: None,
+                promotion: None,
+                en_passant: false,
+                double_step: true,
+                castling: false,
+            }
+            .into(),
+        );
+        assert_eq!(
+            ChessBoard::from_fen("rnbqkbnr/pp1ppppp/8/2p5/4P3/8/PPPP1PPP/RNBQKBNR w KQkq c6 0 2")
+                .unwrap(),
+            position
+        );
+        // Finally, Nf3
+        position.do_move(
+            MoveBuilder {
+                piece: Piece::Knight,
+                start: Square::G1,
+                destination: Square::F3,
+                capture: None,
+                promotion: None,
+                en_passant: false,
+                double_step: false,
+                castling: false,
+            }
+            .into(),
+        );
+        assert_eq!(
+            ChessBoard::from_fen("rnbqkbnr/pp1ppppp/8/2p5/4P3/5N2/PPPP1PPP/RNBQKB1R b KQkq - 1 2 ")
+                .unwrap(),
+            position
+        );
+    }
+}