zonemaster / zonemaster-backend

The Zonemaster Backend - part of the Zonemaster project
Other
14 stars 23 forks source link

Fix ns argument #1147

Closed mattias-p closed 9 months ago

mattias-p commented 10 months ago

Purpose

This PR fixes a bug in get_test_results. The RPCAPI reference specifies that ns arguments should sometimes be included in the results, but this functionality was broken since last release.

Context

The bug was introduced in #1092 and discovered while testing #1145. The cause of the bug is that the database schema was changed to use capitalized names instead of uppercase module names, and get_test_results was not updated accordingly.

Changes

This PR updates get_test_results to understand the new database schema.

How to test this PR

Run zmtest se and see that all entries from the Nameserver module include an ns argument.