Merge pull request #15 from tacobell1896/fix/game-put

add fix to put method for games
tacobell1896 · May 19, 2024 · cd758a4 · cd758a4
2 parents b51926c + c66a6bc
commit cd758a4
Show file tree

Hide file tree

Showing 2 changed files with 50 additions and 43 deletions.
diff --git a/src/Controllers/SavePointGamesController.cs b/src/Controllers/SavePointGamesController.cs
@@ -54,22 +54,29 @@ public async Task<IActionResult> PutSavePointGame(int id, SavePointGameDTO saveP
                 return BadRequest();
             }
 
-            _context.Entry(savePointGameDTO).State = EntityState.Modified;
+            var savePointGame = await _context.SavePointGames.FindAsync(id);
+            if (savePointGame == null)
+            {
+                return NotFound();
+            }
+
+            savePointGame.GameName = savePointGameDTO.GameName;
+            savePointGame.GameConsole = savePointGameDTO.GameConsole;
+            savePointGame.GameGenre = savePointGameDTO.GameGenre;
+            savePointGame.GameDeveloper = savePointGameDTO.GameDeveloper;
+            savePointGame.GamePublisher = savePointGameDTO.GamePublisher;
+            savePointGame.GameReleaseDate = savePointGameDTO.GameReleaseDate;
+            savePointGame.GameDescription = savePointGameDTO.GameDescription;
+            savePointGame.GameRating = savePointGameDTO.GameRating;
+            savePointGame.GameImage = savePointGameDTO.GameImage;
 
             try
             {
                 await _context.SaveChangesAsync();
             }
-            catch (DbUpdateConcurrencyException)
+            catch (DbUpdateConcurrencyException) when (!SavePointGameExists(id))
             {
-                if (!SavePointGameExists(id))
-                {
-                    return NotFound();
-                }
-                else
-                {
-                    throw;
-                }
+                return NotFound();
             }
 
             return NoContent();

diff --git a/test/RequestTests.cs b/test/RequestTests.cs
@@ -66,39 +66,39 @@ public async void TestGetGameById()
     }
 
     // TOOD: Fix this test
-    // [Fact]
-    // public async void TestPutGame()
-    // {
-        // // Arrange
-        // var options = new DbContextOptionsBuilder<SavePointContext>()
-            // .UseInMemoryDatabase(databaseName: "SavePointGames")
-            // .Options;
-        // var context = new SavePointContext(options);
-        // var controller = new SavePointGamesController(context);
-
-        // // Mocking the SavePointGameDTO object
-        // var savePointGameDTO = new SavePointGameDTO
-        // {
-            // SavePointGameId = 1,
-            // GameName = "Test Game",
-            // GameGenre = "Test Genre",
-            // GameConsole = "Test Platform"
-        // };
-        // await controller.PostSavePointGame(savePointGameDTO);
-
-        // savePointGameDTO = new SavePointGameDTO
-        // {
-            // SavePointGameId = 1,
-            // GameName = "Test Game Updated",
-            // GameGenre = "Test Genre Updated",
-            // GameConsole = "Test Platform Updated"
-        // };
-        // // Act
-        // var result = await controller.PutSavePointGame(1, savePointGameDTO);
-
-        // // Assert
-        // Assert.IsType<NoContentResult>(result);
-    // }
+    [Fact]
+    public async void TestPutGame()
+    {
+        // Arrange
+        var options = new DbContextOptionsBuilder<SavePointContext>()
+            .UseInMemoryDatabase(databaseName: "SavePointGames")
+            .Options;
+        var context = new SavePointContext(options);
+        var controller = new SavePointGamesController(context);
+
+        // Mocking the SavePointGameDTO object
+        var savePointGameDTO = new SavePointGameDTO
+        {
+            SavePointGameId = 1,
+            GameName = "Test Game",
+            GameGenre = "Test Genre",
+            GameConsole = "Test Platform"
+        };
+        await controller.PostSavePointGame(savePointGameDTO);
+
+        savePointGameDTO = new SavePointGameDTO
+        {
+            SavePointGameId = 1,
+            GameName = "Test Game Updated",
+            GameGenre = "Test Genre Updated",
+            GameConsole = "Test Platform Updated"
+        };
+        // Act
+        var result = await controller.PutSavePointGame(1, savePointGameDTO);
+
+        // Assert
+        Assert.IsType<NoContentResult>(result);
+    }
 
     [Fact]
     public async void TestPostNote()