Skip to content

chore: move pid/uptime_secs test coverage to e2e test#18

Merged
Nic-dorman merged 1 commit intoWithAutonomi:mainfrom
jacderida:chore-update_end_to_end_test
Mar 31, 2026
Merged

chore: move pid/uptime_secs test coverage to e2e test#18
Nic-dorman merged 1 commit intoWithAutonomi:mainfrom
jacderida:chore-update_end_to_end_test

Conversation

@jacderida
Copy link
Copy Markdown
Contributor

Summary

  • Removed the nodes_status_includes_pid_and_uptime integration test from daemon_integration.rs (added by mistake — this should be tested end-to-end, not as a Rust integration test)
  • Extended the e2e node management test command to verify pid and uptime_secs fields in node status responses (present for running nodes, omitted for stopped nodes)

Test plan

E2E test run on all three platforms against DEV-01 testnet:

Linux (x86_64)
=== E2E Node Management Test Results ===
Platform: Linux 6.18.7-arch1-1 x86_64

Phase 5: Daemon Lifecycle
[PASS] 5.1 Daemon start
[PASS] 5.2 Daemon status
[PASS] 5.3 Daemon info

Phase 6: Node Management
[PASS] 6.1 Add 3 nodes
[PASS] 6.2 Start all nodes
[PASS] 6.3 Node status (all running) — pid & uptime_secs present
[PASS] 6.4 Stop single node
[PASS] 6.5 Status (partial) — stopped node omits pid/uptime_secs, running nodes include them
[PASS] 6.6 Start single node
[PASS] 6.7 Stop all nodes
[PASS] 6.8 Start all nodes

Phase 7: REST API
[PASS] 7.1 GET /status
[PASS] 7.2 GET /nodes/status — pid & uptime_secs present for running nodes
[PASS] 7.3 GET /openapi.json
[PASS] 7.4 GET /console
[PASS] 7.5 CORS headers

Phase 8: Cleanup
[PASS] 8.1 Stop all nodes
[PASS] 8.2 Reset (3 nodes cleared)
[PASS] 8.3 Daemon stop
[PASS] 8.4 Daemon not running

Result: ALL TESTS PASSED
macOS (Apple Silicon)
=== E2E Node Management Test Results ===
Platform: Darwin 24.6.0 arm64 (macOS, Apple Silicon)

Phase 5: Daemon Lifecycle
[PASS] 5.1 Daemon start
[PASS] 5.2 Daemon status
[PASS] 5.3 Daemon info

Phase 6: Node Management
[PASS] 6.1 Add 3 nodes
[PASS] 6.2 Start all nodes
[PASS] 6.3 Node status (all running)
[PASS] 6.4 Stop single node
[PASS] 6.5 Status (partial - 1 stopped, 2 running)
[PASS] 6.6 Start single node
[PASS] 6.7 Stop all nodes
[PASS] 6.8 Start all nodes

Phase 7: REST API
[PASS] 7.1 GET /status
[PASS] 7.2 GET /nodes/status
[PASS] 7.3 GET /openapi.json
[PASS] 7.4 GET /console
[PASS] 7.5 CORS headers

Phase 8: Cleanup
[PASS] 8.1 Stop all nodes
[PASS] 8.2 Reset
[PASS] 8.3 Daemon stop
[PASS] 8.4 Daemon not running

Result: ALL TESTS PASSED
Windows 10 (x86_64)
=== E2E Node Management Test Results ===
Platform: Windows 10 Pro 10.0.19045 x86_64

Phase 5: Daemon Lifecycle
[PASS] 5.1 Daemon start
[PASS] 5.2 Daemon status
[PASS] 5.3 Daemon info

Phase 6: Node Management
[PASS] 6.1 Add 3 nodes
[PASS] 6.2 Start all nodes
[PASS] 6.3 Node status (all running)
[PASS] 6.4 Stop single node
[PASS] 6.5 Status (partial - 1 stopped, 2 running)
[PASS] 6.6 Start single node
[PASS] 6.7 Stop all nodes
[PASS] 6.8 Start all nodes

Phase 7: REST API
[PASS] 7.1 GET /status
[PASS] 7.2 GET /nodes/status
[PASS] 7.3 GET /openapi.json
[PASS] 7.4 GET /console
[PASS] 7.5 CORS headers

Phase 8: Cleanup
[PASS] 8.1 Stop all nodes
[PASS] 8.2 Reset
[PASS] 8.3 Daemon stop
[PASS] 8.4 Daemon not running

Result: ALL TESTS PASSED

🤖 Generated with Claude Code

Remove the integration test for pid and uptime_secs fields from
daemon_integration.rs (added by mistake) and extend the e2e node
management test command to verify these fields instead.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Nic-dorman Nic-dorman merged commit 58a98d2 into WithAutonomi:main Mar 31, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants