canonical / kubeflow-volumes-operator

Volumes Operator
Apache License 2.0
0 stars 3 forks source link

Investigate whether `bcrypt` is required for the unit tests to run #99

Closed phoevos closed 1 year ago

phoevos commented 1 year ago

In requirements-unit.in we specify a dependency on bcrypt: https://github.com/canonical/kubeflow-volumes-operator/blob/608be697543fcccc5df484e2425066c12d29a256/requirements-unit.in#L1

Our unit tests don't look like they depend on it directly. We should investigate and remove the dependency if it's indeed redundant, or add a comment to clarify it's significance in the event that it's not.