Add missing set request interception tests (#1798)

Author: kblok

lib/PuppeteerSharp.TestServer/wwwroot/cached/one-style-font.css

@@ -0,0 +1,9 @@
+@font-face {
+  font-family: 'one-style';
+  src: url('./one-style.woff') format('woff');
+}
+
+body {
+  background-color: pink;
+  font-family: 'one-style', sans-serif;
+}

lib/PuppeteerSharp.TestServer/wwwroot/cached/one-style-font.html

@@ -0,0 +1,2 @@
+<link rel='stylesheet' href='./one-style-font.css'>
+<div>hello, world!</div>

lib/PuppeteerSharp.Tests/RequestInterceptionTests/SetRequestInterceptionTests.cs

@@ -613,5 +613,53 @@ public async Task ShouldWorkWithFileURLs()
             Assert.Contains("one-style.html", urls);
             Assert.Contains("one-style.css", urls);
         }
+
+        [PuppeteerTest("requestinterception.spec.ts", "Page.setRequestInterception", "should not cache if cache disabled")]
+        [SkipBrowserFact(skipFirefox: true)]
+        public async Task ShouldNotCacheIfCacheDisabled()
+        {
+            await Page.GoToAsync(TestConstants.ServerUrl + "/cached/one-style.html");
+            await Page.SetRequestInterceptionAsync(true);
+            await Page.SetCacheEnabledAsync(false);
+            var urls = new List<string>();
+            Page.Request += (_, e) => _ = e.Request.ContinueAsync();
+
+            var cached = new List<Request>();
+            Page.RequestServedFromCache += (_, e) => cached.Add(e.Request);
+
+            await Page.ReloadAsync();
+            Assert.Empty(cached);
+        }
+
+        [PuppeteerTest("requestinterception.spec.ts", "Page.setRequestInterception", "should cache if cache enabled")]
+        [SkipBrowserFact(skipFirefox: true)]
+        public async Task ShouldNotCacheIfCacheEnabled()
+        {
+            await Page.GoToAsync(TestConstants.ServerUrl + "/cached/one-style.html");
+            await Page.SetRequestInterceptionAsync(true);
+            await Page.SetCacheEnabledAsync(true);
+            var urls = new List<string>();
+            Page.Request += (_, e) => _ = e.Request.ContinueAsync();
+
+            var cached = new List<Request>();
+            Page.RequestServedFromCache += (_, e) => cached.Add(e.Request);
+
+            await Page.ReloadAsync();
+            Assert.Single(cached);
+        }
+
+        [PuppeteerTest("requestinterception.spec.ts", "Page.setRequestInterception", "should load fonts if cache enabled")]
+        [SkipBrowserFact(skipFirefox: true)]
+        public async Task ShouldLoadFontsIfCacheEnabled()
+        {
+            await Page.SetRequestInterceptionAsync(true);
+            await Page.SetCacheEnabledAsync(true);
+            var urls = new List<string>();
+            Page.Request += (_, e) => _ = e.Request.ContinueAsync();
+
+            var waitTask = Page.WaitForResponseAsync(response => response.Url.EndsWith("/one-style.woff"));
+            await Page.GoToAsync(TestConstants.ServerUrl + "/cached/one-style-font.html");
+            await waitTask;
+        }
     }
 }

lib/PuppeteerSharp/NetworkManager.cs

@@ -62,6 +62,8 @@ internal NetworkManager(CDPSession client, bool ignoreHTTPSErrors, FrameManager
 
         internal event EventHandler<RequestEventArgs> RequestFailed;
 
+        internal event EventHandler<RequestEventArgs> RequestServedFromCache;
+
         internal FrameManager FrameManager { get; set; }
         #endregion
 
@@ -376,6 +378,7 @@ private void OnRequestServedFromCache(RequestServedFromCacheResponse response)
             {
                 request.FromMemoryCache = true;
             }
+            RequestServedFromCache?.Invoke(this, new RequestEventArgs { Request = request });
         }
 
         private void HandleRequestRedirect(Request request, ResponsePayload responseMessage)

lib/PuppeteerSharp/Page.cs

@@ -202,6 +202,11 @@ private Page(
         /// </summary>
         public event EventHandler<RequestEventArgs> RequestFailed;
 
+        /// <summary>
+        /// Raised when a request ended up loading from cache.
+        /// </summary>
+        public event EventHandler<RequestEventArgs> RequestServedFromCache;
+
         /// <summary>
         /// Raised when an uncaught exception happens within the page.
         /// </summary>
@@ -1855,6 +1860,7 @@ private async Task InitializeAsync(bool ignoreHTTPSErrors)
             networkManager.RequestFailed += (_, e) => RequestFailed?.Invoke(this, e);
             networkManager.Response += (_, e) => Response?.Invoke(this, e);
             networkManager.RequestFinished += (_, e) => RequestFinished?.Invoke(this, e);
+            networkManager.RequestServedFromCache += (_, e) => RequestServedFromCache?.Invoke(this, e);
 
             await Task.WhenAll(
                Client.SendAsync("Target.setAutoAttach", new TargetSetAutoAttachRequest