It's just used to check that GetPendingRequests() isn't empty. Where WaitForAccessTokenRequest* is already used, we can just remove the check entirely, as WaitForAccessTokenRequest* waits for an access token request internally and thus implicitly checks that an access token request occurs. Where it's just used to verify that there's an access token request outstanding, we can replace the usage with a call to WaitForAccessTokenRequest*.

We can rename IsRequestActive() to IsNetworkRequestActive() and make sure that it's only used when we're verifying that there's a network request, not to verify that there's a pending access token request.

Don't spend too much time trying to get this exactly right, as in the end this is not the fundamentals of this test code. We can always refine the changes during code review.