BIND 10 #2679: use value-param tests for database tests, not type-param tests

[BIND 10 Development]

#2679: use value-param tests for database tests, not type-param tests
-------------------------------------+-------------------------------------
            Reporter:  jinmei        |                        Owner:
                Type:  defect        |  jinmei
            Priority:  medium        |                       Status:
           Component:  data source   |  reviewing
            Keywords:                |                    Milestone:
           Sensitive:  0             |  Sprint-20130219
         Sub-Project:  DNS           |                   Resolution:
Estimated Difficulty:  5             |                 CVSS Scoring:
         Total Hours:  0             |              Defect Severity:  N/A
                                     |  Feature Depending on Ticket:
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------
Changes (by vorner):

 * owner:  vorner => jinmei


Comment:

 Hello

 I found some minor things in the branch. I hope I didn't miss anything
 important in the large amount of diff, but it probably couldn't have been
 made shorter.

 Do the commits `79f65e1240e0104f6d425ce63821e1297b78a3ce` and
 `4792e513f5dc087a6dc5e51f72acf0efd0cc4f37` contain anything but
 reindentation and removal of the `#if 0` lines?

 Is this really true?
 {{{#!c++
 /// This is similar to TEST_RECORDS, but the first entry is base32-hex
 encoded
 /// hash value (which is expected to appear as the first label of NSEC3
 /// owner names), not an FQDN.
 }}}

 If so, what is the purpose of the split in the load function? That split
 seems to expect a name and takes the first label, but if it's only the
 hash, it should be passed as it is, shouldn't it?

 Also, I was surprised it is possible to instantiate the tests multiple
 times with different values each time and have only one set of the test
 bodies. Is that really supported by GTEST, or does it work by accident?

 It really seems #1272 can be closed.

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