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

chore(test): prepare test files for refactor #1391

Merged
merged 11 commits into from
Feb 14, 2023
Merged

Conversation

fedekunze
Copy link
Contributor

@fedekunze fedekunze commented Feb 14, 2023

Description


Refactors the current test in preparation for the testing refactor. The new structure is as follows:

  • keeper_test.go: should only contain tests on the keeper.go
  • setup_test.go: setup of the testing suites
  • utils_test.go: all utils functions that are duplicated and that will need to be removed in follow-up PRs

@fedekunze fedekunze marked this pull request as ready for review February 14, 2023 10:55
@fedekunze fedekunze requested review from GAtom22 and removed request for a team February 14, 2023 10:55
@github-actions github-actions bot added the types label Feb 14, 2023
@codecov
Copy link

codecov bot commented Feb 14, 2023

Codecov Report

Merging #1391 (690b0b1) into main (2871b8e) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1391   +/-   ##
=======================================
  Coverage   72.42%   72.42%           
=======================================
  Files         259      260    +1     
  Lines       17684    17685    +1     
=======================================
+ Hits        12807    12808    +1     
  Misses       4308     4308           
  Partials      569      569           
Impacted Files Coverage Δ
app/ante/evm/setup_ctx.go 81.89% <100.00%> (ø)
x/evm/keeper/block_proposer.go 100.00% <100.00%> (ø)
x/evm/keeper/fees.go 81.96% <100.00%> (ø)

@fedekunze fedekunze enabled auto-merge (squash) February 14, 2023 11:31
app/ante/evm/setup_ctx.go Show resolved Hide resolved
Copy link
Contributor

@MalteHerrmann MalteHerrmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good so far! Just some small adjustments from my side

CHANGELOG.md Outdated Show resolved Hide resolved
app/ante/evm/setup_ctx.go Show resolved Hide resolved
app/ante/evm/setup_test.go Outdated Show resolved Hide resolved
app/ante/evm/setup_test.go Outdated Show resolved Hide resolved
x/erc20/keeper/erc20_test.go Outdated Show resolved Hide resolved
Copy link
Contributor

@GAtom22 GAtom22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good! Left a question

app/ante/cosmos/setup_test.go Show resolved Hide resolved
@facs95
Copy link
Contributor

facs95 commented Feb 14, 2023

Lint is failing

@fedekunze fedekunze merged commit c545822 into main Feb 14, 2023
@fedekunze fedekunze deleted the fedekunze/utils-test branch February 14, 2023 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants