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

AVM: Add global GenesisHash #5858

Merged
merged 3 commits into from Dec 7, 2023
Merged

Conversation

jannotti
Copy link
Contributor

@jannotti jannotti commented Dec 7, 2023

Paranoid code (especially logicsigs) can check if they are on testnet.

Added some extra test code that seems superfluous to this change (because it is!) But they are good tests that I needed while pursuing other ways of doing this.

@jannotti jannotti self-assigned this Dec 7, 2023
@jannotti jannotti force-pushed the global-genhash branch 3 times, most recently from d7edd37 to 77aabad Compare December 7, 2023 18:58
Paranoid code (especially logicsigs) can check if they are on testnet.
@jannotti jannotti marked this pull request as ready for review December 7, 2023 19:11
algorandskiy
algorandskiy previously approved these changes Dec 7, 2023
data/transactions/logic/ledger_test.go Outdated Show resolved Hide resolved
ledger/eval/appcow_test.go Show resolved Hide resolved
Copy link

codecov bot commented Dec 7, 2023

Codecov Report

Attention: 8 lines in your changes are missing coverage. Please review.

Comparison is base (ed278b8) 55.90% compared to head (c0cfb9e) 55.89%.
Report is 20 commits behind head on master.

Files Patch % Lines
cmd/tealdbg/localLedger.go 0.00% 2 Missing ⚠️
data/transactions/logic/eval.go 81.81% 2 Missing ⚠️
ledger/eval/cow.go 0.00% 2 Missing ⚠️
ledger/eval/eval.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5858      +/-   ##
==========================================
- Coverage   55.90%   55.89%   -0.02%     
==========================================
  Files         477      477              
  Lines       67333    67350      +17     
==========================================
- Hits        37643    37642       -1     
- Misses      27137    27153      +16     
- Partials     2553     2555       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

algorandskiy
algorandskiy previously approved these changes Dec 7, 2023
jasonpaulos
jasonpaulos previously approved these changes Dec 7, 2023
Copy link
Member

@jasonpaulos jasonpaulos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just a test suggestion

data/transactions/logic/evalAppTxn_test.go Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants