[ ] workarounds in main code and tests have been removed
Details and Comments
I only noticed this recently, after the tag got introduced. I reckon it should be easily fixable since we have full control over the use of that tag.
This code snippet illustrates the problem
if isinstance(private_tag.value, bytes):
# Allow this for the time being, until it has been investigated
logging.error(f"{private_tag.value} should be of type str, but is of type bytes")
assert private_tag.value.decode() == TestFtpsUpload.project_slug
else:
assert private_tag.value == TestFtpsUpload.project_slug
Definition of Done / Acceptance Criteria
Details and Comments
I only noticed this recently, after the tag got introduced. I reckon it should be easily fixable since we have full control over the use of that tag.
This code snippet illustrates the problem
similarly: