fix test

fs/_repr.py

@@ -1,11 +1,32 @@
+"""
+Tools to generate __repr__ strings
+
+"""
+
 from __future__ import unicode_literals
 
 def make_repr(class_name, *args, **kwargs):
-    """Generate a repr string."""
+    """
+    Generate a repr string.
+
+    Positional arguments should be the positional arguments used to
+    construct the class. Keyword arguments should consist of tuples of
+    the attribute value and default. If the value is the default, then
+    it won't be rendered in the output.
+
+    Here's an example::
+
+        def __repr__(self):
+            return make_repr('MyClass', 'foo', name=(self.name, None))
+
+    The output of this would be something line ``MyClass('foo',
+    name='Will')``.
+
+    """
     arguments = [repr(arg) for arg in args]
     arguments.extend([
         "{}={!r}".format(name, value)
-        for name, (value, default) in kwargs.items()
+        for name, (value, default) in sorted(kwargs.items())
         if value != default
     ])
-    return "{}({})".format(class_name, ', '.join(arguments))
+    return "{}({})".format(class_name, ', '.join(arguments))

fs/appfs.py

@@ -85,8 +85,7 @@ class UserConfigFS(_AppFS):
     :param str version: Optional version string, if a unique location
         per version of the application is required.
     :param bool roaming: If ``True``, use a *roaming* profile on
-        Windows, see http://technet.microsoft.com/en-
-        us/library/cc766489(WS.10).aspx
+        Windows.
     :param bool create: If ``True`` (the default) the directory will
         be created if it does not exist.
 

fs/osfs.py

@@ -123,7 +123,7 @@ def _make_details_from_stat(cls, stat_result):
             'accessed': stat_result.st_atime,
             'modified': stat_result.st_mtime,
             'size': stat_result.st_size,
-            'type': int(cls._get_type_from_stat(stat))
+            'type': int(cls._get_type_from_stat(stat_result))
         }
         if hasattr(stat, 'st_birthtime'):
             details['created'] = stat_result.st_birthtime