From 1108534540a2a60ee0f7d6bc19b4648c2e9dd9fc Mon Sep 17 00:00:00 2001 From: "Jokob @NetAlertX" <96159884+jokob-sk@users.noreply.github.com> Date: Sun, 24 May 2026 01:30:33 +0000 Subject: [PATCH] Fix formatting inconsistencies in comments for sync endpoint tests --- .github/skills/code-standards/SKILL.md | 2 +- test/api_endpoints/test_sync_endpoint.py | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/skills/code-standards/SKILL.md b/.github/skills/code-standards/SKILL.md index af1092d3..6d522d39 100644 --- a/.github/skills/code-standards/SKILL.md +++ b/.github/skills/code-standards/SKILL.md @@ -13,7 +13,7 @@ description: NetAlertX coding standards and conventions. Use this when writing c - follow DRY principle - maintainability of code is more important than speed of implementation - code files should be less than 500 LOC for better maintainability - DB columns must not contain underscores, use camelCase instead (e.g., deviceInstanceId, not device_instance_id) -- treat DB as temporary storage for stats, long term configuration should be stored in the `/config` folder, the `/config` folder should allow you to restore most of your functionality (excluding historical data) +- treat DB as temporary storage for stats, long-term configuration should be stored in the `/config` folder, the `/config` folder should allow you to restore most of your functionality (excluding historical data) - never access DB directly from application layers, always use helper functions in `server/db/db_helper.py` and implement new functionality in handlers (e.g., `DeviceInstance` in `server/models/device_instance.py`) - always validate and normalize MAC addresses before writing to DB (use `normalize_mac` from `plugin_helper.py`) - all subprocess calls must set explicit timeouts diff --git a/test/api_endpoints/test_sync_endpoint.py b/test/api_endpoints/test_sync_endpoint.py index cab43e07..da88b591 100644 --- a/test/api_endpoints/test_sync_endpoint.py +++ b/test/api_endpoints/test_sync_endpoint.py @@ -40,7 +40,7 @@ def auth_headers(token): # ======================================================================== -# POST /sync – authentication +# POST /sync - authentication # ======================================================================== def test_sync_post_no_token_is_forbidden(client): @@ -54,7 +54,7 @@ def test_sync_post_invalid_token_is_forbidden(client): # ======================================================================== -# POST /sync – content-type enforcement +# POST /sync - content-type enforcement # Regression: node used to send data= (form-encoded); validation rejects it. # ======================================================================== @@ -111,7 +111,7 @@ def test_sync_post_json_body_writes_encoded_file(client, api_token, tmp_path): # ======================================================================== -# GET /sync – authentication +# GET /sync - authentication # ======================================================================== def test_sync_get_no_token_is_forbidden(client):