zarr-developers / zarr / 1431

100%

Pull #318

travis-ci

Only catch `OSError` if its `errno` equal `EEXIST`

On Python 3, we would only want to catch the race case here by checking
for the `FileExistsError` and suppressing raising for that case. However
there is no `FileExistsError` on Python 2. Only `OSError` exists on
Python 2/3. However that catches some other errors that we might not
want to suppress here. So we check the `errno` of the `OSError` instance
to verify it is caused by an existing file/directory represented by the
`EEXIST` error number. This amounts to catching the `FileExistsError` in
a Python 2/3 friendly way. All other `OSErrors` we raise as before.

4 of 4 new or added lines in 1 file covered. (100.0%)

8696 of 8698 relevant lines covered (99.98%)

3.96 hits per line