110 sats \ 2 replies \ @WeAreAllSatoshi 8 Aug 2023 \ on: SN release: nostr login, nostr zap metadata, singular unit(s), more meta
@k00b @ekzyis Is unit testing on y'all's roadmap? I was feeling a little naked making changes without adding some unit tests to go with them. I'd be happy to take a stab at it in a PR if you're interested.
I'd like to see unit tests around the wallet touching code.
Everything else is kind of nonessential for the time being and the agility is worth the naked feeling. I think the wallet could really benefit from tests though because the stakes are a lot higher yet it'd be nice to iterate there as fast as everywhere else.
reply
Makes sense, thanks for the follow up!
reply