search

Backblaze / b2-sdk-python

Python library to access B2 cloud storage.
Other
179 stars 59 forks source link

test_upload_url_concurrency not in test/ directory #471

Closed Hawk777 closed 7 months ago

Hawk777 commented 7 months ago

I’m not certain if this is intentional or accidental, but test_upload_url_concurrency is in the main package source directory rather than the test/ directory, despite being a test.

vbaltrusaitis-reef commented 7 months ago

This file is ancient and it's not a part of the test suite that is run with nox or otherwise in CI, it's more of a "test script" than an actual test. It should be either incorporated into the test suite or perhaps simply removed.

ppolewicz commented 7 months ago

I don't think anyone used it for many years now.

@Hawk777 since this is your finding, would you like to do the honors?

Hawk777 commented 7 months ago

You’d like the test deleted?

ppolewicz commented 7 months ago

Yes, and the changelog needs to mention it (please see a note about how to use towncrier)