fix: Use correct browser suffixes in constants (#315)

Author: camdecoster

src/choreographer/browser_async.py

@@ -180,13 +180,11 @@ def run() -> subprocess.Popen[bytes] | subprocess.Popen[str]:  # depends on args
                     break
         except (BrowserClosedError, BrowserFailedError, asyncio.CancelledError) as e:
             raise BrowserFailedError(
-                "The browser seemed to close immediately after starting.",
-                "You can set the `logging.Logger` level lower to see more output.",
-                "You may try installing a known working copy of Chrome by running ",
-                "`$ choreo_get_chrome`."
-                ""
-                "It may be your browser auto-updated and will now work upon "
-                "restart. The browser we tried to start is located at "
+                "The browser seemed to close immediately after starting. "
+                "You can set the `logging.Logger` level lower to see more output. "
+                "You may try installing a known working copy of Chrome by running "
+                "`$ choreo_get_chrome`. It may be your browser auto-updated and will "
+                "now work upon restart. The browser we tried to start is located at "
                 f"{self._browser_impl.path}.",
             ) from e
 

src/choreographer/browsers/_chrome_constants.py

@@ -50,10 +50,10 @@
     typical_edge_paths = tuple(_p + _edge_suffix for _p in _windows_app_dirs)
 
     _brave_suffix = r"\BraveSoftware\Brave-Browser\Application\brave.exe"
-    typical_brave_paths = tuple(_p + _edge_suffix for _p in _windows_app_dirs)
+    typical_brave_paths = tuple(_p + _brave_suffix for _p in _windows_app_dirs)
 
     _vivaldi_suffix = r"\Vivaldi\Application\vivaldi.exe"
-    typical_vivaldi_paths = tuple(_p + _brave_suffix for _p in _windows_app_dirs)
+    typical_vivaldi_paths = tuple(_p + _vivaldi_suffix for _p in _windows_app_dirs)
 
 elif platform.system() == "Linux":
     typical_chrome_paths = (