Updates TestUtil.pm for unit tests

[matsduf]

Purpose

This PR introduces two changes to function in TestUtil.pm:

• Support of undelegated data (both NS and DS) for unit tests.
• Makes the function more flexible to avoid long lists of tags when not needed.
• Do not use diag/fail when there is an error in the t file. Use croak() instead.
• More verifications of data.

The unit test files using TestUtil.pm have been updated to match the new format.

Five new scenarios have been added to Consistency05 and two to Consistency06. Both updates are to unit tests that uses the new function in TestUtil.pm that supports undelegated data.

The following new scenario fail:

• MULT-SOA-MNAMES-NO-DEL-UNDEL-2 (Consistency06)

Context

The update is needed for https://github.com/zonemaster/zonemaster/pull/1255.

How to test this PR

Review TestUtil.pm. If the unit test files using TestUtil.pm then they should be fine.

[matsduf]

@mattias-p, please re-review.

[matsduf]

@mattias-p, could your re-review?

[matsduf]

@tgreenx and @mattias-p, please re-review.

[matsduf]

@mattias-p, @tgreenx and @marc-vanderwal, can you approve this? I would like to merge it. It updates TestUtils.pm and the unit tests that use it. They pass as expected.

[marc-vanderwal]

I’ve had a look. I see no issues that haven’t already been brought up by @tgreenx.

[matsduf]

@marc-vanderwal and @tgreenx, please re-review after my latest updates.

[matsduf]

@marc-vanderwal and @tgreenx, please re-review after my latest updates. All tests pass. TestUtil now treats empty-empty as an error. Can this be merged now?

[matsduf]

I have commited the proposal from @tgreenx and I have updated the documentation so that it states that empty-empty is not permitted. @marc-vanderwal and @tgreenx, please re-review.

[matsduf]

@marc-vanderwal and @tgreenx, please re-review.