[libcxx] r273049 - Reorder permissions test so they are not dependent on the processes umask

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 17 14:00:28 PDT 2016


Author: ericwf
Date: Fri Jun 17 16:00:27 2016
New Revision: 273049

URL: http://llvm.org/viewvc/llvm-project?rev=273049&view=rev
Log:
Reorder permissions test so they are not dependent on the processes umask

Modified:
    libcxx/trunk/test/std/experimental/filesystem/fs.op.funcs/fs.op.permissions/permissions.pass.cpp

Modified: libcxx/trunk/test/std/experimental/filesystem/fs.op.funcs/fs.op.permissions/permissions.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/experimental/filesystem/fs.op.funcs/fs.op.permissions/permissions.pass.cpp?rev=273049&r1=273048&r2=273049&view=diff
==============================================================================
--- libcxx/trunk/test/std/experimental/filesystem/fs.op.funcs/fs.op.permissions/permissions.pass.cpp (original)
+++ libcxx/trunk/test/std/experimental/filesystem/fs.op.funcs/fs.op.permissions/permissions.pass.cpp Fri Jun 17 16:00:27 2016
@@ -89,20 +89,20 @@ TEST_CASE(basic_permissions_test)
       perms expected;
     } cases[] = {
         // test file
+        {file, perms::none, perms::none},
         {file, perms::owner_all, perms::owner_all},
         {file, perms::group_all | AP, perms::owner_all | perms::group_all},
         {file, perms::group_all | RP, perms::owner_all},
-        {file, perms::none, perms::none},
         // test directory
+        {dir, perms::none, perms::none},
         {dir, perms::owner_all, perms::owner_all},
         {dir, perms::group_all | AP, perms::owner_all | perms::group_all},
         {dir, perms::group_all | RP, perms::owner_all},
-        {dir, perms::none, perms::none},
         // test symlink with resolve symlinks on symlink
+        {sym, perms::none | RS, perms::none},
         {sym, perms::owner_all | RS, perms::owner_all},
         {sym, perms::group_all | AP | RS, perms::owner_all | perms::group_all},
-        {sym, perms::group_all | RP | RS, perms::owner_all},
-        {sym, perms::none | RS, perms::none}
+        {sym, perms::group_all | RP | RS, perms::owner_all}
     };
     for (auto const& TC : cases) {
         TEST_CHECK(status(TC.p).permissions() != TC.expected);




More information about the cfe-commits mailing list