small test case for abbrev(cidr)
Started by Peter Eisentrautabout 5 years ago2 messageshackers
I noticed that the file src/backend/utils/adt/inet_cidr_ntop.c has no
test coverage at all. The only way to reach this appears to be by
calling abbrev(cidr). It was easy to add a test case for this into the
existing, otherwise pretty complete, cidr tests.
Attachments:
0001-Add-test-case-for-abbrev-cidr.patchtext/plain; charset=UTF-8; name=0001-Add-test-case-for-abbrev-cidr.patch; x-mac-creator=0; x-mac-type=0Download+23-1
Re: small test case for abbrev(cidr)
Peter Eisentraut <peter.eisentraut@enterprisedb.com> writes:
I noticed that the file src/backend/utils/adt/inet_cidr_ntop.c has no
test coverage at all. The only way to reach this appears to be by
calling abbrev(cidr). It was easy to add a test case for this into the
existing, otherwise pretty complete, cidr tests.
Seems reasonable.
regards, tom lane