More generaly I think this guide should illustrate testing a HttpInterceptorFn since this is the recommended implementation for Interceptors.

In general, I agree with this, hence why I added the example for the functional interceptor first.

However, I was not sure how to proceed in this case, given that Angular still supports interceptors from DI.
Because of this, not having any details on how to test these interceptors seems a bit wrong, hence why I added them.

Let me know if these should be removed, or changed in some way.

@JeanMeche any word on this?
Is there anything blocking it?

We'll need you to drop that merge commit, the change needs to be a single commit to be merged.

@JeanMeche I had to force push to remove the unwanted commit. Could you please re-add the labels and merge?

@JeanMeche is there anything pending on my part, or is it just tag updating and merging?

This PR was merged into the repository by commit 6674130.

The changes were merged into the following branches: main, 18.2.x, 19.0.x

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}