I read the patch. It looks good.
Anyway, I have two questions:
1. I saw 3 variants of writing description for skipping test descriptor:
requires os.mkfifo
test needs socket.inet_pton()
needs os.read
Which one is the preferred way to go? "requires" or "test needs" or "needs"? Or it does not matter?
2. I saw 2 variants of writing the function name required for running the test.
test needs socket.inet_pton()
test needs socket.inet_pton
Which one is the preferred way to go? With or without parentheses? Or it does not matter? |