tests: pouch: downlink: test with very long device name

Use very long device name, so that pouch header does not fit into single
transport MTU (19).

Author: mniestroj

tests/pouch/downlink/CMakeLists.txt

@@ -18,6 +18,7 @@ function(pouch_gen_lorem length)
     ${PYTHON_EXECUTABLE}
     ${APPLICATION_SOURCE_DIR}/scripts/pouch-gen-lorem.py
     --length ${length}
+    --device-name ${CONFIG_POUCH_DEVICE_NAME}
     ${gen_dir}/lorem-${length}.bin
     DEPENDS ${APPLICATION_SOURCE_DIR}/scripts/pouch-gen-lorem.py
     WORKING_DIRECTORY ${APPLICATION_SOURCE_DIR}/

tests/pouch/downlink/Kconfig

@@ -0,0 +1,9 @@
+menu "Downlink test configuration"
+
+config POUCH_DEVICE_NAME
+	string "Device name used in encoded pouches"
+	default "id123"
+
+endmenu
+
+source "Kconfig.zephyr"

tests/pouch/downlink/scripts/pouch-gen-lorem.py

@@ -78,9 +78,10 @@ def payload_append(payload: bytearray, data: bytes, stream_id: int, flags: int):
 def main(
         out: Annotated[typer.FileBinaryWrite, typer.Argument()],
         length: Annotated[int, typer.Option(min=0)] = len(LOREM_IPSUM),
+        device_name: Annotated[str, typer.Option()] = "id123",
 ):
     obj = {
-        "device_id": "id123",
+        "device_id": device_name,
         "entries": [
             {
                 "path": "/.s/lorem",

tests/pouch/downlink/src/downlink.c

@@ -12,13 +12,19 @@ LOG_MODULE_REGISTER(downlink_test);
 
 #define BLOCK_SIZE 512
 
-#define LEN_POUCH_HEADER 5
+#define LEN_DEVICE_NAME (sizeof(CONFIG_POUCH_DEVICE_NAME) - 1)
+
+/*
+ * CBOR "device name length" is encoded with more bytes if device name is more
+ * than 23 bytes
+ */
+#define LEN_POUCH_HEADER 5 + (LEN_DEVICE_NAME > 23 ? 1 : 0)
 #define LEN_BLOCK_HEADER 3
 #define LEN_ENTRY_LENGTH 2
 #define LEN_ENTRY_PATH 2
 #define LEN_ENTRY_CONTENT_TYPE 1
 
-#define FIRST_BLOCK_OFFSET (LEN_POUCH_HEADER + sizeof("id123") - 1)
+#define FIRST_BLOCK_OFFSET (LEN_POUCH_HEADER + LEN_DEVICE_NAME)
 
 #define TRANSPORT_MTU 19
 

tests/pouch/downlink/testcase.yaml

@@ -1,9 +1,15 @@
+common:
+  platform_allow:
+    - native_sim
+    - native_sim/native/64
+  integration_platforms:
+    - native_sim
+    - native_sim/native/64
+  tags: test_framework
 tests:
-  pouch.downlink:
-    platform_allow:
-      - native_sim
-      - native_sim/native/64
-    integration_platforms:
-      - native_sim
-      - native_sim/native/64
-    tags: test_framework
+  pouch.downlink.id123:
+    extra_configs:
+      - CONFIG_POUCH_DEVICE_NAME="id123"
+  pouch.downlink.very-long-device-name-with-id123:
+    extra_configs:
+      - CONFIG_POUCH_DEVICE_NAME="very-long-device-name-with-id123"