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

Create a quine to test app_params_get AppApprovalProgram #2682

Merged
merged 3 commits into from
Aug 3, 2021

Conversation

jasonpaulos
Copy link
Contributor

@jasonpaulos jasonpaulos commented Aug 2, 2021

Summary

I made a TEAL quine to test the output of app_params_get AppApprovalProgram for the current program. It's mostly for fun but this does strengthen the tests for app_params_get.

Test Plan

That's all this is.

@jasonpaulos jasonpaulos self-assigned this Aug 2, 2021
@jasonpaulos jasonpaulos requested a review from jannotti August 2, 2021 19:29
@codecov-commenter
Copy link

codecov-commenter commented Aug 2, 2021

Codecov Report

Merging #2682 (4e0388f) into master (8835bfd) will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2682      +/-   ##
==========================================
+ Coverage   46.99%   47.03%   +0.03%     
==========================================
  Files         350      350              
  Lines       55832    55832              
==========================================
+ Hits        26240    26259      +19     
+ Misses      26636    26622      -14     
+ Partials     2956     2951       -5     
Impacted Files Coverage Δ
catchup/peerSelector.go 98.95% <0.00%> (-1.05%) ⬇️
network/requestTracker.go 70.25% <0.00%> (-0.87%) ⬇️
catchup/service.go 69.35% <0.00%> (ø)
network/wsNetwork.go 60.92% <0.00%> (+0.18%) ⬆️
ledger/acctupdates.go 62.30% <0.00%> (+0.41%) ⬆️
cmd/tealdbg/debugger.go 73.86% <0.00%> (+1.00%) ⬆️
ledger/blockqueue.go 82.18% <0.00%> (+1.14%) ⬆️
network/wsPeer.go 74.37% <0.00%> (+2.22%) ⬆️
util/metrics/counter.go 80.95% <0.00%> (+2.38%) ⬆️
util/metrics/gauge.go 70.66% <0.00%> (+2.66%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8835bfd...4e0388f. Read the comment docs.

Copy link
Contributor

@jannotti jannotti left a comment

Choose a reason for hiding this comment

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

Pretty cool test

@tsachiherman tsachiherman merged commit c1b17d9 into algorand:master Aug 3, 2021
@cusma
Copy link

cusma commented Aug 3, 2021

Cool! That's a... fracTEAL

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.

5 participants