search

Unleash / unleash-client-rust

Unleash client SDK for Rust language projects
Apache License 2.0
23 stars 17 forks source link

fix: under-counting metrics for enum is_enabled() #83

Closed rbtcollins closed 7 months ago

rbtcollins commented 7 months ago
rbtcollins commented 7 months ago

I figured out - this was a regression in 26eaac3af39f9ced77743e3745279b0ed48bbade (@thomasheartman )

thomasheartman commented 7 months ago

Oh, wow. Good catch! Might be worth adding some tests for it so that we avoid it happening again?

rbtcollins commented 7 months ago

I think the str case is already tested, this is oversight indeed; we should add some tests; it may be a bit before I have time to do that myself