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.

Sign up for GitHub

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

Jump to bottom

test for stargate queries #2

Open
Kynea0b opened this issue Dec 27, 2023 · 0 comments
Open

test for stargate queries #2

Kynea0b opened this issue Dec 27, 2023 · 0 comments

Comments

@Kynea0b
Copy link

Kynea0b commented Dec 27, 2023 •
edited
Loading

Overview

Integration tests to use finschia-wasm-std library from contracts are required.

Refer to below:

requires stargate query for integration tests

We plan to write integration tests for stargate queries, which are queries for CosmWasm smart contracts, like osmosis-rust.
In Finschia 2.0, stargate queries are not still enabled, so these tests are suspended.

refs

Finschia/wasmd#84
https://github.com/Finschia/wasmd/blob/c83ad2f3e36801060759d83df133ddcbd4106139/x/wasm/keeper/query_plugins.go#L314C1-L317C61
@Kynea0b Kynea0b mentioned this issue Dec 27, 2023
Merged
@Kynea0b Kynea0b changed the title consider and create tests and examples for finschia consider and make tests and examples for finschia Dec 27, 2023
Kynea0b added a commit to Kynea0b/finschia-wasm that referenced this issue Dec 28, 2023
@Kynea0b
chore: delete tests
c1c2d65 
- packages/finschia-std-derive/tests
- packages/finschia-std/tests

The above two tests need to be recreated as tests for finschia.
Finschia#2
Kynea0b added a commit that referenced this issue Jan 10, 2024
@Kynea0b
Use osmosis-rust as finschia-wasm (#1)
31bdb1e 
* feat: change submodule to finschia

* feat: change project settings for finschia

* feat: apply proto-build modified for finschia

* feat: change the code generation output path to finschia

* feat: rename to finschia

* refactor: change local variable name to finschia and delete unnecessary osmosis variable

* chore: commented out to reconsider the test for finschia

Need to consider testing for finschia

* test: delete due to unnecessary testing for finschia

* chore: replace with contracts for finschia

* chore: delete tests

- packages/finschia-std-derive/tests
- packages/finschia-std/tests

The above two tests need to be recreated as tests for finschia.
#2

* test: add encode/decode and serialize/deserialize test using structure which belongs to foundation module

* test: add test for `CosmwasmExt`

* style: cargo fmt
@Kynea0b Kynea0b changed the title consider and make tests and examples for finschia consider integrations tests and examples for finschia Jan 12, 2024
@Kynea0b Kynea0b changed the title consider integrations tests and examples for finschia consider integrations tests for finschia Jan 12, 2024
@Kynea0b Kynea0b changed the title consider integrations tests for finschia test for stargate queries Jan 20, 2024
@Kynea0b Kynea0b mentioned this issue Jan 20, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Kynea0b