search

canonical / ops-scenario

State-transition testing SDK for Operator Framework Juju charms.
Apache License 2.0
10 stars 7 forks source link

Fix Event field name ("name" -> "path") #94

Closed benhoyt closed 8 months ago

benhoyt commented 8 months ago

Without this, when using Secret.changed_event and similar properties, a TypeError is raised:

>       return Event(name="secret_changed", secret=self)
E       TypeError: Event.__init__() got an unexpected keyword argument 'name'

Probably means there should be tests of these code paths too, but I'm too lazy to add that here -- just flagging the issue. :-)

benhoyt commented 8 months ago

Oops, sorry, I'd forgotten about those fixes (and that they were on a different branch). Closing.