diff options
author | Nathan Fritz <fritzy@netflint.net> | 2010-01-23 01:08:21 -0800 |
---|---|---|
committer | Nathan Fritz <fritzy@netflint.net> | 2010-01-23 01:08:21 -0800 |
commit | 3dbd496de8b6d358b657eada91acc6b71455ffbf (patch) | |
tree | ab7615b372d9c1206ea44e16da984abdcb5eb425 | |
parent | 65dd83d4e174fc5f88ea90b32dd9e3ebf1787294 (diff) | |
download | slixmpp-3dbd496de8b6d358b657eada91acc6b71455ffbf.tar.gz slixmpp-3dbd496de8b6d358b657eada91acc6b71455ffbf.tar.bz2 slixmpp-3dbd496de8b6d358b657eada91acc6b71455ffbf.tar.xz slixmpp-3dbd496de8b6d358b657eada91acc6b71455ffbf.zip |
fixed presence['type'] w/ show value bug and added test to prove
-rw-r--r-- | sleekxmpp/stanza/presence.py | 22 | ||||
-rw-r--r-- | testall.py | 4 |
2 files changed, 14 insertions, 12 deletions
diff --git a/sleekxmpp/stanza/presence.py b/sleekxmpp/stanza/presence.py index 6a8247c5..2bf767d4 100644 --- a/sleekxmpp/stanza/presence.py +++ b/sleekxmpp/stanza/presence.py @@ -15,18 +15,18 @@ class Presence(RootStanza): return self.xml.find("{%s}show" % self.namespace) def setType(self, value): + show = self.getShowElement() if value in self.types: - show = self.getShowElement() - if value in self.types: - if show is not None: - self.xml.remove(show) - if value == 'available': - value = '' - self._setAttr('type', value) - elif value in self.showtypes: - if show is None: - show = ET.Element("{%s}show" % self.namespace) - show.text = value + if show is not None: + self.xml.remove(show) + if value == 'available': + value = '' + self._setAttr('type', value) + elif value in self.showtypes: + if show is None: + show = ET.Element("{%s}show" % self.namespace) + self.xml.append(show) + show.text = value return self def setPriority(self, value): @@ -16,6 +16,7 @@ class testoverall(unittest.TestCase): """Invoking the tabnanny""" import tabnanny self.failIf(tabnanny.check("." + os.sep + 'sleekxmpp')) + #raise "Help!" def testMethodLength(self): """Testing for excessive method lengths""" @@ -62,4 +63,5 @@ if __name__ == '__main__': #sys.modules[modname].config = moduleconfig alltests.append(sys.modules[modname].suite) alltests_suite = unittest.TestSuite(alltests) - unittest.TextTestRunner(verbosity=2).run(alltests_suite) + result = unittest.TextTestRunner(verbosity=2).run(alltests_suite) + print("""<tests xmlns='http://andyet.net/protocol/tests' ran='%s' errors='%s' fails='%s' success='%s' />""" % (result.testsRun, len(result.errors), len(result.failures), result.wasSuccessful())) |