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

fix: update version on migrate #15

Merged
merged 2 commits into from
Aug 3, 2023
Merged

fix: update version on migrate #15

merged 2 commits into from
Aug 3, 2023

Conversation

ibrizsabin
Copy link
Collaborator

Description:

Commit Message

fix: update version on migrate

see the guidelines for commit messages.

Changelog Entry

version: <log entry>

Checklist:

  • I have performed a self-review of my own code
  • I have documented my code in accordance with the documentation guidelines
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the unit tests
  • I only have one commit (if not, squash them into one commit).
  • I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

@codecov
Copy link

codecov bot commented Aug 3, 2023

Codecov Report

Merging #15 (e0d4492) into main (d5058d3) will decrease coverage by 0.30%.
Report is 16 commits behind head on main.
The diff coverage is 83.33%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main      #15      +/-   ##
============================================
- Coverage     67.40%   67.10%   -0.30%     
  Complexity      108      108              
============================================
  Files            35       35              
  Lines          1945     1970      +25     
  Branches         55       55              
============================================
+ Hits           1311     1322      +11     
- Misses          610      624      +14     
  Partials         24       24              
Flag Coverage Δ
rust 63.32% <83.33%> (-0.32%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
contracts/cosmwasm-vm/cw-xcall/src/lib.rs 27.77% <0.00%> (-1.64%) ⬇️
contracts/cosmwasm-vm/cw-xcall/src/contract.rs 73.60% <100.00%> (+11.59%) ⬆️

... and 9 files with indirect coverage changes

@ibrizsabin ibrizsabin merged commit 0c57444 into main Aug 3, 2023
10 checks passed
ibrizsabin pushed a commit that referenced this pull request Jan 19, 2024
* feat: add boilerplate for asset manager contract

* feat: add cw-asset-manager in workspace

* feat: add schema for cw-asset-manager

* chore: update version for dependencies

* feat: Implementation structure for cw common (#14)

* build: Added cw-common as a part of this workspace

* feat: Initiated  AssetManager message implementation

* refactor: Changed message code structure

* feat: Add XcallMsg related Data Structures

* feat: Add XcallMsg related Data Structures

* feat: Add XcallMsg related Data Structures

* feat: Remove (from) field from variants of ExecMsg

* feat: Add XcallMsg related Data Structures

* chore: rename package name

* feat: use execute msg from common library

* feat: update schema of cw-asset-manager

* chore: run pre-commit script

---------

Co-authored-by: Night Owl <[email protected]>

* Asset manager implementation (#15)

* build: Added cw-common as a part of this workspace

* feat: Initiated  AssetManager message implementation

* refactor: Changed message code structure

* feat: Add XcallMsg related Data Structures

* feat: Add XcallMsg related Data Structures

* feat: Add XcallMsg related Data Structures

* feat: Remove (from) field from variants of ExecMsg

* feat: Add XcallMsg related Data Structures

* feat: Add states

* feat: Add states

* feat: Replace type for the amount field

* refactor: implementation for the structs

* feat:Add code implementation for initial start

* feat: add state for user's deposits

* feat: Add incomplete withdraw and deposit reply handlers

* feat: Add incomplete withdraw and deposit reply handlers

* feat: add states for xcall integration

* feat: Add starter template for integration test

* build: add cw20 base as a dev-dependency for integration_tests

* refactor: restructured the code for deposit handler

* feat: Add common utilities modules for asset manager

* feat: completed unit test for encoding xdata types

* feat: Add decoder helper with fully tested

* feat: Add implementation for withdraw struct

* feat: Add variant rlp error variat fro contractError

* refactor: restructured the contract's execution logic

* feat: add success constant

* feat: add require error variants

* feat: remove unneccessary files(types & msg)

* Committing changes to Cargo.lock

* build: added new cw20 package

* feat: add item for deposit

* refactor: restructured code

* feat: add unkown err variant

* feat: remove errr handler for xcallhandle

* feat: add err handler for replyhandler

* feat: add err handler for replyhandler

* fix(cw-common,am): rlpStream length:4,depositRevert item count to 4

* fix: fixed rlp value index for depositRevert

* feat: completed unit testing for critical functions

* feat: removed unknown variant

* chore: run pre-commit checks

Signed-off-by: Night Owl <[email protected]>

* feat: add clippy command in pre commit script

Signed-off-by: Night Owl <[email protected]>

* feat: fix clippy warnings

Signed-off-by: Night Owl <[email protected]>

* feat: add asset-manager schema generation script

Signed-off-by: Night Owl <[email protected]>

* feat: import wasm query

Signed-off-by: Night Owl <[email protected]>

* feat: add schema for asset manager

Signed-off-by: Night Owl <[email protected]>

* feat: add wasm query dependency in tests

Signed-off-by: Night Owl <[email protected]>

* feat: update the schema for cw-asset-manager

Signed-off-by: Night Owl <[email protected]>

* fix: signature of reply message

Signed-off-by: Night Owl <[email protected]>

* fix: skip cw-check for cw-common

Signed-off-by: Night Owl <[email protected]>

* fix: wasm file name

Signed-off-by: Night Owl <[email protected]>

* chore: rename variables

Signed-off-by: Night Owl <[email protected]>

* feat: Asset manager implementation for review (#21)

* feat: Initiated  AssetManager message implementation

* refactor: Changed message code structure

* feat: Add XcallMsg related Data Structures

* feat: Add XcallMsg related Data Structures

* feat: Add XcallMsg related Data Structures

* feat: Remove (from) field from variants of ExecMsg

* feat: Add XcallMsg related Data Structures

* feat: Add states

* feat: Add states

* feat: Replace type for the amount field

* refactor: implementation for the structs

* feat:Add code implementation for initial start

* feat: add state for user's deposits

* feat: Add incomplete withdraw and deposit reply handlers

* feat: Add incomplete withdraw and deposit reply handlers

* feat: add states for xcall integration

* feat: Add starter template for integration test

* build: add cw20 base as a dev-dependency for integration_tests

* refactor: restructured the code for deposit handler

* feat: Add common utilities modules for asset manager

* feat: completed unit test for encoding xdata types

* feat: Add decoder helper with fully tested

* feat: Add implementation for withdraw struct

* feat: Add variant rlp error variat fro contractError

* refactor: restructured the contract's execution logic

* feat: add success constant

* feat: add require error variants

* feat: remove unneccessary files(types & msg)

* Committing changes to Cargo.lock

* build: added new cw20 package

* feat: add item for deposit

* refactor: restructured code

* feat: add unkown err variant

* feat: remove errr handler for xcallhandle

* feat: add err handler for replyhandler

* feat: add err handler for replyhandler

* fix(cw-common,am): rlpStream length:4,depositRevert item count to 4

* fix: fixed rlp value index for depositRevert

* feat: completed unit testing for critical functions

* feat: removed unknown variant

* fix: added clippy to pre_commit scripts

* fix: changed source scripth path for asset-manager

* fix: added expected n/w addr for target in unit tests

* feat: add rust codecov badge (#20)

Signed-off-by: Night Owl <[email protected]>

* feat: renamed vars

* chore: fix merge conflicts

Signed-off-by: Night Owl <[email protected]>

* chore: run rust fmt

Signed-off-by: Night Owl <[email protected]>

* fix: errors during merge

Signed-off-by: Night Owl <[email protected]>

* fix: merge conflicts

* feat: add bin package

Signed-off-by: Night Owl <[email protected]>

* fix: unit test for asset manager instantiation

Signed-off-by: Night Owl <[email protected]>

* style: format Cargo.toml file

Signed-off-by: Night Owl <[email protected]>

* feat: add package versioning option for upgrade

Signed-off-by: Night Owl <[email protected]>

* fix: spelling mistakes

Signed-off-by: Night Owl <[email protected]>

---------

Signed-off-by: Night Owl <[email protected]>
Co-authored-by: Night Owl <[email protected]>
Co-authored-by: Night Owl <[email protected]>

* fix : only owner can call configure network and added enum in helper code (#23)

* only owner can call the configure network and test is added

* enum added and also test for it

* chore: add review comments

Signed-off-by: Night Owl <[email protected]>

* fix: Merge bdev/asset-manager to feat (#27)

* fix: changes per review suggestions

* fix: initial reviewed changes applied

* fix: initial reviewed changes applied

* fix: initial reviewed changes applied

* feat:Add unit test for deposit

* some refines

* fix: pass initiaized resp based on specific transfer data

* feat: Add checks for handling messages

* fix: changed network address for deposit test

* feat: Remove withdrawReq related functions and structs

* feat: Add complete unit tests

* test: add additional test mark for configuration

* fix: add additional check for valid  icon addr character setn

* feat: Add test module

* feat: Remove (;) from return statements

* build: add x-call-mock package from cw-hub-bnUSD branch

* feat: Add helpers for am contract and setup func for integration test

* feat: Add archway addr func()

* feat: Add mark fro deposit struct implemntation

* feat: Add setup and contract helper for integration test

* feat: Add setup and contract helper for integration test

* feat: Add setup and contract helper for integration test

* feat: Add implementations for contract helpers and setu func

* feat: Add some changes

* build: Add xcall mocked contract from cw-hub-bnUSD branch

* fix(deposit): Changed expected parsing type to allowanceRsp type for query

* Changes Made

* chore: add dev required dependencies for xcall

* feat: Add tests for deposit and xcall handling msg

* fix(cw20_setup): remove reply access from cw20 boxed instance

* chore: additional reviewing changes

* chore: run pre_commit script

* feat: Add query msg

* feat: Add entry point and handlers for contract query

* test(deposit_sufficient): fixed 'to' field assertion value

* fix(helper,contract): fixed cargo clippy issues

* chore: run pre_commit script

* chore: Improved code using clippy

* fix(deposit,exe_msg): changed contract addr to token addr

* feat: Add balance check helper for test purpose

* fix(am_setup): added reply contract wrapper

* style: Renamed XCallMsg to XcallExecuteMsg

* feat: add tests for handle msg

* style: Renamed  XcallMsg to XcallExecuteMsg

* feat: Add handlemsg test for asset manager

* test(balance): add balance check for manual submsg atomicity execution

* chore(transfer_func): add debug print for execution flow check

* feat: Add variant for xcall nework address check

* fix(handle_call): optmized xcall check

* test(token_adr): added network address for the token

* chore: run pre_commit script

* chore: fix cargo clippy issues

* fix(deposit_cw20): changed 'to' filed with dest_am address

* fix(deposit_encode): appended data to the stream

* fix(transfer): used native addr

* fix: cargo clippy issues

* fix(exec_mod): private visibility for internal  token transfer func

* build: updated dependencies

* feat: add ibc-core setup

* fix(handle_msg): removed sn field

* fix: remove sn field

* chore: run pre_commit

* feat(deposit): add xcall fee fund

* chore: update packages in cargo toml

Signed-off-by: Night Owl <[email protected]>

* chore: run linter checks

Signed-off-by: Night Owl <[email protected]>

* fix: spelling error, import of XCallMsg library

Signed-off-by: Night Owl <[email protected]>

* fix: network address struct usage

Signed-off-by: Night Owl <[email protected]>

* feat: add setup method during contract instantiate

Signed-off-by: Night Owl <[email protected]>

* feat: update asset manager schema

Signed-off-by: Night Owl <[email protected]>

* chore: update packages

Signed-off-by: Night Owl <[email protected]>

* fix: fail on empty network address

Signed-off-by: Night Owl <[email protected]>

* feat: add packages in code cov

Signed-off-by: Night Owl <[email protected]>

* fix: contract setup in integration test

Signed-off-by: Night Owl <[email protected]>

* fix: withdrawal of deposited tokens

Signed-off-by: Night Owl <[email protected]>

* fix: cosm wasm message in deposit tokens

Signed-off-by: Night Owl <[email protected]>

* fix: test for insufficient allowance

Signed-off-by: Night Owl <[email protected]>

* update: cargo packages

Signed-off-by: Night Owl <[email protected]>

* chore: remove TODO comments

Signed-off-by: Night Owl <[email protected]>

* chore: use mock ibc connection

Signed-off-by: Night Owl <[email protected]>

* fix: tests for depositing tokens

Signed-off-by: Night Owl <[email protected]>

* feat: update rlp library to ibc

Signed-off-by: Night Owl <[email protected]>

* fix: decoder error mapped to contract error

* fix: cargo lints fixes

* fix: carg lock updated

* feat: change from address to native address from network address

Signed-off-by: Night Owl <[email protected]>

---------

Signed-off-by: Night Owl <[email protected]>
Co-authored-by: bdev <[email protected]>
Co-authored-by: 0ptimizerr <[email protected]>
Co-authored-by: AntonAndell <[email protected]>
Co-authored-by: qwerty0789 <[email protected]>
ibrizsabin pushed a commit that referenced this pull request Jan 19, 2024
* build: Added cw-common as a part of this workspace

* feat: Initiated  AssetManager message implementation

* refactor: Changed message code structure

* feat: Add XcallMsg related Data Structures

* feat: Add XcallMsg related Data Structures

* feat: Add XcallMsg related Data Structures

* feat: Remove (from) field from variants of ExecMsg

* feat: Add XcallMsg related Data Structures

* feat: Add states

* feat: Add states

* feat: Replace type for the amount field

* refactor: implementation for the structs

* feat:Add code implementation for initial start

* feat: add state for user's deposits

* feat: Add incomplete withdraw and deposit reply handlers

* feat: Add incomplete withdraw and deposit reply handlers

* feat: add states for xcall integration

* feat: Add starter template for integration test

* build: add cw20 base as a dev-dependency for integration_tests

* refactor: restructured the code for deposit handler

* feat: Add common utilities modules for asset manager

* feat: completed unit test for encoding xdata types

* feat: Add decoder helper with fully tested

* feat: Add implementation for withdraw struct

* feat: Add variant rlp error variat fro contractError

* refactor: restructured the contract's execution logic

* feat: add success constant

* feat: add require error variants

* feat: remove unneccessary files(types & msg)

* Committing changes to Cargo.lock

* build: added new cw20 package

* feat: add item for deposit

* refactor: restructured code

* feat: add unkown err variant

* feat: remove errr handler for xcallhandle

* feat: add err handler for replyhandler

* feat: add err handler for replyhandler

* fix(cw-common,am): rlpStream length:4,depositRevert item count to 4

* fix: fixed rlp value index for depositRevert

* feat: completed unit testing for critical functions

* feat: removed unknown variant
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