fix AddGameResults test

WTobor · Aug 26, 2018 · ce66383 · ce66383
1 parent d38cc7d
commit ce66383
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 15 deletions.
diff --git a/BoardGamesNook.Services/GameResultService.cs b/BoardGamesNook.Services/GameResultService.cs
@@ -44,25 +44,20 @@ public IEnumerable<GameResultDto> GetAllByGamerNickname(string nickname)
 
         public void AddGameResult(GameResultDto gameResultDto, Gamer gamer)
         {
-            GameTable gameTable = null;
-            if (gameResultDto.GameTableId.HasValue)
-                gameTable = _gameTableRepository.Get(gameResultDto.GameTableId.Value);
-
-            var boardGame = _boardGameRepository.Get(gameResultDto.BoardGameId);
-
-            var gameResult = Mapper.Map<GameResultDto, GameResult>(gameResultDto);
-            Mapper.Map(gamer, gameResult);
-            Mapper.Map(gameTable, gameResult);
-            Mapper.Map(boardGame, gameResult);
+            var gameResult = MapGameResultDtoToGameResultWithReferences(gameResultDto, gamer);
             _gameResultRepository.Add(gameResult);
         }
 
         public void AddGameResults(List<GameResultDto> gameResultDtoList, Gamer gamer)
         {
-            var gameResults =
-                Mapper.Map<List<GameResultDto>, List<GameResult>>(gameResultDtoList);
-            foreach (var obj in gameResults)
-                Mapper.Map(gamer, obj);
+            var gameResults = new List<GameResult>();
+            foreach (var gameResultDto in gameResultDtoList)
+            {
+                var gameResult = MapGameResultDtoToGameResultWithReferences(gameResultDto, gamer);
+                gameResult.CreatedGamerId = gamer.Id;
+                gameResults.Add(gameResult);
+            }
+
             _gameResultRepository.AddMany(gameResults);
         }
 
@@ -90,6 +85,21 @@ public GameResultDto GetGameResult(int id)
             return gameResultDto;
         }
 
+        private GameResult MapGameResultDtoToGameResultWithReferences(GameResultDto gameResultDto, Gamer gamer)
+        {
+            GameTable gameTable = null;
+            if (gameResultDto.GameTableId.HasValue)
+                gameTable = _gameTableRepository.Get(gameResultDto.GameTableId.Value);
+
+            var boardGame = _boardGameRepository.Get(gameResultDto.BoardGameId);
+
+            var gameResult = Mapper.Map<GameResultDto, GameResult>(gameResultDto);
+            Mapper.Map(gamer, gameResult);
+            Mapper.Map(gameTable, gameResult);
+            Mapper.Map(boardGame, gameResult);
+            return gameResult;
+        }
+
         private IEnumerable<GameResultDto> MapGameResultListToGameResultDtoList(List<GameResult> gameResultList)
         {
             var gameResultDtoList =

diff --git a/BoardGamesNook.Tests/GameResultTest.cs b/BoardGamesNook.Tests/GameResultTest.cs
@@ -64,6 +64,8 @@ public class GameResultTest
             Id = 1,
             GameTableId = 1,
             GamerId = _testGuid1.ToString(),
+            CreatedGamerId = _testGamer1.Id.ToString(),
+            CreatedGamerNickname = _testGamer1.Nickname,
             BoardGameId = 1
         };
 
@@ -74,13 +76,17 @@ public class GameResultTest
                 Id = 1,
                 GameTableId = 1,
                 GamerId = _testGuid1.ToString(),
+                CreatedGamerId = _testGamer1.Id.ToString(),
+                CreatedGamerNickname = _testGamer1.Nickname,
                 BoardGameId = 1
             },
             new GameResultDto
             {
                 Id = 2,
                 GameTableId = 2,
                 GamerId = _testGuid2.ToString(),
+                CreatedGamerId = _testGamer1.Id.ToString(),
+                CreatedGamerNickname = _testGamer1.Nickname,
                 BoardGameId = 1
             }
         };
@@ -169,12 +175,15 @@ public void AddManyGameResultToGameResultsList()
         {
             //Arrange
             _gameResultRepositoryMock.Setup(mock => mock.AddMany(It.IsAny<List<GameResult>>()));
+            _gameTableRepositoryMock.Setup(mock => mock.Get(1)).Returns(_testGameTable1);
+            _gameTableRepositoryMock.Setup(mock => mock.Get(2)).Returns(_testGameTable2);
+            _boardGameRepositoryMock.Setup(mock => mock.Get(It.IsAny<int>())).Returns(_testBoardGame);
             var gameResultService = new GameResultService(_gameResultRepositoryMock.Object, _gamerRepositoryMock.Object,
                 _gameTableRepositoryMock.Object, _boardGameRepositoryMock.Object);
             Mapper.Reset();
             Mapper.Initialize(cfg => { cfg.AddServicesProfiles(); });
             //Act
-            gameResultService.AddGameResults(_testGameResultDtoList, new Gamer());
+            gameResultService.AddGameResults(_testGameResultDtoList, _testGamer1);
             //Assert
             _gameResultRepositoryMock.Verify(
                 mock => mock.AddMany(It.Is<List<GameResult>>(