BIND 10 #527: refactoring auth/query_unittests
BIND 10 Development
do-not-reply at isc.org
Fri Jan 21 09:36:30 UTC 2011
#527: refactoring auth/query_unittests
-------------------------------------+-------------------------------------
Reporter: jinmei | Owner: jinmei
Type: task | Status: new
Priority: major | Milestone: A-Team-
Component: | Sprint-20110126
b10-auth | Resolution:
Keywords: | Sensitive: 0
Estimated Number of Hours: 2.0 | Add Hours to Ticket: 0
Billable?: 1 | Total Hours: 0
Internal?: 0 |
-------------------------------------+-------------------------------------
Comment (by jinmei):
The branch trac527 is ready for review.
It (mainly) contains two sets of changes:
- changes to src/lib/testutils. I've introduced a few new test utility
functions, created a separate .h and .cc files for it, and moved
the existing headerCheck() function to the new files (because it
seems to be the more appropriate place). The added functions are
based on similar tests for sqlite3_unittest, but were more generalized
and complete (eventually we want to merge these, but that would be a
separate task). This is an independent change and can be separately
reviewed if the entire diff is too big.
- changes to auth/tests/query_unittest.cc. This is the main subject
of this ticket. The refactoring is quite substantial, so it's
probably easier to simply compare the old and new tests to check
if these essentially test the same things (rather than taking a diff).
I added some comments about how the new mock zone works. The new
code shouldn't change the point of the tests (although the new ones
actually do more detailed tests using the new tesutils) with one
exception: MXAlias was different from what was actually intended,
so I updated the test scenario. See the log for commit
cd2d518fbcb19a0119553f71593b7a9d908b90c3.
--
Ticket URL: <http://bind10.isc.org/ticket/527#comment:1>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list