You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I can - but frankly, I think my time would be better spent elsewhere for now. These tests will be useful when we get more code relying on this functionality, and it would enable better testing then. But for now, these tests are quite a lot of work to create for what is literally 40 lines of simple code. So I would prefer to defer this work until it becomes more relevant.
@wcoenraads Do you think the first point is also a lot of effort? I would say if we can address the first one, and leave the more complete approach for later, it is a lot better.
Originally posted by @suvayu in #19 (comment)
The text was updated successfully, but these errors were encountered: