Manual cleanup of the remaining set([…]) and set((…)).

docs/create_plugin.rst

@@ -163,7 +163,7 @@ behaviour:
         namespace = 'jabber:iq:register'
         name = 'query'
         plugin_attrib = 'register'
-        interfaces = set(('username', 'password', 'registered', 'remove'))
+        interfaces = {'username', 'password', 'registered', 'remove'}
         sub_interfaces = interfaces
 
         def getRegistered(self):
@@ -535,10 +535,10 @@ with some additional registration fields implemented.
         namespace = 'jabber:iq:register'
         name = 'query'
         plugin_attrib = 'register'
-        interfaces = set(('username', 'password', 'email', 'nick', 'name',
+        interfaces = {'username', 'password', 'email', 'nick', 'name',
                       'first', 'last', 'address', 'city', 'state', 'zip',
                       'phone', 'url', 'date', 'misc', 'text', 'key',
-                          'registered', 'remove', 'instructions'))
+                      'registered', 'remove', 'instructions'}
         sub_interfaces = interfaces
 
         def getRegistered(self):

slixmpp/features/feature_mechanisms/mechanisms.py

@@ -49,7 +49,7 @@ class FeatureMechanisms(BasePlugin):
         if self.security_callback is None:
             self.security_callback = self._default_security
 
-        creds = self.sasl_callback(set(['username']), set())
+        creds = self.sasl_callback({'username'}, set())
         if not self.use_mech and not creds['username']:
             self.use_mech = 'ANONYMOUS'
 

slixmpp/features/feature_mechanisms/stanza/failure.py

@@ -19,10 +19,11 @@ class Failure(StanzaBase):
     interfaces = {'condition', 'text'}
     plugin_attrib = name
     sub_interfaces = {'text'}
-    conditions = set(('aborted', 'account-disabled', 'credentials-expired',
+    conditions = {'aborted', 'account-disabled', 'credentials-expired',
-        'encryption-required', 'incorrect-encoding', 'invalid-authzid',
+                  'encryption-required', 'incorrect-encoding',
-        'invalid-mechanism', 'malformed-request', 'mechansism-too-weak',
+                  'invalid-authzid', 'invalid-mechanism', 'malformed-request',
-        'not-authorized', 'temporary-auth-failure'))
+                  'mechansism-too-weak', 'not-authorized',
+                  'temporary-auth-failure'}
 
     def setup(self, xml=None):
         """

slixmpp/plugins/gmail_notify.py

@@ -37,8 +37,8 @@ class MailBox(ElementBase):
     namespace = 'google:mail:notify'
     name = 'mailbox'
     plugin_attrib = 'mailbox'
-    interfaces = set(('result-time', 'total-matched', 'total-estimate',
+    interfaces = {'result-time', 'total-matched', 'total-estimate',
-                      'url', 'threads', 'matched', 'estimate'))
+                  'url', 'threads', 'matched', 'estimate'}
 
     def get_threads(self):
         threads = []
@@ -58,8 +58,8 @@ class MailThread(ElementBase):
     namespace = 'google:mail:notify'
     name = 'mail-thread-info'
     plugin_attrib = 'thread'
-    interfaces = set(('tid', 'participation', 'messages', 'date',
+    interfaces = {'tid', 'participation', 'messages', 'date',
-                      'senders', 'url', 'labels', 'subject', 'snippet'))
+                  'senders', 'url', 'labels', 'subject', 'snippet'}
     sub_interfaces = {'labels', 'subject', 'snippet'}
 
     def get_senders(self):

slixmpp/plugins/xep_0004/stanza/field.py

@@ -14,21 +14,21 @@ class FormField(ElementBase):
     name = 'field'
     plugin_attrib = 'field'
     plugin_multi_attrib = 'fields'
-    interfaces = set(('answer', 'desc', 'required', 'value',
+    interfaces = {'answer', 'desc', 'required', 'value',
-                      'label', 'type', 'var'))
+                  'label', 'type', 'var'}
     sub_interfaces = {'desc'}
     plugin_tag_map = {}
     plugin_attrib_map = {}
 
-    field_types = set(('boolean', 'fixed', 'hidden', 'jid-multi',
+    field_types = {'boolean', 'fixed', 'hidden', 'jid-multi',
                    'jid-single', 'list-multi', 'list-single',
-                       'text-multi', 'text-private', 'text-single'))
+                   'text-multi', 'text-private', 'text-single'}
 
     true_values = {True, '1', 'true'}
     option_types = {'list-multi', 'list-single'}
     multi_line_types = {'hidden', 'text-multi'}
-    multi_value_types = set(('hidden', 'jid-multi',
+    multi_value_types = {'hidden', 'jid-multi',
-                             'list-multi', 'text-multi'))
+                         'list-multi', 'text-multi'}
 
     def setup(self, xml=None):
         if ElementBase.setup(self, xml):

slixmpp/plugins/xep_0016/stanza.py

@@ -55,8 +55,8 @@ class Item(ElementBase):
     namespace = 'jabber:iq:privacy'
     plugin_attrib = name
     plugin_multi_attrib = 'items'
-    interfaces = set(['type', 'value', 'action', 'order', 'iq',
+    interfaces = {'type', 'value', 'action', 'order', 'iq',
-                      'message', 'presence_in', 'presence_out'])
+                  'message', 'presence_in', 'presence_out'}
     bool_interfaces = {'message', 'iq', 'presence_in', 'presence_out'}
 
     type_values = ('', 'jid', 'group', 'subscription')

slixmpp/plugins/xep_0050/adhoc.py

@@ -322,8 +322,8 @@ class XEP_0050(BasePlugin):
         interfaces = session.get('interfaces', set())
         payload_classes = session.get('payload_classes', set())
 
-        interfaces.update(set([item.plugin_attrib for item in payload]))
+        interfaces.update({item.plugin_attrib for item in payload})
-        payload_classes.update(set([item.__class__ for item in payload]))
+        payload_classes.update({item.__class__ for item in payload})
 
         session['interfaces'] = interfaces
         session['payload_classes'] = payload_classes

slixmpp/plugins/xep_0050/stanza.py

@@ -72,8 +72,8 @@ class Command(ElementBase):
     name = 'command'
     namespace = 'http://jabber.org/protocol/commands'
     plugin_attrib = 'command'
-    interfaces = set(('action', 'sessionid', 'node',
+    interfaces = {'action', 'sessionid', 'node',
-                      'status', 'actions', 'notes'))
+                  'status', 'actions', 'notes'}
     actions = {'cancel', 'complete', 'execute', 'next', 'prev'}
     statuses = {'canceled', 'completed', 'executing'}
     next_actions = {'prev', 'next', 'complete'}

slixmpp/plugins/xep_0054/stanza.py

@@ -105,11 +105,11 @@ class Address(ElementBase):
     namespace = 'vcard-temp'
     plugin_attrib = name
     plugin_multi_attrib = 'addresses'
-    interfaces = set(['HOME', 'WORK', 'POSTAL', 'PARCEL', 'DOM', 'INTL',
+    interfaces = {'HOME', 'WORK', 'POSTAL', 'PARCEL', 'DOM', 'INTL',
                   'PREF', 'POBOX', 'EXTADD', 'STREET', 'LOCALITY',
-                      'REGION', 'PCODE', 'CTRY'])
+                  'REGION', 'PCODE', 'CTRY'}
-    sub_interfaces = set(['POBOX', 'EXTADD', 'STREET', 'LOCALITY',
+    sub_interfaces = {'POBOX', 'EXTADD', 'STREET', 'LOCALITY',
-                          'REGION', 'PCODE', 'CTRY'])
+                      'REGION', 'PCODE', 'CTRY'}
     bool_interfaces = {'HOME', 'WORK', 'DOM', 'INTL', 'PREF'}
 
 
@@ -118,13 +118,13 @@ class Telephone(ElementBase):
     namespace = 'vcard-temp'
     plugin_attrib = name
     plugin_multi_attrib = 'telephone_numbers'
-    interfaces = set(['HOME', 'WORK', 'VOICE', 'FAX', 'PAGER', 'MSG',
+    interfaces = {'HOME', 'WORK', 'VOICE', 'FAX', 'PAGER', 'MSG',
                   'CELL', 'VIDEO', 'BBS', 'MODEM', 'ISDN', 'PCS',
-                      'PREF', 'NUMBER'])
+                  'PREF', 'NUMBER'}
     sub_interfaces = {'NUMBER'}
-    bool_interfaces = set(['HOME', 'WORK', 'VOICE', 'FAX', 'PAGER',
+    bool_interfaces = {'HOME', 'WORK', 'VOICE', 'FAX', 'PAGER',
                        'MSG', 'CELL', 'VIDEO', 'BBS', 'MODEM',
-                           'ISDN', 'PCS', 'PREF'])
+                       'ISDN', 'PCS', 'PREF'}
 
     def setup(self, xml=None):
         super().setup(xml=xml)
@@ -143,10 +143,10 @@ class Label(ElementBase):
     namespace = 'vcard-temp'
     plugin_attrib = name
     plugin_multi_attrib = 'labels'
-    interfaces = set(['HOME', 'WORK', 'POSTAL', 'PARCEL', 'DOM', 'INT',
+    interfaces = {'HOME', 'WORK', 'POSTAL', 'PARCEL', 'DOM', 'INT',
-                      'PREF', 'lines'])
+                  'PREF', 'lines'}
-    bool_interfaces = set(['HOME', 'WORK', 'POSTAL', 'PARCEL', 'DOM',
+    bool_interfaces = {'HOME', 'WORK', 'POSTAL', 'PARCEL', 'DOM',
-                           'INT', 'PREF'])
+                       'INT', 'PREF'}
 
     def add_line(self, value):
         line = ET.Element('{%s}LINE' % self.namespace)

slixmpp/plugins/xep_0059/stanza.py

@@ -64,10 +64,10 @@ class Set(ElementBase):
     namespace = 'http://jabber.org/protocol/rsm'
     name = 'set'
     plugin_attrib = 'rsm'
-    sub_interfaces = set(('first', 'after', 'before', 'count',
+    sub_interfaces = {'first', 'after', 'before', 'count',
-                          'index', 'last', 'max'))
+                      'index', 'last', 'max'}
-    interfaces = set(('first_index', 'first', 'after', 'before',
+    interfaces = {'first_index', 'first', 'after', 'before',
-                      'count', 'index', 'last', 'max'))
+                  'count', 'index', 'last', 'max'}
 
     def set_first_index(self, val):
         fi = self.xml.find("{%s}first" % (self.namespace))

slixmpp/plugins/xep_0060/stanza/pubsub_errors.py

@@ -16,7 +16,7 @@ class PubsubErrorCondition(ElementBase):
     interfaces = {'condition', 'unsupported'}
     plugin_attrib_map = {}
     plugin_tag_map = {}
-    conditions = set(('closed-node', 'configuration-required', 'invalid-jid',
+    conditions = {'closed-node', 'configuration-required', 'invalid-jid',
                   'invalid-options', 'invalid-payload', 'invalid-subid',
                   'item-forbidden', 'item-required', 'jid-required',
                   'max-items-exceeded', 'max-nodes-exceeded',
@@ -24,7 +24,7 @@ class PubsubErrorCondition(ElementBase):
                   'not-subscribed', 'payload-too-big',
                   'payload-required', 'pending-subscription',
                   'presence-subscription-required', 'subid-required',
-                      'too-many-subscriptions', 'unsupported'))
+                  'too-many-subscriptions', 'unsupported'}
     condition_ns = 'http://jabber.org/protocol/pubsub#errors'
 
     def setup(self, xml):

slixmpp/plugins/xep_0077/stanza.py

@@ -16,14 +16,14 @@ class Register(ElementBase):
     namespace = 'jabber:iq:register'
     name = 'query'
     plugin_attrib = 'register'
-    interfaces = set(('username', 'password', 'email', 'nick', 'name',
+    interfaces = {'username', 'password', 'email', 'nick', 'name',
                   'first', 'last', 'address', 'city', 'state', 'zip',
                   'phone', 'url', 'date', 'misc', 'text', 'key',
-                      'registered', 'remove', 'instructions', 'fields'))
+                  'registered', 'remove', 'instructions', 'fields'}
     sub_interfaces = interfaces
-    form_fields = set(('username', 'password', 'email', 'nick', 'name',
+    form_fields = {'username', 'password', 'email', 'nick', 'name',
                    'first', 'last', 'address', 'city', 'state', 'zip',
-                       'phone', 'url', 'date', 'misc', 'text', 'key'))
+                   'phone', 'url', 'date', 'misc', 'text', 'key'}
 
     def get_registered(self):
         present = self.xml.find('{%s}registered' % self.namespace)

slixmpp/plugins/xep_0080/stanza.py

@@ -65,11 +65,11 @@ class Geoloc(ElementBase):
 
     namespace = 'http://jabber.org/protocol/geoloc'
     name = 'geoloc'
-    interfaces = set(('accuracy', 'alt', 'area', 'bearing', 'building',
+    interfaces = {'accuracy', 'alt', 'area', 'bearing', 'building',
                   'country', 'countrycode', 'datum', 'dscription',
                   'error', 'floor', 'lat', 'locality', 'lon',
                   'postalcode', 'region', 'room', 'speed', 'street',
-                      'text', 'timestamp', 'uri'))
+                  'text', 'timestamp', 'uri'}
     sub_interfaces = interfaces
     plugin_attrib = name
 

slixmpp/plugins/xep_0107/stanza.py

@@ -16,7 +16,7 @@ class UserMood(ElementBase):
     plugin_attrib = 'mood'
     interfaces = {'value', 'text'}
     sub_interfaces = {'text'}
-    moods = set(['afraid', 'amazed', 'amorous', 'angry', 'annoyed', 'anxious',
+    moods = {'afraid', 'amazed', 'amorous', 'angry', 'annoyed', 'anxious',
              'aroused', 'ashamed', 'bored', 'brave', 'calm', 'cautious',
              'cold', 'confident', 'confused', 'contemplative', 'contented',
              'cranky', 'crazy', 'creative', 'curious', 'dejected',
@@ -32,7 +32,7 @@ class UserMood(ElementBase):
              'sad', 'sarcastic', 'satisfied', 'serious', 'shocked',
              'shy', 'sick', 'sleepy', 'spontaneous', 'stressed', 'strong',
              'surprised', 'thankful', 'thirsty', 'tired', 'undefined',
-                 'weak', 'worried'])
+             'weak', 'worried'}
 
     def set_value(self, value):
         self.del_value()

slixmpp/plugins/xep_0108/stanza.py

@@ -16,10 +16,10 @@ class UserActivity(ElementBase):
     plugin_attrib = 'activity'
     interfaces = {'value', 'text'}
     sub_interfaces = {'text'}
-    general = set(['doing_chores', 'drinking', 'eating', 'exercising',
+    general = {'doing_chores', 'drinking', 'eating', 'exercising',
                'grooming', 'having_appointment', 'inactive', 'relaxing',
-                   'talking', 'traveling', 'undefined', 'working'])
+               'talking', 'traveling', 'undefined', 'working'}
-    specific = set(['at_the_spa', 'brushing_teeth', 'buying_groceries',
+    specific = {'at_the_spa', 'brushing_teeth', 'buying_groceries',
                 'cleaning', 'coding', 'commuting', 'cooking', 'cycling',
                 'dancing', 'day_off', 'doing_maintenance',
                 'doing_the_dishes', 'doing_the_laundry', 'driving',
@@ -37,7 +37,7 @@ class UserActivity(ElementBase):
                 'socializing', 'studying', 'sunbathing', 'swimming',
                 'taking_a_bath', 'taking_a_shower', 'thinking',
                 'walking', 'walking_the_dog', 'watching_a_movie',
-                    'watching_tv', 'working_out', 'writing'])
+                'watching_tv', 'working_out', 'writing'}
 
     def set_value(self, value):
         self.del_value()

slixmpp/plugins/xep_0118/stanza.py

@@ -14,8 +14,8 @@ class UserTune(ElementBase):
     name = 'tune'
     namespace = 'http://jabber.org/protocol/tune'
     plugin_attrib = 'tune'
-    interfaces = set(['artist', 'length', 'rating', 'source',
+    interfaces = {'artist', 'length', 'rating', 'source',
-                      'title', 'track', 'uri'])
+                  'title', 'track', 'uri'}
     sub_interfaces = interfaces
 
     def set_length(self, value):

slixmpp/plugins/xep_0133.py

@@ -15,7 +15,7 @@ class XEP_0133(BasePlugin):
     name = 'xep_0133'
     description = 'XEP-0133: Service Administration'
     dependencies = {'xep_0030', 'xep_0004', 'xep_0050'}
-    commands = set(['add-user', 'delete-user', 'disable-user',
+    commands = {'add-user', 'delete-user', 'disable-user',
                 'reenable-user', 'end-user-session', 'get-user-password',
                 'change-user-password', 'get-user-roster',
                 'get-user-lastlogin', 'user-stats', 'edit-blacklist',
@@ -26,7 +26,7 @@ class XEP_0133(BasePlugin):
                 'get-online-users-list', 'get-online-users',
                 'get-active-users', 'get-idle-userslist', 'announce',
                 'set-motd', 'edit-motd', 'delete-motd', 'set-welcome',
-                    'delete-welcome', 'edit-admin', 'restart', 'shutdown'])
+                'delete-welcome', 'edit-admin', 'restart', 'shutdown'}
 
     def get_commands(self, jid=None, **kwargs):
         if jid is None:

slixmpp/plugins/xep_0196/stanza.py

@@ -14,7 +14,7 @@ class UserGaming(ElementBase):
     name = 'gaming'
     namespace = 'urn:xmpp:gaming:0'
     plugin_attrib = 'gaming'
-    interfaces = set(['character_name', 'character_profile', 'name',
+    interfaces = {'character_name', 'character_profile', 'name',
-                        'level', 'server_address', 'server_name', 'uri'])
+                  'level', 'server_address', 'server_name', 'uri'}
     sub_interfaces = interfaces
 

slixmpp/plugins/xep_0235/stanza.py

@@ -19,9 +19,9 @@ class OAuth(ElementBase):
     name = 'oauth'
     namespace = 'urn:xmpp:oauth:0'
     plugin_attrib = 'oauth'
-    interfaces = set(['oauth_consumer_key', 'oauth_nonce', 'oauth_signature',
+    interfaces = {'oauth_consumer_key', 'oauth_nonce', 'oauth_signature',
-                      'oauth_signature_method', 'oauth_timestamp',
+                  'oauth_signature_method', 'oauth_timestamp', 'oauth_token',
-                      'oauth_token', 'oauth_version'])
+                  'oauth_version'}
     sub_interfaces = interfaces
 
     def generate_signature(self, stanza, sfrom, sto, consumer_secret,

slixmpp/plugins/xep_0242.py

@@ -13,9 +13,9 @@ class XEP_0242(BasePlugin):
 
     name = 'xep_0242'
     description = 'XEP-0242: XMPP Client Compliance 2009'
-    dependencies = set(['xep_0030', 'xep_0115', 'xep_0054',
+    dependencies = {'xep_0030', 'xep_0115', 'xep_0054',
                     'xep_0045', 'xep_0085', 'xep_0016',
-                        'xep_0191'])
+                    'xep_0191'}
 
 
 register_plugin(XEP_0242)

slixmpp/plugins/xep_0270.py

@@ -13,8 +13,8 @@ class XEP_0270(BasePlugin):
 
     name = 'xep_0270'
     description = 'XEP-0270: XMPP Compliance Suites 2010'
-    dependencies = set(['xep_0030', 'xep_0115', 'xep_0054',
+    dependencies = {'xep_0030', 'xep_0115', 'xep_0054',
-                        'xep_0163', 'xep_0045', 'xep_0085'])
+                    'xep_0163', 'xep_0045', 'xep_0085'}
 
 
 register_plugin(XEP_0270)

slixmpp/plugins/xep_0302.py

@@ -13,9 +13,9 @@ class XEP_0302(BasePlugin):
 
     name = 'xep_0302'
     description = 'XEP-0302: XMPP Compliance Suites 2012'
-    dependencies = set(['xep_0030', 'xep_0115', 'xep_0054',
+    dependencies = {'xep_0030', 'xep_0115', 'xep_0054',
                     'xep_0163', 'xep_0045', 'xep_0085',
-                        'xep_0184', 'xep_0198'])
+                    'xep_0184', 'xep_0198'}
 
 
 register_plugin(XEP_0302)

slixmpp/plugins/xep_0323/stanza/sensordata.py

@@ -17,21 +17,25 @@ class Sensordata(ElementBase):
     namespace = 'urn:xmpp:iot:sensordata'
     name = 'sensordata'
     plugin_attrib = name
-    interfaces = set(tuple())
+    interfaces = set()
 
 class FieldTypes():
     """
     All field types are optional booleans that default to False
     """
-    field_types = set([ 'momentary','peak','status','computed','identity','historicalSecond','historicalMinute','historicalHour', \
+    field_types = {'momentary', 'peak', 'status', 'computed', 'identity',
-                        'historicalDay','historicalWeek','historicalMonth','historicalQuarter','historicalYear','historicalOther'])
+                   'historicalSecond', 'historicalMinute', 'historicalHour',
+                   'historicalDay', 'historicalWeek', 'historicalMonth',
+                   'historicalQuarter', 'historicalYear', 'historicalOther'}
 
 class FieldStatus():
     """
     All field statuses are optional booleans that default to False
     """
-    field_status = set([ 'missing','automaticEstimate','manualEstimate','manualReadout','automaticReadout','timeOffset','warning','error', \
+    field_status = {'missing', 'automaticEstimate', 'manualEstimate',
-                         'signed','invoiced','endOfSeries','powerFailure','invoiceConfirmed'])
+                    'manualReadout', 'automaticReadout', 'timeOffset',
+                    'warning', 'error', 'signed', 'invoiced', 'endOfSeries',
+                    'powerFailure', 'invoiceConfirmed'}
 
 class Request(ElementBase):
     namespace = 'urn:xmpp:iot:sensordata'

slixmpp/plugins/xep_0332/http.py

@@ -40,7 +40,7 @@ class XEP_0332(BasePlugin):
 
     #: TODO: Do we really need to mention the supported_headers?!
     default_config = {
-        'supported_headers': set([
+        'supported_headers': {
             'Content-Length', 'Transfer-Encoding', 'DateTime',
             'Accept-Charset', 'Location', 'Content-ID', 'Description',
             'Content-Language', 'Content-Transfer-Encoding', 'Timestamp',
@@ -48,7 +48,7 @@ class XEP_0332(BasePlugin):
             'WWW-Authenticate', 'Accept-Encoding', 'Server', 'Error-Info',
             'Identifier', 'Content-Location', 'Content-Encoding', 'Distribute',
             'Accept', 'Proxy-Authenticate', 'ETag', 'Expect', 'Content-Type'
-        ])
+        }
     }
 
     def plugin_init(self):

slixmpp/stanza/atom.py

@@ -22,8 +22,7 @@ class AtomEntry(ElementBase):
     name = 'entry'
     plugin_attrib = 'entry'
     interfaces = {'title', 'summary', 'id', 'published', 'updated'}
-    sub_interfaces = set(('title', 'summary', 'id', 'published',
+    sub_interfaces = {'title', 'summary', 'id', 'published', 'updated'}
-                          'updated'))
 
 class AtomAuthor(ElementBase):
 

slixmpp/stanza/error.py

@@ -51,12 +51,12 @@ class Error(ElementBase):
     namespace = 'jabber:client'
     name = 'error'
     plugin_attrib = 'error'
-    interfaces = set(('code', 'condition', 'text', 'type',
+    interfaces = {'code', 'condition', 'text', 'type',
-                      'gone', 'redirect', 'by'))
+                      'gone', 'redirect', 'by'}
     sub_interfaces = {'text'}
     plugin_attrib_map = {}
     plugin_tag_map = {}
-    conditions = set(('bad-request', 'conflict', 'feature-not-implemented',
+    conditions = {'bad-request', 'conflict', 'feature-not-implemented',
                   'forbidden', 'gone', 'internal-server-error',
                   'item-not-found', 'jid-malformed', 'not-acceptable',
                   'not-allowed', 'not-authorized', 'payment-required',
@@ -64,7 +64,7 @@ class Error(ElementBase):
                   'registration-required', 'remote-server-not-found',
                   'remote-server-timeout', 'resource-constraint',
                   'service-unavailable', 'subscription-required',
-                      'undefined-condition', 'unexpected-request'))
+                  'undefined-condition', 'unexpected-request'}
     condition_ns = 'urn:ietf:params:xml:ns:xmpp-stanzas'
     types = {'cancel', 'continue', 'modify', 'auth', 'wait'}
 

slixmpp/stanza/message.py

@@ -47,8 +47,8 @@ class Message(RootStanza):
     name = 'message'
     namespace = 'jabber:client'
     plugin_attrib = name
-    interfaces = set(['type', 'to', 'from', 'id', 'body', 'subject',
+    interfaces = {'type', 'to', 'from', 'id', 'body', 'subject', 'thread',
-                      'thread', 'parent_thread', 'mucroom', 'mucnick'])
+                  'parent_thread', 'mucroom', 'mucnick'}
     sub_interfaces = {'body', 'subject', 'thread'}
     lang_interfaces = sub_interfaces
     types = {'normal', 'chat', 'headline', 'error', 'groupchat'}

slixmpp/stanza/presence.py

@@ -56,13 +56,12 @@ class Presence(RootStanza):
     name = 'presence'
     namespace = 'jabber:client'
     plugin_attrib = name
-    interfaces = set(['type', 'to', 'from', 'id', 'show',
+    interfaces = {'type', 'to', 'from', 'id', 'show', 'status', 'priority'}
-                      'status', 'priority'])
     sub_interfaces = {'show', 'status', 'priority'}
     lang_interfaces = {'status'}
 
-    types = set(['available', 'unavailable', 'error', 'probe', 'subscribe',
+    types = {'available', 'unavailable', 'error', 'probe', 'subscribe',
-                 'subscribed', 'unsubscribe', 'unsubscribed'])
+             'subscribed', 'unsubscribe', 'unsubscribed'}
     showtypes = {'dnd', 'chat', 'xa', 'away'}
 
     def __init__(self, *args, **kwargs):

slixmpp/stanza/roster.py

@@ -118,8 +118,7 @@ class RosterItem(ElementBase):
     namespace = 'jabber:iq:roster'
     name = 'item'
     plugin_attrib = 'item'
-    interfaces = set(('jid', 'name', 'subscription', 'ask',
+    interfaces = {'jid', 'name', 'subscription', 'ask', 'approved', 'groups'}
-                      'approved', 'groups'))
 
     def get_jid(self):
         return JID(self._get_attr('jid', ''))

slixmpp/stanza/stream_error.py

@@ -55,7 +55,7 @@ class StreamError(Error, StanzaBase):
 
     namespace = 'http://etherx.jabber.org/streams'
     interfaces = {'condition', 'text', 'see_other_host'}
-    conditions = set((
+    conditions = {
         'bad-format', 'bad-namespace-prefix', 'conflict',
         'connection-timeout', 'host-gone', 'host-unknown',
         'improper-addressing', 'internal-server-error', 'invalid-from',
@@ -64,7 +64,7 @@ class StreamError(Error, StanzaBase):
         'reset', 'resource-constraint', 'restricted-xml', 'see-other-host',
         'system-shutdown', 'undefined-condition', 'unsupported-encoding',
         'unsupported-feature', 'unsupported-stanza-type',
-        'unsupported-version'))
+        'unsupported-version'}
     condition_ns = 'urn:ietf:params:xml:ns:xmpp-streams'
 
     def get_see_other_host(self):

tests/test_stanza_element.py

@@ -561,12 +561,12 @@ class TestElementBase(SlixTest):
 
         stanza = TestStanza()
 
-        self.failUnless(set(stanza.keys()) == set(('lang', 'bar', 'baz')),
+        self.failUnless(set(stanza.keys()) == {'lang', 'bar', 'baz'},
             "Returned set of interface keys does not match expected.")
 
         stanza.enable('qux')
 
-        self.failUnless(set(stanza.keys()) == set(('lang', 'bar', 'baz', 'qux')),
+        self.failUnless(set(stanza.keys()) == {'lang', 'bar', 'baz', 'qux'},
             "Incorrect set of interface and plugin keys.")
 
     def testGet(self):

tests/test_stanza_xep_0030.py

@@ -254,10 +254,10 @@ class TestDisco(SlixTest):
         iq['disco_info'].add_identity('client', 'pc', lang='en')
         iq['disco_info'].add_identity('client', 'pc', lang='fr')
 
-        expected = set([('client', 'pc', None, None),
+        expected = {('client', 'pc', None, None),
                     ('client', 'pc', 'no', None),
                     ('client', 'pc', 'en', None),
-                        ('client', 'pc', 'fr', None)])
+                    ('client', 'pc', 'fr', None)}
         self.failUnless(iq['disco_info']['identities'] == expected,
                 "Identities do not match:\n%s\n%s" % (
                     expected,
@@ -472,9 +472,9 @@ class TestDisco(SlixTest):
                                    node='bar',
                                    name='Tester')
 
-        expected = set([('user@localhost', None, None),
+        expected = {('user@localhost', None, None),
                     ('user@localhost', 'foo', None),
-                        ('test@localhost', 'bar', 'Tester')])
+                    ('test@localhost', 'bar', 'Tester')}
         self.failUnless(iq['disco_items']['items'] == expected,
                 "Items do not match:\n%s\n%s" % (
                     expected,

tests/test_stream_presence.py

@@ -38,7 +38,7 @@ class TestStreamPresence(SlixTest):
                     to="tester@localhost"/>
         """)
 
-        self.assertEqual(events, set(('unavailable',)),
+        self.assertEqual(events, {'unavailable'},
                 "Got offline incorrectly triggered: %s." % events)
 
     def testGotOffline(self):

tests/test_stream_xep_0030.py

@@ -307,7 +307,7 @@ class TestStreamDisco(SlixTest):
           </iq>
         """)
 
-        self.assertEqual(events, set(('disco_info',)),
+        self.assertEqual(events, {'disco_info'},
                 "Disco info event was not triggered: %s" % events)
 
     def testDynamicItemsJID(self):
@@ -502,9 +502,9 @@ class TestStreamDisco(SlixTest):
           </iq>
         """)
 
-        items = set([('user@localhost', 'bar', 'Test'),
+        items = {('user@localhost', 'bar', 'Test'),
-                     ('user@localhost', 'baz', 'Test 2')])
+                 ('user@localhost', 'baz', 'Test 2')}
-        self.assertEqual(events, set(('disco_items',)),
+        self.assertEqual(events, {'disco_items'},
                 "Disco items event was not triggered: %s" % events)
         self.assertEqual(results, items,
                 "Unexpected items: %s" % results)

tests/test_stream_xep_0047.py

@@ -77,7 +77,7 @@ class TestInBandByteStreams(SlixTest):
               from="tester@localhost/receiver" />
         """)
 
-        self.assertEqual(events, set(['ibb_stream_start', 'callback']))
+        self.assertEqual(events, {'ibb_stream_start', 'callback'})
 
     @asyncio.coroutine
     def testSendData(self):