PyFilesystem / pyfilesystem

Python filesystem abstraction layer
http://pyfilesystem.org/
BSD 3-Clause "New" or "Revised" License
288 stars 63 forks source link

Unit Test Failures on OpenBSD #179

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Hi,

I am working on building a OpenBSD package of fs. It seems like it mostly 
works, although there are a bunch of test failures. I am running the tests as 
suggested in #178.

Looking at the logs, these failures can probably be put into one of categories:

 * Missing modules.
 * Actual errors.

For the former, this is because there are no OpenBSD packages for some of the 
optional dependencies. I propose these should be skip cases if the modules are 
absent (instead of fail cases).

I'm attaching a log for inspection. This is against version 0.5.0. I'll try 
running the tests again trunk shortly. If the problems are gone, I will close 
this bug.

Thanks

Original issue reported on code.google.com by vex...@gmail.com on 29 Jun 2014 at 10:36

Attachments:

GoogleCodeExporter commented 9 years ago
Find attached the test output from trunk. Much the same issues.

Cheers

Original comment by vex...@gmail.com on 30 Jun 2014 at 1:30

Attachments:

GoogleCodeExporter commented 9 years ago
Some of those issues may be permissions related. The tests do some things like 
launch servers on reserved ports. You may get more passes if you run the tests 
as superuser.

dexml is required for webdav support I believe. The setup.py only includes 
dependencies for besic fs support, not every implementation. dexml should 
probably be added to tests_requires in setup.py.

As for FUSE support, does BSD support FUSE? It should probably be made optional 
in the tests.

I think that if these are fixes then you would be left with a handful of test 
fails, if any. I'm not likely to have time to work on this any time soon, but 
I'd be happy to accept a patch to setup.py and tests if you want to attempt 
it...

Original comment by willmcgugan on 30 Jun 2014 at 1:50

GoogleCodeExporter commented 9 years ago
Running tests as root is a bad idea. This won't float in an OpenBSD port :\

In general if a module is missing, I think the test should be skipped. E.g. 
there is no dexml port for OpenBSD yet, so skip these tests.

OpenBSD does have fuse, but I do not have it installed. The tests should have 
been skipped (in my case).

Original comment by vex...@gmail.com on 7 Jul 2014 at 1:35

GoogleCodeExporter commented 9 years ago
My last comment was inaccurate.

OpenBSD has the fuse support by default. The test suite thinks it cannot use it 
for some reason though.

Original comment by vex...@gmail.com on 14 Jul 2014 at 9:53