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 missing methods #93

Merged
merged 8 commits into from
Apr 23, 2024
Merged

Test missing methods #93

merged 8 commits into from
Apr 23, 2024

Conversation

cuteolaf
Copy link
Member

No description provided.

@cuteolaf cuteolaf changed the base branch from main to region-pallet April 22, 2024 16:46
Copy link

codecov bot commented Apr 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.47%. Comparing base (c4c61de) to head (40b7c24).

❗ Current head 40b7c24 differs from pull request most recent head 8eecdf8. Consider uploading reports for the commit 8eecdf8 to get more accurate results

Additional details and impacted files
@@                Coverage Diff                 @@
##           region-pallet      #93       +/-   ##
==================================================
+ Coverage          64.05%   76.47%   +12.41%     
==================================================
  Files                  7        7               
  Lines                153      153               
==================================================
+ Hits                  98      117       +19     
+ Misses                55       36       -19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Szegoo Szegoo merged commit 4d09cb6 into region-pallet Apr 23, 2024
@cuteolaf cuteolaf deleted the test-missing-methods branch April 23, 2024 17:21
Szegoo added a commit that referenced this pull request Apr 24, 2024
* Region pallet

* progress

* tests compiling

* rename

* make request

* getting storage key works

* progress

* messy solution

* improvements

* improved solution

* move types & on_timeout

* runtime implementation progress

* compiles

* compiles

* warnings

* fix runtime

* height provider

* resolve some todos

* init tests

* setting record test

* add todos

* rename request status

* correct logic

* refactor

* fix tests

* mock improvements

* clippy fixes

* fix on_timeout

* util read_value

* ismp custom error

* on_accept sanity test

* merge

* move primitives

* merge

* fix

* new tests

* add license

* some refactors

* more refactor

* fix typo

* test: set_record

* test transfer

* test request_region_record

* fix: request_region_record_works

* remove test on_timeout_handles_get_only

* update request_region_record_works

* add events

* cargo fmt

* add more tests

* fix events problem

* check events

* add utility pallet

* cargo fmt

* cargo fmt again

* cargo fmt nightly

* cargo fmt

* Benchmarks (#85)

* WIP: benchmarking

* fix benchmarks

* check events in benchmarks

* cargo fmt

---------

Co-authored-by: Sergej <[email protected]>

* weights for pallet-regions (#87)

* weights for pallet-regions

* add header and template to weights

---------

Co-authored-by: Sergej <[email protected]>

* remove duplicate item

* fmt

* cargo fixes

* rename to codec

* region_id defensive

* dependency fix

* fmt

* small fixes

* Test missing methods (#93)

* test owner

* refactor

* test method: attribute

* test nonfungible transfer

* cargo fmt

* update test for on_timeout

* WIP: tests for on_response

* fix test for invalid id and record

---------

Co-authored-by: Sergej <[email protected]>

* cargo fmt

* fix cargo.toml

* pre merge improvements & additional tests

---------

Co-authored-by: cuteolaf <[email protected]>
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.

2 participants