From 6063ddcf554802cbdab20f91e62e1673420bcd48 Mon Sep 17 00:00:00 2001
From: "wagic.jeck" <wagic.jeck@01922e74-a4c3-11dd-a5cd-697bfd0a0cef>
Date: Sun, 12 Sep 2010 05:10:10 +0000
Subject: [PATCH] Jeck - Note to self: next time you write a unit test, check
 it better.

---
 projects/mtg/src/GameStateShop.cpp | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/projects/mtg/src/GameStateShop.cpp b/projects/mtg/src/GameStateShop.cpp
index 05e54cc1c..d852845bc 100644
--- a/projects/mtg/src/GameStateShop.cpp
+++ b/projects/mtg/src/GameStateShop.cpp
@@ -806,6 +806,7 @@ bool ShopBooster::unitTest(){
     mP->pool = buf;
     mP->assemblePack(d); //Use the primary packfile. assemblePack deletes pool.
     DeckDataWrapper* ddw = NEW DeckDataWrapper(d);
+    bool res = true;
 
     int u = 0, r = 0;
     int card = 0;
@@ -821,15 +822,17 @@ bool ShopBooster::unitTest(){
     if(r != 1 || u != 3 ){
         sprintf(result, "<span class=\"error\">==Unexpected rarity count==</span><br />");
         TestSuite::Log(result);
-        return false;
+        res = false;
     }
     if(ddw->getCount() < 14) {
         sprintf(result, "<span class=\"error\">==Unexpected card count==</span><br />");
         TestSuite::Log(result);
-        return false;
+        res = false;
     }
     sprintf(result, "<span class=\"success\">==Test Succesful !==</span><br />");
     TestSuite::Log(result);
-    return true;
+    SAFE_DELETE(ddw);
+    SAFE_DELETE(d);
+    return res;
 }
  #endif
\ No newline at end of file