imo it’s violation of Single-Responsobility principle.
One class should request data, another cache it.
If you want to make this request and caching transparent for a client, create some new class, maybe some Repository, and inject request and cache dependenices inside, so you can easily mock Repository or request or cache without mocking of static fields