Thanks for your comments @pelson, ummm yes and no.. I don't know.  I added `test_suitable_permissions` and `test_exist_directory` because those that exist are integrated tests.  I thought they needn't be integration tests.  I'm happy to remove these two if you still think their redundant?