bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: test-symlink failure on MinGW


From: Eric Blake
Subject: Re: test-symlink failure on MinGW
Date: Tue, 29 Dec 2009 07:03:29 -0700
User-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.23) Gecko/20090812 Thunderbird/2.0.0.23 Mnenhy/0.7.6.666

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

According to Simon Josefsson on 12/28/2009 11:51 PM:
> This patch looks a bit cleaner, it moves the system call into the
> function actually responsible for creating the same files.  And it also
> fixes lstat self-tests.

On the one hand, you are correct that it is nicer to put the cleanup in
the same location as the creation.  On the other hand, it means tests like
test-symlinkat will encounter more forks, because they end up calling
system() on every call into the .h file(s) rather than just once at
startup.  Again, the point of the system() call is to ease debugging.  The
.h files already clean up after themselves, if they run successfully to
completion.  It is only when they fail that the cleanup-on-startup is
useful, so repeating the cleanup multiple times during a single test just
adds up to a waste of processes and execution time.  So I'd rather do the
code motion in the opposite direction (out of test-lstat.h and into the
callers).  And I really want to change ALL such uses of system(), not just
the ones you encountered.

Here's what I'm pushing:

- --
Don't work too hard, make some time for fun as well!

Eric Blake             address@hidden
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAks6DDEACgkQ84KuGfSFAYBy1gCcCQOOS6PuoVG2afo3o6iyrO8P
+TsAoJzT2XHa1b3qxrNrgHdwN0673RnG
=Ca3T
-----END PGP SIGNATURE-----
>From 7cea870190160e1899076987664fd9b9859edeaf Mon Sep 17 00:00:00 2001
From: Eric Blake <address@hidden>
Date: Tue, 29 Dec 2009 06:58:18 -0700
Subject: [PATCH] tests: don't require debug system() to pass

When running a cross-compilation to mingw on a Linux host,
but without a valid mingw rm executable, the debug-only
system() call would fail.  This is not fatal to the test,
since the call is merely a debugging aid if a prior call to
the test encountered a failure and left garbage; in general,
the tests should pass and the system() call had nothing to
clean in the first place.  Also, the system() call only
needs to happen once at startup, not once per iteration
into the tests performed by .h files.

* tests/test-lstat.h (test_lstat_func): Move debug cleanup...
* tests/test-rmdir.h (test_rmdir_func): Likewise.
* tests/test-unlink.h (test_unlink_func): Likewise.
* tests/test-fstatat.c (main): ...into callers.
* tests/test-lstat.c (main): Likewise.
* tests/test-rmdir.c (main): Likewise.
* tests/test-unlink.c (main): Likewise.
* tests/test-unlinkat.c (main): Likewise.
* tests/test-areadlink-with-size.c (main): Don't require a
debug-only system call to pass, aiding cross-testing to mingw.
* tests/test-areadlink.c (main): Likewise.
* tests/test-areadlinkat-with-size.c (main): Likewise.
* tests/test-areadlinkat.c (main): Likewise.
* tests/test-canonicalize-lgpl.c (main): Likewise.
* tests/test-canonicalize.c (main): Likewise.
* tests/test-chown.c (main): Likewise.
* tests/test-fchownat.c (main): Likewise.
* tests/test-lchown.c (main): Likewise.
* tests/test-fdutimensat.c (main): Likewise.
* tests/test-futimens.c (main): Likewise.
* tests/test-link.c (main): Likewise.
* tests/test-linkat.c (main): Likewise.
* tests/test-mkdir.c (main): Likewise.
* tests/test-mkdirat.c (main): Likewise.
* tests/test-mkfifo.c (main): Likewise.
* tests/test-mkfifoat.c (main): Likewise.
* tests/test-mknod.c (main): Likewise.
* tests/test-readlink.c (main): Likewise.
* tests/test-remove.c (main): Likewise.
* tests/test-rename.c (main): Likewise.
* tests/test-renameat.c (main): Likewise.
* tests/test-symlink.c (main): Likewise.
* tests/test-symlinkat.c (main): Likewise.
* tests/test-utimens.c (main): Likewise.
* tests/test-utimensat.c (main): Likewise.

Signed-off-by: Eric Blake <address@hidden>
---
 ChangeLog                          |   40 ++++++++++++++++++++++++++++++++++++
 tests/test-areadlink-with-size.c   |    2 +-
 tests/test-areadlink.c             |    2 +-
 tests/test-areadlinkat-with-size.c |    2 +-
 tests/test-areadlinkat.c           |    2 +-
 tests/test-canonicalize-lgpl.c     |    2 +-
 tests/test-canonicalize.c          |    2 +-
 tests/test-chown.c                 |    2 +-
 tests/test-fchownat.c              |    2 +-
 tests/test-fdutimensat.c           |    2 +-
 tests/test-fstatat.c               |    4 +++
 tests/test-futimens.c              |    2 +-
 tests/test-lchown.c                |    2 +-
 tests/test-link.c                  |    2 +-
 tests/test-linkat.c                |    2 +-
 tests/test-lstat.c                 |    3 ++
 tests/test-lstat.h                 |    3 --
 tests/test-mkdir.c                 |    2 +-
 tests/test-mkdirat.c               |    2 +-
 tests/test-mkfifo.c                |    2 +-
 tests/test-mkfifoat.c              |    2 +-
 tests/test-mknod.c                 |    2 +-
 tests/test-readlink.c              |    2 +-
 tests/test-remove.c                |    2 +-
 tests/test-rename.c                |    2 +-
 tests/test-renameat.c              |    2 +-
 tests/test-rmdir.c                 |    3 ++
 tests/test-rmdir.h                 |    3 --
 tests/test-symlink.c               |    2 +-
 tests/test-symlinkat.c             |    2 +-
 tests/test-unlink.c                |    3 ++
 tests/test-unlink.h                |    3 --
 tests/test-unlinkat.c              |    4 +++
 tests/test-utimens.c               |    2 +-
 tests/test-utimensat.c             |    2 +-
 35 files changed, 83 insertions(+), 35 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 60a9b56..41ef6f7 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,4 +1,44 @@
 2009-12-29  Simon Josefsson  <address@hidden>
+       and Eric Blake  <address@hidden>
+
+       tests: don't require debug system() to pass
+       * tests/test-lstat.h (test_lstat_func): Move debug cleanup...
+       * tests/test-rmdir.h (test_rmdir_func): Likewise.
+       * tests/test-unlink.h (test_unlink_func): Likewise.
+       * tests/test-fstatat.c (main): ...into callers.
+       * tests/test-lstat.c (main): Likewise.
+       * tests/test-rmdir.c (main): Likewise.
+       * tests/test-unlink.c (main): Likewise.
+       * tests/test-unlinkat.c (main): Likewise.
+       * tests/test-areadlink-with-size.c (main): Don't require a
+       debug-only system call to pass, aiding cross-testing to mingw.
+       * tests/test-areadlink.c (main): Likewise.
+       * tests/test-areadlinkat-with-size.c (main): Likewise.
+       * tests/test-areadlinkat.c (main): Likewise.
+       * tests/test-canonicalize-lgpl.c (main): Likewise.
+       * tests/test-canonicalize.c (main): Likewise.
+       * tests/test-chown.c (main): Likewise.
+       * tests/test-fchownat.c (main): Likewise.
+       * tests/test-lchown.c (main): Likewise.
+       * tests/test-fdutimensat.c (main): Likewise.
+       * tests/test-futimens.c (main): Likewise.
+       * tests/test-link.c (main): Likewise.
+       * tests/test-linkat.c (main): Likewise.
+       * tests/test-mkdir.c (main): Likewise.
+       * tests/test-mkdirat.c (main): Likewise.
+       * tests/test-mkfifo.c (main): Likewise.
+       * tests/test-mkfifoat.c (main): Likewise.
+       * tests/test-mknod.c (main): Likewise.
+       * tests/test-readlink.c (main): Likewise.
+       * tests/test-remove.c (main): Likewise.
+       * tests/test-rename.c (main): Likewise.
+       * tests/test-renameat.c (main): Likewise.
+       * tests/test-symlink.c (main): Likewise.
+       * tests/test-symlinkat.c (main): Likewise.
+       * tests/test-utimens.c (main): Likewise.
+       * tests/test-utimensat.c (main): Likewise.
+
+2009-12-29  Simon Josefsson  <address@hidden>

        * modules/selinux-h (selinux/selinux.h, selinux/context.h): Depend
        on $(UNUSED_PARAMETER_H) to avoid build failure.
diff --git a/tests/test-areadlink-with-size.c b/tests/test-areadlink-with-size.c
index 8b7ff63..d5f4a0d 100644
--- a/tests/test-areadlink-with-size.c
+++ b/tests/test-areadlink-with-size.c
@@ -39,7 +39,7 @@ int
 main (void)
 {
   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   return test_areadlink (areadlink_with_size, true);
 }
diff --git a/tests/test-areadlink.c b/tests/test-areadlink.c
index 2e8aaa5..272a614 100644
--- a/tests/test-areadlink.c
+++ b/tests/test-areadlink.c
@@ -46,7 +46,7 @@ int
 main (void)
 {
   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   return test_areadlink (do_areadlink, true);
 }
diff --git a/tests/test-areadlinkat-with-size.c 
b/tests/test-areadlinkat-with-size.c
index 67252a7..113c6b4 100644
--- a/tests/test-areadlinkat-with-size.c
+++ b/tests/test-areadlinkat-with-size.c
@@ -50,7 +50,7 @@ main (void)
   int result;

   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   /* Basic tests.  */
   result = test_areadlink (do_areadlinkat_with_size, false);
diff --git a/tests/test-areadlinkat.c b/tests/test-areadlinkat.c
index 54e0745..68d7f58 100644
--- a/tests/test-areadlinkat.c
+++ b/tests/test-areadlinkat.c
@@ -50,7 +50,7 @@ main (void)
   int result;

   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   /* Basic tests.  */
   result = test_areadlink (do_areadlinkat, false);
diff --git a/tests/test-canonicalize-lgpl.c b/tests/test-canonicalize-lgpl.c
index e35d921..2425921 100644
--- a/tests/test-canonicalize-lgpl.c
+++ b/tests/test-canonicalize-lgpl.c
@@ -55,7 +55,7 @@ main (void)
      any leftovers from a previous partial run.  */
   {
     int fd;
-    ASSERT (system ("rm -rf " BASE " ise") == 0);
+    system ("rm -rf " BASE " ise");
     ASSERT (mkdir (BASE, 0700) == 0);
     fd = creat (BASE "/tra", 0600);
     ASSERT (0 <= fd);
diff --git a/tests/test-canonicalize.c b/tests/test-canonicalize.c
index 5830267..b09c1ca 100644
--- a/tests/test-canonicalize.c
+++ b/tests/test-canonicalize.c
@@ -46,7 +46,7 @@ main (void)
      any leftovers from a previous partial run.  */
   {
     int fd;
-    ASSERT (system ("rm -rf " BASE " ise") == 0);
+    system ("rm -rf " BASE " ise");
     ASSERT (mkdir (BASE, 0700) == 0);
     fd = creat (BASE "/tra", 0600);
     ASSERT (0 <= fd);
diff --git a/tests/test-chown.c b/tests/test-chown.c
index 3974903..15a6d75 100644
--- a/tests/test-chown.c
+++ b/tests/test-chown.c
@@ -42,7 +42,7 @@ int
 main (void)
 {
   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   return test_chown (chown, true);
 }
diff --git a/tests/test-fchownat.c b/tests/test-fchownat.c
index a6d177d..f7c5827 100644
--- a/tests/test-fchownat.c
+++ b/tests/test-fchownat.c
@@ -63,7 +63,7 @@ main (void)
   int result2; /* Skip because of no lchown support.  */

   /* Clean up any trash from prior testsuite runs.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   /* Basic tests.  */
   result1 = test_chown (do_chown, true);
diff --git a/tests/test-fdutimensat.c b/tests/test-fdutimensat.c
index ad0eca9..0e76661 100644
--- a/tests/test-fdutimensat.c
+++ b/tests/test-fdutimensat.c
@@ -84,7 +84,7 @@ main (void)
   int fd;

   /* Clean up any trash from prior testsuite runs.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   /* Basic tests.  */
   result1 = test_utimens (do_utimens, true);
diff --git a/tests/test-fstatat.c b/tests/test-fstatat.c
index e6abe38..6160e16 100644
--- a/tests/test-fstatat.c
+++ b/tests/test-fstatat.c
@@ -60,6 +60,10 @@ int
 main (void)
 {
   int result;
+
+  /* Remove any leftovers from a previous partial run.  */
+  system ("rm -rf " BASE "*");
+
   result = test_stat_func (do_stat, false);
   ASSERT (test_lstat_func (do_lstat, false) == result);
   dfd = open (".", O_RDONLY);
diff --git a/tests/test-futimens.c b/tests/test-futimens.c
index 0ee1fe0..af553a9 100644
--- a/tests/test-futimens.c
+++ b/tests/test-futimens.c
@@ -44,7 +44,7 @@ int
 main (void)
 {
   /* Clean up any trash from prior testsuite runs.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   return test_futimens (futimens, true);
 }
diff --git a/tests/test-lchown.c b/tests/test-lchown.c
index 24b7ece..0c0aa11 100644
--- a/tests/test-lchown.c
+++ b/tests/test-lchown.c
@@ -42,7 +42,7 @@ int
 main (void)
 {
   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   return test_lchown (lchown, true);
 }
diff --git a/tests/test-link.c b/tests/test-link.c
index 817c464..9c1ce36 100644
--- a/tests/test-link.c
+++ b/tests/test-link.c
@@ -40,7 +40,7 @@ int
 main (void)
 {
   /* Remove any garbage left from previous partial runs.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   return test_link (link, true);
 }
diff --git a/tests/test-linkat.c b/tests/test-linkat.c
index 98165c0..efe5109 100644
--- a/tests/test-linkat.c
+++ b/tests/test-linkat.c
@@ -81,7 +81,7 @@ main (void)
   int result;

   /* Clean up any trash from prior testsuite runs.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   /* Test basic link functionality, without mentioning symlinks.  */
   result = test_link (do_link, true);
diff --git a/tests/test-lstat.c b/tests/test-lstat.c
index 1940a21..bff40eb 100644
--- a/tests/test-lstat.c
+++ b/tests/test-lstat.c
@@ -52,5 +52,8 @@ do_lstat (char const *name, struct stat *st)
 int
 main (void)
 {
+  /* Remove any leftovers from a previous partial run.  */
+  system ("rm -rf " BASE "*");
+
   return test_lstat_func (do_lstat, true);
 }
diff --git a/tests/test-lstat.h b/tests/test-lstat.h
index 3828316..071626a 100644
--- a/tests/test-lstat.h
+++ b/tests/test-lstat.h
@@ -28,9 +28,6 @@ test_lstat_func (int (*func) (char const *, struct stat *), 
bool print)
   struct stat st1;
   struct stat st2;

-  /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
-
   /* Test for common directories.  */
   ASSERT (func (".", &st1) == 0);
   ASSERT (func ("./", &st2) == 0);
diff --git a/tests/test-mkdir.c b/tests/test-mkdir.c
index 379e90c..6be4486 100644
--- a/tests/test-mkdir.c
+++ b/tests/test-mkdir.c
@@ -40,7 +40,7 @@ int
 main (void)
 {
   /* Clean up any trash from prior testsuite runs.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   return test_mkdir (mkdir, true);
 }
diff --git a/tests/test-mkdirat.c b/tests/test-mkdirat.c
index d3180e6..fe0da25 100644
--- a/tests/test-mkdirat.c
+++ b/tests/test-mkdirat.c
@@ -51,7 +51,7 @@ main (void)
   int result;

   /* Clean up any trash from prior testsuite runs.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   /* Test basic mkdir functionality.  */
   result = test_mkdir (do_mkdir, false);
diff --git a/tests/test-mkfifo.c b/tests/test-mkfifo.c
index 4b4c93b..fc36412 100644
--- a/tests/test-mkfifo.c
+++ b/tests/test-mkfifo.c
@@ -40,7 +40,7 @@ int
 main (void)
 {
   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   return test_mkfifo (mkfifo, true);
 }
diff --git a/tests/test-mkfifoat.c b/tests/test-mkfifoat.c
index ef3e817..0b20181 100644
--- a/tests/test-mkfifoat.c
+++ b/tests/test-mkfifoat.c
@@ -72,7 +72,7 @@ main (void)
   int result;

   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   /* Basic tests.  */
   result = test_mkfifo (do_mkfifoat, true);
diff --git a/tests/test-mknod.c b/tests/test-mknod.c
index a62df6d..ed16b50 100644
--- a/tests/test-mknod.c
+++ b/tests/test-mknod.c
@@ -47,7 +47,7 @@ int
 main (void)
 {
   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   /* We can only portably test creation of fifos.  Anything else
      requires root privileges and knowledge of device numbers.  */
diff --git a/tests/test-readlink.c b/tests/test-readlink.c
index 4ec0fcd..ed30c1d 100644
--- a/tests/test-readlink.c
+++ b/tests/test-readlink.c
@@ -41,7 +41,7 @@ int
 main (void)
 {
   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   return test_readlink (readlink, true);
 }
diff --git a/tests/test-remove.c b/tests/test-remove.c
index 4bcc2ec..5428e37 100644
--- a/tests/test-remove.c
+++ b/tests/test-remove.c
@@ -38,7 +38,7 @@ int
 main (void)
 {
   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   /* Setup.  */
   ASSERT (mkdir (BASE "dir", 0700) == 0);
diff --git a/tests/test-rename.c b/tests/test-rename.c
index 797cc94..17cd025 100644
--- a/tests/test-rename.c
+++ b/tests/test-rename.c
@@ -39,7 +39,7 @@ int
 main (void)
 {
   /* Remove any garbage left from previous partial runs.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   return test_rename (rename, true);
 }
diff --git a/tests/test-renameat.c b/tests/test-renameat.c
index de84fa6..a22af04 100644
--- a/tests/test-renameat.c
+++ b/tests/test-renameat.c
@@ -57,7 +57,7 @@ main (void)
   int result;

   /* Clean up any trash from prior testsuite runs.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   /* Test basic rename functionality, using current directory.  */
   result = test_rename (do_rename, false);
diff --git a/tests/test-rmdir.c b/tests/test-rmdir.c
index 9ca88e2..a270ca7 100644
--- a/tests/test-rmdir.c
+++ b/tests/test-rmdir.c
@@ -39,5 +39,8 @@ SIGNATURE_CHECK (rmdir, int, (char const *));
 int
 main (void)
 {
+  /* Remove any leftovers from a previous partial run.  */
+  system ("rm -rf " BASE "*");
+
   return test_rmdir_func (rmdir, true);
 }
diff --git a/tests/test-rmdir.h b/tests/test-rmdir.h
index 5f1e955..19c6961 100644
--- a/tests/test-rmdir.h
+++ b/tests/test-rmdir.h
@@ -25,9 +25,6 @@
 static int
 test_rmdir_func (int (*func) (char const *name), bool print)
 {
-  /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
-
   /* Setup.  */
   ASSERT (mkdir (BASE "dir", 0700) == 0);
   ASSERT (close (creat (BASE "dir/file", 0600)) == 0);
diff --git a/tests/test-symlink.c b/tests/test-symlink.c
index c84b2bc..f066ee3 100644
--- a/tests/test-symlink.c
+++ b/tests/test-symlink.c
@@ -40,7 +40,7 @@ int
 main (void)
 {
   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   return test_symlink (symlink, true);
 }
diff --git a/tests/test-symlinkat.c b/tests/test-symlinkat.c
index 714dd98..be59d5d 100644
--- a/tests/test-symlinkat.c
+++ b/tests/test-symlinkat.c
@@ -64,7 +64,7 @@ main (void)
   int result;

   /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   /* Perform same checks as counterpart functions.  */
   result = test_readlink (do_readlink, false);
diff --git a/tests/test-unlink.c b/tests/test-unlink.c
index 1525802..b87f7d6 100644
--- a/tests/test-unlink.c
+++ b/tests/test-unlink.c
@@ -41,5 +41,8 @@ SIGNATURE_CHECK (unlink, int, (char const *));
 int
 main (void)
 {
+  /* Remove any leftovers from a previous partial run.  */
+  system ("rm -rf " BASE "*");
+
   return test_unlink_func (unlink, true);
 }
diff --git a/tests/test-unlink.h b/tests/test-unlink.h
index e65057d..05be29c 100644
--- a/tests/test-unlink.h
+++ b/tests/test-unlink.h
@@ -25,9 +25,6 @@
 static int
 test_unlink_func (int (*func) (char const *name), bool print)
 {
-  /* Remove any leftovers from a previous partial run.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
-
   /* Setup.  */
   ASSERT (mkdir (BASE "dir", 0700) == 0);
   ASSERT (close (creat (BASE "dir/file", 0600)) == 0);
diff --git a/tests/test-unlinkat.c b/tests/test-unlinkat.c
index 3d7f118..bd2b48a 100644
--- a/tests/test-unlinkat.c
+++ b/tests/test-unlinkat.c
@@ -60,6 +60,10 @@ main (void)
   /* FIXME: Add tests of fd other than ".".  */
   int result1;
   int result2;
+
+  /* Remove any leftovers from a previous partial run.  */
+  system ("rm -rf " BASE "*");
+
   result1 = test_rmdir_func (rmdirat, false);
   result2 = test_unlink_func (unlinker, false);
   ASSERT (result1 == result2);
diff --git a/tests/test-utimens.c b/tests/test-utimens.c
index 618d941..dccaacc 100644
--- a/tests/test-utimens.c
+++ b/tests/test-utimens.c
@@ -66,7 +66,7 @@ main (void)
   int result3; /* Skip because of no lutimens support.  */

   /* Clean up any trash from prior testsuite runs.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   result1 = test_utimens (utimens, true);
   ASSERT (test_utimens (do_fdutimens, false) == result1);
diff --git a/tests/test-utimensat.c b/tests/test-utimensat.c
index ba4c22d..45aa8c3 100644
--- a/tests/test-utimensat.c
+++ b/tests/test-utimensat.c
@@ -66,7 +66,7 @@ main (void)
   int fd;

   /* Clean up any trash from prior testsuite runs.  */
-  ASSERT (system ("rm -rf " BASE "*") == 0);
+  system ("rm -rf " BASE "*");

   /* Basic tests.  */
   result1 = test_utimens (do_utimensat, true);
-- 
1.6.4.2


reply via email to

[Prev in Thread] Current Thread [Next in Thread]