Skip to content

Conversation

@junghoon-vans
Copy link
Member

@junghoon-vans junghoon-vans commented Nov 27, 2025

Summary

  • Add missing gas values to all 8 gas measurement txtar files
  • Add STORAGE DELTA and STORAGE FEE values to storage_gas_measurement.txtar
  • Enhance comment_gas_measurement with wrapper functions for same-TX caching tests

Changes

Files Updated

  • comment_gas_measurement.txtar - Add gas values, add caching test wrapper functions
  • cross_realm_depth_gas_measurement.txtar - Add gas values
  • data_structure_gas_measurement.txtar - Add gas values
  • pkg_realm_call_gas_measurement.txtar - Add gas values
  • realm_load_cost_measurement.txtar - Add gas values
  • storage_gas_measurement.txtar - Add gas values, add STORAGE DELTA/FEE values
  • strconv_gas_measurement.txtar - Add gas values
  • string_concat_gas_measurement.txtar - Add gas values

Key Improvements

  1. Storage Fee Tracking: storage_gas_measurement.txtar now includes actual storage delta bytes and fee amounts for each write operation
  2. Same-TX Caching Tests: comment_gas_measurement.txtar uses wrapper functions (e.g., CallNoCommentTwice, CallHeavyComment5x) to test caching behavior within a single transaction

…tests

- Enable stdout assertions for all 8 gas measurement txtar files
- Add STORAGE DELTA and STORAGE FEE assertions to storage_gas_measurement
- Update gas values to match actual test outputs
- Enhance comment_gas_measurement with wrapper functions for same-TX caching tests
@junghoon-vans junghoon-vans force-pushed the gas-measurement-assertions branch from cf02f11 to 81ec3e4 Compare November 27, 2025 12:05
@sonarqubecloud
Copy link

@junghoon-vans junghoon-vans self-assigned this Nov 27, 2025
@junghoon-vans junghoon-vans changed the title test(integration): add gas and storage fee assertions to measurement tests test(integration): add missing assertions to measurement tests Nov 27, 2025
@junghoon-vans junghoon-vans changed the title test(integration): add missing assertions to measurement tests test(integration): add missing gas and storage fee values to measurement tests Nov 27, 2025
@junghoon-vans junghoon-vans changed the title test(integration): add missing gas and storage fee values to measurement tests test(integration): add missing fee values to measurement tests Nov 27, 2025
@junghoon-vans junghoon-vans merged commit 38e5175 into main Nov 27, 2025
164 of 166 checks passed
@junghoon-vans junghoon-vans deleted the gas-measurement-assertions branch November 27, 2025 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants