RemoteRF 2.2.2__tar.gz → 2.2.4__tar.gz

{remoterf-2.2.2/src/remoterf.egg-info → remoterf-2.2.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: remoterf
-Version: 2.2.2
+Version: 2.2.4
 Summary: A python API to remotely access signal centric hardware. Client-side only! Courtesy of Wireless Lab @ UCLA & Prof. Ian Roberts.
 Author-email: Ethan Ge <ethoGalaxy@gmail.com>
 License-Expression: GPL-3.0-or-later

{remoterf-2.2.2 → remoterf-2.2.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "remoterf"
-version = "2.2.2"
+version = "2.2.4"
 description = "A python API to remotely access signal centric hardware. Client-side only! Courtesy of Wireless Lab @ UCLA & Prof. Ian Roberts."
 readme = "README.md"
 license = "GPL-3.0-or-later"

{remoterf-2.2.2 → remoterf-2.2.4}/src/remoterf/core/acc_login.py

@@ -14,7 +14,7 @@
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
 def main(*, register=False, enrollment_code=None, server_label=None, show_banner=True,
-         home=None, route=None):
+         home=None, route=None, username=None):
     from .app import run
     return run(
         register=register,
@@ -23,4 +23,5 @@ def main(*, register=False, enrollment_code=None, server_label=None, show_banner
         show_banner=show_banner,
         home=home,
         route=route,
+        username=username,
     )

{remoterf-2.2.2 → remoterf-2.2.4}/src/remoterf/core/app.py

@@ -116,6 +116,8 @@ def _password_confirmation(*, enforce_policy=False):
 
 # The code typed at `remoterf -r`, used once for this registration.
 pending_enrollment_code = None
+# A username already given (`name@lab` at `-l`): the login asks only the password.
+pending_username = None
 # The saved home this shell is attached to (None on a bare direct profile).
 home_name = None
 # The route it was reached over (None on a bare direct profile or an HTTPS home).
@@ -207,6 +209,9 @@ def _account_command(command):
             account.email = _ask('Email') if policy['email_required'] else ''
             if not account.create_user():
                 raise AccountBackendError('Registration failed')
+            if account.is_https_home:
+                from ..deployment import homes
+                printf(f"Log in from anywhere as {account.username}@{homes.GLOBAL_ACCOUNTS}", Sty.GRAY)
         except BaseException:
             # The code was fine; only the details failed. Keep it for the retry.
             pending_enrollment_code = account.enrollment_code or None
@@ -249,7 +254,14 @@ def _account_command(command):
         return False
     if command in ('l', 'login'):
         printf(f"Login to {_home_name()}", (Sty.BOLD, Sty.BLUE))
-        account.username = _ask('Username or Email' if account.is_https_home else 'Username')
+        global pending_username
+        from ..deployment import homes
+        typed = pending_username or _ask('Username or Email' if account.is_https_home else 'Username')
+        pending_username = None
+        # `ethan@ucla` typed at ucla is `ethan` there.
+        name, where = homes.split_account(typed)
+        here = homes.GLOBAL_ACCOUNTS if account.is_https_home else home_name
+        account.username = name if where == here else typed
         account.password = _ask('Password (Hidden)', hidden=True)
         ok = bool(account.login_user(remember=not account.is_https_home))
         if ok:
@@ -1585,8 +1597,9 @@ def interactive_reserve_next_days_auto():
 
 
 def run(backend=None, *, register=False, enrollment_code=None, server_label=None,
-        show_banner=True, home=None, route=None):
-    global account, session, server_addr, pending_enrollment_code, home_name, current_route
+        show_banner=True, home=None, route=None, username=None):
+    global account, session, server_addr, pending_enrollment_code, home_name, current_route, pending_username
+    pending_username = username
     home_name = home
     current_route = route
     from ..deployment.backend import selected_backend

{remoterf-2.2.2 → remoterf-2.2.4}/src/remoterf/deployment/homes.py

@@ -268,6 +268,20 @@ def connect(name: str) -> dict:
     return route
 
 
+# An account is `name@<deployment>`; Global's own accounts are `name@global`.
+GLOBAL_ACCOUNTS = "global"
+
+
+def split_account(text: str) -> tuple[str, str | None]:
+    """`ethan@ucla` -> ("ethan", "ucla"). A bare name, or an email (a dot
+    after the @; deployment names never have one), names no deployment."""
+    text = text.strip()
+    name, at, where = text.rpartition("@")
+    if not at or not name or not where or "." in where:
+        return text, None
+    return name, where.lower()
+
+
 def register_home(host: str) -> tuple[str, dict, dict]:
     """Learn a deployment from its code, save it, and connect to it."""
     document = discover(host)

{remoterf-2.2.2 → remoterf-2.2.4}/src/remoterf/remoterf_cli.py

@@ -150,7 +150,7 @@ def print_help() -> None:
     printf("  remoterf -h | --help", Sty.CYAN, "              Show this help", Sty.DEFAULT)
     print()
     printf("Commands:", (Sty.BOLD, Sty.MAGENTA))
-    printf("  remoterf -l | --login [target]", Sty.CYAN, "    Login: last used, 'global', or a home name", Sty.DEFAULT)
+    printf("  remoterf -l | --login [target]", Sty.CYAN, "    Login: last used, 'global', a home name, or an account (ethan@ucla)", Sty.DEFAULT)
     printf("  remoterf -r | --register [global]", Sty.CYAN, " Register with an enrollment code (or a RemoteRF Global account)", Sty.DEFAULT)
     printf("  remoterf homes", Sty.CYAN, "                    List saved deployment homes", Sty.DEFAULT)
     printf("  remoterf -v | --version", Sty.CYAN, "           Print version", Sty.DEFAULT)
@@ -171,6 +171,7 @@ def print_help() -> None:
     printf("  remoterf --login", Sty.GREEN)
     printf("  remoterf --login ucla", Sty.GREEN)
     printf("  remoterf --login global", Sty.GREEN)
+    printf("  remoterf --login ethan@ucla", Sty.GREEN)
     printf("  remoterf homes", Sty.GREEN)
     printf("  remoterf --version", Sty.GREEN)
     printf("  remoterf --config --addr 203.0.113.10:12321", Sty.GREEN)
@@ -180,7 +181,8 @@ def print_help() -> None:
 
 def _account_shell(*, register: bool, enrollment_code: str | None = None,
                    server_label: str | None = None, show_banner: bool = True,
-                   home: str | None = None, route: dict | None = None) -> int:
+                   home: str | None = None, route: dict | None = None,
+                   username: str | None = None) -> int:
     ok, _ = _ensure_config_present()
     if not ok:
         _print_server_unavailable()
@@ -198,6 +200,7 @@ def _account_shell(*, register: bool, enrollment_code: str | None = None,
                 show_banner=show_banner,
                 home=home,
                 route=route,
+                username=username,
             )
             or 0
         )
@@ -212,7 +215,7 @@ def _global_origin() -> str:
     return os.getenv('REMOTERF_GLOBAL_ORIGIN', 'https://global.remoterf.net')
 
 
-def _use_global(*, register: bool, show_banner: bool = True) -> int:
+def _use_global(*, register: bool, show_banner: bool = True, username: str | None = None) -> int:
     """Register or log in with the RemoteRF Global identity itself.
 
     A Global-native account is complete on its own: it needs no deployment
@@ -238,7 +241,7 @@ def _use_global(*, register: bool, show_banner: bool = True) -> int:
     except (RuntimeError, ValueError) as exc:
         print(f'Could not reach RemoteRF Global: {exc}')
         return 2
-    return _account_shell(register=register, show_banner=show_banner)
+    return _account_shell(register=register, show_banner=show_banner, username=username)
 
 
 @contextmanager
@@ -293,7 +296,8 @@ def _reason(exc: BaseException) -> str:
     return str(exc)
 
 
-def _use_home(name: str, *, register: bool, enrollment_code: str | None = None) -> int:
+def _use_home(name: str, *, register: bool, enrollment_code: str | None = None,
+              username: str | None = None) -> int:
     from remoterf.deployment import homes
 
     try:
@@ -309,6 +313,7 @@ def _use_home(name: str, *, register: bool, enrollment_code: str | None = None)
         server_label=f'{name} ({_route_label(route)})',
         home=name,
         route={**route, 'home': home},
+        username=username,
     )
 
 
@@ -382,6 +387,8 @@ def _login(target: str | None) -> int:
     if not require_tos_for_login():
         return 1
     homes.migrate_legacy_target()
+    if target is not None and '@' in target:
+        return _login_as(target)
     if target is None:
         last = homes.last_target()
         saved = sorted(homes.homes())
@@ -397,7 +404,7 @@ def _login(target: str | None) -> int:
             # Nothing saved. A legacy address in .env still logs in there;
             # a fresh install has nothing to log in to, so `-l` is `-r`.
             if not _ensure_config_present()[0]:
-                return _register()
+                return _login_by_username()
             return _account_shell(register=False)
     if target == 'global':
         # Global's address is built in, so a fresh install can log in to an
@@ -409,6 +416,41 @@ def _login(target: str | None) -> int:
     return _use_home(target, register=False)
 
 
+def _login_by_username() -> int:
+    """A fresh install: an account says where it lives (`ethan@ucla`), so
+    asking for it is enough. Enter registers, as `-l` here always did."""
+    try:
+        name = input('Account, e.g. ethan@ucla (Enter to register): ').strip()
+    except (EOFError, KeyboardInterrupt):
+        print()
+        return 1
+    if not name:
+        return _register()
+    return _login_as(name)
+
+
+def _login_as(account: str) -> int:
+    """Log in to `name@lab` at that lab; a bare name or an email is Global."""
+    from remoterf.deployment import homes
+
+    name, lab = homes.split_account(account)
+    if lab is None or lab == homes.GLOBAL_ACCOUNTS:
+        return _use_global(register=False, username=name)
+    if lab in homes.homes():
+        return _use_home(lab, register=False, username=name)
+    host = f'{lab}.{homes.GLOBAL_DOMAIN}'
+    try:
+        with _working(f"Looking up {host}…"):
+            lab, home, route = homes.register_home(host)
+    except (RuntimeError, ValueError) as exc:
+        printf(f'Could not reach {host}: {_reason(exc)}', Sty.WARNING)
+        return 2
+    return _account_shell(
+        register=False, server_label=f'{lab} ({_route_label(route)})',
+        home=lab, route={**route, 'home': home}, username=name,
+    )
+
+
 def _print_pick_a_target() -> int:
     _print_homes()
     printf('Choose one: ', Sty.GRAY, 'remoterf --login <name>', Sty.CYAN)

{remoterf-2.2.2 → remoterf-2.2.4/src/remoterf.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: remoterf
-Version: 2.2.2
+Version: 2.2.4
 Summary: A python API to remotely access signal centric hardware. Client-side only! Courtesy of Wireless Lab @ UCLA & Prof. Ian Roberts.
 Author-email: Ethan Ge <ethoGalaxy@gmail.com>
 License-Expression: GPL-3.0-or-later

remoterf-2.2.4/tests/test_cli_login_fresh.py

@@ -0,0 +1,203 @@
+"""`remoterf -l` on a fresh install is `remoterf -r`; a configured client logs in."""
+
+import os
+import tempfile
+from unittest import mock
+
+
+def _fresh_home():
+    tmp = tempfile.TemporaryDirectory()
+    env = {k: v for k, v in os.environ.items() if not k.startswith("REMOTERF_")}
+    env["HOME"] = tmp.name
+    return tmp, mock.patch.dict(os.environ, env, clear=True)
+
+
+def test_login_on_a_fresh_install_registers_on_enter():
+    from remoterf import remoterf_cli
+
+    tmp, env = _fresh_home()
+    with tmp, env, \
+            mock.patch("remoterf.config.config.require_tos_for_login", return_value=True), \
+            mock.patch("builtins.input", return_value=""), \
+            mock.patch.object(remoterf_cli, "_register", return_value=0) as register, \
+            mock.patch.object(remoterf_cli, "_account_shell") as shell:
+        assert remoterf_cli._login(None) == 0
+    register.assert_called_once_with()
+    shell.assert_not_called()
+
+
+def test_an_account_name_says_where_it_lives():
+    from remoterf.deployment.homes import split_account
+
+    assert split_account(" ian@ucla ") == ("ian", "ucla")
+    assert split_account("Ian@UCLA") == ("Ian", "ucla")
+    assert split_account("ian@global") == ("ian", "global")
+    assert split_account("ian") == ("ian", None)
+    assert split_account("ian@ucla.edu") == ("ian@ucla.edu", None)  # an email
+    assert split_account("a@b@ucla") == ("a@b", "ucla")
+    assert split_account("@ucla") == ("@ucla", None)
+    assert split_account("ian@") == ("ian@", None)
+
+
+def test_login_on_a_fresh_install_goes_to_the_lab_in_the_account_name():
+    from remoterf import remoterf_cli
+
+    tmp, env = _fresh_home()
+    route = {"kind": "relay", "host": "ucla.global.remoterf.net", "port": 12321}
+    with tmp, env, \
+            mock.patch("remoterf.config.config.require_tos_for_login", return_value=True), \
+            mock.patch("builtins.input", return_value="ian@ucla"), \
+            mock.patch("remoterf.deployment.homes.register_home",
+                       return_value=("ucla", {"display_name": "UCLA"}, route)) as register_home, \
+            mock.patch.object(remoterf_cli, "_account_shell", return_value=0) as shell:
+        assert remoterf_cli._login(None) == 0
+    register_home.assert_called_once_with("ucla.global.remoterf.net")
+    assert shell.call_args.kwargs["register"] is False
+    assert shell.call_args.kwargs["home"] == "ucla"
+    assert shell.call_args.kwargs["username"] == "ian"
+
+
+def test_login_on_a_fresh_install_global_bare_names_and_emails_go_to_global():
+    from remoterf import remoterf_cli
+
+    for typed, username in (("ian@global", "ian"), ("ian", "ian"), ("ian@ucla.edu", "ian@ucla.edu")):
+        tmp, env = _fresh_home()
+        with tmp, env, \
+                mock.patch("remoterf.config.config.require_tos_for_login", return_value=True), \
+                mock.patch("builtins.input", return_value=typed), \
+                mock.patch("remoterf.deployment.homes.register_home") as register_home, \
+                mock.patch.object(remoterf_cli, "_use_global", return_value=0) as use_global:
+            assert remoterf_cli._login(None) == 0
+        use_global.assert_called_once_with(register=False, username=username)
+        register_home.assert_not_called()
+
+
+def test_login_to_a_lab_that_does_not_answer():
+    from remoterf import remoterf_cli
+
+    tmp, env = _fresh_home()
+    with tmp, env, \
+            mock.patch("remoterf.config.config.require_tos_for_login", return_value=True), \
+            mock.patch("builtins.input", return_value="ian@nolab"), \
+            mock.patch("remoterf.deployment.homes.register_home",
+                       side_effect=RuntimeError("no such deployment")) as register_home, \
+            mock.patch.object(remoterf_cli, "_account_shell") as shell:
+        assert remoterf_cli._login(None) == 2
+    register_home.assert_called_once_with("nolab.global.remoterf.net")
+    shell.assert_not_called()
+
+
+def test_login_with_an_account_uses_the_saved_home():
+    from remoterf import remoterf_cli
+
+    tmp, env = _fresh_home()
+    with tmp, env, \
+            mock.patch("remoterf.config.config.require_tos_for_login", return_value=True), \
+            mock.patch("remoterf.deployment.homes.homes", return_value={"ucla": {}}), \
+            mock.patch("remoterf.deployment.homes.register_home") as register_home, \
+            mock.patch.object(remoterf_cli, "_use_home", return_value=0) as use_home:
+        assert remoterf_cli._login("ian@ucla") == 0
+    use_home.assert_called_once_with("ucla", register=False, username="ian")
+    register_home.assert_not_called()
+
+
+def test_login_global_on_a_fresh_install_logs_in_to_global():
+    from remoterf import remoterf_cli
+
+    tmp, env = _fresh_home()
+    with tmp, env, \
+            mock.patch("remoterf.config.config.require_tos_for_login", return_value=True), \
+            mock.patch.object(remoterf_cli, "_use_global", return_value=0) as use_global:
+        assert remoterf_cli._login("global") == 0
+    use_global.assert_called_once_with(register=False)
+
+
+def test_login_with_a_legacy_address_still_logs_in_there():
+    from remoterf import remoterf_cli
+
+    tmp, env = _fresh_home()
+    with tmp, env, \
+            mock.patch("remoterf.config.config.require_tos_for_login", return_value=True), \
+            mock.patch.object(remoterf_cli, "_ensure_config_present", return_value=(True, "")), \
+            mock.patch.object(remoterf_cli, "_register") as register, \
+            mock.patch.object(remoterf_cli, "_account_shell", return_value=0) as shell:
+        assert remoterf_cli._login(None) == 0
+    shell.assert_called_once_with(register=False)
+    register.assert_not_called()
+
+
+def test_register_without_a_code_makes_a_global_account(monkeypatch):
+    from remoterf import remoterf_cli
+
+    monkeypatch.setattr("builtins.input", lambda prompt="": "")
+    with mock.patch("remoterf.config.config._confirm_tos", return_value=True), \
+            mock.patch("remoterf.config.config.forget_tos_agreement"), \
+            mock.patch("remoterf.config.config.remember_tos_agreement"), \
+            mock.patch.object(remoterf_cli, "print_client_banner"), \
+            mock.patch.object(remoterf_cli, "_use_global", return_value=0) as use_global, \
+            mock.patch.object(remoterf_cli, "_account_shell") as shell:
+        assert remoterf_cli._register() == 0
+    use_global.assert_called_once_with(register=True, show_banner=False)
+    shell.assert_not_called()
+
+
+def test_register_global_asks_for_the_tos_like_register(monkeypatch):
+    from remoterf import remoterf_cli
+
+    for arg in ("global", "-global"):
+        monkeypatch.setattr("sys.argv", ["remoterf", "-r", arg])
+        with mock.patch("remoterf.config.config._confirm_tos", return_value=True) as confirm, \
+                mock.patch("remoterf.config.config.forget_tos_agreement") as forget, \
+                mock.patch("remoterf.config.config.remember_tos_agreement") as remember, \
+                mock.patch.object(remoterf_cli, "print_client_banner"), \
+                mock.patch.object(remoterf_cli, "_use_global", return_value=0) as use_global:
+            assert remoterf_cli.main() == 0
+        forget.assert_called_once_with()
+        confirm.assert_called_once()
+        remember.assert_called_once_with()
+        use_global.assert_called_once_with(register=True, show_banner=False)
+
+    # Declining stops before any account is made.
+    monkeypatch.setattr("sys.argv", ["remoterf", "-r", "global"])
+    with mock.patch("remoterf.config.config._confirm_tos", return_value=False), \
+            mock.patch("remoterf.config.config.forget_tos_agreement"), \
+            mock.patch("remoterf.config.config.remember_tos_agreement") as remember, \
+            mock.patch.object(remoterf_cli, "print_client_banner"), \
+            mock.patch.object(remoterf_cli, "_use_global") as use_global:
+        assert remoterf_cli.main() == 1
+    remember.assert_not_called()
+    use_global.assert_not_called()
+
+
+def _login_at(home, typed, *, https=False, pending=None):
+    from remoterf.core import app
+
+    account = mock.Mock(is_https_home=https, backend=None)
+    account.login_user.return_value = True
+    with mock.patch.object(app, "account", account), \
+            mock.patch.object(app, "home_name", home), \
+            mock.patch.object(app, "pending_username", pending), \
+            mock.patch.object(app, "local_capabilities",
+                              return_value={"registration_policy": {}}), \
+            mock.patch.object(app, "_home_name", return_value=home), \
+            mock.patch.object(app, "_remember_login"), \
+            mock.patch.object(app, "_ask", side_effect=[typed, "secret"] if typed else ["secret"]) as ask:
+        if https:
+            account.backend = mock.Mock(capabilities={"registration_policy": {}})
+        assert app._account_command("login") is True
+        assert app.pending_username is None
+    return account.username, ask
+
+
+def test_a_found_username_is_not_asked_again():
+    username, ask = _login_at("ucla", None, pending="ian")
+    assert username == "ian"
+    ask.assert_called_once_with("Password (Hidden)", hidden=True)
+
+
+def test_the_full_account_name_works_at_its_own_home_only():
+    assert _login_at("ucla", "ian@ucla")[0] == "ian"
+    assert _login_at("ucla", "ian")[0] == "ian"
+    assert _login_at("otheruni", "ian@ucla")[0] == "ian@ucla"  # not this lab's: no such name here
+    assert _login_at(None, "ian@global", https=True)[0] == "ian"
+    assert _login_at(None, "ian@ucla.edu", https=True)[0] == "ian@ucla.edu"  # an email

remoterf-2.2.2/tests/test_cli_login_fresh.py

@@ -1,93 +0,0 @@
-"""`remoterf -l` on a fresh install is `remoterf -r`; a configured client logs in."""
-
-import os
-import tempfile
-from unittest import mock
-
-
-def _fresh_home():
-    tmp = tempfile.TemporaryDirectory()
-    env = {k: v for k, v in os.environ.items() if not k.startswith("REMOTERF_")}
-    env["HOME"] = tmp.name
-    return tmp, mock.patch.dict(os.environ, env, clear=True)
-
-
-def test_login_on_a_fresh_install_registers():
-    from remoterf import remoterf_cli
-
-    tmp, env = _fresh_home()
-    with tmp, env, \
-            mock.patch("remoterf.config.config.require_tos_for_login", return_value=True), \
-            mock.patch.object(remoterf_cli, "_register", return_value=0) as register, \
-            mock.patch.object(remoterf_cli, "_account_shell") as shell:
-        assert remoterf_cli._login(None) == 0
-    register.assert_called_once_with()
-    shell.assert_not_called()
-
-
-def test_login_global_on_a_fresh_install_logs_in_to_global():
-    from remoterf import remoterf_cli
-
-    tmp, env = _fresh_home()
-    with tmp, env, \
-            mock.patch("remoterf.config.config.require_tos_for_login", return_value=True), \
-            mock.patch.object(remoterf_cli, "_use_global", return_value=0) as use_global:
-        assert remoterf_cli._login("global") == 0
-    use_global.assert_called_once_with(register=False)
-
-
-def test_login_with_a_legacy_address_still_logs_in_there():
-    from remoterf import remoterf_cli
-
-    tmp, env = _fresh_home()
-    with tmp, env, \
-            mock.patch("remoterf.config.config.require_tos_for_login", return_value=True), \
-            mock.patch.object(remoterf_cli, "_ensure_config_present", return_value=(True, "")), \
-            mock.patch.object(remoterf_cli, "_register") as register, \
-            mock.patch.object(remoterf_cli, "_account_shell", return_value=0) as shell:
-        assert remoterf_cli._login(None) == 0
-    shell.assert_called_once_with(register=False)
-    register.assert_not_called()
-
-
-def test_register_without_a_code_makes_a_global_account(monkeypatch):
-    from remoterf import remoterf_cli
-
-    monkeypatch.setattr("builtins.input", lambda prompt="": "")
-    with mock.patch("remoterf.config.config._confirm_tos", return_value=True), \
-            mock.patch("remoterf.config.config.forget_tos_agreement"), \
-            mock.patch("remoterf.config.config.remember_tos_agreement"), \
-            mock.patch.object(remoterf_cli, "print_client_banner"), \
-            mock.patch.object(remoterf_cli, "_use_global", return_value=0) as use_global, \
-            mock.patch.object(remoterf_cli, "_account_shell") as shell:
-        assert remoterf_cli._register() == 0
-    use_global.assert_called_once_with(register=True, show_banner=False)
-    shell.assert_not_called()
-
-
-def test_register_global_asks_for_the_tos_like_register(monkeypatch):
-    from remoterf import remoterf_cli
-
-    for arg in ("global", "-global"):
-        monkeypatch.setattr("sys.argv", ["remoterf", "-r", arg])
-        with mock.patch("remoterf.config.config._confirm_tos", return_value=True) as confirm, \
-                mock.patch("remoterf.config.config.forget_tos_agreement") as forget, \
-                mock.patch("remoterf.config.config.remember_tos_agreement") as remember, \
-                mock.patch.object(remoterf_cli, "print_client_banner"), \
-                mock.patch.object(remoterf_cli, "_use_global", return_value=0) as use_global:
-            assert remoterf_cli.main() == 0
-        forget.assert_called_once_with()
-        confirm.assert_called_once()
-        remember.assert_called_once_with()
-        use_global.assert_called_once_with(register=True, show_banner=False)
-
-    # Declining stops before any account is made.
-    monkeypatch.setattr("sys.argv", ["remoterf", "-r", "global"])
-    with mock.patch("remoterf.config.config._confirm_tos", return_value=False), \
-            mock.patch("remoterf.config.config.forget_tos_agreement"), \
-            mock.patch("remoterf.config.config.remember_tos_agreement") as remember, \
-            mock.patch.object(remoterf_cli, "print_client_banner"), \
-            mock.patch.object(remoterf_cli, "_use_global") as use_global:
-        assert remoterf_cli.main() == 1
-    remember.assert_not_called()
-    use_global.assert_not_called()