Regression PR 8826 broke cache.reset()

Intended outcome: Before #8826 was introduced we used cache.reset() to not only reset the cache but also recall all active query watchers. (much like client.resetStore()).

However, I noticed that https://github.com/apollographql/apollo-client/pull/8826#discussion_r714166464 seems to break this functionality

This seems to be a regression (Or were we using this wrong) as now the store is reset but no active queries are called again.

Actual outcome: I’d Expect the active query watchers to be called as it worked before

How to reproduce the issue: Call cache.reset() before and after #8826

[tm1000]

@benjamn Thanks for the quick response. I think it fixes the issues but my engineers have told me that cache.reset() actually broke between 3.3.21 and 3.4.0 which was rather unexpected (they first told me it was related to this and then they ran through downgrading until they got to 3.3.21 which they said worked)

Looking at the code though it looks like the watchers should refetch even in 3.4.0 and especially after your work in #8826 so the investigation continues but without the work in #8826 it definitely wouldn’t have worked at all.

Either way this specific issue is resolved. If I find something else I’ll open a new issue in regards to that but again #8826 was still needed because it brings back watchers being able to refetch at least

[tm1000]

@benjamn Thanks for pointing me in the right direction!