BIND 10 #23: Python ccmsg stuff review

[BIND 10 Development]

#23: Python ccmsg stuff review
----------------------------------------+-----------------------------------
 Reporter:  mgraff                      |        Owner:  shane                                                
     Type:  task                        |       Status:  assigned                                             
 Priority:  major                       |    Milestone:  01. Running, non-functional authoritative-only server
Component:  Inter-module communication  |   Resolution:                                                       
 Keywords:                              |    Sensitive:  0                                                    
----------------------------------------+-----------------------------------
Changes (by mgraff):

  * owner:  mgraff => shane


Comment:

 I don't think it had been reviewed.

 Some observations:

   Test coverage of negative situations is poor.  We probably call the
 methods expecting success, but don't call any tests expecting failure due
 to argument problems, badly formatted messages, etc.  Some of this is
 because the doctest thing is used -- which ensures the documentation
 examples work -- but we need more tests as well I think.  I'm quite
 certain parts of this code are not in fact tested in real use due to data
 sizes.

-- 
Ticket URL: <https://bind10.isc.org/ticket/23#comment:3>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development