Changed addcapability to reject all attempts to add the owner capability.

src/Admin.py

@@ -139,8 +139,13 @@ class Admin(privmsgs.CapabilityCheckingPrivmsg):
         Gives the user specified by <name> (or the user to whom <hostmask>
         currently maps) the specified capability <capability>
         """
-        (name, capability) = privmsgs.getArgs(args, 2)
+        (name, capability) = privmsgs.getArgs(args, required=2)
-        # This next check to make sure 'admin's can't hand out 'owner'.
+        if capability == 'owner':
+            irc.error(msg, 'The "owner" capability can\'t be added in the bot.'
+                           '  Use the supybot-adduser program (or edit the '
+                           'users.conf file yourself) to add an owner '
+                           'capability.')
+            return
         if ircdb.checkCapability(msg.prefix, capability) or \
            '!' in capability:
             try:

test/test_Admin.py

@@ -117,6 +117,9 @@ class AdminTestCase(PluginTestCase, PluginDocumentation):
         m = self.getMsg('nick foobar')
         self.assertEqual(m.command, 'NICK')
         self.assertEqual(m.args[0], 'foobar')
+
+    def testAddCapabilityOwner(self):
+        self.assertError('admin addcapability %s owner' % self.nick)