Skip to content

feat: add /logs endpoint verification to smoke tests #51

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
bburda merged 4 commits into from
Mar 22, 2026
+33 −21
Merged

feat: add /logs endpoint verification to smoke tests #51

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
f585f43
feat: add /logs endpoint verification to all smoke tests
bburda Mar 22, 2026
7581ef0
fix: use per-entity /logs endpoints, drop non-existent global /logs
bburda Mar 22, 2026
8c84c0f
refactor: extract assert_non_empty_items helper to smoke_lib.sh
bburda Mar 22, 2026
f85ed01
fix: use medkit-gateway for logs check in sensor demo
bburda Mar 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions tests/smoke_test.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,28 @@ else
fail "GET /apps/lidar-sim/configurations returns 200" "unexpected status code"
fi

section "Logs"

if api_get "/logs"; then
if echo "$RESPONSE" | jq -e '.items | length > 0' > /dev/null 2>&1; then
pass "GET /logs returns non-empty items"
else
fail "GET /logs returns non-empty items" "items is empty"
fi
else
fail "GET /logs returns 200" "unexpected status code"
fi

if api_get "/apps/lidar-sim/logs"; then
if echo "$RESPONSE" | jq -e '.items | length > 0' > /dev/null 2>&1; then
pass "GET /apps/lidar-sim/logs returns non-empty items"
else
fail "GET /apps/lidar-sim/logs returns non-empty items" "items is empty"
fi
else
fail "GET /apps/lidar-sim/logs returns 200" "unexpected status code"
fi
Comment thread
bburda marked this conversation as resolved.
Outdated

section "Fault Injection"

# Inject noise fault via configuration API
Expand Down
22 changes: 22 additions & 0 deletions tests/smoke_test_moveit.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,28 @@ test_entity_discovery "areas" manipulation planning diagnostics bridge
test_entity_discovery "components" panda-arm panda-gripper moveit-planning pick-place-loop gateway fault-manager diagnostic-bridge
test_entity_discovery "apps" joint-state-broadcaster panda-arm-controller panda-hand-controller robot-state-publisher move-group pick-place-node medkit-gateway medkit-fault-manager diagnostic-bridge-app manipulation-monitor

section "Logs"

if api_get "/logs"; then
if echo "$RESPONSE" | jq -e '.items | length > 0' > /dev/null 2>&1; then
pass "GET /logs returns non-empty items"
else
fail "GET /logs returns non-empty items" "items is empty"
fi
else
fail "GET /logs returns 200" "unexpected status code"
fi

if api_get "/apps/medkit-gateway/logs"; then
if echo "$RESPONSE" | jq -e '.items | length > 0' > /dev/null 2>&1; then
pass "GET /apps/medkit-gateway/logs returns non-empty items"
else
fail "GET /apps/medkit-gateway/logs returns non-empty items" "items is empty"
fi
else
fail "GET /apps/medkit-gateway/logs returns 200" "unexpected status code"
fi

Comment thread
bburda marked this conversation as resolved.
# --- Summary ---

print_summary
22 changes: 22 additions & 0 deletions tests/smoke_test_turtlebot3.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,28 @@ test_entity_discovery "areas" robot navigation diagnostics bridge
test_entity_discovery "components" turtlebot3-base lidar-sensor nav2-stack gateway fault-manager diagnostic-bridge-unit
test_entity_discovery "apps" turtlebot3-node robot-state-publisher amcl bt-navigator controller-server planner-server velocity-smoother medkit-gateway medkit-fault-manager diagnostic-bridge anomaly-detector

section "Logs"

if api_get "/logs"; then
if echo "$RESPONSE" | jq -e '.items | length > 0' > /dev/null 2>&1; then
pass "GET /logs returns non-empty items"
else
fail "GET /logs returns non-empty items" "items is empty"
fi
else
fail "GET /logs returns 200" "unexpected status code"
fi

if api_get "/apps/medkit-gateway/logs"; then
if echo "$RESPONSE" | jq -e '.items | length > 0' > /dev/null 2>&1; then
pass "GET /apps/medkit-gateway/logs returns non-empty items"
else
fail "GET /apps/medkit-gateway/logs returns non-empty items" "items is empty"
fi
else
fail "GET /apps/medkit-gateway/logs returns 200" "unexpected status code"
fi

Comment thread
bburda marked this conversation as resolved.
Outdated
# --- Summary ---

print_summary
Loading