BIND 10 #3088: Develop NameRemoveTransaction class for b10-dhcp-ddns

[BIND 10 Development]

#3088: Develop NameRemoveTransaction class for b10-dhcp-ddns
-------------------------------------+-------------------------------------
            Reporter:  tmark         |                        Owner:
                Type:  enhancement   |  UnAssigned
            Priority:  medium        |                       Status:
           Component:  dhcp-ddns     |  reviewing
            Keywords:  Task# 4.3     |                    Milestone:  DHCP-
           Sensitive:  0             |  Kea1.0-alpha
         Sub-Project:  DHCP          |                   Resolution:
Estimated Difficulty:  0             |                 CVSS Scoring:
         Total Hours:  0             |              Defect Severity:  N/A
                                     |  Feature Depending on Ticket:
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------
Changes (by tmark):

 * owner:  tmark => UnAssigned
 * status:  assigned => reviewing


Comment:

 This ticket introduces the initial implementation of
 !NameRemoveTransaction.
 This class derives from !NameChangeTransaction and provides the state
 model
 implementation to remove DNS entries for a given FQDN.   With the
 completion of
 this ticket the state model logic is fully functional, and there are unit
 tests
 for each state's handler that verify all of the state model transitions.

 In addition, the class also has all of the logic necessary to construct
 the
 actual DNS update requests.


 The following class diagram depicts the new class:

 [[Image(nc_remove_trans_class.svg)]]

 and its state model is as shown below:

 [[Image(nc_remove_state_diagram.svg)]]


 Some further refactoring has been done in the unit test code.  A new
 class,
 !TransactionTest, was added to nc_test_utils.  It serves as a common test
 fixture that can be used throughout !NameChangeTransaction class
 hierarchy.
 It is currently only used in nc_remove_unittests.cc.  Using this class in
 nc_trans_unittests.cc and nc_add_unittests.cc will be done under a
 subsequent
 ticket to reduce the amount of work in this ticket.

 Lastly, functions used to verify DNS request contents in nc_test_utils
 were
 renamed to match the build method used to create the request.  This was
 done
 to reduce confusion.

 The following !ChangeLog entry is proposed:

 {{{
 7xx.    [func]          tmark
         Added the initial implementation of the class, NameAddTransaction,
         to b10-dhcp-ddns.  This class provides the state model logic
 described
         in the DHCP_DDNS design to remove forward and reverse DNS entries
 as
         described in RFC 4703, section 5.5 This includes the ability to
 construct
         the necessary DNS requests.
 }}}

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