fix: add fallback for browser opening in explore command

fix: add fallback for browser opening in explore command

Author: lempira

src/algokit/cli/explore.py

@@ -99,10 +99,11 @@ def get_explore_url(network: str) -> str:
 def explore_command(network: str) -> None:
     url = get_explore_url(network)
     logger.info(f"Opening {network} explorer in your default browser")
+    logger.info(f"URL: {url}")
 
-    if is_wsl():
-        import webbrowser
+    import webbrowser
 
+    if is_wsl():
         warning = (
             "Unable to open browser from WSL environment.\n"
             "Ensure 'wslu' is installed: (https://wslutiliti.es/wslu/install.html),\n"
@@ -111,7 +112,13 @@ def explore_command(network: str) -> None:
         try:
             if not webbrowser.open(url):
                 logger.warning(warning)
-        except Exception as e:
-            logger.warning(warning, exc_info=e)
+        except Exception:
+            logger.warning(warning)
     else:
-        click.launch(url)
+        # https://github.com/pallets/click/issues/2868 restore click.launch once bug is fixed
+        try:
+            if not webbrowser.open(url):
+                logger.warning(f"Failed to open browser. Please open this URL manually: {url}")
+        except Exception as e:
+            logger.debug("Error opening browser", exc_info=e)
+            logger.warning(f"Failed to open browser. Please open this URL manually: {url}")

tests/explore/test_explore.py

@@ -11,7 +11,7 @@
 
 @pytest.mark.parametrize("command", ["", "localnet", "testnet", "mainnet"])
 def test_explore(command: str, mocker: MockerFixture) -> None:
-    launch_mock = mocker.patch("click.launch")
+    launch_mock = mocker.patch("webbrowser.open")
     result = invoke(f"explore {command}")
 
     assert result.exit_code == 0
@@ -24,10 +24,22 @@ def test_explore(command: str, mocker: MockerFixture) -> None:
 def test_explore_wsl_exception(mocker: MockerFixture, caplog: pytest.LogCaptureFixture) -> None:
     command = "localnet"
     mocker.patch("algokit.cli.explore.is_wsl", return_value=True)
-    mocker.patch("webbrowser.open", side_effect=Exception("Test Exception"))
+    mocker.patch("webbrowser.open", return_value=False)
 
     with caplog.at_level(logging.WARNING):
         result = invoke(f"explore {command}")
 
     assert result.exit_code == 0
     assert any("Unable to open browser from WSL" in message for message in caplog.messages)
+
+
+def test_explore_webbrowser_exception(mocker: MockerFixture, caplog: pytest.LogCaptureFixture) -> None:
+    command = "localnet"
+    mocker.patch("algokit.cli.explore.is_wsl", return_value=False)
+    mocker.patch("webbrowser.open", side_effect=Exception("Webbrowser Exception"))
+
+    with caplog.at_level(logging.WARNING):
+        result = invoke(f"explore {command}")
+
+    assert result.exit_code == 0
+    assert any("Failed to open browser. Please open this URL manually:" in message for message in caplog.messages)

tests/explore/test_explore.test_explore.localnet.approved.txt

@@ -1,4 +1,5 @@
 Opening localnet explorer in your default browser
+URL: https://explore.algokit.io/localnet
 ----
 launch args:
 ----

tests/explore/test_explore.test_explore.mainnet.approved.txt

@@ -1,4 +1,5 @@
 Opening mainnet explorer in your default browser
+URL: https://explore.algokit.io/mainnet
 ----
 launch args:
 ----

tests/explore/test_explore.test_explore.testnet.approved.txt

@@ -1,4 +1,5 @@
 Opening testnet explorer in your default browser
+URL: https://explore.algokit.io/testnet
 ----
 launch args:
 ----