ldns icon indicating copy to clipboard operation
ldns copied to clipboard

Would it be easy (or not) to test for an issue like #131

Open cpaelzer opened this issue 2 years ago • 1 comments

Hi, in Ubuntu I spotted an issue when testing dns-root-data which ended up being an issue in ldns that wasn't spotted for quite some time. I found that you knew about it #131 and had fixed it in 4d2057f0b5 already. My analysis and driving the fix into Ubuntu is on bug 1966237.

So fixed upstream fixed in Ubuntu - all good right? But a coworker rightfully asked if there could be a test for this kind of behavior to prevent similar issues in the future. And I think this would be best in the upstream build time tests as that way everyone would benefit.

One could run the test as I used it in the ubuntu bug with the different hash styles and check this against expected results.

I had not much time or I'd have provided a PR right away, but maybe it is already obvious for you like:

  • we never add this kind of tests do not try
  • this is easy, already done here
  • any other response

So while I didn't have the time to write the test I wanted to bring it up for your consideration by filing this issue.

Thanks in advance!

cpaelzer avatar Apr 07 '22 07:04 cpaelzer

Thank you @cpaelzer . I have noticed and have been following (but not too closely) the Ubuntu bug report. We do some basic unit testing which are run with github actions on each commit (thanks again @FGasper ), see: https://github.com/NLnetLabs/ldns/blob/develop/.github/workflows/testsuite.yml

Some of those tests are online tests though. Maybe it would be worthwhile to identify which tests can be run offline and have a special Makefile target for them. WDYT?

wtoorop avatar Apr 12 '22 07:04 wtoorop