[Opensync-commits] r5641 - in trunk: opensync/group tests/group-tests

[<svn...@op...>]

Author: dgollub
Date: Sat May  9 20:49:48 2009
New Revision: 5641
URL: http://www.opensync.org/changeset/5641

Log:
Fix testcase member_configdir_deep_path, which got recently added.
Allow deep member configuration directories

Modified:
   trunk/opensync/group/opensync_member.c
   trunk/tests/group-tests/check_member.c

Modified: trunk/opensync/group/opensync_member.c
==============================================================================
--- trunk/opensync/group/opensync_member.c	Sat May  9 12:04:41 2009	(r5640)
+++ trunk/opensync/group/opensync_member.c	Sat May  9 20:49:48 2009	(r5641)
@@ -518,7 +518,10 @@
 	osync_assert(member->configdir);
 	
 	if (!g_file_test(member->configdir, G_FILE_TEST_IS_DIR)) {
-		if (g_mkdir(member->configdir, 0700)) {
+		/* g_mkdir -> g_mkdir_with_parent: Regression testcase:
+		 * member_configdir_deep_path
+		 */
+		if (g_mkdir_with_parents(member->configdir, 0700)) {
 			osync_error_set(error, OSYNC_ERROR_IO_ERROR, "Unable to create directory for member %li\n", member->id);
 			goto error;
 		}

Modified: trunk/tests/group-tests/check_member.c
==============================================================================
--- trunk/tests/group-tests/check_member.c	Sat May  9 12:04:41 2009	(r5640)
+++ trunk/tests/group-tests/check_member.c	Sat May  9 20:49:48 2009	(r5641)
@@ -182,7 +182,6 @@
 }
 END_TEST
 
-/** TODO: are non-existant paths more than one directory deep acceptable? */
 START_TEST (member_configdir_deep_path)
 {
 	char *testbed = setup_testbed("filter_save_and_load");