Skip to content
This repository has been archived by the owner on Oct 25, 2022. It is now read-only.

Test coverage for WebAssembly target on CI #72

Open
creativcoder opened this issue Oct 26, 2021 · 2 comments
Open

Test coverage for WebAssembly target on CI #72

creativcoder opened this issue Oct 26, 2021 · 2 comments
Assignees

Comments

@creativcoder
Copy link
Contributor

creativcoder commented Oct 26, 2021

Issue summary

https://rustwasm.github.io/docs/wasm-bindgen/wasm-bindgen-test/usage.html

This will also require a test harness to be setup for browser environment: https://rustwasm.github.io/docs/wasm-bindgen/wasm-bindgen-test/browsers.html

a/c:

Other information and links

@polarbeargo polarbeargo self-assigned this Nov 8, 2021
polarbeargo added a commit that referenced this issue Nov 9, 2021
Implement wasm-bindgen-test under test serialization directly
@polarbeargo
Copy link
Contributor

polarbeargo commented Nov 10, 2021

Hello @hanabi1224 , I transfer issue #72 to you such that we won't implement the same thing at the same time, thank you 😄!

@hanabi1224
Copy link
Contributor

@polarbeargo Ah sry, did not see ur progress on this. I enabled wasm testing for only 1 project as part of CI setup but there're more to be enabled, appreciate if u can help enabling the rest after the PR is merged, thx!

@polarbeargo polarbeargo self-assigned this Nov 10, 2021
polarbeargo added a commit that referenced this issue Nov 12, 2021
polarbeargo added a commit that referenced this issue Nov 24, 2021
* begin checkpointing funcs

* implement checkpoint algorithms, add impl Chain<ProtocolState> for ProtocolState

* add tests

* fix: Cherry pick conflict

* fix: Update library

* fix: Use of moved value:  #24

* style: Format

* fix: Unneeded unit return type

* Add license

* test: Add run test flag

* style: Format

* style: fix function name

* fix: initCheckpoints & test_init_checkpoints

* fix: x86_64 & wasm CI error

* test: #72 Implement wasm-bindgen-test

* fix: Update slots_per_epoch follow mina spec

* feat: Implement is_short_range() follow new spec

* feat: Update init_checkpoints & test_init_checkpoints follow new spec

* feat: Update init_checkpoints follow new spec

* test: Implement test_is_short_range()

* style: Clean code

* chore: Delete Cargo lock for CI build

* fix: rustc E0616

* fix: Unused import: `Hashable`

* fix: Replace Common to Chain trait

* chore: Add Cargo.lock

* test: Add false test case

* fix: No need use clone() less expensive

* fix: Warning about unused import in test module despite being used in a test

* style: Format

* Disable wasm in consensus

* fmt

* Update Cargo.lock

* fix: Remove ProtocolState

* fix: fFx clippy issues

* fix: Fix clippy issues

* fix: Fix clippy issue

* style: Format

* style: Format

* fix: Fix clippy issues

* test: Add test flag

* fix: Fix clippy issue

* fix: Don't use Copy

* fix: Make func to pub

* fix: Update formula

* Remove tag

* fix: Don't use unwrap()

* Update checkpoint.rs

* fix: No need semicolon

* Update checkpoint.rs

* Update checkpoint.rs

* Update checkpoint.rs

Co-authored-by: noot <[email protected]>
@ChainSafe ChainSafe deleted a comment from polarbeargo Feb 8, 2022
@ChainSafe ChainSafe deleted a comment from polarbeargo Feb 8, 2022
@ChainSafe ChainSafe deleted a comment from polarbeargo Feb 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants