canonical / data-platform-libs

A collection of charm libraries curated by the Data Platform Team
https://charmhub.io/data-platform-libs
Apache License 2.0
9 stars 9 forks source link

Non-existing secret error message shouldn't display set contents such as "{non-existing-secret}" #123

Closed juditnovak closed 6 months ago

juditnovak commented 6 months ago

Steps to reproduce

Use mongodb-operator unittests -> test_delete_password()

Expected behavior

Instead of expected "Non-existing secret {'ca-secret'}" it should be ""Non-existing secret(s) 'ca-secret', 'any-other-secret'..."

Actual behavior

See above

Versions

Operating system: any

Juju CLI: any with secrets

Juju agent: any with secrets

Log output

Juju debug log: None

Additional context

github-actions[bot] commented 6 months ago

https://warthogs.atlassian.net/browse/DPE-3312

juditnovak commented 6 months ago

Fixed in https://github.com/canonical/data-platform-libs/pull/131