refactor : moved board creation to utility class (#16)

Co-authored-by: elfarsif <[email protected]>

Author: frankfarsi

Diff for: minesweeper/tests/unittests/BoardUtilies.py

@@ -0,0 +1,16 @@
+from typing import List
+
+from minesweeper.src.Cell import Cell
+from minesweeper.src.State import State
+
+
+class BoardUtilities:
+    @staticmethod
+    def create_empty_board() -> List[List[Cell]]:
+        expected_cells = []
+        for i in range(9):
+            row = []
+            for j in range(9):
+                row.append(Cell(State.EMPTY))
+            expected_cells.append(row)
+        return expected_cells

Diff for: minesweeper/tests/unittests/EmptyBoardUnitTest.py

@@ -1,12 +1,12 @@
 import unittest
 
-from minesweeper.src.Cell import Cell
 from minesweeper.src.Game import Game
-from minesweeper.src.State import State
+from minesweeper.tests.unittests.BoardUtilies import BoardUtilities
 
 
 class EmptyBoardUnitTest(unittest.TestCase):
 
+
     def setUp(self):
         self.game = Game()
 
@@ -18,14 +18,6 @@ def test_when_game_is_started_it_is_running(self):
         assert self.game.isRunning == True
 
     def test_board_is_empty_on_start(self):
-        expected_cells = []
-        for i in range(9):
-            row= []
-            for j in range(9):
-                row.append(Cell(State.EMPTY))
-            expected_cells.append(row)
-
-        print(self.game.board.cells)
+        assert self.game.board.cells == BoardUtilities.create_empty_board()
+        assert self.game.board.isEmpty() == True;
 
-        assert self.game.board.cells == expected_cells
-        assert self.game.board.isEmpty() == True;