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

refactor(verification): remove VertexVerifier from TransactionVerifier [part 8/9] #837

Merged

Conversation

glevco
Copy link
Contributor

@glevco glevco commented Oct 27, 2023

Depends on #836

Motivation

Simplify code by removing the VertexVerifier composition from TransactionVerifier.

Acceptance Criteria

  • Remove TransactionVerifier.verify_outputs(), moving its only custom verification to a new verify_output_token_indexes() method, and moving its call to VerificationService._verify_without_storage_tx(). This allows us to remove the VertexVerifier composition from TransactionVerifier.
  • Update usages accordingly.
  • No behavior should be changed by this PR.

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

@glevco glevco self-assigned this Oct 27, 2023
@glevco glevco changed the title refactor(verification): remove VertexVerifier from TransactionVerifier refactor(verification): remove VertexVerifier from TransactionVerifier [part 8/9] Oct 27, 2023
@codecov
Copy link

codecov bot commented Oct 27, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (353c6c4) 85.25% compared to head (e21db39) 85.31%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #837      +/-   ##
==========================================
+ Coverage   85.25%   85.31%   +0.05%     
==========================================
  Files         283      283              
  Lines       22416    22415       -1     
  Branches     3397     3397              
==========================================
+ Hits        19111    19123      +12     
+ Misses       2613     2606       -7     
+ Partials      692      686       -6     

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

@glevco glevco force-pushed the refactor/verification-inheritance/7-remove-vertex-block branch 2 times, most recently from ee1d52e to d0f42cc Compare November 10, 2023 19:17
@glevco glevco force-pushed the refactor/verification-inheritance/8-remove-vertex-tx branch from edd0878 to 53a7f36 Compare November 10, 2023 19:23
@glevco glevco marked this pull request as ready for review November 10, 2023 20:05
@glevco glevco force-pushed the refactor/verification-inheritance/7-remove-vertex-block branch from d0f42cc to 5ecf018 Compare November 16, 2023 17:05
@glevco glevco force-pushed the refactor/verification-inheritance/8-remove-vertex-tx branch from 53a7f36 to cdc3256 Compare November 16, 2023 17:05
@glevco glevco force-pushed the refactor/verification-inheritance/7-remove-vertex-block branch from 5ecf018 to 45df1ef Compare November 16, 2023 20:06
@glevco glevco force-pushed the refactor/verification-inheritance/8-remove-vertex-tx branch from cdc3256 to aaef3db Compare November 16, 2023 20:06
@glevco glevco force-pushed the refactor/verification-inheritance/7-remove-vertex-block branch from 45df1ef to cbdf80b Compare November 16, 2023 21:26
@glevco glevco force-pushed the refactor/verification-inheritance/8-remove-vertex-tx branch from aaef3db to 230e953 Compare November 16, 2023 21:26
@glevco glevco force-pushed the refactor/verification-inheritance/8-remove-vertex-tx branch from 230e953 to b3974c0 Compare November 27, 2023 20:22
@glevco glevco force-pushed the refactor/verification-inheritance/7-remove-vertex-block branch from 21a6883 to 2e7d25d Compare November 29, 2023 02:14
@glevco glevco force-pushed the refactor/verification-inheritance/8-remove-vertex-tx branch from b3974c0 to f5048ea Compare November 29, 2023 02:15
@glevco glevco force-pushed the refactor/verification-inheritance/7-remove-vertex-block branch from 2e7d25d to f7245f1 Compare November 29, 2023 03:09
@glevco glevco force-pushed the refactor/verification-inheritance/8-remove-vertex-tx branch from f5048ea to 31630fd Compare November 29, 2023 03:09
@glevco glevco force-pushed the refactor/verification-inheritance/7-remove-vertex-block branch from f7245f1 to 7af542d Compare December 5, 2023 14:33
@glevco glevco force-pushed the refactor/verification-inheritance/8-remove-vertex-tx branch from 31630fd to 9cd52aa Compare December 5, 2023 14:34
jansegre
jansegre previously approved these changes Dec 5, 2023
msbrogli
msbrogli previously approved these changes Dec 6, 2023
Base automatically changed from refactor/verification-inheritance/7-remove-vertex-block to master December 6, 2023 17:04
@glevco glevco dismissed stale reviews from msbrogli and jansegre December 6, 2023 17:04

The base branch was changed.

@glevco glevco force-pushed the refactor/verification-inheritance/8-remove-vertex-tx branch from 9cd52aa to e21db39 Compare December 6, 2023 17:11
@glevco glevco merged commit db91e96 into master Dec 7, 2023
9 checks passed
@glevco glevco deleted the refactor/verification-inheritance/8-remove-vertex-tx branch December 7, 2023 18:24
@jansegre jansegre mentioned this pull request Dec 11, 2023
2 tasks
This was referenced Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants