On 22 January 2013 16:50, Michael Hanselmann <[email protected]> wrote: > This patch moves the functions “_ReadTestData” and “_TestDataFilename” > to module level. They do not depend on a class instance at all. > > Some cases of “utils.ReadFile(self._TestDataFilename(…))” are also > corrected. > > Signed-off-by: Michael Hanselmann <[email protected]>
> +def ReadTestData(name): > + """Returns the contents of a test data file. This was in the original code, but a plural "contents" sounds odd. Except for that, LGTM, thanks. Bernardo
