Add set tests. Test and fix table command.

admin.py:do_set:
  Better document use of classname in set command.

  Remove traceback.print_exc() that generates stderr index error which
  duplicates UsageError and can confuse the user.

admin.py:do_table:

  Fix null width spec to be length of label not len(label)+1

  Handle exception if width is not an int.

  Outdent code to proper location. Was inside loop and should not have
  been.

test_admin.py:

  Add tests for get with bad node designator

  Add disabled test for HelpInitopts. Proper test TBD as valid result
     depends on the environment the test is run in. So making it
     robust it tricky.

  Add tests for set error cases.

  Add test for using set on class rather than designator

  Add new tests for table command

Author: rouilj

roundup/admin.py

@@ -660,10 +660,13 @@ def do_set(self, args):
         A designator is a classname and a nodeid concatenated,
         eg. bug1, user10, ...
 
-        This command sets the properties to the values for all designators
-        given. If the value is missing (ie. "property=") then the property
-        is un-set. If the property is a multilink, you specify the linked
-        ids for the multilink as comma-separated numbers (ie "1,2,3").
+        This command sets the properties to the values for all
+        designators given. If a class is used, the property will be
+        set for all items in the class. If the value is missing
+        (ie. "property=") then the property is un-set. If the property
+        is a multilink, you specify the linked ids for the multilink
+        as comma-separated numbers (ie "1,2,3").
+
         """
         import copy  # needed for copying props list
 
@@ -711,7 +714,6 @@ def do_set(self, args):
             try:
                 cl.set(itemid, **props)
             except (TypeError, IndexError, ValueError) as message:
-                import traceback; traceback.print_exc()
                 raise UsageError(message)
         self.db_uncommitted = True
         return 0
@@ -1073,17 +1075,23 @@ def do_table(self, args):
             if ':' in spec:
                 name, width = spec.split(':')
                 if width == '':
-                    props.append((name, len(spec)))
+                    # spec includes trailing :, use label/name width 
+                    props.append((name, len(name)))
                 else:
-                    props.append((name, int(width)))
+                    try:
+                        props.append((name, int(width)))
+                    except ValueError:
+                        raise UsageError(_('"%(spec)s" does not have an '
+                                           'integer width: "%(width)s"') %
+                                         locals())
             else:
                 # this is going to be slow
                 maxlen = len(spec)
                 for nodeid in cl.list():
                     curlen = len(str(cl.get(nodeid, spec)))
                     if curlen > maxlen:
                         maxlen = curlen
-                    props.append((spec, maxlen))
+                props.append((spec, maxlen))
 
         # now display the heading
         print(' '.join([name.capitalize().ljust(width)