test: added additional algod_client tests#518
Open
lempira wants to merge 19 commits intodecouplingfrom
Open
Conversation
- Add config.ts with mock and localnet algod client configuration - Add schemas.ts with 82 Zod schemas generated from OAS spec - Add scripts/generate-zod-schemas.ts for generating Zod schemas from OpenAPI specs
- Used Zod schema validation for API responses - Update generate-zod-schemas.ts to convert kebab-case and snake_case to camelCase - Add VSCode debug configuration for Vitest tests - Skip version test due to genesisHashB64 type mismatch (Uint8Array vs string)
- Add tests for getSupply, suggestedParams, getPendingTransactions - Add SuggestedParams schema manually (client-derived type not in OAS)
- Add localnetConfig for POST endpoints to hit localnet (port 4001) - Implement get_v2_status_wait_for_block_after_round.test.ts - Implement get_v2_transactions_pending_txid.test.ts (mock server issue pending) - Implement post_v2_transactions.test.ts with proper transaction signing - Implement post_v2_transactions_simulate.test.ts with unsigned transactions - Use Vitest property matchers to handle dynamic data (txId, rounds) in snapshots - Add devDependencies: algokit-algo25, @noble/ed25519, zod
- Implement TEAL compile and disassemble tests (using localnet) - Implement devmode block offset tests (using localnet) - Skip ledger sync tests (require follower mode) - Skip delta endpoint tests (require archival node features) - Skip stateproofs test (requires mainnet/testnet) - Skip teal dryrun test (API method not implemented)
…TATEPROOF constant
neilcampbell
approved these changes
Dec 31, 2025
| // Polytest Group: Common Tests | ||
|
|
||
| // TODO: Fix msgpack response handling in PollyJS mock server | ||
| // Skipped: Requires experimental/archival node features and msgpack response handling |
Contributor
There was a problem hiding this comment.
Doesn't appear to be skipped like the comment implies.
| }) | ||
| }) | ||
| }) | ||
|
|
Contributor
There was a problem hiding this comment.
You can probably remove this.
| describe('Common Tests', () => { | ||
| // Polytest Group: Common Tests | ||
|
|
||
| // Skipping this test because there is mismtach between the oas schema and what is returned for |
Contributor
There was a problem hiding this comment.
Comment can be removed, it's not skipped
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.
Added some additional tests to algod_client. I skipped the 7 tests that still need recordings. Will handle those later.
NOTE: This PR depends on this one merging first. It removes private and experimental api endpoints from the polytest config.