GitLab

* Fix the unittest so that it compares the correct object (tsig6)
* Store canonical long-form HMAC name in case of HMAC-MD5
  + Update TSIG RDATA class (test already present)
  + Update TSIGKey class (new tests added)

To check if there's something inside.

They are not needed, as it makes no sense to call constructor with
different template parameter than one of the class being constructed.
This also confused some versions of cppcheck and it thought there's no
constructor available.

This way cppcheck should recognize it as such.

Instead of assignment in the copy constructor.

It contains pointers, doesn't make sense to copy and writing copy
constructors would be extra work.

also constify one variable.

also constify one other object.

Also removed OldRdataFactory/new_rdata_factory_users stuff, since all Rdata classes now use the new RdataFactory.

- use the unnamed namespace for the entire content to hide as many things as
  possible.
- define rdata_tsig inside the fixture class to avoid initialization fiasco.
  (to make it possible, also make toWireCommonChecks a member function).