Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

add max cu-limit to test transactions #34702

Merged
merged 1 commit into from
Jan 9, 2024

Conversation

tao-stones
Copy link
Contributor

Problem

feature gate #30620 is activated everywhere, forcing all native programs to consume CUs. Some of ZK token programs consume more than default instruction CU limits (eg 200_000 CUs), therefore its tests would fail when feature code is cleaned up.

Summary of Changes

  • set all ZK-token-programs test transactions with MAX cu limits (eg 1_400_000)

Fixes #

Copy link

codecov bot commented Jan 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (30fa449) 81.8% compared to head (469acdb) 81.8%.
Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #34702   +/-   ##
=======================================
  Coverage    81.8%    81.8%           
=======================================
  Files         824      824           
  Lines      222687   222687           
=======================================
+ Hits       182245   182311   +66     
+ Misses      40442    40376   -66     

Copy link
Contributor

@samkim-crypto samkim-crypto 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 to me! Thank you!

@tao-stones tao-stones merged commit a1d3003 into solana-labs:master Jan 9, 2024
35 checks passed
@tao-stones tao-stones deleted the add-cu-limit-zk-tests branch January 9, 2024 02:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants