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

tests(bank): Add query unit tests #2040

Merged
merged 2 commits into from Nov 14, 2023
Merged

Conversation

Vvaradinov
Copy link
Contributor

Description


Adds unit tests for bank extension queries.

Closes ENG-2313

@Vvaradinov Vvaradinov requested a review from a team as a code owner November 14, 2023 13:10
@Vvaradinov Vvaradinov requested review from MalteHerrmann and 0xstepit and removed request for a team November 14, 2023 13:10
Copy link

linear bot commented Nov 14, 2023

Copy link

codecov bot commented Nov 14, 2023

Codecov Report

Merging #2040 (0fd6d30) into main (e87bc1b) will decrease coverage by 0.08%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2040      +/-   ##
==========================================
- Coverage   70.54%   70.46%   -0.08%     
==========================================
  Files         335      338       +3     
  Lines       25089    25201     +112     
==========================================
+ Hits        17698    17757      +59     
- Misses       6485     6534      +49     
- Partials      906      910       +4     
Files Coverage Δ
precompiles/bank/query.go 86.36% <ø> (ø)
precompiles/bank/types.go 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

@fedekunze fedekunze merged commit 273c00b into main Nov 14, 2023
34 checks passed
@fedekunze fedekunze deleted the Vvaradinov/bank-ext-unit-tests branch November 14, 2023 13:39
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.

None yet

2 participants