mirror of
https://github.com/zulip/zulip.git
synced 2026-07-03 21:10:12 +08:00
dfab993e7d
We now use narrow_state directly in pm_list and pm_list_data tests, rather than mocking it with our `override*` helpers. In some places I use an actual Filter() object, but in places where the only testing concern is that the active is narrow, I use a stub value. We will continue to mock narrow_state in most places. In addition to avoiding test-setup complications, we want to avoid incidental line coverage on narrow_state that only indirectly validates its behavior. Part of the trickiness in avoiding narrow_state mocking is that you often would have to introduce "real" Filter objects, and the API for Filter objects is somewhat less than ideal, and its wordiness can distract from the main point of the tests. Hopefully the changes here reflect the correct tradeoffs. |
||
|---|---|---|
| .. | ||
| node_tests | ||
| puppeteer_lib | ||
| puppeteer_tests | ||
| zjsunit | ||