Changeset 129

Show
Ignore:
Timestamp:
12/04/07 18:12:50 (1 year ago)
Author:
David Rousselie <dax@happycoders.org>
Message:

Store error as string in account instead of just a boolean (in_error)

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • src/jmc/jabber/component.py

    r127 r129  
    184184            _account.disconnect() 
    185185            _account.first_check = False 
    186             _account.in_error = False 
     186            _account.error = None 
    187187            return True 
    188188        except Exception, e: 
    …… 
    254254                                            + "\nPlease reconfigure account.") 
    255255                    _account.disconnect() 
    256                     _account.in_error = False 
     256                    _account.error = None 
    257257                    self.__logger.debug("\nCHECK_MAIL ends " + _account.jid) 
    258258                except Exception, e: 

  • src/jmc/jabber/tests/component.py

    r120 r129  
    2323 
    2424import unittest 
    25 import os 
    2625import sys 
    2726import time 
    …… 
    4544     SMTPAccount, NoAccountError 
    4645from jmc.jabber import MailHandler 
    47 from jmc.jabber.message import SendMailMessageHandler, \ 
    48      RootSendMailMessageHandler 
     46from jmc.jabber.message import SendMailMessageHandler 
    4947from jmc.jabber.presence import MailSubscribeHandler, \ 
    5048     MailUnsubscribeHandler, MailPresenceHandler 
    …… 
    188186        account11.status = account.ONLINE 
    189187        self.assertTrue(account11.first_check) 
    190         self.assertFalse(account11.in_error) 
     188        self.assertEquals(account11.error, None) 
    191189        self.assertFalse(account11.waiting_password_reply) 
    192190        account11.live_email_only = True 
    …… 
    200198        self.assertTrue(account11.first_check) 
    201199        self.assertTrue(account11.waiting_password_reply) 
    202         self.assertFalse(account11.in_error) 
     200        self.assertEquals(account11.error, None) 
    203201        self.assertFalse(account11.connected) 
    204202        self.assertFalse(account11.has_connected) 
    …… 
    213211        account11.status = account.ONLINE 
    214212        self.assertTrue(account11.first_check) 
    215         self.assertFalse(account11.in_error) 
     213        self.assertEquals(account11.error, None) 
    216214        account11.waiting_password_reply = True 
    217215        account11.live_email_only = True 
    …… 
    221219        self.assertTrue(account11.first_check) 
    222220        self.assertTrue(account11.waiting_password_reply) 
    223         self.assertFalse(account11.in_error) 
     221        self.assertEquals(account11.error, None) 
    224222        self.assertFalse(account11.connected) 
    225223        self.assertFalse(account11.has_connected) 
    …… 
    269267        self.assertFalse(account11.waiting_password_reply) 
    270268        result = self.comp.handler.feeder.feed(account11) 
    271         self.assertFalse(account11.in_error) 
     269        self.assertEquals(account11.error, None) 
    272270        self.assertEquals(len(result), 0) 
    273271        sent = self.comp.stream.sent 
    …… 
    293291        account11.get_mail_list = lambda: [] 
    294292        result = self.comp.handler.feeder.feed(account11) 
    295         self.assertTrue(account11.in_error) 
     293        self.assertNotEquals(account11.error, None) 
    296294        self.assertEquals(len(result), 0) 
    297295        sent = self.comp.stream.sent 
    …… 
    317315        account11.get_mail_list = lambda: [] 
    318316        result = self.comp.handler.feeder.feed(account11) 
    319         self.assertFalse(account11.in_error) 
     317        self.assertEquals(account11.error, None) 
    320318        self.assertEquals(result, []) 
    321319        self.assertEquals(account11.lastcheck, 0) 
    …… 
    342340        account11.get_mail = mock_get_mail 
    343341        result = self.comp.handler.feeder.feed(account11) 
    344         self.assertFalse(account11.in_error) 
     342        self.assertEquals(account11.error, None) 
    345343        self.assertEquals(account11.lastcheck, 0) 
    346344        self.assertFalse(account11.connected) 
    …… 
    375373        account11.get_mail_list = lambda: [] 
    376374        result = self.comp.handler.feeder.feed(account11) 
    377         self.assertFalse(account11.in_error) 
     375        self.assertEquals(account11.error, None) 
    378376        self.assertEquals(result, []) 
    379377        self.assertEquals(account11.lastcheck, 0) 
    …… 
    400398        account11.get_mail_summary = mock_get_mail_summary 
    401399        result = self.comp.handler.feeder.feed(account11) 
    402         self.assertFalse(account11.in_error) 
     400        self.assertEquals(account11.error, None) 
    403401        self.assertEquals(account11.lastcheck, 0) 
    404402        self.assertFalse(account11.connected) 
    …… 
    423421        account11.status = account.ONLINE 
    424422        self.assertTrue(account11.first_check) 
    425         self.assertFalse(account11.in_error) 
     423        self.assertEquals(account11.error, None) 
    426424        account11.live_email_only = True 
    427425        account11.password = "password" 
    …… 
    430428        self.assertFalse(account11.first_check) 
    431429        self.assertFalse(account11.waiting_password_reply) 
    432         self.assertFalse(account11.in_error) 
     430        self.assertEquals(account11.error, None) 
    433431        self.assertFalse(account11.connected) 
    434432        self.assertTrue(account11.has_connected) 
    …… 
    446444        account11.status = account.ONLINE 
    447445        self.assertTrue(account11.first_check) 
    448         self.assertFalse(account11.in_error) 
     446        self.assertEquals(account11.error, None) 
    449447        account11.live_email_only = True 
    450448        account11.password = "password" 
    …… 
    457455        self.assertTrue(account11.first_check) 
    458456        self.assertFalse(account11.waiting_password_reply) 
    459         self.assertTrue(account11.in_error) 
     457        self.assertEquals(account11.error, "") 
    460458        self.assertFalse(account11.connected) 
    461459        self.assertFalse(account11.has_connected) 
    …… 
    473471        account11.status = account.ONLINE 
    474472        self.assertTrue(account11.first_check) 
    475         self.assertFalse(account11.in_error) 
     473        self.assertEquals(account11.error, None) 
    476474        account11.live_email_only = True 
    477475        account11.password = "password" 
    …… 
    484482        self.assertTrue(account11.first_check) 
    485483        self.assertFalse(account11.waiting_password_reply) 
    486         self.assertTrue(account11.in_error) 
     484        self.assertEquals(account11.error, "") 
    487485        self.assertFalse(account11.connected) 
    488486        self.assertTrue(account11.has_connected) 
    …… 
    500498        account11.status = account.ONLINE 
    501499        self.assertTrue(account11.first_check) 
    502         self.assertFalse(account11.in_error) 
     500        self.assertEquals(account11.error, None) 
    503501        account11.live_email_only = True 
    504502        account11.password = "password" 
    …… 
    512510        self.assertTrue(account11.first_check) 
    513511        self.assertFalse(account11.waiting_password_reply) 
    514         self.assertTrue(account11.in_error) 
     512        self.assertEquals(account11.error, "") 
    515513        self.assertFalse(account11.connected) 
    516514        self.assertTrue(account11.has_connected)