Skip to content

Commit

Permalink
feat: remove unused wasm_key_manager (tari-project#5622)
Browse files Browse the repository at this point in the history
Description
---
Remove unused wasm wrapper for key manager

Motivation and Context
---
The wasm key manager wrapper is unused in the code base

How Has This Been Tested?
---
unit tests
  • Loading branch information
SWvheerden committed Aug 10, 2023
1 parent 41244a3 commit 508c971
Show file tree
Hide file tree
Showing 6 changed files with 0 additions and 311 deletions.
6 changes: 0 additions & 6 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -194,12 +194,6 @@ jobs:
restore-keys: |
tari-${{ runner.os }}-${{ runner.cpu-model }}-${{ env.toolchain }}-nightly-${{ hashFiles('**/Cargo.lock') }}
tari-${{ runner.os }}-${{ runner.cpu-model }}-${{ env.toolchain }}-nightly
- name: test key manager wasm
run: |
cd base_layer/key_manager
rustup target add wasm32-unknown-unknown
cargo install wasm-pack --force
make test
- name: Install cargo-nextest
run: cargo install cargo-nextest --locked --force
- name: cargo test compile
Expand Down
57 changes: 0 additions & 57 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 0 additions & 5 deletions base_layer/key_manager/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ edition = "2021"
[lib]
crate-type = ["lib", "cdylib"]

# NB: All dependencies must support or be gated for the WASM target.
[dependencies]
tari_crypto = { version = "0.17" }
tari_utilities = "0.4.10"
Expand All @@ -29,7 +28,6 @@ diesel_migrations = {version = "2.0.0"}
argon2 = { version = "0.4.1", features = ["std", "alloc"] }
blake2 = "0.9.1"
chacha20 = "0.7.1"
console_error_panic_hook = { version = "0.1.7", optional = true }
crc32fast = "1.2.1"
derivative = "2.2.0"
digest = "0.9.0"
Expand All @@ -39,18 +37,15 @@ serde = "1.0.89"
thiserror = "1.0.26"
strum_macros = "0.22"
strum = { version = "0.22", features = ["derive"] }
wasm-bindgen = { version = "0.2", features = ["serde-serialize", "nightly"], optional = true }
zeroize = "1"
subtle = "2.4.1"

[dev-dependencies]
sha2 = "0.9.8"
wasm-bindgen-test = "0.3.28"
tempfile = "3.1.0"

[features]
default = []
key_manager_service = []
avx2 = ["tari_crypto/simd_backend"]
js = [ "js-sys"]
wasm = ["tari_crypto/wasm", "wasm-bindgen", "js", "console_error_panic_hook"]
17 changes: 0 additions & 17 deletions base_layer/key_manager/Makefile

This file was deleted.

4 changes: 0 additions & 4 deletions base_layer/key_manager/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,6 @@ pub mod mnemonic;
pub mod mnemonic_wordlists;
#[cfg(feature = "key_manager_service")]
pub mod schema;
// https://github.com/rustwasm/wasm-bindgen/issues/2774
#[allow(clippy::unused_unit)]
#[cfg(feature = "wasm")]
pub mod wasm;

hash_domain!(KeyManagerDomain, "com.tari.base_layer.key_manager", 1);

Expand Down
Loading

0 comments on commit 508c971

Please sign in to comment.