BIND 10 #2338: Add tests for ZoneDataUpdater

[BIND 10 Development]

#2338: Add tests for ZoneDataUpdater
-------------------------------------+-------------------------------------
                   Reporter:  muks   |                 Owner:
                       Type:         |                Status:  new
  defect                             |             Milestone:  New Tasks
                   Priority:         |            Resolution:
  medium                             |             Sensitive:  0
                  Component:  data   |           Sub-Project:  DNS
  source                             |  Estimated Difficulty:  5
                   Keywords:         |           Total Hours:  0
            Defect Severity:  N/A    |
Feature Depending on Ticket:         |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Description changed by muks:

Old description:

> Bug #2268 introduced the `ZoneDataUpdater` class. It did not add tests as
> these are covered already by `InMemoryClient`'s tests. There are many
> testcases to do with validation that are in `InMemoryClient`'s tests (the
> `ZoneDataUpdater` code used to first be a part of `InMemoryClient`).
>
> This bug is to add direct tests for `ZoneDataUpdater` by porting all the
> data related tests from `InMemoryClient`'s tests.

New description:

 Bug #2268 introduced the `ZoneDataUpdater` class. It did not add tests as
 these are covered already by `InMemoryClient`'s tests. There are many
 testcases to do with validation that are in `InMemoryClient`'s tests (the
 `ZoneDataUpdater` code used to first be a part of `InMemoryClient`).

 This bug is to add direct tests for `ZoneDataUpdater` by porting all the
 data related tests from `InMemoryClient`'s tests.

 In addition to these tests, be sure to add a test that checks that
 `ZoneDataUpdater::add()` throws when a NULL `ConstRRSetPtr` is passed in
 the `rrset` arg. This test was removed in the memory client as it's not
 possible to get this condition there. But you can get this condition when
 directly using the `ZoneDataUpdater::add()` method.

--

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