Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test Case Refactoring & add btc trx hashes to process through tests #278

Open
reinamora137 opened this issue May 21, 2024 · 0 comments
Open
Labels
indexer Indexer related issues release_requirement Required for Public Release / Production
Milestone

Comments

@reinamora137
Copy link
Collaborator

Reference issues:

#252

#257

there is a flaw in the test case logic that needs to be addressed. Currently it runs through parse_stamp and parse_src20 logic independently. while iin the typical codebase flow these are processed serially with parse_src20 relying on the output data from parse_stamp. There is also loads of edge cases in there with zlib compressed versions, and a need to process real tx_hashes all the way through blocks.py to validate things on the actual trx decoding. There are things in blocks that need to be adjusted irt strings vs bytestrings that gets annoying later on both in parse_stamp and parse_src that requires flipping between data types all the time and creates confusion. There is a variance in SRC-20 vs CP transaction among other things.

@reinamora137 reinamora137 added this to the OS Release milestone May 22, 2024
@reinamora137 reinamora137 added release_requirement Required for Public Release / Production indexer Indexer related issues labels May 22, 2024
@reinamora137 reinamora137 changed the title Test Case Refactoring Test Case Refactoring & add btc trx hashes to process through tests May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
indexer Indexer related issues release_requirement Required for Public Release / Production
Projects
None yet
Development

No branches or pull requests

1 participant