Add SCP only tests

.github/workflows/ci.yml

@@ -83,7 +83,7 @@ jobs:
 
       - name: Wait for containers to start
         run: |
-          SSH_PORTS="2222 2223"
+          SSH_PORTS="2222 2223 2224"
           for p in $SSH_PORTS; do
             echo "::group::Wait for container $p to start"
             for i in {1..20}; do ssh-keyscan -p $p localhost && break || echo -e "\033[0;33mNot started yet ($i)\033[0m" && sleep 1; done

tests/Dockerfile

@@ -0,0 +1,5 @@
+FROM alpine:latest
+RUN apk add --no-cache dropbear dropbear-scp
+RUN mkdir /root/.ssh/ \
+    && echo "ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIEq2cLqs++uwJh6xLTy4nX20UDfbgb6cDZl2nNkJEKud" > /root/.ssh/authorized_keys
+ENTRYPOINT ["dropbear", "-EFR"]

tests/conftest.py

@@ -34,4 +34,6 @@ def nr_dry_run():
 
 
 def pytest_generate_tests(metafunc: pytest.Metafunc) -> None:
+    if "host" not in metafunc.fixturenames:
+        return
     metafunc.parametrize("host", ("password", "key"))

tests/docker-compose.yml

@@ -17,3 +17,10 @@ services:
       PUBLIC_KEY: ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIEq2cLqs++uwJh6xLTy4nX20UDfbgb6cDZl2nNkJEKud
     ports:
       - 2223:2222
+  alpinescp:
+    image: alpinescp
+    build:
+      context: .
+      dockerfile: Dockerfile
+    ports:
+      - 2224:2222

tests/inventory/hosts.yml

@@ -12,3 +12,11 @@ key:
     paramiko:
       extras:
         key_filename: tests/id_ed25519
+alpinescp:
+  hostname: 127.0.0.1
+  port: 2224
+  username: root
+  connection_options:
+    paramiko:
+      extras:
+        key_filename: tests/id_ed25519

tests/test_paramiko_sftp.py

@@ -7,7 +7,6 @@
 import pytest
 from nornir.core import Nornir
 
-from nornir_paramiko.exceptions import CommandError
 from nornir_paramiko.plugins.tasks import paramiko_command, paramiko_sftp
 
 
@@ -110,3 +109,34 @@ def test_paramiko_sftp_dry_run(host: str, nr_dry_run: Nornir, temp_dir: Path) ->
     # Check that the file was not written
     result = nr_filtered.run(task=paramiko_command, command=f"test -f '{host_file}'")
     assert result.failed is True  # because the file wasn't written
+
+
+def test_paramiko_scp_only_host(nr_dry_run: Nornir, temp_dir: Path) -> None:
+    host = "alpinescp"
+    nr_filtered = nr_dry_run.filter(name=host)
+
+    test_content = random_str(20)
+
+    host_file = f"/tmp/test_{random_str(5)}"
+
+    test_put_file = temp_dir.joinpath("test_put")
+
+    with open(test_put_file, "wt") as f:
+        f.write(test_content)
+    result = nr_filtered.run(
+        task=paramiko_sftp,
+        action="put",
+        src=str(test_put_file),
+        dst=host_file,
+        compare=False,
+    )
+
+    assert result.failed is False
+    assert len(result[host]) == 1
+    host_result = result[host][0]
+    assert host_result.changed is True
+    assert host_result.files_changed == [host_file]
+
+    # Check that the file was not written
+    result = nr_filtered.run(task=paramiko_command, command=f"test -f '{host_file}'")
+    assert result.failed is False  # because the file wasn't written