test(browser): lock in domcontentloaded defaults for browser_navigate_back/go_forward

[Copilot]

No unit tests existed to guard the wait_until="domcontentloaded" default introduced in #68 for browser_navigate_back and browser_go_forward, leaving the change vulnerable to silent regression.

Changes

• New TestBrowserNavigationDefaults class in test_browser_toolset_error_handling.py — 4 tests asserting wait_until="domcontentloaded" and timeout=None defaults for both methods.

• Inspection approach — inspect.signature cannot introspect Tool objects produced by the @tool decorator. Tests use Tool.args_schema.model_fields instead, which is the Pydantic model the decorator builds directly from the source function's signature:

def test_browser_navigate_back_default_wait_until(self):
    fields = BrowserToolSet.browser_navigate_back.args_schema.model_fields
    assert fields["wait_until"].default == "domcontentloaded"

---

📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.