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

build / submit transaction test #622

Merged
merged 5 commits into from
Dec 8, 2022
Merged

build / submit transaction test #622

merged 5 commits into from
Dec 8, 2022

Conversation

cxloe
Copy link
Contributor

@cxloe cxloe commented Dec 5, 2022

In this PR

Testing the build transaction and submit transaction endpoints independently.

This is derived off of basic.py and includes some changes that have been moved to other PRs.

@codecov-commenter
Copy link

Codecov Report

Base: 59.50% // Head: 59.50% // No change to project coverage 👍

Coverage data is based on head (db40fa3) compared to base (f67c69b).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #622   +/-   ##
========================================
  Coverage    59.50%   59.50%           
========================================
  Files           83       83           
  Lines        11540    11540           
  Branches      1866     1866           
========================================
  Hits          6867     6867           
  Misses        3124     3124           
  Partials      1549     1549           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@cxloe cxloe marked this pull request as ready for review December 6, 2022 00:38
@cxloe cxloe requested review from pouneh and queenbdubs December 6, 2022 00:38
@cxloe cxloe merged commit cfbd326 into develop Dec 8, 2022
@cxloe cxloe deleted the zd/build-submit-tx-test branch December 8, 2022 02:10
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.

3 participants