test: infrastructure and deployment verification tests for issues #919-#922 - #968
Merged
fejilaup-cloud merged 4 commits intoAug 31, 2026
Merged
Conversation
Implements tests for Issue AtomicIP#919 to verify that deploy.sh and deploy_testnet.sh validate critical config values (treasury address, admin address, notary public key) before deployment, rejecting placeholder/zero addresses with clear error messages. Tests verify: - Zero address rejection - Placeholder value rejection - Validation before deployment operations - Consistent validation between scripts
Implements tests for Issue AtomicIP#920 to verify Redis-backed state handling and document backup/restore procedures. Tests categorize state as either safe-to-lose (cache) or critical (dedup, rate-limit). Tests verify: - Cache state is safe to lose without data corruption - Critical dedup state must be persisted - Rate-limit counters require backup - Persistence configuration recommendations - Cross-instance consistency requirements
Implements tests for Issue AtomicIP#921 to verify Dockerfile structure, build optimization, and CI workflow configuration for building and publishing container images. Tests verify: - Dockerfile exists with proper multi-stage build - Uses lean base images with security best practices - Exposes required API ports - Optimized build layers and caching strategy - Environment variables and build arguments documented - CI workflow triggers on tags and main branch
Implements tests for Issue AtomicIP#922 to verify smoke-test configuration, CI workflow triggers, and deployment verification process. Tests verify: - Smoke-test script coverage of critical endpoints - Proper exit code handling - Environment-specific configuration - CI workflow post-deploy triggers - Failure alerting configuration - Response validation and timeout handling - Independent executability and reusability
|
@owennashdev-ctrl Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements comprehensive test suites for four infrastructure issues covering deployment configuration validation, Redis backup/restore procedures, container image building, and post-deployment smoke testing.
Changes
Issue Add environment-specific configuration validation to
scripts/deploy.sh#919 - Deployment Configuration Validation Testsscripts/test_deploy_config_validation.shwith tests for verifying treasury address, admin address, and notary public key validationIssue Add a documented, tested backup/restore procedure for API-server Redis state #920 - Redis Backup/Restore Integration Tests
api-server/tests/redis_backup_restore.rswith Redis integration testsIssue Add container image build and publish workflow #921 - Container Image Build Tests
api-server/tests/container_image_build.rswith Dockerfile structure validationIssue Add a staging environment smoke-test workflow triggered on deploy #922 - Staging Smoke-Test Workflow Tests
scripts/test_smoke_test_workflow.shwith smoke-test verificationTest Plan
./scripts/test_deploy_config_validation.shcargo test --features redis-integration-tests --test redis_backup_restorecargo test --test container_image_build./scripts/test_smoke_test_workflow.shAll tests are designed to be implementation-agnostic and focus on validating requirements rather than testing implementation details.
Closes #919
Closes #920
Closes #921
Closes #922