Skip to content

Commit 5894cb5

Browse files
committed
autotest: remove greylisting for undocumented parameters
.... now that we have them all documented
1 parent 3dce3e2 commit 5894cb5

File tree

1 file changed

+0
-15
lines changed

1 file changed

+0
-15
lines changed

Tools/autotest/vehicle_test_suite.py

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -3088,12 +3088,6 @@ def all_log_format_ids(self):
30883088

30893089
return ids
30903090

3091-
def LoggerDocumentation_greylist(self):
3092-
'''returns a set of messages should should be documented but
3093-
are currently known as undocumented'''
3094-
return set([
3095-
])
3096-
30973091
def LoggerDocumentation_whitelist(self):
30983092
'''returns a set of messages which we do not want to see
30993093
documentation for'''
@@ -3192,7 +3186,6 @@ def LoggerDocumentation(self):
31923186
tree = objectify.fromstring(xml)
31933187

31943188
whitelist = self.LoggerDocumentation_whitelist()
3195-
greylist = self.LoggerDocumentation_greylist()
31963189

31973190
docco_ids = {}
31983191
for thing in tree.logformat:
@@ -3219,9 +3212,6 @@ def LoggerDocumentation(self):
32193212
overdocumented = set()
32203213
for name in sorted(code_ids.keys()):
32213214
if name not in docco_ids:
3222-
if name in greylist:
3223-
self.progress(f"{name} should be documented but isn't")
3224-
continue
32253215
if name not in whitelist:
32263216
undocumented.add(name)
32273217
continue
@@ -3272,11 +3262,6 @@ def LoggerDocumentation(self):
32723262
if len(missing) > 0:
32733263
raise NotAchievedException("Documented messages (%s) not in code" % missing)
32743264

3275-
# ensure things in the whitelist are not documented:
3276-
for g in greylist:
3277-
if g in docco_ids:
3278-
raise NotAchievedException(f"greylisted ({g}) is actually documented")
3279-
32803265
def initialise_after_reboot_sitl(self):
32813266

32823267
# after reboot stream-rates may be zero. Request streams.

0 commit comments

Comments
 (0)