[tracker/wip/carlosg/libtracker-common-cleanup: 2/6] libtracker-common: Drop unused file util functions




commit b46fe81f08e21071582af5a9b85bfa5f2bad7c46
Author: Carlos Garnacho <carlosg gnome org>
Date:   Tue Sep 8 13:34:01 2020 +0200

    libtracker-common: Drop unused file util functions
    
    We use a very limited portion of this API in the Tracker repo, throw
    away the remainders.

 src/libtracker-common/tracker-file-utils.c        | 576 +---------------------
 src/libtracker-common/tracker-file-utils.h        |  23 -
 tests/libtracker-common/tracker-file-utils-test.c | 351 -------------
 3 files changed, 1 insertion(+), 949 deletions(-)
---
diff --git a/src/libtracker-common/tracker-file-utils.c b/src/libtracker-common/tracker-file-utils.c
index 1962a23d2..6dbb416da 100644
--- a/src/libtracker-common/tracker-file-utils.c
+++ b/src/libtracker-common/tracker-file-utils.c
@@ -43,64 +43,6 @@
 
 #define TEXT_SNIFF_SIZE 4096
 
-int
-tracker_file_open_fd (const gchar *path)
-{
-       int fd;
-
-       g_return_val_if_fail (path != NULL, -1);
-
-#if defined(__linux__)
-       fd = g_open (path, O_RDONLY | O_NOATIME, 0);
-       if (fd == -1 && errno == EPERM) {
-               fd = g_open (path, O_RDONLY, 0);
-       }
-#else
-       fd = g_open (path, O_RDONLY, 0);
-#endif
-
-       return fd;
-}
-
-FILE *
-tracker_file_open (const gchar *path)
-{
-       FILE *file;
-       int fd;
-
-       g_return_val_if_fail (path != NULL, NULL);
-
-       fd = tracker_file_open_fd (path);
-
-       if (fd == -1) {
-               return NULL;
-       }
-
-       file = fdopen (fd, "r");
-
-       if (!file) {
-               return NULL;
-       }
-
-       return file;
-}
-
-void
-tracker_file_close (FILE     *file,
-                    gboolean  need_again_soon)
-{
-       g_return_if_fail (file != NULL);
-
-#ifdef HAVE_POSIX_FADVISE
-       if (!need_again_soon) {
-               if (posix_fadvise (fileno (file), 0, 0, POSIX_FADV_DONTNEED) != 0)
-                       g_warning ("posix_fadvise() call failed: %m");
-       }
-#endif /* HAVE_POSIX_FADVISE */
-
-       fclose (file);
-}
-
 goffset
 tracker_file_get_size (const gchar *path)
 {
@@ -138,106 +80,6 @@ tracker_file_get_size (const gchar *path)
        return size;
 }
 
-static
-guint64
-file_get_mtime (GFile *file)
-{
-       GFileInfo *info;
-       GError    *error = NULL;
-       guint64    mtime;
-
-       info = g_file_query_info (file,
-                                 G_FILE_ATTRIBUTE_TIME_MODIFIED,
-                                 G_FILE_QUERY_INFO_NONE,
-                                 NULL,
-                                 &error);
-
-       if (G_UNLIKELY (error)) {
-               gchar *uri;
-
-               uri = g_file_get_uri (file);
-               g_message ("Could not get mtime for '%s': %s",
-                          uri,
-                          error->message);
-               g_free (uri);
-               g_error_free (error);
-               mtime = 0;
-       } else {
-               mtime = g_file_info_get_attribute_uint64 (info, G_FILE_ATTRIBUTE_TIME_MODIFIED);
-               g_object_unref (info);
-       }
-
-       return mtime;
-}
-
-guint64
-tracker_file_get_mtime (const gchar *path)
-{
-       GFile     *file;
-       guint64    mtime;
-
-       g_return_val_if_fail (path != NULL, 0);
-
-       file = g_file_new_for_path (path);
-
-       mtime = file_get_mtime (file);
-
-       g_object_unref (file);
-
-       return mtime;
-}
-
-
-guint64
-tracker_file_get_mtime_uri (const gchar *uri)
-{
-       GFile     *file;
-       guint64    mtime;
-
-       g_return_val_if_fail (uri != NULL, 0);
-
-       file = g_file_new_for_uri (uri);
-
-       mtime = file_get_mtime (file);
-
-       g_object_unref (file);
-
-       return mtime;
-}
-
-gchar *
-tracker_file_get_mime_type (GFile *file)
-{
-       GFileInfo *info;
-       GError    *error = NULL;
-       gchar     *content_type;
-
-       g_return_val_if_fail (G_IS_FILE (file), NULL);
-
-       info = g_file_query_info (file,
-                                 G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE,
-                                 G_FILE_QUERY_INFO_NONE,
-                                 NULL,
-                                 &error);
-
-       if (G_UNLIKELY (error)) {
-               gchar *uri;
-
-               uri = g_file_get_uri (file);
-               g_message ("Could not guess mimetype for '%s', %s",
-                          uri,
-                          error->message);
-               g_free (uri);
-               g_error_free (error);
-               content_type = NULL;
-       } else {
-               content_type = g_strdup (g_file_info_get_content_type (info));
-               g_object_unref (info);
-       }
-
-       return content_type ? content_type : g_strdup ("unknown");
-}
-
 #ifdef __linux__
 
 #define __bsize f_bsize
@@ -290,7 +132,7 @@ statvfs_helper (const gchar *path, struct __statvfs *st)
        return (retval == 0);
 }
 
-guint64
+static guint64
 tracker_file_system_get_remaining_space (const gchar *path)
 {
        struct __statvfs st;
@@ -305,20 +147,6 @@ tracker_file_system_get_remaining_space (const gchar *path)
        }
 }
 
-gdouble
-tracker_file_system_get_remaining_space_percentage (const gchar *path)
-{
-       struct __statvfs st;
-       guint64 available;
-
-       if (statvfs_helper (path, &st)) {
-               available = (geteuid () == 0) ? st.f_bfree : st.f_bavail;
-               return (((gdouble) available * 100) / st.f_blocks);
-       } else {
-               return 0.0;
-       }
-}
-
 gboolean
 tracker_file_system_has_enough_space (const gchar *path,
                                       gulong       required_bytes,
@@ -356,405 +184,3 @@ tracker_file_system_has_enough_space (const gchar *path,
 
        return enough;
 }
-
-gboolean
-tracker_path_is_in_path (const gchar *path,
-                         const gchar *in_path)
-{
-       gchar    *new_path;
-       gchar    *new_in_path;
-       gboolean  is_in_path = FALSE;
-
-       g_return_val_if_fail (path != NULL, FALSE);
-       g_return_val_if_fail (in_path != NULL, FALSE);
-
-       if (!g_str_has_suffix (path, G_DIR_SEPARATOR_S)) {
-               new_path = g_strconcat (path, G_DIR_SEPARATOR_S, NULL);
-       } else {
-               new_path = g_strdup (path);
-       }
-
-       if (!g_str_has_suffix (in_path, G_DIR_SEPARATOR_S)) {
-               new_in_path = g_strconcat (in_path, G_DIR_SEPARATOR_S, NULL);
-       } else {
-               new_in_path = g_strdup (in_path);
-       }
-
-       if (g_str_has_prefix (new_path, new_in_path)) {
-               is_in_path = TRUE;
-       }
-
-       g_free (new_in_path);
-       g_free (new_path);
-
-       return is_in_path;
-}
-
-GSList *
-tracker_path_list_filter_duplicates (GSList      *roots,
-                                     const gchar *basename_exception_prefix,
-                                     gboolean     is_recursive)
-{
-       GSList *l1, *l2;
-       GSList *new_list;
-
-       new_list = tracker_gslist_copy_with_string_data (roots);
-       l1 = new_list;
-
-       while (l1) {
-               const gchar *path;
-               gchar       *p;
-               gboolean     reset = FALSE;
-
-               path = l1->data;
-
-               l2 = new_list;
-
-               while (l2 && !reset) {
-                       const gchar *in_path;
-
-                       in_path = l2->data;
-
-                       if (path == in_path) {
-                               /* Do nothing */
-                               l2 = l2->next;
-                               continue;
-                       }
-
-                       if (basename_exception_prefix) {
-                               gchar *lbasename;
-                               gboolean has_prefix = FALSE;
-
-                               lbasename = g_path_get_basename (path);
-                               if (!g_str_has_prefix (lbasename, basename_exception_prefix)) {
-                                       g_free (lbasename);
-
-                                       lbasename = g_path_get_basename (in_path);
-                                       if (g_str_has_prefix (lbasename, basename_exception_prefix)) {
-                                               has_prefix = TRUE;
-                                       }
-                               } else {
-                                       has_prefix = TRUE;
-                               }
-
-                               g_free (lbasename);
-
-                               /* This is so we can ignore this check
-                                * on files which prefix with ".".
-                                */
-                               if (has_prefix) {
-                                       l2 = l2->next;
-                                       continue;
-                               }
-                       }
-
-                       if (is_recursive && tracker_path_is_in_path (path, in_path)) {
-                               g_debug ("Removing path:'%s', it is in path:'%s'",
-                                        path, in_path);
-
-                               g_free (l1->data);
-                               new_list = g_slist_delete_link (new_list, l1);
-                               l1 = new_list;
-
-                               reset = TRUE;
-
-                               continue;
-                       } else if (is_recursive && tracker_path_is_in_path (in_path, path)) {
-                               g_debug ("Removing path:'%s', it is in path:'%s'",
-                                        in_path, path);
-
-                               g_free (l2->data);
-                               new_list = g_slist_delete_link (new_list, l2);
-                               l2 = new_list;
-
-                               reset = TRUE;
-
-                               continue;
-                       }
-
-                       l2 = l2->next;
-               }
-
-               if (G_LIKELY (!reset)) {
-                       p = strrchr (path, G_DIR_SEPARATOR);
-
-                       /* Make sure the path doesn't have the '/' suffix. */
-                       if (p && !p[1]) {
-                               *p = '\0';
-                       }
-
-                       l1 = l1->next;
-               }
-       }
-
-#ifdef TESTING
-       g_debug ("GSList paths were filtered down to:");
-
-       if (TRUE) {
-               GSList *l;
-
-               for (l = new_list; l; l = l->next) {
-                       g_debug ("  %s", (gchar*) l->data);
-               }
-       }
-#endif /* TESTING */
-
-       return new_list;
-}
-
-const struct {
-       const gchar *symbol;
-       GUserDirectory user_dir;
-} special_dirs[] = {
-       {"&DESKTOP",      G_USER_DIRECTORY_DESKTOP},
-       {"&DOCUMENTS",    G_USER_DIRECTORY_DOCUMENTS},
-       {"&DOWNLOAD",     G_USER_DIRECTORY_DOWNLOAD},
-       {"&MUSIC",        G_USER_DIRECTORY_MUSIC},
-       {"&PICTURES",     G_USER_DIRECTORY_PICTURES},
-       {"&PUBLIC_SHARE", G_USER_DIRECTORY_PUBLIC_SHARE},
-       {"&TEMPLATES",    G_USER_DIRECTORY_TEMPLATES},
-       {"&VIDEOS",       G_USER_DIRECTORY_VIDEOS}
-};
-
-
-static gboolean
-get_user_special_dir_if_not_home (const gchar  *path,
-                                  gchar       **special_dir)
-{
-       int i;
-       const gchar *real_path;
-       GFile *home, *file;
-
-       real_path = NULL;
-       *special_dir = NULL;
-
-       for (i = 0; i < G_N_ELEMENTS(special_dirs); i++) {
-               if (strcmp (path, special_dirs[i].symbol) == 0) {
-                       real_path = g_get_user_special_dir (special_dirs[i].user_dir);
-
-                       if (real_path == NULL) {
-                               g_warning ("Unable to get XDG user directory path for special "
-                                          "directory %s. Ignoring this location.", path);
-                       }
-
-                       break;
-               }
-       }
-
-       if (real_path == NULL)
-               return FALSE;
-
-       file = g_file_new_for_path (real_path);
-       home = g_file_new_for_path (g_get_home_dir ());
-
-       /* ignore XDG directories set to $HOME */
-       if (!g_file_equal (file, home)) {
-               *special_dir = g_strdup (real_path);
-       }
-
-       g_object_unref (file);
-       g_object_unref (home);
-
-       return TRUE;
-}
-
-
-gchar *
-tracker_path_evaluate_name (const gchar *path)
-{
-       gchar        *special_dir_path;
-       gchar        *final_path;
-       gchar       **tokens;
-       gchar       **token;
-       gchar        *start;
-       gchar        *end;
-       const gchar  *env;
-       gchar        *expanded;
-
-       if (!path || path[0] == '\0') {
-               return NULL;
-       }
-
-       /* See if it is a special directory name. */
-       if (get_user_special_dir_if_not_home (path, &special_dir_path))
-               return special_dir_path;
-
-       /* First check the simple case of using tilde */
-       if (path[0] == '~') {
-               const gchar *home;
-
-               home = g_getenv ("HOME");
-               if (! home) {
-                       home = g_get_home_dir ();
-               }
-
-               if (!home || home[0] == '\0') {
-                       return NULL;
-               }
-
-               return g_build_path (G_DIR_SEPARATOR_S,
-                                    home,
-                                    path + 1,
-                                    NULL);
-       }
-
-       /* Second try to find any environment variables and expand
-        * them, like $HOME or ${FOO}
-        */
-       tokens = g_strsplit (path, G_DIR_SEPARATOR_S, -1);
-
-       for (token = tokens; *token; token++) {
-               if (**token != '$') {
-                       continue;
-               }
-
-               start = *token + 1;
-
-               if (*start == '{') {
-                       start++;
-                       end = start + (strlen (start)) - 1;
-                       *end='\0';
-               }
-
-               env = g_getenv (start);
-               g_free (*token);
-
-               /* Don't do g_strdup (s?s1:s2) as that doesn't work
-                * with certain gcc 2.96 versions.
-                */
-               *token = env ? g_strdup (env) : g_strdup ("");
-       }
-
-       /* Third get the real path removing any "../" and other
-        * symbolic links to other places, returning only the REAL
-        * location.
-        */
-       expanded = g_strjoinv (G_DIR_SEPARATOR_S, tokens);
-       g_strfreev (tokens);
-
-       /* Only resolve relative paths if there is a directory
-        * separator in the path, otherwise it is just a name.
-        */
-       if (strchr (expanded, G_DIR_SEPARATOR)) {
-               GFile *file;
-
-               file = g_file_new_for_commandline_arg (expanded);
-               final_path = g_file_get_path (file);
-               g_object_unref (file);
-               g_free (expanded);
-       } else {
-               final_path = expanded;
-       }
-
-       return final_path;
-}
-
-gboolean
-tracker_file_is_hidden (GFile *file)
-{
-       GFileInfo *file_info;
-       gboolean is_hidden = FALSE;
-
-       file_info = g_file_query_info (file,
-                                      G_FILE_ATTRIBUTE_STANDARD_IS_HIDDEN,
-                                      G_FILE_QUERY_INFO_NOFOLLOW_SYMLINKS,
-                                      NULL, NULL);
-       if (file_info) {
-               /* Check if GIO says the file is hidden */
-               is_hidden = g_file_info_get_is_hidden (file_info);
-               g_object_unref (file_info);
-       } else {
-               gchar *basename;
-
-               /* Resort last to basename checks, this might happen on
-                * already deleted files.
-                */
-               basename = g_file_get_basename (file);
-               is_hidden = basename[0] == '.';
-               g_free (basename);
-       }
-
-       return is_hidden;
-}
-
-gint
-tracker_file_cmp (GFile *file_a,
-                  GFile *file_b)
-{
-       /* Returns 0 if files are equal.
-        * Useful to be used in g_list_find_custom() or g_queue_find_custom() */
-       return !g_file_equal (file_a, file_b);
-}
-
-/**
- * tracker_filename_casecmp_without_extension:
- * @a: a string containing a file name
- * @b: filename to be compared with @a
- *
- * This function performs a case-insensitive comparison of @a and @b.
- * Additionally, text beyond the last '.' in a string is not considered
- * part of the match, so for example given the inputs "file.mp3" and
- * "file.wav" this function will return %TRUE.
- *
- * Internally, the g_ascii_tolower() function is used - this means that
- * @a and @b must be in an encoding in which ASCII characters always
- * represent themselves, such as UTF-8 or the ISO-8859-* charsets.
- *
- * Returns: %TRUE if the two file names match.
- **/
-gboolean
-tracker_filename_casecmp_without_extension (const gchar *a,
-                                            const gchar *b)
-{
-       gchar *pa;
-       gchar *pb;
-       gint len_a;
-       gint len_b;
-
-       g_return_val_if_fail (a != NULL, FALSE);
-       g_return_val_if_fail (b != NULL, FALSE);
-
-       pa = strrchr (a, '.');
-       pb = strrchr (b, '.');
-
-       /* Did we find a "." */
-       if (pa) {
-               len_a = pa - a;
-       } else {
-               len_a = -1;
-       }
-
-       if (pb) {
-               len_b = pb - b;
-       } else {
-               len_b = -1;
-       }
-
-       /* If one has a "." and the other doesn't, we do length
-        * comparison with strlen() which is less optimal but this is
-        * not a case we consider common operation.
-        */
-       if (len_a == -1 && len_b > -1) {
-               len_a = strlen (a);
-       } else if (len_b == -1 && len_a > -1) {
-               len_b = strlen (b);
-       }
-
-       /* If we have length for both and it's different then these
-        * strings are not the same. If we have no length for the
-        * strings then it's a simple -1 != -1 comparison.
-        */
-       if (len_a != len_b) {
-               return FALSE;
-       }
-
-       /* Now we know we either have the same length string or no
-        * extension in a and b, meaning it's a strcmp() of the
-        * string only. We test only len_a or len_b here for that:
-        */
-       if (G_UNLIKELY (len_a == -1)) {
-               return g_ascii_strcasecmp (a, b) == 0;
-       }
-
-       return g_ascii_strncasecmp (a, b, len_a) == 0;
-}
diff --git a/src/libtracker-common/tracker-file-utils.h b/src/libtracker-common/tracker-file-utils.h
index 45c5f9374..679a4543e 100644
--- a/src/libtracker-common/tracker-file-utils.h
+++ b/src/libtracker-common/tracker-file-utils.h
@@ -32,35 +32,12 @@ G_BEGIN_DECLS
 #endif
 
 /* File utils */
-int      tracker_file_open_fd                               (const gchar *path);
-FILE*    tracker_file_open                                  (const gchar *path);
-void     tracker_file_close                                 (FILE        *file,
-                                                             gboolean     need_again_soon);
 goffset  tracker_file_get_size                              (const gchar *path);
-guint64  tracker_file_get_mtime                             (const gchar *path);
-guint64  tracker_file_get_mtime_uri                         (const gchar *uri);
-gchar *  tracker_file_get_mime_type                         (GFile       *file);
-gboolean tracker_file_is_locked                             (GFile       *file);
-gboolean tracker_file_is_hidden                             (GFile       *file);
-gint     tracker_file_cmp                                   (GFile       *file_a,
-                                                             GFile       *file_b);
-
-/* Path utils */
-gboolean tracker_path_is_in_path                            (const gchar *path,
-                                                             const gchar *in_path);
-GSList * tracker_path_list_filter_duplicates                (GSList      *roots,
-                                                             const gchar *basename_exception_prefix,
-                                                             gboolean     is_recursive);
-gchar *  tracker_path_evaluate_name                         (const gchar *uri);
-gboolean tracker_filename_casecmp_without_extension         (const gchar *a,
-                                                             const gchar *b);
 
 /* File system utils */
 gboolean tracker_file_system_has_enough_space               (const gchar *path,
                                                              gulong       required_bytes,
                                                              gboolean     creating_db);
-guint64  tracker_file_system_get_remaining_space            (const gchar *path);
-gdouble  tracker_file_system_get_remaining_space_percentage (const gchar *path);
 
 G_END_DECLS
 
diff --git a/tests/libtracker-common/tracker-file-utils-test.c 
b/tests/libtracker-common/tracker-file-utils-test.c
index a508386fb..1286aa6e8 100644
--- a/tests/libtracker-common/tracker-file-utils-test.c
+++ b/tests/libtracker-common/tracker-file-utils-test.c
@@ -50,251 +50,6 @@ remove_file (const gchar *filename)
         g_assert_cmpint (g_remove (filename), ==, 0);
 }
 
-static GSList *
-array_as_list (const gchar **array)
-{
-       gint i;
-       GSList *result = NULL;
-
-       for (i = 0; array[i] != NULL; i++) {
-               result = g_slist_prepend (result, g_strdup(array[i]));
-
-       }
-
-       return result;
-}
-
-static gboolean
-string_in_list (GSList *list, const gchar *string)
-{
-       GSList *it;
-       for ( it = list; it != NULL; it = it->next) {
-               if (strcmp (it->data, string) == 0) {
-                       return TRUE;
-               }
-       }
-       return FALSE;
-}
-
-static void
-test_path_list_filter_duplicates (void)
-{
-       const gchar *input_roots [] = {"/home/ivan",
-                                       "/home",
-                                       "/tmp",
-                                       "/usr/",
-                                       "/usr/share/local", NULL};
-
-       GSList *input_as_list = NULL;
-       GSList *result;
-
-       input_as_list = array_as_list (input_roots);
-
-       result = tracker_path_list_filter_duplicates (input_as_list, ".", TRUE);
-       g_assert_cmpint (3, ==, g_slist_length (result));
-
-       g_assert_true (string_in_list (result, "/home"));
-       g_assert_true (string_in_list (result, "/tmp"));
-       g_assert_true (string_in_list (result, "/usr"));
-
-       g_slist_foreach (input_as_list, (GFunc) g_free, NULL);
-       g_slist_foreach (result, (GFunc) g_free, NULL);
-}
-
-static void
-test_path_list_filter_duplicates_with_exceptions ()
-{
-        const gchar *input_roots [] = { "/home/user/MyDocs",
-                                        "/home/user/MyDocs/.sounds",
-                                        "/home/user/MyDocs/visible",
-                                        NULL};
-        GSList *input_as_list = NULL, *result = NULL;
-
-        input_as_list = array_as_list (input_roots);
-
-        result = tracker_path_list_filter_duplicates (input_as_list, "/home/user/MyDocs", FALSE);
-        g_assert_cmpint (g_slist_length (result), ==, 3);
-       g_assert_true (string_in_list (result, "/home/user/MyDocs"));
-       g_assert_true (string_in_list (result, "/home/user/MyDocs/.sounds"));
-       g_assert_true (string_in_list (result, "/home/user/MyDocs/visible"));
-       g_slist_foreach (result, (GFunc) g_free, NULL);
-
-
-        result = tracker_path_list_filter_duplicates (input_as_list, "/home/user/MyDocs", TRUE);
-        g_assert_cmpint (g_slist_length (result), ==, 1);
-       g_assert_true (string_in_list (result, "/home/user/MyDocs"));
-       g_slist_foreach (result, (GFunc) g_free, NULL);
-
-       g_slist_foreach (input_as_list, (GFunc) g_free, NULL);
-}
-
-static void
-test_path_evaluate_name (void)
-{
-       gchar *result, *expected, *pwd, *home;
-
-
-       const gchar *test = "/one/two";
-       gchar *parent_dir;
-
-       home = g_strdup (g_getenv ("HOME"));
-       pwd = g_get_current_dir ();
-       g_setenv ("TEST_TRACKER_DIR", test, TRUE);
-
-
-       result = tracker_path_evaluate_name ("/home/user/all/ok");
-       g_assert_cmpstr (result, ==, "/home/user/all/ok");
-       g_free (result);
-
-       /* The result of this test and the next one are not consistent!
-        * Must it remove the end '/' or not?
-        */
-       result = tracker_path_evaluate_name ("/home/user/all/dir/");
-       g_assert_cmpstr (result, ==, "/home/user/all/dir");
-       g_free (result);
-
-
-       /*
-        * TODO: In valgrind this test shows a memory leak
-        */
-       result = tracker_path_evaluate_name ("~/all/dir/");
-       expected = g_build_path (G_DIR_SEPARATOR_S, home, "/all/dir/", NULL);
-       g_assert_cmpstr (result, ==, expected);
-       g_free (result);
-       g_free (expected);
-
-       result = tracker_path_evaluate_name ("just-a-filename");
-       g_assert_cmpstr (result, ==, "just-a-filename");
-       g_free (result);
-
-       result = tracker_path_evaluate_name ("$HOME/all/dir/");
-       expected = g_build_path (G_DIR_SEPARATOR_S, home, "/all/dir", NULL);
-       g_assert_cmpstr (result, ==, expected);
-       g_free (result);
-       g_free (expected);
-
-       result = tracker_path_evaluate_name ("${HOME}/all/dir/");
-       expected = g_build_path (G_DIR_SEPARATOR_S, home, "/all/dir", NULL);
-       g_assert_cmpstr (result, ==, expected);
-       g_free (result);
-       g_free (expected);
-
-       result = tracker_path_evaluate_name ("./test/current/dir");
-       expected = g_build_path (G_DIR_SEPARATOR_S, pwd, "/test/current/dir", NULL);
-       g_assert_cmpstr (result, ==, expected);
-       g_free (result);
-       g_free (expected);
-
-       result = tracker_path_evaluate_name ("$TEST_TRACKER_DIR/test/dir");
-       expected = g_build_path (G_DIR_SEPARATOR_S, test, "/test/dir", NULL);
-       g_assert_cmpstr (result, ==, expected);
-       g_free (result);
-       g_free (expected);
-
-       result = tracker_path_evaluate_name ("../test/dir");
-       parent_dir = g_path_get_dirname (pwd);
-       expected = g_build_path (G_DIR_SEPARATOR_S, parent_dir, "/test/dir", NULL);
-       g_assert_cmpstr (result, ==, expected);
-       g_free (result);
-       g_free (parent_dir);
-       g_free (expected);
-
-       result = tracker_path_evaluate_name ("");
-       g_assert_true (!result);
-       g_free (result);
-
-       result = tracker_path_evaluate_name (NULL);
-       g_assert_true (!result);
-       g_free (result);
-
-        g_setenv ("HOME", "", TRUE);
-        result = tracker_path_evaluate_name ("~/but-no-home.txt");
-        g_assert_true (!result);
-        g_free (result);
-        g_setenv ("HOME", home, TRUE);
-
-        result = tracker_path_evaluate_name ("$UNDEFINED/something");
-        g_assert_cmpstr (result, ==, "/something");
-        g_free (result);
-
-       result = tracker_path_evaluate_name (tracker_test_helpers_get_nonutf8 ());
-       g_assert_cmpstr (result, ==, tracker_test_helpers_get_nonutf8 ());
-       g_free (result);
-
-       g_free (home);
-       g_free (pwd);
-       g_unsetenv ("TEST_TRACKER_DIR");
-}
-
-
-static void
-test_file_get_mime_type (void)
-{
-       gchar *result;
-       GFile *f;
-
-        f = g_file_new_for_path (TEST_FILENAME);
-        result = tracker_file_get_mime_type (f);
-        g_assert_true (g_strcmp0 (result, "text/plain") == 0 ||
-                       g_strcmp0 (result, "application/x-zerosize") == 0);
-
-        g_object_unref (f);
-        g_free (result);
-
-        f = g_file_new_for_path ("./file-does-NOT-exist");
-        result = tracker_file_get_mime_type (f);
-        g_assert_cmpstr (result, ==, "unknown");
-
-        g_object_unref (f);
-        g_free (result);
-
-}
-
-#define assert_filename_match(a, b) { \
-       g_assert_cmpint (tracker_filename_casecmp_without_extension (a, b), ==, TRUE); \
-       g_assert_cmpint (tracker_filename_casecmp_without_extension (b, a), ==, TRUE); }
-
-#define assert_no_filename_match(a, b) { \
-       g_assert_cmpint (tracker_filename_casecmp_without_extension (a, b), ==, FALSE); \
-       g_assert_cmpint (tracker_filename_casecmp_without_extension (b, a), ==, FALSE); }
-
-static void
-test_case_match_filename_without_extension ()
-{
-       assert_filename_match ("test.mp3", "test.mp3");
-       assert_filename_match ("test.mp3", "test.wav");
-       assert_filename_match ("test.mp3", "test.mp");
-       assert_filename_match ("test.mp3", "test.");
-       assert_filename_match ("test.mp3", "test");
-       assert_filename_match ("01 - Song 1 (Remix).wav", "01 - Song 1 (Remix).flac");
-
-       assert_no_filename_match ("test.mp3", "bacon.mp3");
-
-       /* Pathological cases, mainly testing that nothing crashes */
-       assert_no_filename_match (".", "\n");
-       assert_no_filename_match ("as", "as..");
-       assert_no_filename_match ("...as", "...as..");
-       assert_no_filename_match (".", "test.");
-       assert_filename_match ("", ".");
-}
-
-static void
-test_file_utils_open_close ()
-{
-        FILE *f;
-
-        f = tracker_file_open (TEST_FILENAME);
-        g_assert_true (f);
-        tracker_file_close (f, TRUE);
-
-        f = tracker_file_open (TEST_FILENAME);
-        g_assert_true (f);
-        tracker_file_close (f, FALSE);
-
-        f = tracker_file_open ("./file-does-NOT-exist");
-        g_assert_null (f);
-}
-
 static void
 test_file_utils_get_size ()
 {
@@ -312,58 +67,6 @@ test_file_utils_get_size ()
         g_assert_cmpint (size, ==, 0);
 }
 
-static void
-test_file_utils_get_mtime ()
-{
-        guint64 mtime;
-        struct stat st;
-        gchar *pwd, *uri;
-
-        mtime = tracker_file_get_mtime (TEST_FILENAME);
-        g_assert_cmpint (mtime, >, 0);
-
-        g_assert_cmpint (stat (TEST_FILENAME, &st), ==, 0);
-        // This comparison could lead a problem in 32/64 bits?
-        g_assert_cmpint (mtime, ==, st.st_mtime);
-
-        pwd = g_get_current_dir ();
-        uri = g_strdup_printf ("file://%s/%s", pwd, TEST_FILENAME);
-        mtime = tracker_file_get_mtime_uri (uri);
-        // This comparison could lead a problem in 32/64 bits?
-        g_assert_cmpint (mtime, ==, st.st_mtime);
-
-        g_free (pwd);
-        g_free (uri);
-
-        mtime = tracker_file_get_mtime_uri ("./file-does-NOT-exist");
-        g_assert_cmpint (mtime, ==, 0);
-}
-
-static void
-test_file_system_get_remaining_space ()
-{
-        guint64 space;
-
-        space = tracker_file_system_get_remaining_space ("/home");
-        g_assert_cmpint (space, >, 0);
-
-        // This is a critical (aborts the process)
-        //space = tracker_file_system_get_remaining_space ("/unlikely/to/have/this/folder");
-}
-
-static void
-test_file_system_get_remaining_space_percentage ()
-{
-        gdouble space;
-
-        space = tracker_file_system_get_remaining_space_percentage ("/home");
-        g_assert_cmpfloat (space, >=, 0);
-        g_assert_cmpfloat (space, <=, 100);
-
-        // This is a critical (aborts the process)
-        //space = tracker_file_system_get_remaining_space_percentage ("/unlikely/to/have/this/folder");
-}
-
 static void
 test_file_system_has_enough_space ()
 {
@@ -375,37 +78,6 @@ test_file_system_has_enough_space ()
         //g_assert_true (!tracker_file_system_has_enough_space ("/home", G_MAXULONG, FALSE));
 }
 
-static void
-test_file_utils_is_hidden ()
-{
-        GFile *f;
-
-        ensure_file_exists ("./non-hidden-test-file");
-
-        f = g_file_new_for_path (TEST_HIDDEN_FILENAME);
-        g_assert_true (tracker_file_is_hidden (f));
-        g_object_unref (f);
-
-        f = g_file_new_for_path ("./non-hidden-test-file");
-        g_assert_true (!tracker_file_is_hidden (f));
-        g_object_unref (f);
-
-        remove_file ("./non-hidden-test-file");
-}
-
-static void
-test_file_utils_cmp ()
-{
-        GFile *one, *two, *three;
-
-        one = g_file_new_for_path (TEST_FILENAME);
-        two = g_file_new_for_path (TEST_FILENAME);
-        three = g_file_new_for_path (TEST_HIDDEN_FILENAME);
-
-        g_assert_true (!tracker_file_cmp (one, two));
-        g_assert_true (tracker_file_cmp (two, three));
-}
-
 int
 main (int argc, char **argv)
 {
@@ -418,33 +90,10 @@ main (int argc, char **argv)
         ensure_file_exists (TEST_FILENAME);
         ensure_file_exists (TEST_HIDDEN_FILENAME);
 
-       g_test_add_func ("/libtracker-common/file-utils/path_evaluate_name",
-                        test_path_evaluate_name);
-       g_test_add_func ("/libtracker-common/file-utils/path_list_filter_duplicates",
-                        test_path_list_filter_duplicates);
-       g_test_add_func ("/libtracker-common/file-utils/path_list_filter_duplicates_with_exceptions",
-                        test_path_list_filter_duplicates_with_exceptions);
-       g_test_add_func ("/libtracker-common/file-utils/file_get_mime_type",
-                        test_file_get_mime_type);
-       g_test_add_func ("/libtracker-common/file-utils/case_match_filename_without_extension",
-                        test_case_match_filename_without_extension);
-
-        g_test_add_func ("/libtracker-common/file-utils/open_close",
-                         test_file_utils_open_close);
         g_test_add_func ("/libtracker-common/file-utils/get_size",
                          test_file_utils_get_size);
-        g_test_add_func ("/libtracker-common/file-utils/get_mtime",
-                         test_file_utils_get_mtime);
-        g_test_add_func ("/libtracker-common/file-utils/get_remaining_space",
-                         test_file_system_get_remaining_space);
-        g_test_add_func ("/libtracker-common/file-utils/get_remaining_space_percentage",
-                         test_file_system_get_remaining_space_percentage);
         g_test_add_func ("/libtracker-common/file-utils/has_enough_space",
                          test_file_system_has_enough_space);
-        g_test_add_func ("/libtracker-common/file-utils/is_hidden",
-                         test_file_utils_is_hidden);
-        g_test_add_func ("/libtracker-common/file-utils/cmp",
-                         test_file_utils_cmp);
 
        result = g_test_run ();
 


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