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

test: add an integration test for UpdateAdmin #68

Merged
merged 7 commits into from
Aug 23, 2023
Merged

Conversation

loloicci
Copy link

@loloicci loloicci commented Aug 10, 2023

This PR adds an integration test for UpdateAdmin.

This test is based on https://github.com/CosmWasm/wasmd/blob/a2373f019ddf34b08bf16ba035f299e119291e66/x/wasm/keeper/msg_server_integration_test.go#L613.

Checklist:

  • I followed the contributing guidelines and code of conduct.
  • I have added a relevant changelog to CHANGELOG.md
  • I have added tests to cover my changes. (not needed)
  • I have updated the documentation accordingly. (not needed)
  • I have updated API documentation client/docs/swagger-ui/swagger.yaml (not needed)

@loloicci loloicci self-assigned this Aug 10, 2023
@codecov
Copy link

codecov bot commented Aug 10, 2023

Codecov Report

Merging #68 (32ef612) into main (025a70b) will increase coverage by 0.15%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #68      +/-   ##
==========================================
+ Coverage   63.54%   63.69%   +0.15%     
==========================================
  Files          80       80              
  Lines        9994     9994              
==========================================
+ Hits         6351     6366      +15     
+ Misses       3318     3299      -19     
- Partials      325      329       +4     

see 1 file with indirect coverage changes

loloicci and others added 2 commits August 23, 2023 11:51
Co-authored-by: 170210 <85928898+170210@users.noreply.github.com>
Co-authored-by: 170210 <85928898+170210@users.noreply.github.com>
Kynea0b
Kynea0b previously approved these changes Aug 23, 2023
dudong2
dudong2 previously approved these changes Aug 23, 2023
@loloicci loloicci merged commit 8ff66d6 into main Aug 23, 2023
12 checks passed
@zemyblue zemyblue mentioned this pull request Oct 19, 2023
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