testsuite: Add a test for listing the root directory...

...and don't treat Path("/") like Path("").

1 files changed, 15 insertions, 1 deletions

diff --git a/src/libstd/os.rs b/src/libstd/os.rs
index e612c8cfeff..44acdd4d617 100644
--- a/src/libstd/os.rs
+++ b/src/libstd/os.rs
@@ -675,7 +675,7 @@ pub fn mkdir_recursive(p: &Path, mode: c_int) -> bool {
 /// Lists the contents of a directory
 #[allow(non_implicitly_copyable_typarams)]
 pub fn list_dir(p: &Path) -> ~[~str] {
-    if p.components.is_empty() {
+    if p.components.is_empty() && !p.is_absolute() {
         // Not sure what the right behavior is here, but this
         // prevents a bounds check failure later
         return ~[];
@@ -1608,6 +1608,20 @@ mod tests {
     }
 
     #[test]
+    #[cfg(not(windows))]
+    fn list_dir_root() {
+        let dirs = os::list_dir(&Path("/"));
+        assert!(dirs.len() > 1);
+    }
+    #[test]
+    #[cfg(windows)]
+    fn list_dir_root() {
+        let dirs = os::list_dir(&Path("C:\\"));
+        assert!(dirs.len() > 1);
+    }
+
+
+    #[test]
     fn path_is_dir() {
         assert!((os::path_is_dir(&Path("."))));
         assert!((!os::path_is_dir(&Path("test/stdtest/fs.rs"))));