-

cool-RR · cool-RR · commit 5784e529e71c · 2016-01-12T21:05:27.000+02:00
diff --git a/source_py3/python_toolbox/context_management/abstract_context_manager.py b/source_py3/python_toolbox/context_management/abstract_context_manager.py
@@ -30,9 +30,9 @@ def __subclasshook__(cls, candidate_class):
         if cls is AbstractContextManager:
             return (
                 hasattr(candidate_class, '__enter__') and
-                candidate_class.__enter__ is not none and 
+                candidate_class.__enter__ is not None and 
                 hasattr(candidate_class, '__exit__') and
-                candidate_class.__exit__ is not none 
+                candidate_class.__exit__ is not None 
             )
         else:
             return NotImplemented
diff --git a/source_py3/test_python_toolbox/test_temp_file_tools/test_create_temp_folder.py b/source_py3/test_python_toolbox/test_temp_file_tools/test_create_temp_folder.py
@@ -97,10 +97,15 @@ def test_parent_folder():
             assert str(tf2).startswith(str(tf1))
     
 def test_chmod():
-    with create_temp_folder(chomd=0o777) as liberal_temp_folder, \
-                   create_temp_folder(chomd=0o000) as conservative_temp_folder:
-        liberal_temp_folder.stat().st_mode & 0o777
-        1 / 0 # blocktodo write this test
+    with create_temp_folder(chmod=0o777) as liberal_temp_folder, \
+                   create_temp_folder(chmod=0o000) as conservative_temp_folder:
+        # Doing a very weak test of chmod because not everything is supported
+        # on Windows.
+        assert (liberal_temp_folder.stat().st_mode & 0o777) > \
+                              (conservative_temp_folder.stat().st_mode & 0o777)
         
+        # Making `conservative_temp_folder` writeable again so it could be
+        # deleted in cleanup:
+        conservative_temp_folder.chmod(0o777)
         
     
