cornflex 1.0.1__tar.gz

cornflex-1.0.1/.github/workflows/pr.yml

@@ -0,0 +1,26 @@
+name: PR Build
+on:
+  pull_request:
+    branches:
+      - main
+jobs:
+  build-and-test:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+      - name: Set up Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.x'
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install build pytest
+          pip install -e .
+      - name: Run tests
+        run: |
+          pytest
+      - name: Build package
+        run: |
+          python -m build

cornflex-1.0.1/.github/workflows/publish.yml

@@ -0,0 +1,41 @@
+name: Publish
+
+on:
+  push:
+    tags:
+      - '*'
+
+jobs:
+  build-and-publish:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+
+      - name: Set up Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.x'
+
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install build twine
+          pip install -e ".[dev]"
+
+      - name: Run tests
+        run: |
+          pip install pytest
+          pytest
+
+      - name: Build package
+        run: |
+          python -m build
+
+      - name: Publish package to PyPI
+        env:
+          TWINE_USERNAME: ${{ secrets.PYPI_USERNAME }}
+          TWINE_PASSWORD: ${{ secrets.PYPI_PASSWORD }}
+        run: |
+          python -m twine upload dist/*

cornflex-1.0.1/.gitignore

@@ -0,0 +1,165 @@
+# Byte-compiled / optimized / DLL files
+__pycache__/
+*.py[cod]
+*$py.class
+
+# C extensions
+*.so
+
+# Distribution / packaging
+.Python
+build/
+develop-eggs/
+dist/
+downloads/
+eggs/
+.eggs/
+lib/
+lib64/
+parts/
+sdist/
+var/
+wheels/
+share/python-wheels/
+*.egg-info/
+.installed.cfg
+*.egg
+MANIFEST
+
+# PyInstaller
+#  Usually these files are written by a python script from a template
+#  before PyInstaller builds the exe, so as to inject date/other infos into it.
+*.manifest
+*.spec
+
+# Installer logs
+pip-log.txt
+pip-delete-this-directory.txt
+
+# Unit test / coverage reports
+htmlcov/
+.tox/
+.nox/
+.coverage
+.coverage.*
+.cache
+nosetests.xml
+coverage.xml
+*.cover
+*.py,cover
+.hypothesis/
+.pytest_cache/
+cover/
+
+# Translations
+*.mo
+*.pot
+
+# Django stuff:
+*.log
+local_settings.py
+db.sqlite3
+db.sqlite3-journal
+
+# Flask stuff:
+instance/
+.webassets-cache
+
+# Scrapy stuff:
+.scrapy
+
+# Sphinx documentation
+docs/_build/
+
+# PyBuilder
+.pybuilder/
+target/
+
+# Jupyter Notebook
+.ipynb_checkpoints
+
+# IPython
+profile_default/
+ipython_config.py
+
+# pyenv
+#   For a library or package, you might want to ignore these files since the code is
+#   intended to run in multiple environments; otherwise, check them in:
+# .python-version
+
+# pipenv
+#   According to pypa/pipenv#598, it is recommended to include Pipfile.lock in version control.
+#   However, in case of collaboration, if having platform-specific dependencies or dependencies
+#   having no cross-platform support, pipenv may install dependencies that don't work, or not
+#   install all needed dependencies.
+#Pipfile.lock
+
+# poetry
+#   Similar to Pipfile.lock, it is generally recommended to include poetry.lock in version control.
+#   This is especially recommended for binary packages to ensure reproducibility, and is more
+#   commonly ignored for libraries.
+#   https://python-poetry.org/docs/basic-usage/#commit-your-poetrylock-file-to-version-control
+#poetry.lock
+
+# pdm
+#   Similar to Pipfile.lock, it is generally recommended to include pdm.lock in version control.
+#pdm.lock
+#   pdm stores project-wide configurations in .pdm.toml, but it is recommended to not include it
+#   in version control.
+#   https://pdm.fming.dev/latest/usage/project/#working-with-version-control
+.pdm.toml
+.pdm-python
+.pdm-build/
+
+# PEP 582; used by e.g. github.com/David-OConnor/pyflow and github.com/pdm-project/pdm
+__pypackages__/
+
+# Celery stuff
+celerybeat-schedule
+celerybeat.pid
+
+# SageMath parsed files
+*.sage.py
+
+# Environments
+.env
+.venv
+env/
+venv/
+ENV/
+env.bak/
+venv.bak/
+
+# Spyder project settings
+.spyderproject
+.spyproject
+
+# Rope project settings
+.ropeproject
+
+# mkdocs documentation
+/site
+
+# mypy
+.mypy_cache/
+.dmypy.json
+dmypy.json
+
+# Pyre type checker
+.pyre/
+
+# pytype static type analyzer
+.pytype/
+
+# Cython debug symbols
+cython_debug/
+
+# PyCharm
+#  JetBrains specific template is maintained in a separate JetBrains.gitignore that can
+#  be found at https://github.com/github/gitignore/blob/main/Global/JetBrains.gitignore
+#  and can be added to the global gitignore or merged into this file.  For a more nuclear
+#  option (not recommended) you can uncomment the following to ignore the entire idea folder.
+#.idea/
+
+
+.DS_Store

cornflex-1.0.1/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2024 Erik Bremstedt
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

cornflex-1.0.1/PKG-INFO

@@ -0,0 +1,133 @@
+Metadata-Version: 2.4
+Name: cornflex
+Version: 1.0.1
+Summary: simple sftp reader
+Project-URL: Homepage, https://github.com/ebremstedt/cornflex
+Project-URL: Issues, https://github.com/ebremstedt/cornflex/issues
+Author-email: Erik Bremstedt <erik.bremstedt@gmail.com>
+License-File: LICENSE
+Classifier: License :: OSI Approved :: MIT License
+Classifier: Operating System :: OS Independent
+Classifier: Programming Language :: Python :: 3
+Requires-Python: >=3.11.0
+Requires-Dist: chardet>=5.0.0
+Requires-Dist: croniter>0.3.0
+Requires-Dist: paramiko>=3.0.0
+Requires-Dist: polars>=0.20.0
+Description-Content-Type: text/markdown
+
+# 🌽💪 Cornflex 🌽💪
+
+A lightweight SFTP reader built on top of `paramiko` and `polars`. Connects to remote servers and pulls files down as strings, Polars DataFrames, or raw text.
+
+---
+
+## Installation
+
+```bash
+pip install paramiko polars chardet
+```
+
+Then drop `cornflex/` into your project.
+
+---
+
+## Usage
+
+```python
+from cornflex import SFTPReader
+```
+
+### Connect with password
+
+```python
+reader = SFTPReader(
+    hostname="sftp.example.com",
+    username="myuser",
+    password="mypassword",
+)
+```
+
+### Connect with PEM key
+
+```python
+reader = SFTPReader(
+    hostname="sftp.example.com",
+    username="myuser",
+    pem_file="/path/to/key.pem",
+)
+```
+
+---
+
+## Methods
+
+### `get_files(remote_path, file_pattern)`
+
+Lists files in a remote directory. Supports glob-style patterns.
+
+```python
+reader.get_files(remote_path="/data", file_pattern="*.csv")
+# ["orders_2024.csv", "users_2024.csv"]
+```
+
+---
+
+### `get_csv_file(file_name, remote_path, column_names)`
+
+Fetches a CSV and returns it as a Polars DataFrame.
+
+```python
+reader.connect()
+df = reader.get_csv_file("orders.csv", remote_path="/data")
+
+# Override column names (useful when file has no header)
+df = reader.get_csv_file("orders.csv", remote_path="/data", column_names=["id", "amount", "date"])
+reader.close()
+```
+
+---
+
+### `get_xml_file_to_string(file_name, remote_path)`
+
+Fetches an XML file and returns it as a string.
+
+```python
+reader.connect()
+xml = reader.get_xml_file_to_string("feed.xml", remote_path="/exports")
+reader.close()
+```
+
+---
+
+### `file_to_string(file_name, remote_path, encoding)`
+
+Fetches any file as a string. Auto-detects encoding via `chardet` if not specified.
+
+```python
+reader.connect()
+content = reader.file_to_string("report.txt", remote_path="/reports")
+
+# Force encoding
+content = reader.file_to_string("report.txt", encoding="latin-1")
+reader.close()
+```
+
+---
+
+## Notes
+
+- `get_files()` handles connect/close internally.
+- For all other methods, call `connect()` before and `close()` after.
+- Either `password` or `pem_file` must be provided — not both, not neither.
+- `get_csv_file` assumes UTF-8. Use `file_to_string` for other encodings.
+
+---
+
+## Dependencies
+
+| Package | Purpose |
+|---|---|
+| `paramiko` | SSH/SFTP connection |
+| `polars` | DataFrame output |
+| `chardet` | Encoding detection |

cornflex-1.0.1/README.md

@@ -0,0 +1,115 @@
+# 🌽💪 Cornflex 🌽💪
+
+A lightweight SFTP reader built on top of `paramiko` and `polars`. Connects to remote servers and pulls files down as strings, Polars DataFrames, or raw text.
+
+---
+
+## Installation
+
+```bash
+pip install paramiko polars chardet
+```
+
+Then drop `cornflex/` into your project.
+
+---
+
+## Usage
+
+```python
+from cornflex import SFTPReader
+```
+
+### Connect with password
+
+```python
+reader = SFTPReader(
+    hostname="sftp.example.com",
+    username="myuser",
+    password="mypassword",
+)
+```
+
+### Connect with PEM key
+
+```python
+reader = SFTPReader(
+    hostname="sftp.example.com",
+    username="myuser",
+    pem_file="/path/to/key.pem",
+)
+```
+
+---
+
+## Methods
+
+### `get_files(remote_path, file_pattern)`
+
+Lists files in a remote directory. Supports glob-style patterns.
+
+```python
+reader.get_files(remote_path="/data", file_pattern="*.csv")
+# ["orders_2024.csv", "users_2024.csv"]
+```
+
+---
+
+### `get_csv_file(file_name, remote_path, column_names)`
+
+Fetches a CSV and returns it as a Polars DataFrame.
+
+```python
+reader.connect()
+df = reader.get_csv_file("orders.csv", remote_path="/data")
+
+# Override column names (useful when file has no header)
+df = reader.get_csv_file("orders.csv", remote_path="/data", column_names=["id", "amount", "date"])
+reader.close()
+```
+
+---
+
+### `get_xml_file_to_string(file_name, remote_path)`
+
+Fetches an XML file and returns it as a string.
+
+```python
+reader.connect()
+xml = reader.get_xml_file_to_string("feed.xml", remote_path="/exports")
+reader.close()
+```
+
+---
+
+### `file_to_string(file_name, remote_path, encoding)`
+
+Fetches any file as a string. Auto-detects encoding via `chardet` if not specified.
+
+```python
+reader.connect()
+content = reader.file_to_string("report.txt", remote_path="/reports")
+
+# Force encoding
+content = reader.file_to_string("report.txt", encoding="latin-1")
+reader.close()
+```
+
+---
+
+## Notes
+
+- `get_files()` handles connect/close internally.
+- For all other methods, call `connect()` before and `close()` after.
+- Either `password` or `pem_file` must be provided — not both, not neither.
+- `get_csv_file` assumes UTF-8. Use `file_to_string` for other encodings.
+
+---
+
+## Dependencies
+
+| Package | Purpose |
+|---|---|
+| `paramiko` | SSH/SFTP connection |
+| `polars` | DataFrame output |
+| `chardet` | Encoding detection |

cornflex-1.0.1/cornflex/__init__.py

@@ -0,0 +1,3 @@
+from cornflex.reader import SFTPReader
+
+__all__ = ["SFTPReader"]

cornflex-1.0.1/cornflex/reader.py

@@ -0,0 +1,128 @@
+from dataclasses import dataclass
+from io import StringIO
+from typing import Any, List, Optional
+import chardet
+import fnmatch
+import paramiko
+import polars as pl
+
+
+@dataclass
+class SFTPReader:
+    hostname: str
+    username: str
+    pem_file: Optional[str] = None
+    password: Optional[str] = None
+    port: int = 22
+
+    def __post_init__(self) -> None:
+        self._client: Optional[paramiko.SSHClient] = None
+        self._sftp: Optional[paramiko.SFTPClient] = None
+
+    def connect(self) -> None:
+        if not self.password and not self.pem_file:
+            raise ValueError("Either password or pem_file must be provided")
+
+        self._client = paramiko.SSHClient()
+        self._client.set_missing_host_key_policy(paramiko.AutoAddPolicy())
+
+        connect_kwargs: dict[str, Any] = {
+            "hostname": self.hostname,
+            "port": self.port,
+            "username": self.username,
+            "timeout": 30,
+        }
+
+        if self.pem_file:
+            private_key = paramiko.RSAKey.from_private_key_file(self.pem_file)
+            connect_kwargs["pkey"] = private_key
+        else:
+            connect_kwargs["password"] = self.password
+
+        self._client.connect(**connect_kwargs)
+        self._sftp = self._client.open_sftp()
+
+    def close(self) -> None:
+        if self._sftp:
+            self._sftp.close()
+        if self._client:
+            self._client.close()
+
+    def get_files(
+        self,
+        remote_path: str = ".",
+        file_pattern: str = "*",
+    ) -> List[str]:
+        self.connect()
+        try:
+            all_files: List[str] = self._sftp.listdir(remote_path)
+            return [f for f in all_files if fnmatch.fnmatch(name=f, pat=file_pattern)]
+        finally:
+            self.close()
+
+    def get_csv_file(
+        self,
+        file_name: str,
+        remote_path: str = ".",
+        column_names: Optional[list[str]] = None,
+    ) -> Optional[pl.DataFrame]:
+        if not self._sftp:
+            raise RuntimeError("Not connected. Call connect() first.")
+
+        try:
+            remote_file_path = f"{remote_path.rstrip('/')}/{file_name}"
+            with self._sftp.file(remote_file_path, "r") as file:
+                content = file.read().decode("utf-8")
+
+            if column_names:
+                return pl.read_csv(
+                    source=StringIO(content),
+                    has_header=False,
+                    new_columns=column_names,
+                )
+            return pl.read_csv(source=StringIO(content))
+        except Exception as e:
+            print(f"Error getting {file_name}: {e}")
+            return None
+
+    def get_xml_file_to_string(
+        self,
+        file_name: str,
+        remote_path: str = ".",
+    ) -> Optional[str]:
+        if not self._sftp:
+            raise RuntimeError("Not connected. Call connect() first.")
+
+        try:
+            remote_file_path = f"{remote_path.rstrip('/')}/{file_name}"
+            with self._sftp.file(remote_file_path, "r") as file:
+                content = file.read().decode("utf-8")
+            return content
+        except Exception as e:
+            print(f"Error getting {file_name}: {e}")
+            return None
+
+    def file_to_string(
+        self,
+        file_name: str,
+        remote_path: str = ".",
+        encoding: Optional[str] = None,
+    ) -> Optional[str]:
+        if not self._sftp:
+            raise RuntimeError("Not connected. Call connect() first.")
+
+        try:
+            remote_file_path = f"{remote_path.rstrip('/')}/{file_name}"
+            with self._sftp.file(remote_file_path, "rb") as file:
+                raw_bytes = file.read()
+
+            if encoding is None:
+                detected = chardet.detect(raw_bytes)
+                encoding = detected["encoding"]
+
+            content = raw_bytes.decode(encoding)
+            return content
+
+        except Exception as e:
+            print(f"Error getting {file_name}: {e}")
+            return None

cornflex-1.0.1/pyproject.toml

@@ -0,0 +1,28 @@
+[build-system]
+requires = ["hatchling"]
+build-backend = "hatchling.build"
+
+[project]
+name = "cornflex"
+version = "1.0.1"
+authors = [
+  { name="Erik Bremstedt", email="erik.bremstedt@gmail.com" },
+]
+description = "simple sftp reader"
+readme = "README.md"
+requires-python = ">=3.11.0"
+classifiers = [
+  "Programming Language :: Python :: 3",
+  "License :: OSI Approved :: MIT License",
+  "Operating System :: OS Independent",
+]
+dependencies = [
+  "croniter>0.3.0",
+  "paramiko>=3.0.0",
+  "polars>=0.20.0",
+  "chardet>=5.0.0",
+]
+
+[project.urls]
+Homepage = "https://github.com/ebremstedt/cornflex"
+Issues = "https://github.com/ebremstedt/cornflex/issues"

cornflex-1.0.1/tests/test_reader.py

@@ -0,0 +1,198 @@
+from io import StringIO
+from unittest.mock import MagicMock, patch, PropertyMock
+import pytest
+import polars as pl
+from polars.testing import assert_frame_equal
+
+from cornflex.reader import SFTPReader
+
+
+@pytest.fixture
+def reader_password() -> SFTPReader:
+    return SFTPReader(hostname="host", username="user", password="pass")
+
+
+@pytest.fixture
+def reader_pem() -> SFTPReader:
+    return SFTPReader(hostname="host", username="user", pem_file="/path/to/key.pem")
+
+
+@pytest.fixture
+def connected_reader(reader_password: SFTPReader) -> SFTPReader:
+    reader_password._sftp = MagicMock()
+    return reader_password
+
+
+# --- connect ---
+
+
+def test_connect_raises_without_auth() -> None:
+    reader = SFTPReader(hostname="host", username="user")
+    with pytest.raises(
+        ValueError, match="Either password or pem_file must be provided"
+    ):
+        reader.connect()
+
+
+@patch("cornflex.reader.paramiko.SSHClient")
+def test_connect_with_password(
+    mock_ssh: MagicMock, reader_password: SFTPReader
+) -> None:
+    reader_password.connect()
+    mock_ssh().connect.assert_called_once()
+    call_kwargs = mock_ssh().connect.call_args.kwargs
+    assert call_kwargs["password"] == "pass"
+    assert "pkey" not in call_kwargs
+
+
+@patch("cornflex.reader.paramiko.RSAKey.from_private_key_file")
+@patch("cornflex.reader.paramiko.SSHClient")
+def test_connect_with_pem(
+    mock_ssh: MagicMock, mock_rsa: MagicMock, reader_pem: SFTPReader
+) -> None:
+    reader_pem.connect()
+    mock_rsa.assert_called_once_with("/path/to/key.pem")
+    call_kwargs = mock_ssh().connect.call_args.kwargs
+    assert "pkey" in call_kwargs
+    assert "password" not in call_kwargs
+
+
+# --- close ---
+
+
+def test_close_calls_sftp_and_client(reader_password: SFTPReader) -> None:
+    reader_password._sftp = MagicMock()
+    reader_password._client = MagicMock()
+    reader_password.close()
+    reader_password._sftp.close.assert_called_once()
+    reader_password._client.close.assert_called_once()
+
+
+def test_close_without_connection_does_not_raise(reader_password: SFTPReader) -> None:
+    reader_password.close()
+
+
+# --- get_files ---
+
+
+@patch("cornflex.reader.paramiko.SSHClient")
+def test_get_files_returns_filtered(
+    mock_ssh: MagicMock, reader_password: SFTPReader
+) -> None:
+    mock_sftp = MagicMock()
+    mock_sftp.listdir.return_value = ["orders.csv", "users.csv", "config.json"]
+    mock_ssh().open_sftp.return_value = mock_sftp
+
+    result = reader_password.get_files(remote_path="/data", file_pattern="*.csv")
+    assert result == ["orders.csv", "users.csv"]
+
+
+@patch("cornflex.reader.paramiko.SSHClient")
+def test_get_files_no_match(mock_ssh: MagicMock, reader_password: SFTPReader) -> None:
+    mock_sftp = MagicMock()
+    mock_sftp.listdir.return_value = ["config.json", "notes.txt"]
+    mock_ssh().open_sftp.return_value = mock_sftp
+
+    result = reader_password.get_files(file_pattern="*.csv")
+    assert result == []
+
+
+# --- get_csv_file ---
+
+
+def test_get_csv_file_raises_when_not_connected(reader_password: SFTPReader) -> None:
+    with pytest.raises(RuntimeError, match="Not connected"):
+        reader_password.get_csv_file("file.csv")
+
+
+def test_get_csv_file_returns_dataframe(connected_reader: SFTPReader) -> None:
+    csv_content = b"id,name\n1,Alice\n2,Bob"
+    mock_file = MagicMock()
+    mock_file.read.return_value = csv_content
+    mock_file.__enter__ = lambda s: s
+    mock_file.__exit__ = MagicMock(return_value=False)
+    connected_reader._sftp.file.return_value = mock_file
+
+    result = connected_reader.get_csv_file("data.csv", remote_path="/data")
+    expected = pl.DataFrame({"id": [1, 2], "name": ["Alice", "Bob"]})
+    assert_frame_equal(result, expected)
+
+
+def test_get_csv_file_with_column_names(connected_reader: SFTPReader) -> None:
+    csv_content = b"1,Alice\n2,Bob"
+    mock_file = MagicMock()
+    mock_file.read.return_value = csv_content
+    mock_file.__enter__ = lambda s: s
+    mock_file.__exit__ = MagicMock(return_value=False)
+    connected_reader._sftp.file.return_value = mock_file
+
+    result = connected_reader.get_csv_file("data.csv", column_names=["id", "name"])
+    assert result.columns == ["id", "name"]
+
+
+def test_get_csv_file_returns_none_on_error(connected_reader: SFTPReader) -> None:
+    connected_reader._sftp.file.side_effect = Exception("boom")
+    result = connected_reader.get_csv_file("bad.csv")
+    assert result is None
+
+
+# --- get_xml_file_to_string ---
+
+
+def test_get_xml_raises_when_not_connected(reader_password: SFTPReader) -> None:
+    with pytest.raises(RuntimeError, match="Not connected"):
+        reader_password.get_xml_file_to_string("file.xml")
+
+
+def test_get_xml_returns_string(connected_reader: SFTPReader) -> None:
+    xml_content = b"<root><item>1</item></root>"
+    mock_file = MagicMock()
+    mock_file.read.return_value = xml_content
+    mock_file.__enter__ = lambda s: s
+    mock_file.__exit__ = MagicMock(return_value=False)
+    connected_reader._sftp.file.return_value = mock_file
+
+    result = connected_reader.get_xml_file_to_string("feed.xml", remote_path="/exports")
+    assert result == "<root><item>1</item></root>"
+
+
+def test_get_xml_returns_none_on_error(connected_reader: SFTPReader) -> None:
+    connected_reader._sftp.file.side_effect = Exception("boom")
+    result = connected_reader.get_xml_file_to_string("bad.xml")
+    assert result is None
+
+
+# --- file_to_string ---
+
+
+def test_file_to_string_raises_when_not_connected(reader_password: SFTPReader) -> None:
+    with pytest.raises(RuntimeError, match="Not connected"):
+        reader_password.file_to_string("file.txt")
+
+
+def test_file_to_string_with_explicit_encoding(connected_reader: SFTPReader) -> None:
+    mock_file = MagicMock()
+    mock_file.read.return_value = "héllo".encode("latin-1")
+    mock_file.__enter__ = lambda s: s
+    mock_file.__exit__ = MagicMock(return_value=False)
+    connected_reader._sftp.file.return_value = mock_file
+
+    result = connected_reader.file_to_string("report.txt", encoding="latin-1")
+    assert result == "héllo"
+
+
+def test_file_to_string_auto_detects_encoding(connected_reader: SFTPReader) -> None:
+    mock_file = MagicMock()
+    mock_file.read.return_value = "hello world".encode("utf-8")
+    mock_file.__enter__ = lambda s: s
+    mock_file.__exit__ = MagicMock(return_value=False)
+    connected_reader._sftp.file.return_value = mock_file
+
+    result = connected_reader.file_to_string("report.txt")
+    assert result == "hello world"
+
+
+def test_file_to_string_returns_none_on_error(connected_reader: SFTPReader) -> None:
+    connected_reader._sftp.file.side_effect = Exception("boom")
+    result = connected_reader.file_to_string("bad.txt")
+    assert result is None