scmrepo 3.3.10__tar.gz → 3.3.12__tar.gz

{scmrepo-3.3.10 → scmrepo-3.3.12}/.github/workflows/tests.yaml

@@ -20,7 +20,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-20.04, windows-latest, macos-latest]
+        os: [ubuntu-latest, windows-latest, macos-latest]
         pyv: ['3.9', '3.10', '3.11', '3.12', '3.13']
 
     steps:
@@ -47,7 +47,7 @@ jobs:
       run: nox -s tests-${{ matrix.pyv }} -- --slow --cov-report=xml
 
     - name: Upload coverage report
-      uses: codecov/codecov-action@v5.3.1
+      uses: codecov/codecov-action@v5
 
     - name: Build package
       run: nox -s build

{scmrepo-3.3.10 → scmrepo-3.3.12}/.pre-commit-config.yaml

@@ -20,13 +20,13 @@ repos:
       - id: sort-simple-yaml
       - id: trailing-whitespace
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: 'v0.9.3'
+    rev: 'v0.12.3'
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]
       - id: ruff-format
   - repo: https://github.com/codespell-project/codespell
-    rev: v2.4.0
+    rev: v2.4.1
     hooks:
       - id: codespell
         additional_dependencies: ["tomli"]

{scmrepo-3.3.10/src/scmrepo.egg-info → scmrepo-3.3.12}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.2
+Metadata-Version: 2.4
 Name: scmrepo
-Version: 3.3.10
+Version: 3.3.12
 Summary: scmrepo
 Author-email: Iterative <support@dvc.org>
 License: Apache-2.0
@@ -17,7 +17,7 @@ Requires-Python: >=3.9
 Description-Content-Type: text/x-rst
 License-File: LICENSE
 Requires-Dist: gitpython>3
-Requires-Dist: dulwich>=0.22.1
+Requires-Dist: dulwich>=0.23.1
 Requires-Dist: pygit2>=1.14.0
 Requires-Dist: pygtrie>=2.3.2
 Requires-Dist: fsspec[tqdm]>=2024.2.0
@@ -30,7 +30,7 @@ Provides-Extra: tests
 Requires-Dist: aioresponses<0.8,>=0.7; extra == "tests"
 Requires-Dist: paramiko<4,>=3.4.0; extra == "tests"
 Requires-Dist: pytest<9,>=7; extra == "tests"
-Requires-Dist: pytest-asyncio<1,>=0.23.2; extra == "tests"
+Requires-Dist: pytest-asyncio<2,>=0.23.2; extra == "tests"
 Requires-Dist: pytest-cov>=4.1.0; extra == "tests"
 Requires-Dist: pytest-docker<4,>=1; extra == "tests"
 Requires-Dist: pytest-mock; extra == "tests"
@@ -38,12 +38,13 @@ Requires-Dist: pytest-sugar; extra == "tests"
 Requires-Dist: pytest-test-utils<0.2,>=0.1.0; extra == "tests"
 Requires-Dist: proxy.py; extra == "tests"
 Provides-Extra: dev
-Requires-Dist: mypy==1.14.1; extra == "dev"
+Requires-Dist: mypy==1.17.0; extra == "dev"
 Requires-Dist: scmrepo[tests]; extra == "dev"
 Requires-Dist: types-certifi; extra == "dev"
 Requires-Dist: types-mock; extra == "dev"
 Requires-Dist: types-paramiko; extra == "dev"
 Requires-Dist: types-tqdm; extra == "dev"
+Dynamic: license-file
 
 scmrepo
 =======

{scmrepo-3.3.10 → scmrepo-3.3.12}/pyproject.toml

@@ -23,7 +23,7 @@ requires-python = ">=3.9"
 dynamic = ["version"]
 dependencies = [
     "gitpython>3",
-    "dulwich>=0.22.1",
+    "dulwich>=0.23.1",
     "pygit2>=1.14.0",
     "pygtrie>=2.3.2",
     "fsspec[tqdm]>=2024.2.0",
@@ -43,7 +43,7 @@ tests = [
     "aioresponses>=0.7,<0.8",
     "paramiko>=3.4.0,<4",
     "pytest>=7,<9",
-    "pytest-asyncio>=0.23.2,<1",
+    "pytest-asyncio>=0.23.2,<2",
     "pytest-cov>=4.1.0",
     "pytest-docker>=1,<4",
     "pytest-mock",
@@ -52,7 +52,7 @@ tests = [
     "proxy.py",
 ]
 dev = [
-    "mypy==1.14.1",
+    "mypy==1.17.0",
     "scmrepo[tests]",
     "types-certifi",
     "types-mock",
@@ -147,6 +147,7 @@ ignore = [
     "RET503", # implicit-return
     "SIM117", # multiple-with-statements
     "N818", # error-suffix-on-exception-name
+    "PLC0415", # import-outside-top-level
 ]
 select = [
     "A", # flake8-buitlins

{scmrepo-3.3.10 → scmrepo-3.3.12}/src/scmrepo/asyn.py

@@ -6,7 +6,6 @@ import threading
 from typing import Any, Optional
 
 from fsspec.asyn import (  # noqa: F401, pylint:disable=unused-import
-    _selector_policy,
     sync,
     sync_wrapper,
 )
@@ -23,8 +22,7 @@ def get_loop() -> asyncio.AbstractEventLoop:
     if default_loop[0] is None:
         with lock:
             if default_loop[0] is None:
-                with _selector_policy():
-                    default_loop[0] = asyncio.new_event_loop()
+                default_loop[0] = asyncio.new_event_loop()
                 loop = default_loop[0]
                 th = threading.Thread(
                     target=loop.run_forever,  # type: ignore[attr-defined]

{scmrepo-3.3.10 → scmrepo-3.3.12}/src/scmrepo/git/__init__.py

@@ -10,6 +10,7 @@ from contextlib import contextmanager
 from functools import partialmethod
 from typing import (
     TYPE_CHECKING,
+    Any,
     Callable,
     ClassVar,
     Optional,
@@ -295,7 +296,7 @@ class Git(Base):
     # See:
     # https://github.com/iterative/dvc/issues/5641
     # https://github.com/iterative/dvc/issues/7458
-    def _backend_func(self, name, *args, **kwargs):
+    def _backend_func(self, name, *args, **kwargs) -> Any:
         backends: Iterable[str] = kwargs.pop("backends", self.backends)
         for key in backends:
             if self._last_backend is not None and key != self._last_backend:

{scmrepo-3.3.10 → scmrepo-3.3.12}/src/scmrepo/git/backend/dulwich/__init__.py

@@ -39,7 +39,7 @@ logger = logging.getLogger(__name__)
 
 
 class DulwichObject(GitObject):
-    def __init__(self, repo, name, mode, sha):
+    def __init__(self, repo, name, mode, sha) -> None:
         self.repo = repo
         self._name = name
         self._mode = mode
@@ -136,7 +136,7 @@ def _get_ssh_vendor() -> "SSHVendor":
 
 
 class DulwichConfig(Config):
-    def __init__(self, config: Union["ConfigFile", "StackedConfig"]):
+    def __init__(self, config: Union["ConfigFile", "StackedConfig"]) -> None:
         self._config = config
 
     @property
@@ -183,7 +183,7 @@ class DulwichBackend(BaseGitBackend):  # pylint:disable=abstract-method
 
     def __init__(  # pylint:disable=W0231
         self, root_dir=os.curdir, search_parent_directories=True
-    ):
+    ) -> None:
         from dulwich.errors import NotGitRepository
         from dulwich.repo import Repo
 
@@ -990,7 +990,7 @@ def ls_remote(url: str) -> dict[str, str]:
     from dulwich.client import HTTPUnauthorized
 
     try:
-        refs = porcelain.ls_remote(url)
+        refs = porcelain.ls_remote(url).refs
         return {os.fsdecode(ref): sha.decode("ascii") for ref, sha in refs.items()}
     except HTTPUnauthorized as exc:
         raise AuthError(url) from exc

{scmrepo-3.3.10 → scmrepo-3.3.12}/src/scmrepo/git/backend/dulwich/asyncssh_vendor.py

@@ -2,7 +2,7 @@
 
 import asyncio
 import os
-from collections.abc import Coroutine, Iterator, Sequence
+from collections.abc import Coroutine, Iterator
 from typing import (
     TYPE_CHECKING,
     Any,
@@ -18,6 +18,7 @@ from scmrepo.asyn import BaseAsyncObject, sync_wrapper
 from scmrepo.exceptions import AuthError
 
 if TYPE_CHECKING:
+    from collections.abc import Sequence
     from pathlib import Path
 
     from asyncssh.auth import KbdIntPrompts, KbdIntResponse
@@ -40,6 +41,12 @@ async def _read_all(read: Callable[[int], Coroutine], n: Optional[int] = None) -
     return b"".join(result)
 
 
+async def _getpass(*args, **kwargs) -> str:
+    from getpass import getpass
+
+    return await asyncio.to_thread(getpass, *args, **kwargs)
+
+
 class _StderrWrapper:
     def __init__(self, stderr: "SSHReader", loop: asyncio.AbstractEventLoop) -> None:
         self.stderr = stderr
@@ -64,7 +71,9 @@ class _StderrWrapper:
 
 
 class AsyncSSHWrapper(BaseAsyncObject):
-    def __init__(self, conn: "SSHClientConnection", proc: "SSHClientProcess", **kwargs):
+    def __init__(
+        self, conn: "SSHClientConnection", proc: "SSHClientProcess", **kwargs
+    ) -> None:
         super().__init__(**kwargs)
         self.conn: SSHClientConnection = conn
         self.proc: SSHClientProcess = proc
@@ -97,51 +106,12 @@ class AsyncSSHWrapper(BaseAsyncObject):
     close = sync_wrapper(_close)
 
 
-# NOTE: Github's SSH server does not strictly comply with the SSH protocol.
-# When validating a public key using the rsa-sha2-256 or rsa-sha2-512
-# signature algorithms, RFC4252 + RFC8332 state that the server should respond
-# with the same algorithm in SSH_MSG_USERAUTH_PK_OK. Github's server always
-# returns "ssh-rsa" rather than the correct sha2 algorithm name (likely for
-# backwards compatibility with old SSH client reasons). This behavior causes
-# asyncssh to fail with a key-mismatch error (since asyncssh expects the server
-# to behave properly).
-#
-# See also:
-#   https://www.ietf.org/rfc/rfc4252.txt
-#   https://www.ietf.org/rfc/rfc8332.txt
-def _process_public_key_ok_gh(self, _pkttype, _pktid, packet):
-    from asyncssh.misc import ProtocolError
-
-    algorithm = packet.get_string()
-    key_data = packet.get_string()
-    packet.check_end()
-
-    # pylint: disable=protected-access
-    if (
-        (
-            algorithm == b"ssh-rsa"
-            and self._keypair.algorithm
-            not in (
-                b"ssh-rsa",
-                b"rsa-sha2-256",
-                b"rsa-sha2-512",
-            )
-        )
-        or (algorithm not in (b"ssh-rsa", self._keypair.algorithm))
-        or key_data != self._keypair.public_data
-    ):
-        raise ProtocolError("Key mismatch")
-
-    self.create_task(self._send_signed_request())
-    return True
-
-
 class InteractiveSSHClient(SSHClient):
     _conn: Optional["SSHClientConnection"] = None
     _keys_to_try: Optional[list["FilePath"]] = None
     _passphrases: dict[str, str]
 
-    def __init__(self, *args, **kwargs):
+    def __init__(self, *args, **kwargs) -> None:
         super(*args, **kwargs)
         self._passphrases: dict[str, str] = {}
 
@@ -171,7 +141,7 @@ class InteractiveSSHClient(SSHClient):
             self._keys_to_try = []
             options = self._conn._options  # pylint: disable=protected-access
             config = options.config
-            client_keys = cast(Sequence["FilePath"], config.get("IdentityFile", ()))
+            client_keys = cast("Sequence[FilePath]", config.get("IdentityFile", ()))
             if not client_keys:
                 client_keys = [
                     os.path.expanduser(os.path.join("~", ".ssh", path))
@@ -202,8 +172,6 @@ class InteractiveSSHClient(SSHClient):
         return None
 
     async def _read_private_key_interactive(self, path: "FilePath") -> "SSHKey":
-        from getpass import getpass
-
         from asyncssh.public_key import (
             KeyEncryptionError,
             KeyImportError,
@@ -215,11 +183,8 @@ class InteractiveSSHClient(SSHClient):
         if passphrase:
             return read_private_key(path, passphrase=passphrase)
 
-        loop = asyncio.get_running_loop()
         for _ in range(3):
-            passphrase = await loop.run_in_executor(
-                None, getpass, f"Enter passphrase for key '{path}': "
-            )
+            passphrase = await _getpass(f"Enter passphrase for key {path!r}: ")
             if passphrase:
                 try:
                     key = read_private_key(path, passphrase=passphrase)
@@ -239,23 +204,20 @@ class InteractiveSSHClient(SSHClient):
         lang: str,
         prompts: "KbdIntPrompts",
     ) -> Optional["KbdIntResponse"]:
-        from getpass import getpass
-
         if os.environ.get("GIT_TERMINAL_PROMPT") == "0":
             return None
 
-        def _getpass(prompt: str) -> str:
-            return getpass(prompt=prompt).rstrip()
-
         if instructions:
             pass
-        loop = asyncio.get_running_loop()
-        return [
-            await loop.run_in_executor(
-                None, _getpass, f"({name}) {prompt}" if name else prompt
-            )
-            for prompt, _ in prompts
-        ]
+
+        response: list[str] = []
+        for prompt, _echo in prompts:
+            p = await _getpass(f"({name}) {prompt}" if name else prompt)
+            response.append(p.rstrip())
+        return response
+
+    async def password_auth_requested(self) -> str:
+        return await _getpass()
 
 
 class AsyncSSHVendor(BaseAsyncObject, SSHVendor):
@@ -286,12 +248,6 @@ class AsyncSSHVendor(BaseAsyncObject, SSHVendor):
           key_filename: Optional path to private keyfile
         """
         import asyncssh
-        from asyncssh.auth import MSG_USERAUTH_PK_OK, _ClientPublicKeyAuth
-
-        # pylint: disable=protected-access
-        _ClientPublicKeyAuth._packet_handlers[MSG_USERAUTH_PK_OK] = (
-            _process_public_key_ok_gh
-        )
 
         try:
             conn = await asyncssh.connect(

{scmrepo-3.3.10 → scmrepo-3.3.12}/src/scmrepo/git/backend/dulwich/client.py

@@ -14,7 +14,7 @@ class GitCredentialsHTTPClient(Urllib3HttpGitClient):  # pylint: disable=abstrac
         password=None,
         config=None,
         **kwargs,
-    ):
+    ) -> None:
         super().__init__(
             base_url=base_url,
             username=username,
@@ -29,6 +29,7 @@ class GitCredentialsHTTPClient(Urllib3HttpGitClient):  # pylint: disable=abstrac
         url: str,
         headers: Optional[dict[str, str]] = None,
         data: Optional[Union[bytes, Iterator[bytes]]] = None,
+        raise_for_status: bool = True,
     ):
         cached_chunks: list[bytes] = []
 
@@ -48,7 +49,10 @@ class GitCredentialsHTTPClient(Urllib3HttpGitClient):  # pylint: disable=abstrac
 
         try:
             result = super()._http_request(
-                url, headers=headers, data=None if data is None else _cached_data()
+                url,
+                headers=headers,
+                data=None if data is None else _cached_data(),
+                raise_for_status=raise_for_status,
             )
         except HTTPUnauthorized:
             auth_header = self._get_auth()
@@ -59,7 +63,10 @@ class GitCredentialsHTTPClient(Urllib3HttpGitClient):  # pylint: disable=abstrac
             else:
                 headers = auth_header
             result = super()._http_request(
-                url, headers=headers, data=None if data is None else _cached_data()
+                url,
+                headers=headers,
+                data=None if data is None else _cached_data(),
+                raise_for_status=raise_for_status,
             )
         if self._store_credentials is not None:
             self._store_credentials.approve()

{scmrepo-3.3.10 → scmrepo-3.3.12}/src/scmrepo/git/backend/pygit2/__init__.py

@@ -289,7 +289,9 @@ class Pygit2Backend(BaseGitBackend):  # pylint:disable=abstract-method
             bare = True
         try:
             with RemoteCallbacks(progress=progress) as cb:
-                repo = clone_repository(url, to_path, callbacks=cb, bare=bare)
+                repo = clone_repository(
+                    url, os.fspath(to_path), callbacks=cb, bare=bare
+                )
                 if mirror:
                     cls._set_mirror(repo, progress=progress)
         except GitError as exc:

{scmrepo-3.3.10 → scmrepo-3.3.12}/src/scmrepo/git/backend/pygit2/callbacks.py

@@ -66,6 +66,8 @@ class RemoteCallbacks(_RemoteCallbacks, AbstractContextManager):
                 else:
                     creds = Credential(username=username_from_url, url=url).fill()
                     self._store_credentials = creds
+                assert creds.username is not None
+                assert creds.password is not None
                 return UserPass(creds.username, creds.password)
             except CredentialNotFoundError:
                 pass

{scmrepo-3.3.10 → scmrepo-3.3.12}/src/scmrepo/git/backend/pygit2/filter.py

@@ -13,7 +13,7 @@ logger = logging.getLogger(__name__)
 class LFSFilter(Filter):
     attributes = "filter=*"
 
-    def __init__(self, *args, **kwargs):
+    def __init__(self, *args, **kwargs) -> None:
         self._smudge_buf: Optional[io.BytesIO] = None
         self._smudge_root: Optional[str] = None
 

{scmrepo-3.3.10 → scmrepo-3.3.12}/src/scmrepo/git/credentials.py

@@ -105,7 +105,7 @@ class GitCredentialHelper(CredentialHelper):
     >>> password = credentials.password
     """
 
-    def __init__(self, command: str, use_http_path: bool = False):
+    def __init__(self, command: str, use_http_path: bool = False) -> None:
         super().__init__()
         self._command = command
         self._run_kwargs: dict[str, Any] = {}
@@ -136,7 +136,7 @@ class GitCredentialHelper(CredentialHelper):
         if not shutil.which(executable) and shutil.which("git"):
             # If the helper cannot be found in PATH, it might be
             # a C git helper in GIT_EXEC_PATH
-            git_exec_path = subprocess.check_output(  # noqa: S603
+            git_exec_path = subprocess.check_output(
                 ("git", "--exec-path"),
                 text=True,
             ).strip()
@@ -353,7 +353,7 @@ def _input_tty(prompt: str = "Username: ") -> str:
 class MemoryCredentialHelper(CredentialHelper):
     """Memory credential helper that supports optional interactive input."""
 
-    def __init__(self):
+    def __init__(self) -> None:
         super().__init__()
         self._credentials: dict[_CredentialKey, Credential] = {}
 
@@ -535,7 +535,7 @@ class Credential(Mapping[str, str]):
         path: Optional[str] = None,
         username: Optional[str] = None,
         password: Optional[str] = None,
-        password_expiry_utc: Optional[int] = None,
+        password_expiry_utc: Optional[str] = None,
         url: Optional[str] = None,
     ):
         self.protocol = protocol

{scmrepo-3.3.10 → scmrepo-3.3.12}/src/scmrepo/git/lfs/smudge.py

@@ -17,7 +17,7 @@ def smudge(
     batch_size: Optional[int] = None,
 ) -> BinaryIO:
     """Wrap the specified binary IO stream and run LFS smudge if necessary."""
-    reader = io.BufferedReader(fobj)  # type: ignore[arg-type]
+    reader = io.BufferedReader(fobj)  # type: ignore[type-var]
     data = reader.peek(100)
     if any(data.startswith(header) for header in HEADERS):
         # read the pointer data into memory since the raw stream is unseekable

{scmrepo-3.3.10 → scmrepo-3.3.12}/src/scmrepo/git/objects.py

@@ -160,7 +160,7 @@ class GitTrie:
             }
         )
 
-        return cast(dict, ret)
+        return cast("dict", ret)
 
 
 @dataclass

{scmrepo-3.3.10 → scmrepo-3.3.12/src/scmrepo.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.2
+Metadata-Version: 2.4
 Name: scmrepo
-Version: 3.3.10
+Version: 3.3.12
 Summary: scmrepo
 Author-email: Iterative <support@dvc.org>
 License: Apache-2.0
@@ -17,7 +17,7 @@ Requires-Python: >=3.9
 Description-Content-Type: text/x-rst
 License-File: LICENSE
 Requires-Dist: gitpython>3
-Requires-Dist: dulwich>=0.22.1
+Requires-Dist: dulwich>=0.23.1
 Requires-Dist: pygit2>=1.14.0
 Requires-Dist: pygtrie>=2.3.2
 Requires-Dist: fsspec[tqdm]>=2024.2.0
@@ -30,7 +30,7 @@ Provides-Extra: tests
 Requires-Dist: aioresponses<0.8,>=0.7; extra == "tests"
 Requires-Dist: paramiko<4,>=3.4.0; extra == "tests"
 Requires-Dist: pytest<9,>=7; extra == "tests"
-Requires-Dist: pytest-asyncio<1,>=0.23.2; extra == "tests"
+Requires-Dist: pytest-asyncio<2,>=0.23.2; extra == "tests"
 Requires-Dist: pytest-cov>=4.1.0; extra == "tests"
 Requires-Dist: pytest-docker<4,>=1; extra == "tests"
 Requires-Dist: pytest-mock; extra == "tests"
@@ -38,12 +38,13 @@ Requires-Dist: pytest-sugar; extra == "tests"
 Requires-Dist: pytest-test-utils<0.2,>=0.1.0; extra == "tests"
 Requires-Dist: proxy.py; extra == "tests"
 Provides-Extra: dev
-Requires-Dist: mypy==1.14.1; extra == "dev"
+Requires-Dist: mypy==1.17.0; extra == "dev"
 Requires-Dist: scmrepo[tests]; extra == "dev"
 Requires-Dist: types-certifi; extra == "dev"
 Requires-Dist: types-mock; extra == "dev"
 Requires-Dist: types-paramiko; extra == "dev"
 Requires-Dist: types-tqdm; extra == "dev"
+Dynamic: license-file
 
 scmrepo
 =======

{scmrepo-3.3.10 → scmrepo-3.3.12}/src/scmrepo.egg-info/requires.txt

@@ -1,5 +1,5 @@
 gitpython>3
-dulwich>=0.22.1
+dulwich>=0.23.1
 pygit2>=1.14.0
 pygtrie>=2.3.2
 fsspec[tqdm]>=2024.2.0
@@ -10,7 +10,7 @@ aiohttp-retry>=2.5.0
 tqdm
 
 [dev]
-mypy==1.14.1
+mypy==1.17.0
 scmrepo[tests]
 types-certifi
 types-mock
@@ -21,7 +21,7 @@ types-tqdm
 aioresponses<0.8,>=0.7
 paramiko<4,>=3.4.0
 pytest<9,>=7
-pytest-asyncio<1,>=0.23.2
+pytest-asyncio<2,>=0.23.2
 pytest-cov>=4.1.0
 pytest-docker<4,>=1
 pytest-mock

{scmrepo-3.3.10 → scmrepo-3.3.12}/tests/test_dulwich.py

@@ -8,6 +8,7 @@ from unittest.mock import AsyncMock
 import asyncssh
 import paramiko
 import pytest
+from paramiko.server import InteractiveQuery
 from pytest_mock import MockerFixture
 from pytest_test_utils.waiters import wait_until
 
@@ -52,13 +53,24 @@ class Server(paramiko.ServerInterface):
     """http://docs.paramiko.org/en/2.4/api/server.html."""
 
     def __init__(self, commands, *args, **kwargs) -> None:
-        super().__init__(*args, **kwargs)
+        super().__init__()
         self.commands = commands
+        self.allowed_auths = kwargs.get("allowed_auths", "publickey,password")
 
     def check_channel_exec_request(self, channel, command):
         self.commands.append(command)
         return True
 
+    def check_auth_interactive(self, username: str, submethods: str):
+        return InteractiveQuery(
+            "Password", "Enter the password", f"Password for user {USER}:"
+        )
+
+    def check_auth_interactive_response(self, responses):
+        if responses[0] == PASSWORD:
+            return paramiko.AUTH_SUCCESSFUL
+        return paramiko.AUTH_FAILED
+
     def check_auth_password(self, username, password):
         if username == USER and password == PASSWORD:
             return paramiko.AUTH_SUCCESSFUL
@@ -76,12 +88,12 @@ class Server(paramiko.ServerInterface):
         return paramiko.OPEN_FAILED_ADMINISTRATIVELY_PROHIBITED
 
     def get_allowed_auths(self, username):
-        return "password,publickey"
+        return self.allowed_auths
 
 
 @pytest.fixture
 def ssh_conn(request: pytest.FixtureRequest) -> dict[str, Any]:
-    server = Server([])
+    server = Server([], **getattr(request, "param", {}))
 
     socket.setdefaulttimeout(10)
     request.addfinalizer(lambda: socket.setdefaulttimeout(None))
@@ -133,7 +145,8 @@ def test_run_command_password(server: Server, ssh_port: int):
     assert b"test_run_command_password" in server.commands
 
 
-def test_run_command_no_password(server: Server, ssh_port: int):
+@pytest.mark.parametrize("ssh_conn", [{"allowed_auths": "publickey"}], indirect=True)
+def test_run_command_no_password(ssh_port: int):
     vendor = AsyncSSHVendor()
     with pytest.raises(AuthError):
         vendor.run_command(
@@ -145,6 +158,28 @@ def test_run_command_no_password(server: Server, ssh_port: int):
         )
 
 
+@pytest.mark.parametrize(
+    "ssh_conn",
+    [{"allowed_auths": "password"}, {"allowed_auths": "keyboard-interactive"}],
+    indirect=True,
+    ids=["password", "interactive"],
+)
+def test_should_prompt_for_password_when_no_password_passed(
+    mocker: MockerFixture, server: Server, ssh_port: int
+):
+    mocked_getpass = mocker.patch("getpass.getpass", return_value=PASSWORD)
+    vendor = AsyncSSHVendor()
+    vendor.run_command(
+        "127.0.0.1",
+        "test_run_command_password",
+        username=USER,
+        port=ssh_port,
+        password=None,
+    )
+    assert server.commands == [b"test_run_command_password"]
+    mocked_getpass.asssert_called_once()
+
+
 def test_run_command_with_privkey(server: Server, ssh_port: int):
     key = asyncssh.import_private_key(CLIENT_KEY)
 
@@ -212,28 +247,6 @@ def test_run_command_partial_transfer(ssh_port: int, mocker: MockerFixture):
     assert mock_stderr.call_count == 3
 
 
-@pytest.mark.parametrize("algorithm", [b"ssh-rsa", b"rsa-sha2-256", b"rsa-sha2-512"])
-def test_dulwich_github_compat(mocker: MockerFixture, algorithm: bytes):
-    from asyncssh.misc import ProtocolError
-
-    from scmrepo.git.backend.dulwich.asyncssh_vendor import _process_public_key_ok_gh
-
-    key_data = b"foo"
-    auth = mocker.Mock(
-        _keypair=mocker.Mock(algorithm=algorithm, public_data=key_data),
-    )
-    packet = mocker.Mock()
-
-    strings = iter((b"ed21556", key_data))
-    packet.get_string = lambda: next(strings)
-    with pytest.raises(ProtocolError):
-        _process_public_key_ok_gh(auth, None, None, packet)
-
-    strings = iter((b"ssh-rsa", key_data))
-    packet.get_string = lambda: next(strings)
-    _process_public_key_ok_gh(auth, None, None, packet)
-
-
 @pytest.mark.skipif(os.name != "nt", reason="Windows only")
 def test_git_bash_ssh_vendor(mocker):
     from dulwich.client import SubprocessSSHVendor

{scmrepo-3.3.10 → scmrepo-3.3.12}/tests/test_git.py

@@ -771,7 +771,7 @@ def test_ignored(tmp_dir: TmpDir, scm: Git, git: Git, git_backend: str):
     assert not git.is_ignored(tmp_dir / "dir1" / "file2.txt")
 
 
-@pytest.mark.skip_git_backend("pygit2", "gitpython")
+@pytest.mark.skip_git_backend("pygit2", "gitpython", "dulwich")
 def test_ignored_dir_unignored_subdirs(tmp_dir: TmpDir, scm: Git, git: Git):
     tmp_dir.gen({".gitignore": "data/**\n!data/**/\n!data/**/*.csv"})
     scm.add([".gitignore"])