test: add failure handling tests for deployment scripts#508
Merged
Jagadeeshftw merged 6 commits intoJagadeeshftw:masterfrom Feb 26, 2026
Merged
Conversation
- Add comprehensive test suite (test_all_script_failures.sh) covering all deployment scripts - Enhance existing deploy.sh and upgrade.sh test files with additional edge cases - Add tests for configuration file missing/invalid scenarios - Add tests for environment variable validation failures - Create failure tests for rollback.sh and verify-deployment.sh scripts - Update CI to run comprehensive test suite - Add detailed documentation for tested failure scenarios Closes Jagadeeshftw#468
|
@ekumamatthew is attempting to deploy a commit to the Jagadeesh B's projects Team on Vercel. A member of the Team first needs to authorize it. |
Owner
|
@ekumamatthew fix the pipeline |
Owner
1 similar comment
Owner
Contributor
Author
|
Just got to see your message now |
Contributor
Author
|
I'm doing that right now |
- Fix WASM file path in Stellar build check to use correct relative path - Add Stellar identity setup step to create grainlify-deployer identity for script tests - Update stellar keys command to use correct 'ls' subcommand
- Fix WASM file path in Stellar build check to use correct relative path - Add Stellar identity setup step to create grainlify-deployer identity for script tests - Update stellar keys command to use correct 'ls' subcommand
- Update test_upgrade_scenarios.rs to use current Soroban SDK API - Replace env.ledger().with_mut(|l| l.timestamp += 200) with env.ledger().set_timestamp() - Fixes compilation error with soroban-sdk v21.7.7
Owner
Contributor
Author
|
Hi @Jagadeeshftw ci fails but runs on my end |
Contributor
Author
|
Well please have a look, I may be missing something I'm nt attentive to. Please help out |
Owner
|
@ekumamatthew, the cargo test is failing. I believe some of the changes you made impacted the test cases. Please check them. It’s passing in the main branch. |
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.
Closes #468