BIND 10 #184: cfgmgr test coverage

BIND 10 Development do-not-reply at isc.org
Tue May 18 15:21:19 UTC 2010


#184: cfgmgr test coverage
--------------------------+-------------------------------------------------
 Reporter:  jinmei        |        Owner:  jinmei                     
     Type:  task          |       Status:  reviewing                  
 Priority:  major         |    Milestone:  04. 2nd Incremental Release
Component:  Unclassified  |   Resolution:                             
 Keywords:                |    Sensitive:  0                          
--------------------------+-------------------------------------------------
Changes (by jelte):

  * owner:  jelte => jinmei


Comment:

 Ok, i've made it use tempfile.NamedTempFile (for which the online
 documentation turns out to be wrong/incomplete, sigh, if i remember i need
 to submit that).

 Please review attached patch too :)

 It doesn't use the auto-delete function (because we need to move the file
 if everything is ok), so there is an extra check+delete later.

 I'm not sure we still need the second of these new tests if we use this
 patch (kind of what you said already). Having the first seems ok.

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


More information about the bind10-tickets mailing list