From acaa5321b6b340f635b584297b93c46983805f7e Mon Sep 17 00:00:00 2001
From: ADato88 <da-maschine@hotmail.de>
Date: Wed, 26 Jan 2022 17:23:18 +0100
Subject: [PATCH] refactor von ZiehEineKarteTest() und der Methode dazu

---
 .../MiniGames/Client/ViewModel/IUno.cs        |  3 ++-
 .../MiniGames/Client/ViewModel/Uno.cs         | 10 ++++----
 BlazorSolution/MiniGamesTests/UnoTest.cs      | 23 +++++++------------
 3 files changed, 15 insertions(+), 21 deletions(-)

diff --git a/BlazorSolution/MiniGames/Client/ViewModel/IUno.cs b/BlazorSolution/MiniGames/Client/ViewModel/IUno.cs
index bb4b870..0e5a0d5 100644
--- a/BlazorSolution/MiniGames/Client/ViewModel/IUno.cs
+++ b/BlazorSolution/MiniGames/Client/ViewModel/IUno.cs
@@ -1,4 +1,5 @@
 using MiniGames.Shared.Models;
+using System.Collections.Generic;
 
 namespace MiniGames.Client.ViewModel
 {
@@ -8,6 +9,6 @@ namespace MiniGames.Client.ViewModel
 
         void ZiehKartenStapelZusammenstellen();
         bool IstAblageStabelFarbeGleichHand(string ablegeStapelKarte, string neueKarte);
-        HandKartenModel ZiehEineKarte(HandKartenModel handStapel, StapelModel stapelModel);
+        List<KartenModel> ZiehEineKarte(List<KartenModel> zuZiehendeHand);
     }
 }
\ No newline at end of file
diff --git a/BlazorSolution/MiniGames/Client/ViewModel/Uno.cs b/BlazorSolution/MiniGames/Client/ViewModel/Uno.cs
index cc1bf83..d6f3cb6 100644
--- a/BlazorSolution/MiniGames/Client/ViewModel/Uno.cs
+++ b/BlazorSolution/MiniGames/Client/ViewModel/Uno.cs
@@ -22,14 +22,14 @@ namespace MiniGames.Client.ViewModel
             }
         }
 
-        public HandKartenModel ZiehEineKarte(HandKartenModel handStapel, StapelModel stapelModel)
+        public List<KartenModel> ZiehEineKarte(List<KartenModel> zuZiehendeHand)
         {
-            HandKartenModel _handStapel = handStapel;
-            StapelModel _stapelModel = stapelModel;
+            List<KartenModel> _zuZiehendeHand = zuZiehendeHand;
 
-            _handStapel.KartenModels.Add(_stapelModel.KartenModels.FirstOrDefault());
+            _zuZiehendeHand.Add(ZiehKartenStapel.KartenModels.FirstOrDefault());
+            ZiehKartenStapel.KartenModels.RemoveAt(0);
 
-            return _handStapel;
+            return _zuZiehendeHand;
         }
 
         public List<KartenModel> KarteAufStapelLegen(List<KartenModel> naechstenHand, KartenModel zuLegendeKarte)
diff --git a/BlazorSolution/MiniGamesTests/UnoTest.cs b/BlazorSolution/MiniGamesTests/UnoTest.cs
index 0d5c73a..b654123 100644
--- a/BlazorSolution/MiniGamesTests/UnoTest.cs
+++ b/BlazorSolution/MiniGamesTests/UnoTest.cs
@@ -1,6 +1,7 @@
 using MiniGames.Client.ViewModel;
 using MiniGames.Shared.Enums;
 using MiniGames.Shared.Models;
+using System.Collections.Generic;
 using System.Linq;
 using Xunit;
 
@@ -87,28 +88,20 @@ namespace MiniGamesTests
             Assert.Equal(_erwartet, erhalten);
         }
 
-        [Theory]
-        [InlineData(Kartenbezeichnung.PlusZweiKarte, Farbe.Blau, true)]
-        [InlineData((Kartenbezeichnung)1, Farbe.Gelb, false)]
-        [InlineData((Kartenbezeichnung)3, Farbe.Blau, false)]
-        [InlineData((Kartenbezeichnung)7, Farbe.Gruen, false)]
-        [InlineData(Kartenbezeichnung.FarbwahlKarte, Farbe.Keine, false)]
-        private void ZiehEineKarteTest(Kartenbezeichnung kartenbezeichnung, Farbe farbe, bool spezial)
+        [Fact]
+        private void ZiehEineKarteTest()
         {
             //arrange
-            var _kartenbezeichnung = kartenbezeichnung.ToString();
-            var _farbe = farbe.ToString();
-            var _spezial = spezial;
-            StapelModel zuFüllend = new();
+            UnoRegeln.ZiehKartenStapelZusammenstellen();
+            List<KartenModel> zuFüllend = new();
 
-            var beispielStapel = BeispielStapelModelKarten(_kartenbezeichnung, _farbe, _spezial);
             var beispielHand = BeispielHand();
+            var erwartet = UnoRegeln.ZiehKartenStapel.KartenModels.FirstOrDefault();
 
             //act
-            zuFüllend.KartenModels = UnoRegeln.ZiehEineKarte(beispielHand, beispielStapel).KartenModels;
+            zuFüllend = UnoRegeln.ZiehEineKarte(beispielHand.KartenModels);
 
-            var erwartet = beispielStapel.KartenModels.FirstOrDefault();
-            var erhalten = beispielHand.KartenModels.LastOrDefault();
+            var erhalten = zuFüllend.LastOrDefault();
 
             //assert
             Assert.Equal(erwartet, erhalten);