[PATCH] tests: chdir.test
Dmitry V. Levin
ldv at altlinux.org
Sat Mar 12 02:23:49 UTC 2016
Hi,
On Fri, Mar 11, 2016 at 11:22:23PM +0530, Jay Joshi wrote:
> Hello,
>
> This is my first attempt to add test for syscall (here, chdir). Any
> suggestion, comment, criticism is welcomed.
chdir is already well covered, see
$ git grep chdir tests
You must have missed my recommendation to check current coverage report
at https://codecov.io/github/strace/strace before adding new tests.
I think you'd rather take something that's not tested at all.
> +#if defined __NR_chdir
Why do you think that libc wrapper for chdir syscall is not suitable?
> + if (errno == ENOSYS) {
> + printf("chdir(\"%s\") = -1 ENOSYS (%m)\n", sample_dir);
chdir failing with ENOSYS? Really?
--
ldv
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.strace.io/pipermail/strace-devel/attachments/20160312/ae5eefd9/attachment.bin>
More information about the Strace-devel
mailing list