X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/698581160d470eb3e3f2ff67aebb04668642dcb9..c29c95fe24973b94fd724db767193171ca7c513d:/tests/filename/filenametest.cpp?ds=sidebyside diff --git a/tests/filename/filenametest.cpp b/tests/filename/filenametest.cpp index 813079ca8e..a1ea735acd 100644 --- a/tests/filename/filenametest.cpp +++ b/tests/filename/filenametest.cpp @@ -23,78 +23,89 @@ #include "wx/filename.h" #include "wx/filefn.h" +#include "wx/stdpaths.h" -// ---------------------------------------------------------------------------- -// local functions -// ---------------------------------------------------------------------------- +#ifdef __WXMSW__ + #include "wx/msw/registry.h" +#endif // __WXMSW__ -// define stream inserter for wxFileName to use it in CPPUNIT_ASSERT_EQUAL() -inline std::ostream& operator<<(std::ostream& o, const wxFileName& fn) -{ - return o << fn.GetFullPath(); -} +#include "testfile.h" // ---------------------------------------------------------------------------- // test data // ---------------------------------------------------------------------------- -static struct FileNameInfo +static struct TestFileNameInfo { - const wxChar *fullname; - const wxChar *volume; - const wxChar *path; - const wxChar *name; - const wxChar *ext; + const char *fullname; + const char *volume; + const char *path; + const char *name; + const char *ext; bool isAbsolute; wxPathFormat format; } filenames[] = { // the empty string - { _T(""), _(""), _(""), _(""), _(""), false, wxPATH_UNIX }, - { _T(""), _(""), _(""), _(""), _(""), false, wxPATH_DOS }, - { _T(""), _(""), _(""), _(""), _(""), false, wxPATH_VMS }, + { "", "", "", "", "", false, wxPATH_UNIX }, + { "", "", "", "", "", false, wxPATH_DOS }, + { "", "", "", "", "", false, wxPATH_VMS }, // Unix file names - { _T("/usr/bin/ls"), _T(""), _T("/usr/bin"), _T("ls"), _T(""), true, wxPATH_UNIX }, - { _T("/usr/bin/"), _T(""), _T("/usr/bin"), _T(""), _T(""), true, wxPATH_UNIX }, - { _T("~/.zshrc"), _T(""), _T("~"), _T(".zshrc"), _T(""), true, wxPATH_UNIX }, - { _T("../../foo"), _T(""), _T("../.."), _T("foo"), _T(""), false, wxPATH_UNIX }, - { _T("foo.bar"), _T(""), _T(""), _T("foo"), _T("bar"), false, wxPATH_UNIX }, - { _T("~/foo.bar"), _T(""), _T("~"), _T("foo"), _T("bar"), true, wxPATH_UNIX }, - { _T("/foo"), _T(""), _T("/"), _T("foo"), _T(""), true, wxPATH_UNIX }, - { _T("Mahogany-0.60/foo.bar"), _T(""), _T("Mahogany-0.60"), _T("foo"), _T("bar"), false, wxPATH_UNIX }, - { _T("/tmp/wxwin.tar.bz"), _T(""), _T("/tmp"), _T("wxwin.tar"), _T("bz"), true, wxPATH_UNIX }, + { "/usr/bin/ls", "", "/usr/bin", "ls", "", true, wxPATH_UNIX }, + { "/usr/bin/", "", "/usr/bin", "", "", true, wxPATH_UNIX }, + { "~/.zshrc", "", "~", ".zshrc", "", true, wxPATH_UNIX }, + { "../../foo", "", "../..", "foo", "", false, wxPATH_UNIX }, + { "foo.bar", "", "", "foo", "bar", false, wxPATH_UNIX }, + { "~/foo.bar", "", "~", "foo", "bar", true, wxPATH_UNIX }, + { "~user/foo.bar", "", "~user", "foo", "bar", true, wxPATH_UNIX }, + { "~user/", "", "~user", "", "", true, wxPATH_UNIX }, + { "/foo", "", "/", "foo", "", true, wxPATH_UNIX }, + { "Mahogany-0.60/foo.bar", "", "Mahogany-0.60", "foo", "bar", false, wxPATH_UNIX }, + { "/tmp/wxwin.tar.bz", "", "/tmp", "wxwin.tar", "bz", true, wxPATH_UNIX }, // Windows file names - { _T("foo.bar"), _T(""), _T(""), _T("foo"), _T("bar"), false, wxPATH_DOS }, - { _T("\\foo.bar"), _T(""), _T("\\"), _T("foo"), _T("bar"), false, wxPATH_DOS }, - { _T("c:foo.bar"), _T("c"), _T(""), _T("foo"), _T("bar"), false, wxPATH_DOS }, - { _T("c:\\foo.bar"), _T("c"), _T("\\"), _T("foo"), _T("bar"), true, wxPATH_DOS }, - { _T("c:\\Windows\\command.com"), _T("c"), _T("\\Windows"), _T("command"), _T("com"), true, wxPATH_DOS }, + { "foo.bar", "", "", "foo", "bar", false, wxPATH_DOS }, + { "\\foo.bar", "", "\\", "foo", "bar", false, wxPATH_DOS }, + { "c:foo.bar", "c", "", "foo", "bar", false, wxPATH_DOS }, + { "c:\\foo.bar", "c", "\\", "foo", "bar", true, wxPATH_DOS }, + { "c:\\Windows\\command.com", "c", "\\Windows", "command", "com", true, wxPATH_DOS }, + { "\\\\?\\Volume{8089d7d7-d0ac-11db-9dd0-806d6172696f}\\", + "Volume{8089d7d7-d0ac-11db-9dd0-806d6172696f}", "\\", "", "", true, wxPATH_DOS }, + { "\\\\?\\Volume{8089d7d7-d0ac-11db-9dd0-806d6172696f}\\Program Files\\setup.exe", + "Volume{8089d7d7-d0ac-11db-9dd0-806d6172696f}", "\\Program Files", "setup", "exe", true, wxPATH_DOS }, - // NB: when using the wxFileName::GetLongPath() function on these two strings, - // the program will hang various seconds. All those time is taken by the - // call to the win32 API GetLongPathName()... - { _T("\\\\server\\foo.bar"), _T("server"), _T("\\"), _T("foo"), _T("bar"), true, wxPATH_DOS }, - { _T("\\\\server\\dir\\foo.bar"), _T("server"), _T("\\dir"), _T("foo"), _T("bar"), true, wxPATH_DOS }, +#if 0 + // NB: when using the wxFileName::GetLongPath() function on these two + // strings, the program will hang for several seconds blocking inside + // Win32 GetLongPathName() function + { "\\\\server\\foo.bar", "server", "\\", "foo", "bar", true, wxPATH_DOS }, + { "\\\\server\\dir\\foo.bar", "server", "\\dir", "foo", "bar", true, wxPATH_DOS }, +#endif + // consecutive [back]slashes should be treated as single occurrences of + // them and not interpreted as share names if there is a volume name + { "c:\\aaa\\bbb\\ccc", "c", "\\aaa\\bbb", "ccc", "", true, wxPATH_DOS }, + { "c:\\\\aaa\\bbb\\ccc", "c", "\\\\aaa\\bbb", "ccc", "", true, wxPATH_DOS }, // wxFileName support for Mac file names is broken currently #if 0 // Mac file names - { _T("Volume:Dir:File"), _T("Volume"), _T("Dir"), _T("File"), _T(""), true, wxPATH_MAC }, - { _T("Volume:Dir:Subdir:File"), _T("Volume"), _T("Dir:Subdir"), _T("File"), _T(""), true, wxPATH_MAC }, - { _T("Volume:"), _T("Volume"), _T(""), _T(""), _T(""), true, wxPATH_MAC }, - { _T(":Dir:File"), _T(""), _T("Dir"), _T("File"), _T(""), false, wxPATH_MAC }, - { _T(":File.Ext"), _T(""), _T(""), _T("File"), _T(".Ext"), false, wxPATH_MAC }, - { _T("File.Ext"), _T(""), _T(""), _T("File"), _T(".Ext"), false, wxPATH_MAC }, + { "Volume:Dir:File", "Volume", "Dir", "File", "", true, wxPATH_MAC }, + { "Volume:Dir:Subdir:File", "Volume", "Dir:Subdir", "File", "", true, wxPATH_MAC }, + { "Volume:", "Volume", "", "", "", true, wxPATH_MAC }, + { ":Dir:File", "", "Dir", "File", "", false, wxPATH_MAC }, + { ":File.Ext", "", "", "File", ".Ext", false, wxPATH_MAC }, + { "File.Ext", "", "", "File", ".Ext", false, wxPATH_MAC }, #endif // 0 +#if 0 // VMS file names // NB: on Windows they have the same effect of the \\server\\ strings // (see the note above) - { _T("device:[dir1.dir2.dir3]file.txt"), _T("device"), _T("dir1.dir2.dir3"), _T("file"), _T("txt"), true, wxPATH_VMS }, - { _T("file.txt"), _T(""), _T(""), _T("file"), _T("txt"), false, wxPATH_VMS }, + { "device:[dir1.dir2.dir3]file.txt", "device", "dir1.dir2.dir3", "file", "txt", true, wxPATH_VMS }, +#endif + { "file.txt", "", "", "file", "txt", false, wxPATH_VMS }, }; // ---------------------------------------------------------------------------- @@ -114,9 +125,15 @@ private: CPPUNIT_TEST( TestSetPath ); CPPUNIT_TEST( TestStrip ); CPPUNIT_TEST( TestNormalize ); + CPPUNIT_TEST( TestReplace ); + CPPUNIT_TEST( TestGetHumanReadable ); #ifdef __WINDOWS__ CPPUNIT_TEST( TestShortLongPath ); #endif // __WINDOWS__ + CPPUNIT_TEST( TestUNC ); + CPPUNIT_TEST( TestVolumeUniqueName ); + CPPUNIT_TEST( TestCreateTempFileName ); + CPPUNIT_TEST( TestGetTimes ); CPPUNIT_TEST_SUITE_END(); void TestConstruction(); @@ -125,9 +142,15 @@ private: void TestSetPath(); void TestStrip(); void TestNormalize(); + void TestReplace(); + void TestGetHumanReadable(); #ifdef __WINDOWS__ void TestShortLongPath(); #endif // __WINDOWS__ + void TestUNC(); + void TestVolumeUniqueName(); + void TestCreateTempFileName(); + void TestGetTimes(); DECLARE_NO_COPY_CLASS(FileNameTestCase) }; @@ -135,27 +158,56 @@ private: // register in the unnamed registry so that these tests are run by default CPPUNIT_TEST_SUITE_REGISTRATION( FileNameTestCase ); -// also include in it's own registry so that these tests can be run alone +// also include in its own registry so that these tests can be run alone CPPUNIT_TEST_SUITE_NAMED_REGISTRATION( FileNameTestCase, "FileNameTestCase" ); void FileNameTestCase::TestConstruction() { for ( size_t n = 0; n < WXSIZEOF(filenames); n++ ) { - const FileNameInfo& fni = filenames[n]; + const TestFileNameInfo& fni = filenames[n]; wxFileName fn(fni.fullname, fni.format); + // the original full name could contain consecutive [back]slashes, + // squeeze them except for the double backslash in the beginning in + // Windows filenames where it has special meaning + wxString fullnameOrig; + if ( fni.format == wxPATH_DOS ) + { + // copy the backslashes at beginning unchanged + const char *p = fni.fullname; + while ( *p == '\\' ) + fullnameOrig += *p++; + + // replace consecutive slashes with single ones in the rest + for ( char chPrev = '\0'; *p; p++ ) + { + if ( *p == '\\' && chPrev == '\\' ) + continue; + + chPrev = *p; + fullnameOrig += chPrev; + } + } + else // !wxPATH_DOS + { + fullnameOrig = fni.fullname; + } + + fullnameOrig.Replace("//", "/"); + + wxString fullname = fn.GetFullPath(fni.format); - CPPUNIT_ASSERT_EQUAL( wxString(fni.fullname), fullname ); + CPPUNIT_ASSERT_EQUAL( fullnameOrig, fullname ); // notice that we use a dummy working directory to ensure that paths // with "../.." in them could be normalized, otherwise this would fail // if the test is run from root directory or its direct subdirectory CPPUNIT_ASSERT_MESSAGE ( - (const char *)wxString::Format(_T("Normalize(%s) failed"), fni.fullname).mb_str(), - fn.Normalize(wxPATH_NORM_ALL, _T("/foo/bar/baz"), fni.format) + (const char *)wxString::Format("Normalize(%s) failed", fni.fullname).mb_str(), + fn.Normalize(wxPATH_NORM_ALL, "/foo/bar/baz", fni.format) ); if ( *fni.volume && *fni.path ) @@ -205,7 +257,7 @@ void FileNameTestCase::TestSplit() { for ( size_t n = 0; n < WXSIZEOF(filenames); n++ ) { - const FileNameInfo& fni = filenames[n]; + const TestFileNameInfo& fni = filenames[n]; wxString volume, path, name, ext; wxFileName::SplitPath(fni.fullname, &volume, &path, &name, &ext, fni.format); @@ -217,19 +269,19 @@ void FileNameTestCase::TestSplit() } // special case of empty extension - wxFileName fn(_T("foo.")); - CPPUNIT_ASSERT_EQUAL( wxString(_T("foo.")), fn.GetFullPath() ); + wxFileName fn("foo."); + CPPUNIT_ASSERT_EQUAL( wxString("foo."), fn.GetFullPath() ); } void FileNameTestCase::TestSetPath() { - wxFileName fn(_T("d:\\test\\foo.bar"), wxPATH_DOS); - fn.SetPath(_T("c:\\temp"), wxPATH_DOS); - CPPUNIT_ASSERT( fn.SameAs(wxFileName(_T("c:\\temp\\foo.bar"), wxPATH_DOS)) ); + wxFileName fn("d:\\test\\foo.bar", wxPATH_DOS); + fn.SetPath("c:\\temp", wxPATH_DOS); + CPPUNIT_ASSERT( fn.SameAs(wxFileName("c:\\temp\\foo.bar", wxPATH_DOS)) ); - fn = wxFileName(_T("/usr/bin/ls"), wxPATH_UNIX); - fn.SetPath(_T("/usr/local/bin"), wxPATH_UNIX); - CPPUNIT_ASSERT( fn.SameAs(wxFileName(_T("/usr/local/bin/ls"), wxPATH_UNIX)) ); + fn = wxFileName("/usr/bin/ls", wxPATH_UNIX); + fn.SetPath("/usr/local/bin", wxPATH_UNIX); + CPPUNIT_ASSERT( fn.SameAs(wxFileName("/usr/local/bin/ls", wxPATH_UNIX)) ); } void FileNameTestCase::TestNormalize() @@ -253,82 +305,243 @@ void FileNameTestCase::TestNormalize() if (cwd.Contains(wxT(':'))) cwd = cwd.AfterFirst(wxT(':')); - static struct FileNameTest + static const struct FileNameTest { - const wxChar *original; + const char *original; int flags; - wxString expected; + const char *expected; + wxPathFormat fmt; } tests[] = { // test wxPATH_NORM_ENV_VARS #ifdef __WXMSW__ - { wxT("%ABCDEF%/g/h/i"), wxPATH_NORM_ENV_VARS, wxT("abcdef/g/h/i") }, + { "%ABCDEF%/g/h/i", wxPATH_NORM_ENV_VARS, "abcdef/g/h/i", wxPATH_UNIX }, #else - { wxT("$(ABCDEF)/g/h/i"), wxPATH_NORM_ENV_VARS, wxT("abcdef/g/h/i") }, + { "$(ABCDEF)/g/h/i", wxPATH_NORM_ENV_VARS, "abcdef/g/h/i", wxPATH_UNIX }, #endif // test wxPATH_NORM_DOTS - { wxT("a/.././b/c/../../"), wxPATH_NORM_DOTS, wxT("") }, - - // test wxPATH_NORM_TILDE - // NB: do the tilde expansion also under Windows to test if it works there too - { wxT("/a/b/~"), wxPATH_NORM_TILDE, wxT("/a/b/~") }, - { wxT("/~/a/b"), wxPATH_NORM_TILDE, home + wxT("a/b") }, - { wxT("~/a/b"), wxPATH_NORM_TILDE, home + wxT("a/b") }, + { "a/.././b/c/../../", wxPATH_NORM_DOTS, "", wxPATH_UNIX }, + { "", wxPATH_NORM_DOTS, "", wxPATH_UNIX }, + { "./foo", wxPATH_NORM_DOTS, "foo", wxPATH_UNIX }, + { "b/../bar", wxPATH_NORM_DOTS, "bar", wxPATH_UNIX }, + { "c/../../quux", wxPATH_NORM_DOTS, "../quux", wxPATH_UNIX }, + { "/c/../../quux", wxPATH_NORM_DOTS, "/quux", wxPATH_UNIX }, + + // test wxPATH_NORM_TILDE: notice that ~ is only interpreted specially + // when it is the first character in the file name + { "/a/b/~", wxPATH_NORM_TILDE, "/a/b/~", wxPATH_UNIX }, + { "/~/a/b", wxPATH_NORM_TILDE, "/~/a/b", wxPATH_UNIX }, + { "~/a/b", wxPATH_NORM_TILDE, "HOME/a/b", wxPATH_UNIX }, + + // test wxPATH_NORM_CASE + { "Foo", wxPATH_NORM_CASE, "Foo", wxPATH_UNIX }, + { "Foo", wxPATH_NORM_CASE, "foo", wxPATH_DOS }, + { "C:\\Program Files\\wx", wxPATH_NORM_CASE, + "c:\\program files\\wx", wxPATH_DOS }, + { "C:/Program Files/wx", wxPATH_NORM_ALL | wxPATH_NORM_CASE, + "c:\\program files\\wx", wxPATH_DOS }, + { "C:\\Users\\zeitlin", wxPATH_NORM_ALL | wxPATH_NORM_CASE, + "c:\\users\\zeitlin", wxPATH_DOS }, // test wxPATH_NORM_ABSOLUTE - { wxT("a/b/"), wxPATH_NORM_ABSOLUTE, cwd + wxT("a/b/") }, - { wxT("a/b/c.ext"), wxPATH_NORM_ABSOLUTE, cwd + wxT("a/b/c.ext") }, - { wxT("/a"), wxPATH_NORM_ABSOLUTE, wxT("/a") }, + { "a/b/", wxPATH_NORM_ABSOLUTE, "CWD/a/b/", wxPATH_UNIX }, + { "a/b/c.ext", wxPATH_NORM_ABSOLUTE, "CWD/a/b/c.ext", wxPATH_UNIX }, + { "/a", wxPATH_NORM_ABSOLUTE, "/a", wxPATH_UNIX }, // test giving no flags at all to Normalize() - { wxT("a/b/"), 0, wxT("a/b/") }, - { wxT("a/b/c.ext"), 0, wxT("a/b/c.ext") }, - { wxT("/a"), 0, wxT("/a") } + { "a/b/", 0, "a/b/", wxPATH_UNIX }, + { "a/b/c.ext", 0, "a/b/c.ext", wxPATH_UNIX }, + { "/a", 0, "/a", wxPATH_UNIX }, + + // test handling dots without wxPATH_NORM_DOTS and wxPATH_NORM_ABSOLUTE + // for both existing and non-existent files (this is important under + // MSW where GetLongPathName() works only for the former) + { "./foo", wxPATH_NORM_LONG, "./foo", wxPATH_UNIX }, + { "../foo", wxPATH_NORM_LONG, "../foo", wxPATH_UNIX }, + { ".\\test.bkl", wxPATH_NORM_LONG, ".\\test.bkl", wxPATH_DOS }, + { ".\\foo", wxPATH_NORM_LONG, ".\\foo", wxPATH_DOS }, + { "..\\Makefile.in", wxPATH_NORM_LONG, "..\\Makefile.in", wxPATH_DOS }, + { "..\\foo", wxPATH_NORM_LONG, "..\\foo", wxPATH_DOS }, }; // set the env var ABCDEF - wxSetEnv(_T("ABCDEF"), _T("abcdef")); + wxSetEnv("ABCDEF", "abcdef"); for ( size_t i = 0; i < WXSIZEOF(tests); i++ ) { - wxFileName fn(tests[i].original, wxPATH_UNIX); + const FileNameTest& fnt = tests[i]; + wxFileName fn(fnt.original, fnt.fmt); // be sure this normalization does not fail - CPPUNIT_ASSERT_MESSAGE + WX_ASSERT_MESSAGE + ( + ("#%d: Normalize(%s) failed", (int)i, fnt.original), + fn.Normalize(fnt.flags, cwd, fnt.fmt) + ); + + // compare result with expected string + wxString expected(tests[i].expected); + expected.Replace("HOME/", home); + expected.Replace("CWD/", cwd); + WX_ASSERT_EQUAL_MESSAGE ( - (const char *)wxString::Format(_T("Normalize(%s) failed"), tests[i].original).mb_str(), - fn.Normalize(tests[i].flags, cwd, wxPATH_UNIX) + ("array element #%d", (int)i), + expected, fn.GetFullPath(fnt.fmt) + ); + } + + // MSW-only test for wxPATH_NORM_LONG: notice that we only run it if short + // names generation is not disabled for this system as otherwise the file + // MKINST~1 doesn't exist at all and normalizing it fails (it's possible + // that we're on a FAT partition in which case the test would still succeed + // and also that the registry key was changed recently and didn't take + // effect yet but these are marginal cases which we consciously choose to + // ignore for now) +#ifdef __WXMSW__ + long shortNamesDisabled; + if ( wxRegKey + ( + wxRegKey::HKLM, + "SYSTEM\\CurrentControlSet\\Control\\FileSystem" + ).QueryValue("NtfsDisable8dot3NameCreation", &shortNamesDisabled) && + !shortNamesDisabled ) + { + wxFileName fn("..\\MKINST~1"); + CPPUNIT_ASSERT( fn.Normalize(wxPATH_NORM_LONG, cwd) ); + CPPUNIT_ASSERT_EQUAL( "..\\mkinstalldirs", fn.GetFullPath() ); + } + //else: when in doubt, don't run the test +#endif // __WXMSW__ +} + +void FileNameTestCase::TestReplace() +{ + static const struct FileNameTest + { + const char *original; + const char *env_contents; + const char *replace_fmtstring; + const char *expected; + wxPathFormat fmt; + } tests[] = + { + { "/usr/a/strange path/lib/someFile.ext", "/usr/a/strange path", "$%s", "$TEST_VAR/lib/someFile.ext", wxPATH_UNIX }, + { "/usr/a/path/lib/someFile.ext", "/usr/a/path", "$%s", "$TEST_VAR/lib/someFile.ext", wxPATH_UNIX }, + { "/usr/a/path/lib/someFile", "/usr/a/path/", "$%s", "$TEST_VARlib/someFile", wxPATH_UNIX }, + { "/usr/a/path/lib/", "/usr/a/path/", "$(%s)", "$(TEST_VAR)lib/", wxPATH_UNIX }, + { "/usr/a/path/lib/", "/usr/a/path/", "${{%s}}", "${{TEST_VAR}}lib/", wxPATH_UNIX }, + { "/usr/a/path/lib/", "/usr/a/path/", "%s", "TEST_VARlib/", wxPATH_UNIX }, + { "/usr/a/path/lib/", "/usr/a/path/", "%s//", "TEST_VAR/lib/", wxPATH_UNIX }, + // note: empty directory components are automatically removed by wxFileName thus + // using // in the replace format string has no effect + + { "/usr/../a/path/lib/", "/usr/a/path/", "%s", "/usr/../a/path/lib/", wxPATH_UNIX }, + { "/usr/a/path/usr/usr", "/usr", "%s", "TEST_VAR/a/pathTEST_VAR/usr", wxPATH_UNIX }, + { "/usr/a/path/usr/usr", "/usr", "$%s", "$TEST_VAR/a/path$TEST_VAR/usr", wxPATH_UNIX }, + { "/a/b/c/d", "a/", "%s", "/TEST_VARb/c/d", wxPATH_UNIX }, + + { "C:\\A\\Strange Path\\lib\\someFile", "C:\\A\\Strange Path", "%%%s%%", "%TEST_VAR%\\lib\\someFile", wxPATH_WIN }, + { "C:\\A\\Path\\lib\\someFile", "C:\\A\\Path", "%%%s%%", "%TEST_VAR%\\lib\\someFile", wxPATH_WIN }, + { "C:\\A\\Path\\lib\\someFile", "C:\\A\\Path", "$(%s)", "$(TEST_VAR)\\lib\\someFile", wxPATH_WIN } + }; + + for ( size_t i = 0; i < WXSIZEOF(tests); i++ ) + { + const FileNameTest& fnt = tests[i]; + wxFileName fn(fnt.original, fnt.fmt); + + // set the environment variable + wxSetEnv("TEST_VAR", fnt.env_contents); + + // be sure this ReplaceEnvVariable does not fail + WX_ASSERT_MESSAGE + ( + ("#%d: ReplaceEnvVariable(%s) failed", (int)i, fnt.replace_fmtstring), + fn.ReplaceEnvVariable("TEST_VAR", fnt.replace_fmtstring, fnt.fmt) ); // compare result with expected string - CPPUNIT_ASSERT_EQUAL( tests[i].expected, fn.GetFullPath(wxPATH_UNIX) ); + wxString expected(fnt.expected); + WX_ASSERT_EQUAL_MESSAGE + ( + ("array element #%d", (int)i), + expected, fn.GetFullPath(fnt.fmt) + ); } + + // now test ReplaceHomeDir + + wxFileName fn = wxFileName::DirName(wxGetHomeDir()); + fn.AppendDir("test1"); + fn.AppendDir("test2"); + fn.AppendDir("test3"); + fn.SetName("some file"); + + WX_ASSERT_MESSAGE + ( + ("ReplaceHomeDir(%s) failed", fn.GetFullPath()), + fn.ReplaceHomeDir() + ); + + CPPUNIT_ASSERT_EQUAL( wxString("~/test1/test2/test3/some file"), + fn.GetFullPath(wxPATH_UNIX) ); } -wxString wxTestStripExtension(wxString szFile) +void FileNameTestCase::TestGetHumanReadable() { - wxStripExtension(szFile); - return szFile; + static const struct TestData + { + const char *result; + int size; + int prec; + wxSizeConvention conv; + } testData[] = + { + { "NA", 0, 1, wxSIZE_CONV_TRADITIONAL }, + { "2.0 KB", 2000, 1, wxSIZE_CONV_TRADITIONAL }, + { "1.953 KiB", 2000, 3, wxSIZE_CONV_IEC }, + { "2.000 KB", 2000, 3, wxSIZE_CONV_SI }, + { "297 KB", 304351, 0, wxSIZE_CONV_TRADITIONAL }, + { "304 KB", 304351, 0, wxSIZE_CONV_SI }, + }; + + CLocaleSetter loc; // we want to use "C" locale for LC_NUMERIC + // so that regardless of the system's locale + // the decimal point used by GetHumanReadableSize() + // is always '.' + for ( unsigned n = 0; n < WXSIZEOF(testData); n++ ) + { + const TestData& td = testData[n]; + + // take care of using the decimal point for the current locale before + // the actual comparison + CPPUNIT_ASSERT_EQUAL + ( + td.result, + wxFileName::GetHumanReadableSize(td.size, "NA", td.prec, td.conv) + ); + } + + // also test the default convention value + CPPUNIT_ASSERT_EQUAL( "1.4 MB", wxFileName::GetHumanReadableSize(1512993, "") ); } void FileNameTestCase::TestStrip() { - //test a crash - CPPUNIT_ASSERT_EQUAL( wxString(_T("")), wxTestStripExtension(_T("")) ); - - //others - CPPUNIT_ASSERT_EQUAL( wxString(_T("")), wxTestStripExtension(_T(".")) ); - CPPUNIT_ASSERT_EQUAL( wxString(_T("")), wxTestStripExtension(_T(".wav")) ); - CPPUNIT_ASSERT_EQUAL( wxString(_T("good")), wxTestStripExtension(_T("good.wav")) ); - CPPUNIT_ASSERT_EQUAL( wxString(_T("good.wav")), wxTestStripExtension(_T("good.wav.wav")) ); + CPPUNIT_ASSERT_EQUAL( "", wxFileName::StripExtension("") ); + CPPUNIT_ASSERT_EQUAL( ".", wxFileName::StripExtension(".") ); + CPPUNIT_ASSERT_EQUAL( ".vimrc", wxFileName::StripExtension(".vimrc") ); + CPPUNIT_ASSERT_EQUAL( "bad", wxFileName::StripExtension("bad") ); + CPPUNIT_ASSERT_EQUAL( "good", wxFileName::StripExtension("good.wav") ); + CPPUNIT_ASSERT_EQUAL( "good.wav", wxFileName::StripExtension("good.wav.wav") ); } #ifdef __WINDOWS__ void FileNameTestCase::TestShortLongPath() { - wxFileName fn(_T("C:\\Program Files\\Windows NT\\Accessories\\wordpad.exe")); + wxFileName fn("C:\\Program Files\\Windows NT\\Accessories\\wordpad.exe"); // incredibly enough, GetLongPath() used to return different results during // the first and subsequent runs, test for this @@ -337,3 +550,100 @@ void FileNameTestCase::TestShortLongPath() } #endif // __WINDOWS__ + +void FileNameTestCase::TestUNC() +{ + wxFileName fn("//share/path/name.ext", wxPATH_DOS); + CPPUNIT_ASSERT_EQUAL( "share", fn.GetVolume() ); + CPPUNIT_ASSERT_EQUAL( "\\path", fn.GetPath(wxPATH_NO_SEPARATOR, wxPATH_DOS) ); + + fn.Assign("\\\\share2\\path2\\name.ext", wxPATH_DOS); + CPPUNIT_ASSERT_EQUAL( "share2", fn.GetVolume() ); + CPPUNIT_ASSERT_EQUAL( "\\path2", fn.GetPath(wxPATH_NO_SEPARATOR, wxPATH_DOS) ); +} + +void FileNameTestCase::TestVolumeUniqueName() +{ + wxFileName fn("\\\\?\\Volume{8089d7d7-d0ac-11db-9dd0-806d6172696f}\\", + wxPATH_DOS); + CPPUNIT_ASSERT_EQUAL( "Volume{8089d7d7-d0ac-11db-9dd0-806d6172696f}", + fn.GetVolume() ); + CPPUNIT_ASSERT_EQUAL( "\\", fn.GetPath(wxPATH_NO_SEPARATOR, wxPATH_DOS) ); + CPPUNIT_ASSERT_EQUAL( "\\\\?\\Volume{8089d7d7-d0ac-11db-9dd0-806d6172696f}\\", + fn.GetFullPath(wxPATH_DOS) ); + + fn.Assign("\\\\?\\Volume{8089d7d7-d0ac-11db-9dd0-806d6172696f}\\" + "Program Files\\setup.exe", wxPATH_DOS); + CPPUNIT_ASSERT_EQUAL( "Volume{8089d7d7-d0ac-11db-9dd0-806d6172696f}", + fn.GetVolume() ); + CPPUNIT_ASSERT_EQUAL( "\\Program Files", + fn.GetPath(wxPATH_NO_SEPARATOR, wxPATH_DOS) ); + CPPUNIT_ASSERT_EQUAL( "\\\\?\\Volume{8089d7d7-d0ac-11db-9dd0-806d6172696f}\\" + "Program Files\\setup.exe", + fn.GetFullPath(wxPATH_DOS) ); +} + +void FileNameTestCase::TestCreateTempFileName() +{ + static const struct TestData + { + const char *prefix; + const char *expectedFolder; + bool shouldSucceed; + } testData[] = + { + { "", "$SYSTEM_TEMP", true }, + { "foo", "$SYSTEM_TEMP", true }, + { "..", "$SYSTEM_TEMP", true }, + { "../bar", "..", true }, +#ifdef __WXMSW__ + { "$USER_DOCS_DIR\\", "$USER_DOCS_DIR", true }, + { "c:\\a\\directory\\which\\does\\not\\exist", "", false }, +#elif defined( __UNIX__ ) + { "$USER_DOCS_DIR/", "$USER_DOCS_DIR", true }, + { "/tmp/foo", "/tmp", true }, + { "/tmp/a/directory/which/does/not/exist", "", false }, +#endif // __UNIX__ + }; + + for ( size_t n = 0; n < WXSIZEOF(testData); n++ ) + { + wxString prefix = testData[n].prefix; + prefix.Replace("$USER_DOCS_DIR", wxStandardPaths::Get().GetDocumentsDir()); + + std::string errDesc = wxString::Format("failed on prefix '%s'", prefix).ToStdString(); + + wxString path = wxFileName::CreateTempFileName(prefix); + CPPUNIT_ASSERT_EQUAL_MESSAGE( errDesc, !testData[n].shouldSucceed, path.empty() ); + + if (testData[n].shouldSucceed) + { + errDesc += "; path is " + path.ToStdString(); + + // test the place where the temp file has been created + wxString expected = testData[n].expectedFolder; + expected.Replace("$SYSTEM_TEMP", wxStandardPaths::Get().GetTempDir()); + expected.Replace("$USER_DOCS_DIR", wxStandardPaths::Get().GetDocumentsDir()); + CPPUNIT_ASSERT_EQUAL_MESSAGE( errDesc, expected, wxFileName(path).GetPath() ); + + // the temporary file is created with full permissions for the current process + // so we should always be able to remove it: + CPPUNIT_ASSERT_MESSAGE( errDesc, wxRemoveFile(path) ); + } + } +} + +void FileNameTestCase::TestGetTimes() +{ + wxFileName fn(wxFileName::CreateTempFileName("filenametest")); + CPPUNIT_ASSERT( fn.IsOk() ); + + wxDateTime dtAccess, dtMod, dtCreate; + CPPUNIT_ASSERT( fn.GetTimes(&dtAccess, &dtMod, &dtCreate) ); + + // make sure all retrieved dates are equal to the current date&time + // with an accuracy up to 1 minute + CPPUNIT_ASSERT(dtCreate.IsEqualUpTo(wxDateTime::Now(), wxTimeSpan(0,1))); + CPPUNIT_ASSERT(dtMod.IsEqualUpTo(wxDateTime::Now(), wxTimeSpan(0,1))); + CPPUNIT_ASSERT(dtAccess.IsEqualUpTo(wxDateTime::Now(), wxTimeSpan(0,1))); +}