From 7639956edcbf9041b7a9d0143e13f280bd69eb48 Mon Sep 17 00:00:00 2001
From: Erik Ogenvik <erik@ogenvik.org>
Date: Sat, 25 Feb 2012 19:57:27 +0100
Subject: [PATCH] Don't test variants of all params.

---
 test/ElementExerciser.h      | 21 ++++++++++++---------
 test/ServerInfo_unittest.cpp |  1 +
 2 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/test/ElementExerciser.h b/test/ElementExerciser.h
index 49cae653..e03e95d1 100644
--- a/test/ElementExerciser.h
+++ b/test/ElementExerciser.h
@@ -22,9 +22,11 @@ class ElementParam
 {
 public:
     Atlas::Message::Element defaultElement;
+    bool testVariants;
 
-    ElementParam(Atlas::Message::Element defaultElement) :
-            defaultElement(defaultElement)
+    ElementParam(Atlas::Message::Element defaultElement, bool testVariants =
+            true) :
+            defaultElement(defaultElement), testVariants(testVariants)
     {
     }
 
@@ -136,14 +138,15 @@ class ElementExerciser
         //throwing an exception or crashing.
         for (std::map<std::string, ElementParam>::const_iterator I =
                 mElementParams.begin(); I != mElementParams.end(); ++I) {
-            for (size_t i = 0; i < (sizeof(types) / sizeof(*types)); ++i) {
-                Atlas::Message::Element::Type type = types[i];
-                EntityType entity;
-                fillAllCorrectParams(entity);
-                fillParam(entity, I->first, type);
-                mExerciseSlot(entity);
+            if (I->second.testVariants) {
+                for (size_t i = 0; i < (sizeof(types) / sizeof(*types)); ++i) {
+                    Atlas::Message::Element::Type type = types[i];
+                    EntityType entity;
+                    fillAllCorrectParams(entity);
+                    fillParam(entity, I->first, type);
+                    mExerciseSlot(entity);
+                }
             }
-
         }
 
     }
diff --git a/test/ServerInfo_unittest.cpp b/test/ServerInfo_unittest.cpp
index eca1fc16..064d2c00 100644
--- a/test/ServerInfo_unittest.cpp
+++ b/test/ServerInfo_unittest.cpp
@@ -59,6 +59,7 @@ int main()
             sigc::mem_fun(info, &Eris::TestServerInfo::injectAtlas));
     ElementExerciser<Atlas::Objects::Entity::RootEntity> exerciser(slot);
     exerciser.addParam("ruleset", ElementParam("mason"));
+    exerciser.addParam("name", ElementParam("a server", false));
     exerciser.addParam("clients", ElementParam(2));
     exerciser.addParam("server", ElementParam("localhost"));
     exerciser.addParam("uptime", ElementParam(1000.0f));