Refactor lazy tests #13383
Refactor lazy tests #13383
Conversation
|
| let { lazyStub: lazyJsonStub, lazyDeferred: lazyJsonDeferred } = | ||
| createLazyStub(); | ||
| let { lazyStub: lazyTextStub, lazyDeferred: lazyTextDeferred } = | ||
| createLazyStub(); |
There was a problem hiding this comment.
This sort of aliased destructuring became so common in these tests, I decided it's cleaner to make this function return an array instead.
|
🤖 Hello there, We just published version Thanks! |
|
🤖 Hello there, We just published version Thanks! |
With the introduction of granular
route.lazy, the patterns used in the tests became a bit inconsistent, but later PRs started landing on better patterns. This PR cleans things up and uses more consistent naming and patterns across all tests. Most notably, we're now avoiding the wordlazyin variable names unless it refers to aroute.lazyfunction.