On Thu, Apr 17, 2014 at 11:06:16AM +0300, Nikolai Kondrashov wrote:
On 04/17/2014 02:29 AM, Lukas Slebodnik wrote:
>It does not make sense to argue.
>
>test-io was not ideal unit test and adding another workaroud(fix/hack) is not
>good. I could sleep, but ...
>
>Patch is attached.
Thanks, Lukas. This fixes my problem.
I have one more request -- don't use strdup and free, but rather create
a test context in the setup function and just free the test context in
the teardown function. I know the current code works, but it's not the
best example for future developers (who tend to look at unit tests for
examples on how to use code).
Conversely, get_random_filepath should accept a TALLOC_CTX as the first
parameter.
You can also consider creating a destructor for the contents of struct
dir_state, but here I think the teardown function is semantically close
enough to a destructor.