From d62579d6c4bf3d2ae0a5d7e696812dc8202de8bd Mon Sep 17 00:00:00 2001
From: "Chris St. Pierre" <chris.a.st.pierre@gmail.com>
Date: Thu, 16 Aug 2012 15:58:53 -0400
Subject: fixed some egregiously wrong Options tests

---
 src/lib/Bcfg2/Options.py | 21 ++++++++++-----------
 1 file changed, 10 insertions(+), 11 deletions(-)

(limited to 'src')

diff --git a/src/lib/Bcfg2/Options.py b/src/lib/Bcfg2/Options.py
index 3af5885bb..ef6807255 100644
--- a/src/lib/Bcfg2/Options.py
+++ b/src/lib/Bcfg2/Options.py
@@ -53,14 +53,6 @@ class DefaultConfigParser(ConfigParser.ConfigParser):
 
 
 class Option(object):
-    def get_cooked_value(self, value):
-        if self.boolean:
-            return True
-        if self.cook:
-            return self.cook(value)
-        else:
-            return value
-
     def __init__(self, desc, default, cmd=False, odesc=False,
                  env=False, cf=False, cook=False, long_arg=False,
                  deprecated_cf=None):
@@ -71,9 +63,8 @@ class Option(object):
         if not self.long:
             if cmd and (cmd[0] != '-' or len(cmd) != 2):
                 raise OptionFailure("Poorly formed command %s" % cmd)
-        else:
-            if cmd and (not cmd.startswith('--')):
-                raise OptionFailure("Poorly formed command %s" % cmd)
+        elif cmd and (not cmd.startswith('--')):
+            raise OptionFailure("Poorly formed command %s" % cmd)
         self.odesc = odesc
         self.env = env
         self.cf = cf
@@ -83,6 +74,14 @@ class Option(object):
             self.boolean = True
         self.cook = cook
 
+    def get_cooked_value(self, value):
+        if self.boolean:
+            return True
+        if self.cook:
+            return self.cook(value)
+        else:
+            return value
+
     def buildHelpMessage(self):
         vals = []
         if not self.cmd:
-- 
cgit v1.2.3-1-g7c22