search

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
10 stars 9 forks source link

[DPE-3876][BUGFIX] DataPeerOtherUnit #150

Closed juditnovak closed 6 months ago

juditnovak commented 6 months ago

There was a major copy-paste mistake on the new DataPeerOtherUnit class (that will be required typically for this functionality demonstarted in Zookeeper).

juditnovak commented 6 months ago

@welpaolo @deusebio +1 commit added (https://github.com/canonical/data-platform-libs/pull/150/commits/1758e05db76de9ef8253aa07d156e2fdeac29027) with extending interfaces unittests to all interfaces.

This is slightly redundant, as most (BUT not all!!!) of this functionality has been tested specifically for various use cases. Yet this "consistent redundancy" won't hurt. (Rather the opposite: it will cover missing bits.)