Announcement reactions query spec improvement and refactor #28768
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In the previous coverage I added for this we do exercise both account-present and account-not-present paths -- but we didn't have an assertion about the difference between those paths (the value of the
me
column in the query). This adds that coverage, and a few other details in coverage to get more precise.The method change mainly pulls both the starting scope and the sql out to private methods, and tightens up return values with
tap
. I verified in console logs that the preloader still runs correctly here.