Skip to content

Distinguish optional service error state#101

Merged
codeforester merged 1 commit into
mainfrom
bugfix/89-20260620-service-error-state
Jun 20, 2026
Merged

Distinguish optional service error state#101
codeforester merged 1 commit into
mainfrom
bugfix/89-20260620-service-error-state

Conversation

@codeforester

Copy link
Copy Markdown
Collaborator

Summary

  • Add an error state for optional services that have start evidence but fail health checks.
  • Preserve stopped for optional services with no process state file or compose state.
  • Report optional service errors from services check with a non-zero exit.
  • Add BATS coverage for process state, compose state, and deterministic no-state checks.

Validation

  • ./tests/validate.sh
  • bats tests/services_test.bats
  • bats tests/infra_test.bats
  • bats tests/native_services_test.bats
  • BASE_DEMO_SERVICES_STATE_DIR=$(mktemp -d "${TMPDIR:-/tmp}/base-demo-status.XXXXXX") BASE_DEMO_SERVICES_DRY_RUN=1 ./bin/base-demo-services status
  • env GRADLE_USER_HOME=/private/tmp/base-demo-gradle-89 bats tests/demo_test.bats tests/services_test.bats tests/environments_test.bats tests/infra_test.bats tests/go_api_test.bats tests/python_api_test.bats tests/java_services_test.bats tests/native_services_test.bats tests/demo_console_test.bats
  • git diff --check

Note: the full BATS run used a fresh Gradle user home with escalated local execution because sandboxed Gradle initialization hit a local socket restriction.

Fixes #89

@codeforester codeforester merged commit 1479b22 into main Jun 20, 2026
2 checks passed
@codeforester codeforester deleted the bugfix/89-20260620-service-error-state branch June 20, 2026 19:19
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.

services status conflates not-started and error states as stopped

1 participant