[PR #16] [MERGED] Add system utils CI workflow and integration tests #20

New issue

Closed

opened 2026-03-02 11:44:09 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 11:44:09 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jhd3197/ServerKit/pull/16
Author: @jhd3197
Created: 2/10/2026
Status: ✅ Merged
Merged: 2/10/2026
Merged by: @jhd3197

Base: main ← Head: dev

📝 Commits (7)

02797c8 Add system utils CI workflow and integration tests
7f80932 chore: bump version to 1.2.82 [skip ci]
d5f6e13 Move EXPECTED_MANAGER check into pytest test
926c2f3 CI: set noninteractive env and harden install step
deb29d3 chore: bump version to 1.2.83 [skip ci]
e1f732c Update CI images and package install steps
34ca8de chore: bump version to 1.2.84 [skip ci]

📊 Changes

3 files changed (+316 additions, -1 deletions)

View changed files

➕ .github/workflows/test-system-utils.yml (+168 -0)
📝 VERSION (+1 -1)
➕ backend/tests/test_utils_system_integration.py (+147 -0)

📄 Description

Introduce a new GitHub Actions workflow (.github/workflows/test-system-utils.yml) that runs unit tests, distro-based integration tests in container images (ubuntu, debian, fedora, rocky) to validate package manager and systemctl detection, and an audit job that greps migrated service files for raw subprocess patterns. Add backend/tests/test_utils_system_integration.py: Linux-only integration smoke tests that directly import backend/app/utils/system.py (avoiding Flask deps) to verify PackageManager detection, command availability, and basic ServiceControl behavior. Designed for CI verification of system utilities across real distros.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/jhd3197/ServerKit/pull/16 **Author:** [@jhd3197](https://github.com/jhd3197) **Created:** 2/10/2026 **Status:** ✅ Merged **Merged:** 2/10/2026 **Merged by:** [@jhd3197](https://github.com/jhd3197) **Base:** `main` ← **Head:** `dev` --- ### 📝 Commits (7) - [`02797c8`](https://github.com/jhd3197/ServerKit/commit/02797c891f1923b1ccbed223f872141f6d743230) Add system utils CI workflow and integration tests - [`7f80932`](https://github.com/jhd3197/ServerKit/commit/7f80932c87f36e2ad6078dc38e423aa987657802) chore: bump version to 1.2.82 [skip ci] - [`d5f6e13`](https://github.com/jhd3197/ServerKit/commit/d5f6e13d045d957210b605cd68803a477470fc48) Move EXPECTED_MANAGER check into pytest test - [`926c2f3`](https://github.com/jhd3197/ServerKit/commit/926c2f3afdbf5a9ec1e0c7eea8520a2b06c7acba) CI: set noninteractive env and harden install step - [`deb29d3`](https://github.com/jhd3197/ServerKit/commit/deb29d3678ee4458c87b9a7bf6cb54ef3fa77451) chore: bump version to 1.2.83 [skip ci] - [`e1f732c`](https://github.com/jhd3197/ServerKit/commit/e1f732c7e970e0bfbba177f2d69b920eaf9274fe) Update CI images and package install steps - [`34ca8de`](https://github.com/jhd3197/ServerKit/commit/34ca8dee8fc5b1d03379144f1484be9227eaab35) chore: bump version to 1.2.84 [skip ci] ### 📊 Changes **3 files changed** (+316 additions, -1 deletions) <details> <summary>View changed files</summary> ➕ `.github/workflows/test-system-utils.yml` (+168 -0) 📝 `VERSION` (+1 -1) ➕ `backend/tests/test_utils_system_integration.py` (+147 -0) </details> ### 📄 Description Introduce a new GitHub Actions workflow (.github/workflows/test-system-utils.yml) that runs unit tests, distro-based integration tests in container images (ubuntu, debian, fedora, rocky) to validate package manager and systemctl detection, and an audit job that greps migrated service files for raw subprocess patterns. Add backend/tests/test_utils_system_integration.py: Linux-only integration smoke tests that directly import backend/app/utils/system.py (avoiding Flask deps) to verify PackageManager detection, command availability, and basic ServiceControl behavior. Designed for CI verification of system utilities across real distros. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>