pykoplenti 1.3.0__tar.gz → 1.4.0__tar.gz

{pykoplenti-1.3.0 → pykoplenti-1.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.4
 Name: pykoplenti
-Version: 1.3.0
+Version: 1.4.0
 Summary: Python REST-Client for Kostal Plenticore Solar Inverters
 Home-page: https://github.com/stegm/pyclient_koplenti
 Author: @stegm
@@ -25,7 +25,8 @@ Requires-Dist: pycryptodome~=3.19
 Requires-Dist: pydantic>=1.10
 Provides-Extra: cli
 Requires-Dist: prompt_toolkit>=3.0; extra == "cli"
-Requires-Dist: click>=7.1; extra == "cli"
+Requires-Dist: click>=8.0; extra == "cli"
+Dynamic: license-file
 
 # Python Library for Accessing Kostal Plenticore Inverters
 
@@ -77,22 +78,28 @@ Installing the libray with `CLI` provides a new command.
 
 ```shell
 $ pykoplenti --help
-Usage: pykoplenti [OPTIONS] COMMAND [ARGS]...
+Usage: python -m pykoplenti.cli [OPTIONS] COMMAND [ARGS]...
 
   Handling of global arguments with click
 
 Options:
-  --host TEXT           hostname or ip of the inverter
-  --port INTEGER        port of the inverter (default 80)
-  --password TEXT       the password
-  --password-file TEXT  password file (default "secrets" in the current
-                        working directory)
-
+  --host TEXT           Hostname or IP of the inverter
+  --port INTEGER        Port of the inverter  [default: 80]
+  --password TEXT       Password or master key (also device id)
+  --service-code TEXT   service code for installer access
+  --password-file FILE  Path to password file - deprecated, use --credentials
+                        [default: secrets]
+  --credentials FILE    Path to the credentials file. This has a simple ini-
+                        format without sections. For user access, use the
+                        'password'. For installer access, use the 'master-key'
+                        and 'service-key'.
   --help                Show this message and exit.
 
 Commands:
   all-processdata   Returns a list of all available process data.
   all-settings      Returns the ids of all settings.
+  download-log      Download the log data from the inverter to a file.
+  read-events       Returns the last events
   read-processdata  Returns the values of the given process data.
   read-settings     Read the value of the given settings.
   repl              Provides a simple REPL for executing API requests to...

{pykoplenti-1.3.0 → pykoplenti-1.4.0}/README.md

@@ -48,22 +48,28 @@ Installing the libray with `CLI` provides a new command.
 
 ```shell
 $ pykoplenti --help
-Usage: pykoplenti [OPTIONS] COMMAND [ARGS]...
+Usage: python -m pykoplenti.cli [OPTIONS] COMMAND [ARGS]...
 
   Handling of global arguments with click
 
 Options:
-  --host TEXT           hostname or ip of the inverter
-  --port INTEGER        port of the inverter (default 80)
-  --password TEXT       the password
-  --password-file TEXT  password file (default "secrets" in the current
-                        working directory)
-
+  --host TEXT           Hostname or IP of the inverter
+  --port INTEGER        Port of the inverter  [default: 80]
+  --password TEXT       Password or master key (also device id)
+  --service-code TEXT   service code for installer access
+  --password-file FILE  Path to password file - deprecated, use --credentials
+                        [default: secrets]
+  --credentials FILE    Path to the credentials file. This has a simple ini-
+                        format without sections. For user access, use the
+                        'password'. For installer access, use the 'master-key'
+                        and 'service-key'.
   --help                Show this message and exit.
 
 Commands:
   all-processdata   Returns a list of all available process data.
   all-settings      Returns the ids of all settings.
+  download-log      Download the log data from the inverter to a file.
+  read-events       Returns the last events
   read-processdata  Returns the values of the given process data.
   read-settings     Read the value of the given settings.
   repl              Provides a simple REPL for executing API requests to...

{pykoplenti-1.3.0 → pykoplenti-1.4.0}/pykoplenti/cli.py

@@ -1,13 +1,16 @@
 from ast import literal_eval
 import asyncio
 from collections import defaultdict
+from dataclasses import dataclass
 from inspect import iscoroutinefunction
 import os
+from pathlib import Path
 from pprint import pprint
 import re
 import tempfile
 import traceback
-from typing import Any, Awaitable, Callable, Dict, Union
+from typing import Any, Awaitable, Callable, Dict, Optional, Union
+import warnings
 
 from aiohttp import ClientSession, ClientTimeout
 import click
@@ -20,40 +23,43 @@ from pykoplenti.extended import ExtendedApiClient
 class SessionCache:
     """Persistent the session in a temporary file."""
 
-    def __init__(self, host):
-        self.host = host
+    def __init__(self, host: str, user: str):
+        self._cache_file = Path(
+            tempfile.gettempdir(), f"pykoplenti-session-{host}-{user}"
+        )
 
     def read_session_id(self) -> Union[str, None]:
-        file = os.path.join(tempfile.gettempdir(), f"pykoplenti-session-{self.host}")
-        if os.path.isfile(file):
-            with open(file, "rt") as f:
+        if self._cache_file.is_file():
+            with self._cache_file.open("rt") as f:
                 return f.readline(256)
         else:
             return None
 
     def write_session_id(self, id: str):
-        file = os.path.join(tempfile.gettempdir(), f"pykoplenti-session-{self.host}")
-        f = os.open(file, os.O_WRONLY | os.O_TRUNC | os.O_CREAT, mode=0o600)
+        f = os.open(self._cache_file, os.O_WRONLY | os.O_TRUNC | os.O_CREAT, mode=0o600)
         try:
             os.write(f, id.encode("ascii"))
         finally:
             os.close(f)
 
+    def remove(self):
+        self._cache_file.unlink(missing_ok=True)
+
 
 class ApiShell:
     """Provides a shell-like access to the inverter."""
 
-    def __init__(self, client: ApiClient):
+    def __init__(self, client: ApiClient, user: str):
         super().__init__()
         self.client = client
-        self._session_cache = SessionCache(self.client.host)
+        self._session_cache = SessionCache(self.client.host, user)
 
-    async def prepare_client(self, passwd):
+    async def prepare_client(self, key: Optional[str], service_code: Optional[str]):
         # first try to reuse existing session
         session_id = self._session_cache.read_session_id()
         if session_id is not None:
             self.client.session_id = session_id
-            print_formatted_text("Trying to reuse existing session... ", end=None)
+            print_formatted_text("Trying to reuse existing session... ", end="")
             me = await self.client.get_me()
             if me.is_authenticated:
                 print_formatted_text("Success")
@@ -61,18 +67,21 @@ class ApiShell:
 
             print_formatted_text("Failed")
 
-        if passwd is not None:
-            print_formatted_text("Logging in... ", end=None)
-            await self.client.login(passwd)
-            self._session_cache.write_session_id(self.client.session_id)
+        if key is not None:
+            print_formatted_text("Logging in... ", end="")
+            await self.client.login(key=key, service_code=service_code)
+            if self.client.session_id is not None:
+                self._session_cache.write_session_id(self.client.session_id)
             print_formatted_text("Success")
+        else:
+            print_formatted_text("Session could not be reused and no key given")
 
     def print_exception(self):
         """Prints an excpetion from executing a method."""
         print_formatted_text(traceback.format_exc())
 
-    async def run(self, passwd):
-        session = PromptSession()
+    async def run(self, key: Optional[str], service_code: Optional[str]):
+        session = PromptSession[str]()
         print_formatted_text(flush=True)  # Initialize output
 
         # Test commands:
@@ -83,7 +92,7 @@ class ApiShell:
         # get_setting_values 'scb:time'
         # set_setting_values 'devices:local' {'Battery:MinSoc':'15'}
 
-        await self.prepare_client(passwd)
+        await self.prepare_client(key, service_code)
 
         while True:
             try:
@@ -153,82 +162,171 @@ class ApiShell:
             self.print_exception()
 
 
-async def repl_main(host, port, passwd):
+async def repl_main(
+    host: str, port: int, key: Optional[str], service_code: Optional[str]
+):
     async with ClientSession(timeout=ClientTimeout(total=10)) as session:
         client = ExtendedApiClient(session, host=host, port=port)
 
-        shell = ApiShell(client)
-        await shell.run(passwd)
+        shell = ApiShell(client, "user" if service_code is None else "master")
+        await shell.run(key, service_code)
 
 
 async def command_main(
-    host: str, port: int, passwd: str, fn: Callable[[ApiClient], Awaitable[Any]]
+    host: str,
+    port: int,
+    key: Optional[str],
+    service_code: Optional[str],
+    fn: Callable[[ApiClient], Awaitable[Any]],
 ):
     async with ClientSession(timeout=ClientTimeout(total=10)) as session:
         client = ExtendedApiClient(session, host=host, port=port)
-        session_cache = SessionCache(host)
+        session_cache = SessionCache(host, "user" if service_code is None else "master")
 
         # Try to reuse an existing session
         client.session_id = session_cache.read_session_id()
         me = await client.get_me()
         if not me.is_authenticated:
+            if key is None:
+                raise ValueError("Could not reuse session and no login key is given.")
+
             # create a new session
-            await client.login(passwd)
+            await client.login(key=key, service_code=service_code)
+
             if client.session_id is not None:
                 session_cache.write_session_id(client.session_id)
 
         await fn(client)
 
 
+@dataclass
 class GlobalArgs:
     """Global arguments over all sub commands."""
 
-    def __init__(self):
-        self.host = None
-        self.port = None
-        self.password = None
-        self.password_file = None
+    host: str = ""
+    """The hostname or ip of the inverter."""
+
+    port: int = 0
+    """The port on which the API listens on the inverter."""
+
+    key: Optional[str] = None
+    """The key (password or master key) to login into the API.
+
+    If None, a previous session cache is used. If the session
+    cache has no valid session, no login is executed.
+    """
+
+    service_code: Optional[str] = None
+    """The service code for master access.
+
+    Only necessary for master access. If missing, user acess is used.
+    """
 
 
 pass_global_args = click.make_pass_decorator(GlobalArgs, ensure=True)
 
 
+def _parse_credentials_file(path: Path) -> tuple[Optional[str], Optional[str]]:
+    """Parse credentials file returning (key, service_code)"""
+    key = service_code = None
+    for line in path.read_text().splitlines():
+        if "=" not in line:
+            return line.strip(), None
+
+        name, _, value = line.partition("=")
+        name = name.strip()
+        if name in ("password", "key", "master-key"):
+            key = value.strip()
+        elif name == "service-code":
+            service_code = value.strip()
+    return key, service_code
+
+
 @click.group()
-@click.option("--host", help="hostname or ip of the inverter")
-@click.option("--port", default=80, help="port of the inverter (default 80)")
-@click.option("--password", default=None, help="the password")
+@click.option("--host", help="Hostname or IP of the inverter")
+@click.option("--port", default=80, help="Port of the inverter", show_default=True)
+@click.option(
+    "--password", default=None, help="Password or master key (also device id)"
+)
+@click.option("--service-code", default=None, help="service code for installer access")
 @click.option(
     "--password-file",
     default="secrets",
-    help='password file (default "secrets" in the current working directory)',
+    help="Path to password file - deprecated, use --credentials",
+    show_default=True,
+    type=click.Path(exists=False, dir_okay=False, readable=True, path_type=Path),
+)
+@click.option(
+    "--credentials",
+    default=None,
+    help="Path to the credentials file. This has a simple ini-format without sections. "
+    "For user access, use the 'password'. For installer access, use the 'master-key' "
+    "and 'service-key'.",
+    type=click.Path(exists=True, dir_okay=False, readable=True, path_type=Path),
 )
 @pass_global_args
-def cli(global_args, host, port, password, password_file):
+def cli(
+    global_args: GlobalArgs,
+    host: str,
+    port: int,
+    password: Optional[str],
+    service_code: Optional[str],
+    password_file: Path,
+    credentials: Path,
+):
     """Handling of global arguments with click"""
-    if password is not None:
-        global_args.passwd = password
-    elif os.path.isfile(password_file):
-        with open(password_file, "rt") as f:
-            global_args.passwd = f.readline()
-    else:
-        global_args.passwd = None
-
     global_args.host = host
     global_args.port = port
 
+    if password is not None:
+        global_args.key = password
+    elif password_file.is_file():
+        with password_file.open("rt") as f:
+            global_args.key = f.readline()
+        warnings.warn(
+            "--password-file is deprecated. Use --credentials instead.",
+            DeprecationWarning,
+        )
+
+    if service_code is not None:
+        global_args.service_code = service_code
+
+    if credentials is not None:
+        if password is not None:
+            raise click.BadOptionUsage(
+                "password", "password cannot be used with credentials"
+            )
+        if password_file is not None and password_file.is_file():
+            raise click.BadOptionUsage(
+                "password-file", "password-file cannot be used with credentials"
+            )
+        if service_code is not None:
+            raise click.BadOptionUsage(
+                "service_code", "service_code cannot be used with credentials"
+            )
+
+        global_args.key, global_args.service_code = _parse_credentials_file(credentials)
+
 
 @cli.command()
 @pass_global_args
-def repl(global_args):
+def repl(global_args: GlobalArgs):
     """Provides a simple REPL for executing API requests to the inverter."""
-    asyncio.run(repl_main(global_args.host, global_args.port, global_args.passwd))
+    asyncio.run(
+        repl_main(
+            global_args.host,
+            global_args.port,
+            global_args.key,
+            global_args.service_code,
+        )
+    )
 
 
 @cli.command()
 @click.option("--lang", default=None, help="language for events")
 @click.option("--count", default=10, help="number of events to read")
 @pass_global_args
-def read_events(global_args, lang, count):
+def read_events(global_args: GlobalArgs, lang, count):
     """Returns the last events"""
 
     async def fn(client: ApiClient):
@@ -240,7 +338,13 @@ def read_events(global_args, lang, count):
             )
 
     asyncio.run(
-        command_main(global_args.host, global_args.port, global_args.passwd, fn)
+        command_main(
+            global_args.host,
+            global_args.port,
+            global_args.key,
+            global_args.service_code,
+            fn,
+        )
     )
 
 
@@ -254,20 +358,26 @@ def read_events(global_args, lang, count):
 @click.option("--begin", type=click.DateTime(["%Y-%m-%d"]), help="first day to export")
 @click.option("--end", type=click.DateTime(["%Y-%m-%d"]), help="last day to export")
 @pass_global_args
-def download_log(global_args, out, begin, end):
+def download_log(global_args: GlobalArgs, out, begin, end):
     """Download the log data from the inverter to a file."""
 
     async def fn(client: ApiClient):
         await client.download_logdata(writer=out, begin=begin, end=end)
 
     asyncio.run(
-        command_main(global_args.host, global_args.port, global_args.passwd, fn)
+        command_main(
+            global_args.host,
+            global_args.port,
+            global_args.key,
+            global_args.service_code,
+            fn,
+        )
     )
 
 
 @cli.command()
 @pass_global_args
-def all_processdata(global_args):
+def all_processdata(global_args: GlobalArgs):
     """Returns a list of all available process data."""
 
     async def fn(client: ApiClient):
@@ -277,14 +387,20 @@ def all_processdata(global_args):
                 print(f"{k}/{x}")
 
     asyncio.run(
-        command_main(global_args.host, global_args.port, global_args.passwd, fn)
+        command_main(
+            global_args.host,
+            global_args.port,
+            global_args.key,
+            global_args.service_code,
+            fn,
+        )
     )
 
 
 @cli.command()
 @click.argument("ids", required=True, nargs=-1)
 @pass_global_args
-def read_processdata(global_args, ids):
+def read_processdata(global_args: GlobalArgs, ids):
     """Returns the values of the given process data.
 
     IDS is the identifier (<module_id>/<processdata_id>) of one or more processdata
@@ -318,7 +434,13 @@ def read_processdata(global_args, ids):
                 print(f"{k}/{x.id}={x.value}")
 
     asyncio.run(
-        command_main(global_args.host, global_args.port, global_args.passwd, fn)
+        command_main(
+            global_args.host,
+            global_args.port,
+            global_args.key,
+            global_args.service_code,
+            fn,
+        )
     )
 
 
@@ -327,7 +449,7 @@ def read_processdata(global_args, ids):
     "--rw", is_flag=True, default=False, help="display only writable settings"
 )
 @pass_global_args
-def all_settings(global_args, rw):
+def all_settings(global_args: GlobalArgs, rw: bool):
     """Returns the ids of all settings."""
 
     async def fn(client: ApiClient):
@@ -338,14 +460,20 @@ def all_settings(global_args, rw):
                     print(f"{k}/{x.id}")
 
     asyncio.run(
-        command_main(global_args.host, global_args.port, global_args.passwd, fn)
+        command_main(
+            global_args.host,
+            global_args.port,
+            global_args.key,
+            global_args.service_code,
+            fn,
+        )
     )
 
 
 @cli.command()
 @click.argument("ids", required=True, nargs=-1)
 @pass_global_args
-def read_settings(global_args, ids):
+def read_settings(global_args: GlobalArgs, ids):
     """Read the value of the given settings.
 
     IDS is the identifier (<module_id>/<setting_id>) of one or more settings to read
@@ -376,14 +504,20 @@ def read_settings(global_args, ids):
                 print(f"{k}/{i}={v}")
 
     asyncio.run(
-        command_main(global_args.host, global_args.port, global_args.passwd, fn)
+        command_main(
+            global_args.host,
+            global_args.port,
+            global_args.key,
+            global_args.service_code,
+            fn,
+        )
     )
 
 
 @cli.command()
 @click.argument("id_values", required=True, nargs=-1)
 @pass_global_args
-def write_settings(global_args, id_values):
+def write_settings(global_args: GlobalArgs, id_values):
     """Write the values of the given settings.
 
     ID_VALUES is the identifier plus the the value to write
@@ -412,7 +546,13 @@ def write_settings(global_args, id_values):
             await client.set_setting_values(module_id, setting_values)
 
     asyncio.run(
-        command_main(global_args.host, global_args.port, global_args.passwd, fn)
+        command_main(
+            global_args.host,
+            global_args.port,
+            global_args.key,
+            global_args.service_code,
+            fn,
+        )
     )
 
 

{pykoplenti-1.3.0 → pykoplenti-1.4.0}/pykoplenti.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.4
 Name: pykoplenti
-Version: 1.3.0
+Version: 1.4.0
 Summary: Python REST-Client for Kostal Plenticore Solar Inverters
 Home-page: https://github.com/stegm/pyclient_koplenti
 Author: @stegm
@@ -25,7 +25,8 @@ Requires-Dist: pycryptodome~=3.19
 Requires-Dist: pydantic>=1.10
 Provides-Extra: cli
 Requires-Dist: prompt_toolkit>=3.0; extra == "cli"
-Requires-Dist: click>=7.1; extra == "cli"
+Requires-Dist: click>=8.0; extra == "cli"
+Dynamic: license-file
 
 # Python Library for Accessing Kostal Plenticore Inverters
 
@@ -77,22 +78,28 @@ Installing the libray with `CLI` provides a new command.
 
 ```shell
 $ pykoplenti --help
-Usage: pykoplenti [OPTIONS] COMMAND [ARGS]...
+Usage: python -m pykoplenti.cli [OPTIONS] COMMAND [ARGS]...
 
   Handling of global arguments with click
 
 Options:
-  --host TEXT           hostname or ip of the inverter
-  --port INTEGER        port of the inverter (default 80)
-  --password TEXT       the password
-  --password-file TEXT  password file (default "secrets" in the current
-                        working directory)
-
+  --host TEXT           Hostname or IP of the inverter
+  --port INTEGER        Port of the inverter  [default: 80]
+  --password TEXT       Password or master key (also device id)
+  --service-code TEXT   service code for installer access
+  --password-file FILE  Path to password file - deprecated, use --credentials
+                        [default: secrets]
+  --credentials FILE    Path to the credentials file. This has a simple ini-
+                        format without sections. For user access, use the
+                        'password'. For installer access, use the 'master-key'
+                        and 'service-key'.
   --help                Show this message and exit.
 
 Commands:
   all-processdata   Returns a list of all available process data.
   all-settings      Returns the ids of all settings.
+  download-log      Download the log data from the inverter to a file.
+  read-events       Returns the last events
   read-processdata  Returns the values of the given process data.
   read-settings     Read the value of the given settings.
   repl              Provides a simple REPL for executing API requests to...

{pykoplenti-1.3.0 → pykoplenti-1.4.0}/pykoplenti.egg-info/SOURCES.txt

@@ -15,6 +15,7 @@ pykoplenti.egg-info/dependency_links.txt
 pykoplenti.egg-info/entry_points.txt
 pykoplenti.egg-info/requires.txt
 pykoplenti.egg-info/top_level.txt
+tests/test_cli.py
 tests/test_extendedapiclient.py
 tests/test_pykoplenti.py
 tests/test_smoketest.py

{pykoplenti-1.3.0 → pykoplenti-1.4.0}/pykoplenti.egg-info/requires.txt

@@ -4,4 +4,4 @@ pydantic>=1.10
 
 [CLI]
 prompt_toolkit>=3.0
-click>=7.1
+click>=8.0

{pykoplenti-1.3.0 → pykoplenti-1.4.0}/setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = pykoplenti
-version = 1.3.0
+version = 1.4.0
 description = Python REST-Client for Kostal Plenticore Solar Inverters
 long_description = file: README.md
 long_description_content_type = text/markdown
@@ -36,7 +36,7 @@ pykoplenti = py.typed
 [options.extras_require]
 CLI = 
 	prompt_toolkit >= 3.0
-	click >= 7.1
+	click >= 8.0
 
 [options.entry_points]
 console_scripts = 

pykoplenti-1.4.0/tests/test_cli.py

@@ -0,0 +1,151 @@
+from pathlib import Path
+from click.testing import CliRunner
+import pytest
+from pykoplenti.cli import cli, SessionCache
+import os
+from conftest import only_smoketest
+
+
+@pytest.fixture
+def credentials(tmp_path: Path, smoketest_config: tuple[str, int, str]):
+    _, _, password = smoketest_config
+    credentials_path = tmp_path / "credentials"
+    credentials_path.write_text(f"password={password}")
+    return credentials_path
+
+
+@pytest.fixture
+def dummy_credentials(tmp_path: Path):
+    credentials_path = tmp_path / "credentials"
+    credentials_path.write_text("password=dummy")
+    return credentials_path
+
+
+@pytest.fixture
+def session_cache(smoketest_config: tuple[str, int, str]):
+    host, _, _ = smoketest_config
+    session_cache = SessionCache(host, "user")
+    session_cache.remove()
+    yield session_cache
+    session_cache.remove()
+
+
+class TestInvalidGlobalOptions:
+    """Test invalid global options."""
+
+    def test_crendentials_and_password(self, dummy_credentials: Path):
+        runner = CliRunner()
+        result = runner.invoke(
+            cli,
+            [
+                "--credentials",
+                str(dummy_credentials),
+                "--password",
+                "topsecret",
+                "all-processdata",
+            ],
+        )
+        assert result.exit_code == 2
+        assert "password cannot be used with credentials" in result.output
+
+    @pytest.mark.filterwarnings(
+        "ignore:--password-file is deprecated. Use --credentials instead."
+    )
+    def test_crendentials_and_password_file(self, dummy_credentials: Path):
+        runner = CliRunner()
+        result = runner.invoke(
+            cli,
+            [
+                "--credentials",
+                str(dummy_credentials),
+                "--password-file",
+                str(dummy_credentials),
+                "all-processdata",
+            ],
+        )
+
+        assert result.exit_code == 2
+        assert "password-file cannot be used with credentials" in result.output
+
+    def test_crendentials_and_service_code(
+        self, dummy_credentials: Path, tmp_path: Path
+    ):
+        # As --password-file has a default value, this ensures
+        # that no default password-file exists.
+        os.chdir(tmp_path)
+
+        runner = CliRunner()
+        result = runner.invoke(
+            cli,
+            [
+                "--credentials",
+                str(dummy_credentials),
+                "--service-code",
+                "topsecret",
+                "all-processdata",
+            ],
+        )
+        assert result.exit_code == 2
+        assert "service_code cannot be used with credentials" in result.output
+
+
+@only_smoketest
+def test_read_process_data(
+    credentials: Path,
+    session_cache: SessionCache,
+    smoketest_config: tuple[str, int, str],
+):
+    # As --password-file has a default value, this ensures
+    # that no default password-file exists.
+    os.chdir(credentials.parent)
+
+    host, port, _ = smoketest_config
+
+    runner = CliRunner()
+    result = runner.invoke(
+        cli,
+        [
+            "--host",
+            host,
+            "--port",
+            str(port),
+            "--credentials",
+            str(credentials),
+            "all-processdata",
+        ],
+    )
+    assert result.exit_code == 0
+    # check any data which is most likely present on most inverter
+    assert "devices:local/Inverter:State" in result.stdout.splitlines()
+    assert session_cache.read_session_id() is not None
+
+
+@only_smoketest
+def test_read_settings_data(
+    credentials: Path,
+    session_cache: SessionCache,
+    smoketest_config: tuple[str, int, str],
+):
+    # As --password-file has a default value, this ensures
+    # that no default password-file exists.
+    os.chdir(credentials.parent)
+
+    host, port, _ = smoketest_config
+
+    runner = CliRunner()
+    result = runner.invoke(
+        cli,
+        [
+            "--host",
+            host,
+            "--port",
+            str(port),
+            "--credentials",
+            str(credentials),
+            "all-settings",
+        ],
+    )
+    assert result.exit_code == 0
+    # check any data which is most likely present on most inverter
+    assert "devices:local/Branding:ProductName1" in result.stdout.splitlines()
+    assert session_cache.read_session_id() is not None