fix: Initialize query on updateObservableQuery even if skip is true

[mu29]

Solves #6816, #6796

If skip option is true at the first rendering, the initializeObservableQuery will not be triggered. And even if the skip value changes to false later, calling refetch throws the following error:

Uncaught TypeError: Cannot read property 'refetch' of undefined
    at QueryData._this.obsRefetch

Checklist:

• If this PR is a new feature, please reference an issue where a consensus about the design was reached (not necessary for small changes)
• Make sure all of the significant new logic is covered by tests

[benjamn]

This makes sense to me, but I'd like to get @hwillson's take, since he added that return case in #6752. In the meantime, I will merge this into release-3.2 and put out another beta release for you to try. Also happy to port this change back to 3.1.x if that's helpful.

[benjamn]

@mu29 This change is available now in @apollo/[email protected], if you want to give it a try!

[hwillson]

Looks good to me @mu29 @benjamn - we really just need that line to be called anytime before setOptions; I added it to the top to short circuit things a bit faster, but clearly that was the wrong decision. Thanks!

[mu29]

Thank you for your quick response! @benjamn @hwillson 👍

[watadarkstar]

Might work on patching this for v3.1.5 as well! Thanks!