Skip to content

feat(dpp): extra methods for state transitions in wasm #5094

feat(dpp): extra methods for state transitions in wasm

feat(dpp): extra methods for state transitions in wasm #5094

Triggered via pull request January 26, 2025 13:01
Status Failure
Total duration 14m 43s
Artifacts 4

tests.yml

on: pull_request
Build JS packages  /  Build JS
5m 56s
Build JS packages / Build JS
Matrix: Build Docker images
Determine changed packages
16s
Determine changed packages
Rust crates security audit
1m 0s
Rust crates security audit
JS dependency versions check
12s
JS dependency versions check
JS NPM security audit
19s
JS NPM security audit
JS code security audit  /  Run Code QL
2m 44s
JS code security audit / Run Code QL
Packages functional tests  /  Run functional tests
3m 45s
Packages functional tests / Run functional tests
Matrix: Test Suite
Matrix: Dashmate E2E tests
Matrix: JS packages
Matrix: Rust packages
Fit to window
Zoom out
Zoom in

Annotations

3 errors, 84 warnings, and 20 notices
Rust packages (wasm-dpp) / Formatting
Process completed with exit code 1.
Build JS packages / Build JS
Unable to download artifact(s): Artifact not found for name: js-build-7e47e6ff40992cae00f11ee660b5b7f267f179eb Please ensure that your artifact is not expired and the artifact was uploaded using a compatible version of toolkit/upload-artifact. For more information, visit the GitHub Artifacts FAQ: https://github.com/actions/toolkit/blob/main/packages/artifact/docs/faq.md
JS code security audit / Run Code QL
CodeQL Action major versions v1 and v2 have been deprecated. Please update all occurrences of the CodeQL Action in your workflow files to v3. For more information, see https://github.blog/changelog/2025-01-10-code-scanning-codeql-action-v2-is-now-deprecated/
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L174
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:174:41 | 174 | DocumentTransition::Replace(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t` | = note: `#[warn(unused_variables)]` on by default
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L175
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:175:40 | 175 | DocumentTransition::Delete(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L176
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:176:42 | 176 | DocumentTransition::Transfer(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L177
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:177:45 | 177 | DocumentTransition::UpdatePrice(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L178
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:178:42 | 178 | DocumentTransition::Purchase(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
the following explicit lifetimes could be elided: 'a: packages/rs-dpp/src/data_contract/document_type/accessors/mod.rs#L153
warning: the following explicit lifetimes could be elided: 'a --> packages/rs-dpp/src/data_contract/document_type/accessors/mod.rs:153:6 | 153 | impl<'a> DocumentTypeV0Getters for DocumentTypeRef<'a> { | ^^ ^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes = note: `#[warn(clippy::needless_lifetimes)]` on by default help: elide the lifetimes | 153 - impl<'a> DocumentTypeV0Getters for DocumentTypeRef<'a> { 153 + impl DocumentTypeV0Getters for DocumentTypeRef<'_> { |
the following explicit lifetimes could be elided: 'a: packages/rs-dpp/src/data_contract/document_type/accessors/mod.rs#L287
warning: the following explicit lifetimes could be elided: 'a --> packages/rs-dpp/src/data_contract/document_type/accessors/mod.rs:287:6 | 287 | impl<'a> DocumentTypeV0Getters for DocumentTypeMutRef<'a> { | ^^ ^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes help: elide the lifetimes | 287 - impl<'a> DocumentTypeV0Getters for DocumentTypeMutRef<'a> { 287 + impl DocumentTypeV0Getters for DocumentTypeMutRef<'_> { |
unneeded `return` statement: packages/rs-dpp/src/data_contract/document_type/property/array.rs#L73
warning: unneeded `return` statement --> packages/rs-dpp/src/data_contract/document_type/property/array.rs:73:9 | 73 | / return match self { 74 | | ArrayItemType::String(_, _) => { 75 | | let value_as_text = value.as_text().ok_or_else(get_field_type_matching_error)?; 76 | | let vec = value_as_text.as_bytes().to_vec(); ... | 118 | | } 119 | | }; | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return = note: `#[warn(clippy::needless_return)]` on by default help: remove `return` | 73 ~ match self { 74 + ArrayItemType::String(_, _) => { 75 + let value_as_text = value.as_text().ok_or_else(get_field_type_matching_error)?; 76 + let vec = value_as_text.as_bytes().to_vec(); 77 + let mut r_vec = vec.len().encode_var_vec(); 78 + r_vec.extend(vec); 79 + Ok(r_vec) 80 + } 81 + ArrayItemType::Date => { 82 + let value_as_f64 = value.to_float().map_err(ProtocolError::ValueError)?; 83 + let value_bytes = value_as_f64.to_be_bytes().to_vec(); 84 + Ok(value_bytes) 85 + } 86 + ArrayItemType::Integer => { 87 + let value_as_i64: i64 = value.to_integer().map_err(ProtocolError::ValueError)?; 88 + let value_bytes = value_as_i64.to_be_bytes().to_vec(); 89 + Ok(value_bytes) 90 + } 91 + ArrayItemType::Number => { 92 + let value_as_f64 = value.to_float().map_err(ProtocolError::ValueError)?; 93 + let value_bytes = value_as_f64.to_be_bytes().to_vec(); 94 + Ok(value_bytes) 95 + } 96 + ArrayItemType::ByteArray(_, _) => { 97 + let mut bytes = value.to_binary_bytes()?; 98 + 99 + let mut r_vec = bytes.len().encode_var_vec(); 100 + r_vec.append(&mut bytes); 101 + Ok(r_vec) 102 + } 103 + ArrayItemType::Identifier => { 104 + let mut bytes = value.to_identifier_bytes()?; 105 + 106 + let mut r_vec = bytes.len().encode_var_vec(); 107 + r_vec.append(&mut bytes); 108 + Ok(r_vec) 109 + } 110 + ArrayItemType::Boolean => { 111 + let value_as_boolean = value.as_bool().ok_or_else(get_field_type_matching_error)?; 112 + // 0 means does not exist 113 + if value_as_boolean { 114 + Ok(vec![1]) // 1 is true 115 + } else { 116 + Ok(vec![0]) // 2 is false 117 + } 118 + } 119 ~ } |
unneeded `return` statement: packages/rs-dpp/src/data_contract/document_type/property/mod.rs#L986
warning: unneeded `return` statement --> packages/rs-dpp/src/data_contract/document_type/property/mod.rs:986:9 | 986 | / return match self { 987 | | DocumentPropertyType::String(_) => { 988 | | let value_as_text = value 989 | | .as_text() ... | 1128 | | )), 1129 | | }; | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return help: remove `return` | 986 ~ match self { 987 + DocumentPropertyType::String(_) => { 988 + let value_as_text = value 989 + .as_text() 990 + .ok_or_else(|| get_field_type_matching_error(value))?; 991 + let vec = value_as_text.as_bytes().to_vec(); 992 + let mut r_vec = vec.len().encode_var_vec(); 993 + r_vec.extend(vec); 994 + Ok(r_vec) 995 + } 996 + // TODO: Make the same as in https://github.com/dashpay/platform/blob/8d2a9e54d62b77581c44a15a09a2c61864af37d3/packages/rs-dpp/src/document/v0/serialize.rs#L161 997 + // it must be u64 BE. Markers are wrong here as well 998 + DocumentPropertyType::Date => { 999 + let value_as_f64 = value.to_float().map_err(ProtocolError::ValueError)?; 1000 + let mut value_bytes = value_as_f64.to_be_bytes().to_vec(); 1001 + if required { 1002 + Ok(value_bytes) 1003 + } else { 1004 + // if the value wasn't required we need to add a byte to prove it existed 1005 + let mut r_vec = vec![255u8]; 1006 + r_vec.append(&mut value_bytes); 1007 + Ok(r_vec) 1008 + } 1009 + } 1010 + DocumentPropertyType::U128 => { 1011 + let value_as_u128: u128 = value.to_integer().map_err(ProtocolError::ValueError)?; 1012 + Ok(value_as_u128.to_be_bytes().to_vec()) 1013 + } 1014 + DocumentPropertyType::I128 => { 1015 + let value_as_i128: i128 = value.to_integer().map_err(ProtocolError::ValueError)?; 1016 + Ok(value_as_i128.to_be_bytes().to_vec()) 1017 + } 1018 + DocumentPropertyType::U64 => { 1019 + let value_as_u64: u64 = value.to_integer().map_err(ProtocolError::ValueError)?; 1020 + Ok(value_as_u64.to_be_bytes().to_vec()) 1021 + } 1022 + DocumentPropertyType::I64 => { 1023 + let value_as_i64: i64 = value.to_integer().map_err(ProtocolError::ValueError)?; 1024 + Ok(value_as_i64.to_be_bytes().to_vec()) 1025 + } 1026 + DocumentPropertyType::U32 => { 1027 + let value_as_u32: u32 = value.to_integer().map_err(ProtocolError::ValueError)?; 1028 + Ok(value_as_u32.to_be_bytes().to_vec()) 1029 + } 1030 + DocumentPropertyType::I32 => { 1031 + let value_as_i32: i32 = value.to_integer().map_err(ProtocolError::ValueError)?; 1032 + Ok(value_as_i32.to_be_bytes().to_vec()) 1033 + } 1034 + DocumentPropertyType::U16 => { 1035 + let value_as_u16: u16 = value.to_integer().map_err(ProtocolError::ValueError)?; 1036 + Ok(value_as_u16.to_be_bytes().to_vec()) 1037 + } 1038 + DocumentPropertyType::I16 => { 1039 + let value_as_i16: i16 = value.to_integer().map_err(ProtocolError::ValueError)?; 1040 + Ok(value_as_i16.to_be_bytes().to_vec()) 1041 + } 1042 + DocumentPropertyType::U8 => { 1043 + let value_as_u8: u8 = value.to_integer().map_err(ProtocolError::ValueError)?; 1044 + Ok(value_as_u8.to_be_bytes().to_vec()) 1045 + } 1046 + DocumentPropertyType::I8 => {
taken reference of right operand: packages/rs-dpp/src/data_contract/document_type/property/mod.rs#L1234
warning: taken reference of right operand --> packages/rs-dpp/src/data_contract/document_type/property/mod.rs:1234:20 | 1234 | if value == &vec![0] { | ^^^^^^^^^-------- | | | help: use the right value directly: `...` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#op_ref = note: `#[warn(clippy::op_ref)]` on by default
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L174
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:174:41 | 174 | DocumentTransition::Replace(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t` | = note: `#[warn(unused_variables)]` on by default
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L175
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:175:40 | 175 | DocumentTransition::Delete(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L176
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:176:42 | 176 | DocumentTransition::Transfer(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L177
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:177:45 | 177 | DocumentTransition::UpdatePrice(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L178
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:178:42 | 178 | DocumentTransition::Purchase(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused import: `crate::util::type_constants::DEFAULT_HASH_SIZE_U8`: packages/rs-drive/src/drive/document/paths.rs#L2
warning: unused import: `crate::util::type_constants::DEFAULT_HASH_SIZE_U8` --> packages/rs-drive/src/drive/document/paths.rs:2:5 | 2 | use crate::util::type_constants::DEFAULT_HASH_SIZE_U8; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: `#[warn(unused_imports)]` on by default
unused import: `dpp::data_contract::document_type::accessors::DocumentTypeV0Getters`: packages/rs-drive/src/drive/document/paths.rs#L3
warning: unused import: `dpp::data_contract::document_type::accessors::DocumentTypeV0Getters` --> packages/rs-drive/src/drive/document/paths.rs:3:5 | 3 | use dpp::data_contract::document_type::accessors::DocumentTypeV0Getters; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
unused import: `dpp::data_contract::document_type::methods::DocumentTypeV0Methods`: packages/rs-drive/src/drive/document/paths.rs#L4
warning: unused import: `dpp::data_contract::document_type::methods::DocumentTypeV0Methods` --> packages/rs-drive/src/drive/document/paths.rs:4:5 | 4 | use dpp::data_contract::document_type::methods::DocumentTypeV0Methods; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
unused import: `dpp::data_contract::document_type::DocumentTypeRef`: packages/rs-drive/src/drive/document/paths.rs#L5
warning: unused import: `dpp::data_contract::document_type::DocumentTypeRef` --> packages/rs-drive/src/drive/document/paths.rs:5:5 | 5 | use dpp::data_contract::document_type::DocumentTypeRef; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
function `contract_documents_primary_key_path` is never used: packages/rs-drive/src/drive/document/paths.rs#L41
warning: function `contract_documents_primary_key_path` is never used --> packages/rs-drive/src/drive/document/paths.rs:41:15 | 41 | pub(crate) fn contract_documents_primary_key_path<'a>( | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: `#[warn(dead_code)]` on by default
unused import: `platform_value::Identifier`: packages/rs-dpp/src/state_transition/state_transitions/identity/identity_credit_transfer_transition/v0/v0_methods.rs#L11
warning: unused import: `platform_value::Identifier` --> packages/rs-dpp/src/state_transition/state_transitions/identity/identity_credit_transfer_transition/v0/v0_methods.rs:11:5 | 11 | use platform_value::Identifier; | ^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: `#[warn(unused_imports)]` on by default
unused import: `crate::state_transition::GetDataContractSecurityLevelRequirementFn`: packages/rs-dpp/src/state_transition/state_transitions/identity/identity_credit_transfer_transition/v0/v0_methods.rs#L15
warning: unused import: `crate::state_transition::GetDataContractSecurityLevelRequirementFn` --> packages/rs-dpp/src/state_transition/state_transitions/identity/identity_credit_transfer_transition/v0/v0_methods.rs:15:5 | 15 | use crate::state_transition::GetDataContractSecurityLevelRequirementFn; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L174
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:174:41 | 174 | DocumentTransition::Replace(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t` | = note: `#[warn(unused_variables)]` on by default
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L175
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:175:40 | 175 | DocumentTransition::Delete(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L176
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:176:42 | 176 | DocumentTransition::Transfer(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L177
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:177:45 | 177 | DocumentTransition::UpdatePrice(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L178
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:178:42 | 178 | DocumentTransition::Purchase(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
this `else { if .. }` block can be collapsed: packages/rs-drive/src/query/mod.rs#L1324
warning: this `else { if .. }` block can be collapsed --> packages/rs-drive/src/query/mod.rs:1324:16 | 1324 | } else { | ________________^ 1325 | | if let Some(start_at_key) = start_at_key { 1326 | | inner_query.insert_range_to(..start_at_key); 1327 | | } else { ... | 1331 | | } 1332 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_else_if = note: `#[warn(clippy::collapsible_else_if)]` on by default help: collapse nested if block | 1324 ~ } else if let Some(start_at_key) = start_at_key { 1325 + inner_query.insert_range_to(..start_at_key); 1326 + } else { 1327 + //todo: really not sure if this is correct 1328 + // Should investigate more 1329 + inner_query.insert_key(vec![]); 1330 + } |
doc list item without indentation: packages/rs-drive/src/drive/contract/get_fetch/fetch_contract_with_history/v0/mod.rs#L26
warning: doc list item without indentation --> packages/rs-drive/src/drive/contract/get_fetch/fetch_contract_with_history/v0/mod.rs:26:9 | 26 | /// start fetching the contract's history. | ^ | = help: if this is supposed to be its own paragraph, add a blank line = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_lazy_continuation = note: `#[warn(clippy::doc_lazy_continuation)]` on by default help: indent this line | 26 | /// start fetching the contract's history. | ++
doc list item without indentation: packages/rs-drive/src/drive/contract/get_fetch/fetch_contract_with_history/v0/mod.rs#L29
warning: doc list item without indentation --> packages/rs-drive/src/drive/contract/get_fetch/fetch_contract_with_history/v0/mod.rs:29:9 | 29 | /// to return. If `None`, the limit is set to 10. Should be between 1 and 10. | ^ | = help: if this is supposed to be its own paragraph, add a blank line = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_lazy_continuation help: indent this line | 29 | /// to return. If `None`, the limit is set to 10. Should be between 1 and 10. | ++
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L174
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:174:41 | 174 | DocumentTransition::Replace(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t` | = note: `#[warn(unused_variables)]` on by default
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L175
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:175:40 | 175 | DocumentTransition::Delete(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L176
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:176:42 | 176 | DocumentTransition::Transfer(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L177
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:177:45 | 177 | DocumentTransition::UpdatePrice(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L178
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:178:42 | 178 | DocumentTransition::Purchase(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unneeded unit return type: packages/wasm-dpp/src/document/state_transition/document_batch_transition/document_transition/document_create_transition.rs#L97
warning: unneeded unit return type --> packages/wasm-dpp/src/document/state_transition/document_batch_transition/document_transition/document_create_transition.rs:97:80 | 97 | pub fn set_identity_contract_nonce(&mut self, identity_contract_nonce: u64) -> () { | ^^^^^^ help: remove the `-> ()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_unit = note: `#[warn(clippy::unused_unit)]` on by default
casting to the same type is unnecessary (`u64` -> `u64`): packages/wasm-dpp/src/document/state_transition/document_batch_transition/document_transition/document_create_transition.rs#L93
warning: casting to the same type is unnecessary (`u64` -> `u64`) --> packages/wasm-dpp/src/document/state_transition/document_batch_transition/document_transition/document_create_transition.rs:93:9 | 93 | self.inner.base().identity_contract_nonce() as u64 | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `self.inner.base().identity_contract_nonce()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_cast
unneeded unit return type: packages/wasm-dpp/src/identity/state_transition/identity_credit_transfer_transition/transition.rs#L118
warning: unneeded unit return type --> packages/wasm-dpp/src/identity/state_transition/identity_credit_transfer_transition/transition.rs:118:44 | 118 | pub fn set_nonce(&mut self, nonce: u64) -> () { | ^^^^^^ help: remove the `-> ()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_unit
casting to the same type is unnecessary (`u16` -> `u16`): packages/wasm-dpp/src/identity/state_transition/identity_credit_transfer_transition/transition.rs#L104
warning: casting to the same type is unnecessary (`u16` -> `u16`) --> packages/wasm-dpp/src/identity/state_transition/identity_credit_transfer_transition/transition.rs:104:9 | 104 | self.0.user_fee_increase() as u16 | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `self.0.user_fee_increase()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_cast
unneeded unit return type: packages/wasm-dpp/src/identity/state_transition/identity_update_transition/identity_update_transition.rs#L179
warning: unneeded unit return type --> packages/wasm-dpp/src/identity/state_transition/identity_update_transition/identity_update_transition.rs:179:71 | 179 | pub fn set_identity_contract_nonce(&mut self, identity_nonce: u64) -> () { | ^^^^^^ help: remove the `-> ()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_unit
JS packages (dash) / Linting: packages/js-dash-sdk/src/SDK/Client/Client.ts#L10
Dependency cycle via ./Platform:1
JS packages (dash) / Linting: packages/js-dash-sdk/src/SDK/Client/Client.ts#L166
Assignment to function parameter 'options'
JS packages (dash) / Linting: packages/js-dash-sdk/src/SDK/Client/Platform/Fetcher/Fetcher.ts#L143
Returning an awaited value that is not a promise is not allowed
JS packages (dash) / Linting: packages/js-dash-sdk/src/SDK/Client/Platform/Platform.spec.ts#L5
Using exported name 'Client' as identifier for default export
JS packages (dash) / Linting: packages/js-dash-sdk/src/SDK/Client/Platform/Platform.ts#L5
Using exported name 'Client' as identifier for default export
JS packages (dash) / Linting: packages/js-dash-sdk/src/SDK/Client/Platform/Platform.ts#L8
Using exported name 'createAssetLockTransaction' as identifier for default export
unused import: `platform_value::Identifier`: packages/rs-dpp/src/state_transition/state_transitions/identity/identity_credit_transfer_transition/v0/v0_methods.rs#L11
warning: unused import: `platform_value::Identifier` --> packages/rs-dpp/src/state_transition/state_transitions/identity/identity_credit_transfer_transition/v0/v0_methods.rs:11:5 | 11 | use platform_value::Identifier; | ^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: `#[warn(unused_imports)]` on by default
unused import: `crate::state_transition::GetDataContractSecurityLevelRequirementFn`: packages/rs-dpp/src/state_transition/state_transitions/identity/identity_credit_transfer_transition/v0/v0_methods.rs#L15
warning: unused import: `crate::state_transition::GetDataContractSecurityLevelRequirementFn` --> packages/rs-dpp/src/state_transition/state_transitions/identity/identity_credit_transfer_transition/v0/v0_methods.rs:15:5 | 15 | use crate::state_transition::GetDataContractSecurityLevelRequirementFn; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L174
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:174:41 | 174 | DocumentTransition::Replace(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t` | = note: `#[warn(unused_variables)]` on by default
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L175
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:175:40 | 175 | DocumentTransition::Delete(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L176
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:176:42 | 176 | DocumentTransition::Transfer(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L177
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:177:45 | 177 | DocumentTransition::UpdatePrice(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
unused variable: `t`: packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs#L178
warning: unused variable: `t` --> packages/rs-dpp/src/state_transition/state_transitions/document/documents_batch_transition/document_transition/mod.rs:178:42 | 178 | DocumentTransition::Purchase(t) => None, | ^ help: if this is intentional, prefix it with an underscore: `_t`
this `else { if .. }` block can be collapsed: packages/rs-drive-abci/src/execution/platform_events/core_based_updates/update_masternode_identities/update_operator_identity/v0/mod.rs#L268
warning: this `else { if .. }` block can be collapsed --> packages/rs-drive-abci/src/execution/platform_events/core_based_updates/update_masternode_identities/update_operator_identity/v0/mod.rs:268:24 | 268 | } else { | ________________________^ 269 | | if let Some((_, found_old_key)) = identity_to_enable_old_keys 270 | | .iter() 271 | | .find(|(_, key)| key.purpose() == Purpose::TRANSFER) ... | 282 | | } 283 | | }; | |_________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_else_if = note: `#[warn(clippy::collapsible_else_if)]` on by default help: collapse nested if block | 268 ~ } else if let Some((_, found_old_key)) = identity_to_enable_old_keys 269 + .iter() 270 + .find(|(_, key)| key.purpose() == Purpose::TRANSFER) 271 + { 272 + Some(found_old_key.data().to_vec().try_into().map_err(|_| { 273 + Error::Execution(ExecutionError::CorruptedDriveResponse( 274 + "old payout address should be 20 bytes".to_string(), 275 + )) 276 + })?) 277 + } else { 278 + // finally we just use the old masternode payout address 279 + // we need to use the old pub_key_operator 280 + old_masternode.state.operator_payout_address 281 ~ }; |
the `Err`-variant returned from this function is very large: packages/rs-drive-abci/src/execution/platform_events/core_based_updates/update_masternode_identities/update_operator_identity/v0/mod.rs#L44
warning: the `Err`-variant returned from this function is very large --> packages/rs-drive-abci/src/execution/platform_events/core_based_updates/update_masternode_identities/update_operator_identity/v0/mod.rs:44:10 | 44 | ) -> Result<(), Error> { | ^^^^^^^^^^^^^^^^^ | ::: packages/rs-drive-abci/src/error/mod.rs:28:5 | 28 | Abci(#[from] AbciError), | ----------------------- the largest variant contains at least 208 bytes | = help: try reducing the size of `error::Error`, for example by boxing large elements or replacing it with `Box<error::Error>` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
taken reference of right operand: packages/rs-drive-abci/src/execution/platform_events/core_based_updates/update_masternode_identities/update_operator_identity/v0/mod.rs#L323
warning: taken reference of right operand --> packages/rs-drive-abci/src/execution/platform_events/core_based_updates/update_masternode_identities/update_operator_identity/v0/mod.rs:323:42 | 323 | .find(|(_, key)| key.data().as_slice() == &new_payout_address) | ^^^^^^^^^^^^^^^^^^^^^^^^^------------------- | | | help: use the right value directly: `new_payout_address` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#op_ref = note: `#[warn(clippy::op_ref)]` on by default
sccache stats
100% - 290 hits, 0 misses, 0 errors
sccache stats
100% - 60 hits, 0 misses, 0 errors
sccache stats
100% - 60 hits, 0 misses, 0 errors
sccache stats
100% - 60 hits, 0 misses, 0 errors
sccache stats
100% - 196 hits, 0 misses, 0 errors
sccache stats
100% - 428 hits, 0 misses, 0 errors
sccache stats
100% - 214 hits, 1 misses, 0 errors
sccache stats
98% - 533 hits, 13 misses, 0 errors
sccache stats
100% - 60 hits, 0 misses, 0 errors
sccache stats
99% - 406 hits, 4 misses, 0 errors
sccache stats
100% - 202 hits, 1 misses, 0 errors
sccache stats
99% - 348 hits, 4 misses, 0 errors
sccache stats
100% - 434 hits, 0 misses, 0 errors
sccache stats
100% - 60 hits, 0 misses, 0 errors
sccache stats
98% - 1022 hits, 18 misses, 0 errors
sccache stats
100% - 202 hits, 0 misses, 0 errors
sccache stats
100% - 202 hits, 1 misses, 0 errors
sccache stats
100% - 887 hits, 0 misses, 0 errors
sccache stats
99% - 430 hits, 4 misses, 0 errors
sccache stats
99% - 835 hits, 5 misses, 0 errors

Artifacts

Produced during runtime
Name Size
dashpay~platform~82P79S.dockerbuild
87.2 KB
dashpay~platform~PQGM2I.dockerbuild
69.8 KB
dashpay~platform~WMCLOM.dockerbuild
99.9 KB
js-build-7e47e6ff40992cae00f11ee660b5b7f267f179eb Expired
17.5 MB