Added a test to make sure JOINs happen before WHOs, and changed code to make that the case.

src/irclib.py

@@ -122,7 +122,7 @@ class IrcMsgQueue(object):
             self.msgs.add(msg)
             if msg.command in ('MODE', 'KICK', 'PONG'):
                 self.highpriority.enqueue(msg)
-            elif msg.command in ('PING',):
+            elif msg.command in ('PING', 'WHO'):
                 self.lowpriority.enqueue(msg)
             else:
                 self.normal.enqueue(msg)

test/test_irclib.py

@@ -46,6 +46,8 @@ class IrcMsgQueueTestCase(unittest.TestCase):
     pong = ircmsgs.pong('123')
     ping = ircmsgs.ping('123')
     notice = ircmsgs.notice('jemfinch', 'supybot here')
+    join = ircmsgs.join('#foo')
+    who = ircmsgs.who('#foo')
 
     def testEmpty(self):
         q = irclib.IrcMsgQueue()
@@ -97,6 +99,17 @@ class IrcMsgQueueTestCase(unittest.TestCase):
         self.assertEqual(self.msg, q.dequeue())
         self.failIf(q)
 
+    def testJoinBeforeWho(self):
+        q = irclib.IrcMsgQueue()
+        q.enqueue(self.join)
+        q.enqueue(self.who)
+        self.assertEqual(self.join, q.dequeue())
+        self.assertEqual(self.who, q.dequeue())
+        q.enqueue(self.who)
+        q.enqueue(self.join)
+        self.assertEqual(self.join, q.dequeue())
+        self.assertEqual(self.who, q.dequeue())
+
 
 class ChannelTestCase(unittest.TestCase):
     def testPickleCopy(self):