kombu icon indicating copy to clipboard operation
kombu copied to clipboard

Used a context manager for locks and files in filesystem transport.

Open atombrella opened this issue 4 years ago • 5 comments

atombrella avatar Jan 01 '20 11:01 atombrella

It seems that the filesystem transport is not tested under Windows platforms. test_filesystem.py is decorated with @skip.if_win32(). The use of a context manager for the files should not require additional tests; it may be different with the locks.

atombrella avatar Jan 01 '20 16:01 atombrella

I think we should remove win32 bit support

auvipy avatar Jan 01 '20 17:01 auvipy

@skip.if_win32() doesn't have anything to do with either 32 or 64 bit. If you see the output from AppVeyor, the two tests from test_filesystem are skipped on all Windows platforms, e.g., also with PYTHON_ARCH=64.

atombrella avatar Jan 01 '20 18:01 atombrella

I am hoping to keep supporting windows build related to issues -- I finally got pycurl to build locally on my Windows ecosystem -- libcurl, and pycurl building solidly now. I agree that the 32 here on the pytest decorator does not signify 32 bit support, its just old nomenclature.

matteius avatar Jan 01 '20 20:01 matteius

would you mind adding some unit tests & doc/release note entry for the changes? And add your name to author/contributor list if not already?

In any case, the doc-string needs to be added, since there's a failing check for this. I'll see what I can do about unit tests; my changes are mostly updates to newer Python syntax/constructs.

atombrella avatar Jan 24 '22 10:01 atombrella