self review

hardkoded · Sep 4, 2024 · 088a344 · 088a344
1 parent 691a010
commit 088a344
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/lib/PuppeteerSharp.Tests/LauncherTests/PuppeteerLaunchTests.cs b/lib/PuppeteerSharp.Tests/LauncherTests/PuppeteerLaunchTests.cs
@@ -396,8 +396,12 @@ await Assert.MultipleAsync(async () =>
             {
                 Assert.That(userDataDirInfo.GetFiles(), Is.Not.Empty);
 
-                Assert.That(await File.ReadAllTextAsync(Path.Combine(userDataDir.Path, "prefs.js")), Is.EqualTo(prefsJSContent));
-                Assert.That(await File.ReadAllTextAsync(Path.Combine(userDataDir.Path, "user.js")), Is.EqualTo(prefsJSContent));
+                Assert.That(
+                    await File.ReadAllTextAsync(Path.Combine(userDataDir.Path, "prefs.js")),
+                    Is.EqualTo(prefsJSContent));
+                Assert.That(
+                    await File.ReadAllTextAsync(Path.Combine(userDataDir.Path, "user.js")),
+                    Is.EqualTo(prefsJSContent));
             });
         }
     }

diff --git a/lib/PuppeteerSharp/FirefoxLauncher.cs b/lib/PuppeteerSharp/FirefoxLauncher.cs
@@ -83,6 +83,7 @@ internal static string[] GetDefaultArgs(LaunchOptions options)
 
         internal override void OnExit()
         {
+            // If TempUserDataDir is null it means that the user provided their own userDataDir
             if (TempUserDataDir is null)
             {
                 var backupSuffix = ".puppeteer";