Looks good. My only concern is whether this will limit packet/test function
visibility by pushing people to explicitly import individual functions from
the API. I don't think this will happen because it will be very clear
developers should review the API files before writing a testsuite. And, if
they don't want to do an individual function import they can always do a
"import api.test as test" or similar and have autocomplete for all the
packet functions. So, I think there is not an issue and we are good to
proceed, but I figured I'd just mention my thoughts.

Reviewed-by: Patrick Robb <[email protected]>

Reply via email to