diff --git a/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/README.md b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/README.md new file mode 100644 index 00000000..c0ae9030 --- /dev/null +++ b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/README.md @@ -0,0 +1,170 @@ +Grant Proposal | [318 - Cross Chain Token Bridge-Staking as a Service Solutions](https://portal.devxdao.com/public-proposals/318) +------------ | ------------- +Milestone | 6 +Milestone Title | Build, Deploy, and Internally QA Staking Shell App +OP | nickodio_ferrum +Reviewer | Hatice Kaya + +# Milestone Details + +## Details & Acceptance Criteria + +**Details of what will be delivered in milestone:** + +- Build and deploy a shell app that will allow traditional staking including: +1. Deployment of pool +2. Staking Period +3. Mandatory Lock +4. Early Withdrawal +5. Redistribution of Early Withdrawal +6. Maturity +- Integrate bridge with one of the Casper’s compatible wallets +- Share Staking shell app with internal team for QA and incorporate QA feedback + +**Acceptance criteria:** + +- UI deployment for staking +- Smart Contract Deployment for traditional staking +- Casper compatible wallet integration +1. Ability to proceed through staking flow as described below: +2. Ability to deploy the pool. +3. Ability to set and test staking Periods +4. Ability to set mandatory lock and early withdrawal +5. Ability to set and test reward redistribution of Early Withdrawal +6. Ability to set and test rewards distribution for full maturity +- Internal QA feedback incorporation + +**Additional notes regarding submission from OP:** + +Regarding Milestone 6, we wanted to provide you with an update that Staking App is completed and is working as per the expectation. However, the URL (https://casper-staking.ferrumnetwork.io/) where the App is deployed needs to be whitelisted by Casper team, a request is already submitted. In the meantime, we have put together a step-by-step guide that will allow you to set up staking on your own end. The staking guide can be found here: https://github.com/ferrumnet/staking-casper-frontend Also, sharing the repos for staking for review: Casper staking frontend: https://github.com/ferrumnet/staking-casper-frontend/releases/tag/v0.5.0 Casper staking SmartContracts: https://github.com/ferrumnet/casper_staking/releases/tag/v0.3.1 + +## Milestone Submission + +The following milestone assets/artifacts were submitted for review: + +Repository | Revision Reviewed +------------ | ------------- +https://github.com/ferrumnet/casper_staking/releases/tag/v0.3.1 | eacb280 +https://github.com/ferrumnet/staking-casper-frontend | 827be5c + +# Install & Usage Testing Procedure and Findings + +A Gitpod cloud environment running Ubuntu 22.04.2 LTS was used for building and running the project accross two separate repositories. + +The smart contract repository (`casper_staking`) was reviewed first. The README at the root level of the repository was empty, leaving the reviewer with no instructions for installing and running that part of the project. After checking the subdirectories, it was seen that there was a basic README under the `erc20` folder, and a more comprehensive one under the `staking_contract` folder. Following the instructions in the `staking_contract` folder, the staking contracts were built successfully: [staking_contract_build.md](assets/staking_contract_build.md) + +It was not possible to deploy or run the staking smart contract by following the instructions on the README.md file under the `staking_contract` directory due to multiple errors: [staking_contract_deploy_command_error.md](assets/staking_contract_deploy_command_error.md) + +It was not possible to build, deploy or run the smart contract under the `erc20` directory due to a build error: [erc20_build_failure.md](assets/erc20_build_failure.md) + +The frontend part was installed successfully following the instructions on the README.md file of the frontend repository: [frontend_npm_install.md](assets/frontend_npm_install.md) + +It was not possible to run the frontend app by following the instructions due to an error: [frontend_start_error.md](assets/frontend_start_error.md) + +When attempting to do usage testing on the OP's deployment at the given URL (https://casper-staking.ferrumnetwork.io/), an empty page was encountered. + +![Frontend usage review blank screen](assets/usage_review_blank_screen.png) + +After communication with the OP, a new url (https://casper-staking.ferrumnetwork.io/b9e3b671e577a7d7a4c53aa7010449b47fb8a811c76582dcc41f65a67a16e23d) was provided for usage testing. The app was loaded and visible on the new url. When the Connect Wallet button was clicked, nothing happened on the user interface, and an error was seen on the developer console of the browser. + +![Frontend usage review Connect Wallet error](assets/usage_review_error_on_connect_wallet.png) + +## Overall Impression of usage testing + +A number of errors were encountered during the build, installation, and usage of the project. It was not possible to do a complete usage testing due to the encountered errors mentioned above. +The documentation does not provide sufficient installation/execution instructions, and containes errors. +Due to the issues mentioned above, and because this is the late milestone of the project, the submission should FAIL on all of the criteria in this section. + +Requirement | Finding +------------ | ------------- +Project builds without errors | FAIL +Documentation provides sufficient installation/execution instructions | FAIL +Project functionality meets/exceeds acceptance criteria and operates without error | FAIL + +# Unit / Automated Testing + +The smart contract repository has 6 tests in total under the `staking_contract` directory, covering the positive paths, but there are no tests for negatives paths: [staking_contract_test.md](assets/staking_contract_test.md) + +It was not possible to check the tests under the `erc20` directory due to the build error encountered. + +The frontend repository does not have any tests apart from a single default test of the React framework. + +The OP needs to improve test coverage by adding more tests to the smart contracts under the `staking_contract` directory to cover the negative paths, and make sure the smart contracts under the `erc20` directory also meets the same criteria. + +The OP also needs to add tests to the frontend repository. + +Due to the issues mentioned above, and because this is the late milestone of the project, the submission should FAIL on all criteria of this section. + +Requirement | Finding +------------ | ------------- +Unit Tests - At least one positive path test | FAIL +Unit Tests - At least one negative path test | FAIL +Unit Tests - Additional path tests | FAIL + +# Documentation + +### Code Documentation + +The code-level documentation of the project is very limited, lacking standard inline comments which allow auto-generation of the documentation throughout the code-base. The OP needs to improve the code-level documentation significantly by adding standard code comments. + +Due to the issues mentioned above, and because this is the late milestone of the project, this submission should fail on the criterion of this section. + +Requirement | Finding +------------ | ------------- +Code Documented | FAIL + +### Project Documentation + +The project documentation is not at an acceptable level for this late milestone. OP needs to improve the project level documentation significantly, and fix the errors in the existing documentation. Although there is a good attempt of documentation on the frontend repository, especially the given examples on the smart contract repository are not working. + +Due to the issues mentioned above, and because this is the late milestone of the project, this submission should fail on all of the criteria of this section. + +Requirement | Finding +------------ | ------------- +Usage Documented | FAIL +Example Documented | FAIL + +## Overall Conclusion on Documentation + +Based on the reviewer's findings, this review should FAIL. + +# Open Source Practices + +## Licenses + +The Project is released under the MIT License. + +Requirement | Finding +------------ | ------------- +OSI-approved open source software license | PASS + +## Contribution Policies + +Pull requests and Issues are enabled on the repositories, but the project lacks CONTRIBUTING.md and SECURITY.md. + +Due to the issues mentioned above, and because this is the late milestone of the project, this submission should fail in this section. + +Requirement | Finding +------------ | ------------- +OSS contribution best practices | FAIL + +# Coding Standards + +## General Observations + +Code is generally well-structured and readable. The project is committed to GitHub, but the automated tests are not sufficient, and manual tests were not possible to be done. + +The project also has a number of security vulnerabilities dicovered by the dependabot alerts and the CodeQL analysis features of the code hosting platform. Both the smart contract and the frontend repositories have a high number of high and critical severity vulnerability alerts. + +![Frontend security alerts](assets/frontend_security_alerts.png) + +Due to the security issues on the repository, and because this is the late milestone of the project, the submission should fail in this section. OP needs to fix the security vulnerabilities for this submission to pass the next review. + +# Final Conclusion + +Due to all of the issues mentioned in the earlier sections, in the reviewers opinion, this submission should FAIL. + +# Recommendation + +Recommendation | FAIL +------------ | ------------- diff --git a/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/erc20_build_failure.md b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/erc20_build_failure.md new file mode 100644 index 00000000..7abb8832 --- /dev/null +++ b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/erc20_build_failure.md @@ -0,0 +1,95 @@ +```bash +$ make prepare +rustup target add wasm32-unknown-unknown +info: syncing channel updates for 'nightly-2023-01-16-x86_64-unknown-linux-gnu' +info: latest update on 2023-01-16, rust version 1.68.0-nightly (9e75dddf6 2023-01-15) +info: downloading component 'cargo' +info: downloading component 'rust-std' +info: downloading component 'rustc' +info: installing component 'cargo' +info: installing component 'rust-std' + 29.8 MiB / 29.8 MiB (100 %) 14.7 MiB/s in 2s ETA: 0s +info: installing component 'rustc' + 67.6 MiB / 67.6 MiB (100 %) 16.2 MiB/s in 4s ETA: 0s +info: downloading component 'rust-std' for 'wasm32-unknown-unknown' +info: installing component 'rust-std' for 'wasm32-unknown-unknown' + 19.2 MiB / 19.2 MiB (100 %) 14.2 MiB/s in 1s ETA: 0s +gitpod /workspace/casper_staking/erc20 (main) $ make build-contracts +cargo build --release --target wasm32-unknown-unknown -p erc20-token -p erc20-test -p erc20-test-call + Updating crates.io index + Downloaded num-bigint v0.4.3 + Downloaded serde_bytes v0.11.7 + Downloaded num-derive v0.3.3 + Downloaded num-complex v0.4.2 + Downloaded hex v0.4.3 + Downloaded rand v0.7.3 + Downloaded proc-macro2 v1.0.43 + Downloaded hex_fmt v0.3.0 + Downloaded memory_units v0.4.0 + Downloaded funty v1.1.0 + Downloaded subtle v2.4.1 + Downloaded autocfg v1.1.0 + Downloaded ff v0.8.0 + Downloaded crypto-mac v0.10.1 + Downloaded crunchy v0.2.2 + Downloaded cpufeatures v0.2.2 + Downloaded once_cell v1.13.0 + Downloaded ecdsa v0.10.2 + Downloaded num-iter v0.1.43 + Downloaded wyz v0.2.0 + Downloaded rand_core v0.6.3 + Downloaded hmac v0.10.1 + Downloaded ed25519 v1.2.0 + Downloaded num-integer v0.1.45 + Downloaded ppv-lite86 v0.2.16 + Downloaded ed25519-dalek v1.0.1 + Downloaded crypto-mac v0.8.0 + Downloaded zeroize_derive v1.3.2 + Downloaded digest v0.9.0 + Downloaded block-buffer v0.9.0 + Downloaded itoa v1.0.3 + Downloaded ryu v1.0.11 + Downloaded num-traits v0.2.15 + Downloaded k256 v0.7.3 + Downloaded serde_derive v1.0.143 + Downloaded num-rational v0.4.1 + Downloaded unicode-ident v1.0.3 + Downloaded bitflags v1.3.2 + Downloaded elliptic-curve v0.8.5 + Downloaded cfg-if v1.0.0 + Downloaded zeroize v1.3.0 + Downloaded num v0.4.0 + Downloaded base64 v0.13.0 + Downloaded rand v0.8.5 + Downloaded bitvec v0.18.5 + Downloaded byteorder v1.4.3 + Downloaded blake2 v0.9.2 + Downloaded group v0.8.0 + Downloaded wee_alloc v0.4.5 + Downloaded unicode-xid v0.2.3 + Downloaded typenum v1.15.0 + Downloaded serde_json v1.0.83 + Downloaded cfg-if v0.1.10 + Downloaded version_check v0.9.4 + Downloaded opaque-debug v0.3.0 + Downloaded radium v0.3.0 + Downloaded synstructure v0.12.6 + Downloaded sha2 v0.9.9 + Downloaded rand_core v0.5.1 + Downloaded generic-array v0.14.6 + Downloaded base16 v0.2.1 + Downloaded quote v1.0.21 + Downloaded static_assertions v1.1.0 + Downloaded serde v1.0.143 + Downloaded uint v0.9.3 + Downloaded rand_chacha v0.2.2 + Downloaded syn v1.0.99 + Downloaded libc v0.2.131 + Downloaded signature v1.2.2 + Downloaded casper-contract v1.4.4 + Downloaded curve25519-dalek v3.2.1 + Downloaded casper-types v1.5.0 + Downloaded 72 crates (3.4 MB) in 0.68s +error: package ID specification `erc20-test` did not match any packages +make: *** [Makefile:9: build-contracts] Error 101 +``` diff --git a/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/frontend_npm_install.md b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/frontend_npm_install.md new file mode 100644 index 00000000..8aa44b52 --- /dev/null +++ b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/frontend_npm_install.md @@ -0,0 +1,72 @@ +```bash +$ npm install +npm WARN ERESOLVE overriding peer dependency +npm WARN While resolving: react-color-gradient-picker@0.1.2 +npm WARN Found: react@17.0.2 +npm WARN node_modules/react +npm WARN react@"^17.0.2" from the root project +npm WARN 34 more (@emotion/react, ...) +npm WARN +npm WARN Could not resolve dependency: +npm WARN peer react@"^16.13.1" from react-color-gradient-picker@0.1.2 +npm WARN node_modules/react-color-gradient-picker +npm WARN react-color-gradient-picker@"^0.1.2" from ferrum-design-system@1.0.82 +npm WARN node_modules/ferrum-design-system +npm WARN +npm WARN Conflicting peer dependency: react@16.14.0 +npm WARN node_modules/react +npm WARN peer react@"^16.13.1" from react-color-gradient-picker@0.1.2 +npm WARN node_modules/react-color-gradient-picker +npm WARN react-color-gradient-picker@"^0.1.2" from ferrum-design-system@1.0.82 +npm WARN node_modules/ferrum-design-system +npm WARN ERESOLVE overriding peer dependency +npm WARN While resolving: react-color-gradient-picker@0.1.2 +npm WARN Found: react-dom@17.0.2 +npm WARN node_modules/react-dom +npm WARN react-dom@"17.0.2" from the root project +npm WARN 15 more (@testing-library/react, react-bootstrap, @restart/ui, ...) +npm WARN +npm WARN Could not resolve dependency: +npm WARN peer react-dom@"^16.13.1" from react-color-gradient-picker@0.1.2 +npm WARN node_modules/react-color-gradient-picker +npm WARN react-color-gradient-picker@"^0.1.2" from ferrum-design-system@1.0.82 +npm WARN node_modules/ferrum-design-system +npm WARN +npm WARN Conflicting peer dependency: react-dom@16.14.0 +npm WARN node_modules/react-dom +npm WARN peer react-dom@"^16.13.1" from react-color-gradient-picker@0.1.2 +npm WARN node_modules/react-color-gradient-picker +npm WARN react-color-gradient-picker@"^0.1.2" from ferrum-design-system@1.0.82 +npm WARN node_modules/ferrum-design-system +npm WARN ERESOLVE overriding peer dependency +npm WARN While resolving: terser-webpack-plugin@1.4.5 +npm WARN Found: webpack@4.46.0 +npm WARN node_modules/terser-webpack-plugin/node_modules/webpack +npm WARN +npm WARN Could not resolve dependency: +npm WARN peer webpack@"^4.0.0" from terser-webpack-plugin@1.4.5 +npm WARN node_modules/terser-webpack-plugin +npm WARN terser-webpack-plugin@"^1.4.3" from webpack@4.46.0 +npm WARN node_modules/ferrum-design-system/node_modules/webpack +npm WARN 2 more (webpack, webpack) +npm WARN EBADENGINE Unsupported engine { +npm WARN EBADENGINE package: 'react-bs-datatable@2.2.3', +npm WARN EBADENGINE required: { node: '14' }, +npm WARN EBADENGINE current: { node: 'v18.16.0', npm: '9.5.1' } +npm WARN EBADENGINE } + +up to date, audited 2310 packages in 6s + +225 packages are looking for funding + run `npm fund` for details + +40 vulnerabilities (1 low, 7 moderate, 22 high, 10 critical) + +To address issues that do not require attention, run: + npm audit fix + +To address all issues (including breaking changes), run: + npm audit fix --force + +Run `npm audit` for details. +``` \ No newline at end of file diff --git a/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/frontend_security_alerts.png b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/frontend_security_alerts.png new file mode 100644 index 00000000..57359f7b Binary files /dev/null and b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/frontend_security_alerts.png differ diff --git a/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/frontend_start_error.md b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/frontend_start_error.md new file mode 100644 index 00000000..8f9cead2 --- /dev/null +++ b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/frontend_start_error.md @@ -0,0 +1,37 @@ +```bash +Starting the development server... + +Error: error:0308010C:digital envelope routines::unsupported + at new Hash (node:internal/crypto/hash:71:19) + at Object.createHash (node:crypto:133:10) + at module.exports (/workspace/staking-casper-frontend/node_modules/webpack/lib/util/createHash.js:135:53) + at NormalModule._initBuildHash (/workspace/staking-casper-frontend/node_modules/webpack/lib/NormalModule.js:417:16) + at handleParseError (/workspace/staking-casper-frontend/node_modules/webpack/lib/NormalModule.js:471:10) + at /workspace/staking-casper-frontend/node_modules/webpack/lib/NormalModule.js:503:5 + at /workspace/staking-casper-frontend/node_modules/webpack/lib/NormalModule.js:358:12 + at /workspace/staking-casper-frontend/node_modules/loader-runner/lib/LoaderRunner.js:373:3 + at iterateNormalLoaders (/workspace/staking-casper-frontend/node_modules/loader-runner/lib/LoaderRunner.js:214:10) + at iterateNormalLoaders (/workspace/staking-casper-frontend/node_modules/loader-runner/lib/LoaderRunner.js:221:10) +/workspace/staking-casper-frontend/node_modules/react-scripts/scripts/start.js:19 + throw err; + ^ + +Error: error:0308010C:digital envelope routines::unsupported + at new Hash (node:internal/crypto/hash:71:19) + at Object.createHash (node:crypto:133:10) + at module.exports (/workspace/staking-casper-frontend/node_modules/webpack/lib/util/createHash.js:135:53) + at NormalModule._initBuildHash (/workspace/staking-casper-frontend/node_modules/webpack/lib/NormalModule.js:417:16) + at /workspace/staking-casper-frontend/node_modules/webpack/lib/NormalModule.js:452:10 + at /workspace/staking-casper-frontend/node_modules/webpack/lib/NormalModule.js:323:13 + at /workspace/staking-casper-frontend/node_modules/loader-runner/lib/LoaderRunner.js:367:11 + at /workspace/staking-casper-frontend/node_modules/loader-runner/lib/LoaderRunner.js:233:18 + at context.callback (/workspace/staking-casper-frontend/node_modules/loader-runner/lib/LoaderRunner.js:111:13) + at /workspace/staking-casper-frontend/node_modules/react-scripts/node_modules/babel-loader/lib/index.js:59:103 { + opensslErrorStack: [ 'error:03000086:digital envelope routines::initialization error' ], + library: 'digital envelope routines', + reason: 'unsupported', + code: 'ERR_OSSL_EVP_UNSUPPORTED' +} + +Node.js v18.16.0 +``` \ No newline at end of file diff --git a/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/staking_contract_build.md b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/staking_contract_build.md new file mode 100644 index 00000000..0837810e --- /dev/null +++ b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/staking_contract_build.md @@ -0,0 +1,333 @@ +```bash +gitpod /workspace/casper_staking ((v0.3.1)) $ make prepare +rustup target add wasm32-unknown-unknown +info: component 'rust-std' for target 'wasm32-unknown-unknown' is up to date +gitpod /workspace/casper_staking ((v0.3.1)) $ make build-contract +cargo build --release -p cep47 --target wasm32-unknown-unknown +error: could not find `Cargo.toml` in `/workspace/casper_staking` or any parent directory +make: *** [Makefile:5: build-contract] Error 101 +gitpod /workspace/casper_staking ((v0.3.1)) $ ls +erc20 keys LICENSE Makefile README.md rust-toolchain staking_contract Staking.sol utils +gitpod /workspace/casper_staking ((v0.3.1)) $ cd staking_contract/ +gitpod /workspace/casper_staking/staking_contract ((v0.3.1)) $ ls +Makefile README.md rust-toolchain staking_contract staking_contract_tests utils +gitpod /workspace/casper_staking/staking_contract ((v0.3.1)) $ make +build-contract clean copy-wasm-file-to-test prepare test-only +check-lint clippy lint test +gitpod /workspace/casper_staking/staking_contract ((v0.3.1)) $ make prepare +rustup target add wasm32-unknown-unknown +info: syncing channel updates for 'nightly-2023-01-16-x86_64-unknown-linux-gnu' +info: latest update on 2023-01-16, rust version 1.68.0-nightly (9e75dddf6 2023-01-15) +info: downloading component 'cargo' +info: downloading component 'rust-std' +info: downloading component 'rustc' +info: installing component 'cargo' +info: installing component 'rust-std' + 29.8 MiB / 29.8 MiB (100 %) 12.9 MiB/s in 2s ETA: 0s +info: installing component 'rustc' + 67.6 MiB / 67.6 MiB (100 %) 15.4 MiB/s in 4s ETA: 0s +info: downloading component 'rust-std' for 'wasm32-unknown-unknown' +info: installing component 'rust-std' for 'wasm32-unknown-unknown' + 19.2 MiB / 19.2 MiB (100 %) 13.2 MiB/s in 1s ETA: 0s +gitpod /workspace/casper_staking/staking_contract ((v0.3.1)) $ make build-contract +cd staking_contract && cargo build --release --target wasm32-unknown-unknown + Updating crates.io index + Downloaded bitflags v1.3.2 + Downloaded ed25519 v1.2.0 + Downloaded hex v0.4.3 + Downloaded ecdsa v0.10.2 + Downloaded generic-array v0.14.7 + Downloaded ed25519-dalek v1.0.1 + Downloaded js-sys v0.3.61 + Downloaded serde_derive v1.0.160 + Downloaded ppv-lite86 v0.2.17 + Downloaded serde_json v1.0.96 + Downloaded unicode-ident v1.0.8 + Downloaded itoa v1.0.6 + Downloaded ff v0.8.0 + Downloaded subtle v2.4.1 + Downloaded wyz v0.2.0 + Downloaded autocfg v1.1.0 + Downloaded crunchy v0.2.2 + Downloaded cpufeatures v0.2.7 + Downloaded cfg-if v0.1.10 + Downloaded hmac v0.10.1 + Downloaded digest v0.9.0 + Downloaded wasm-bindgen-shared v0.2.84 + Downloaded serde v1.0.160 + Downloaded uint v0.9.5 + Downloaded num-complex v0.4.3 + Downloaded typenum v1.16.0 + Downloaded zeroize v1.3.0 + Downloaded curve25519-dalek v3.2.1 + Downloaded crypto-mac v0.10.1 + Downloaded crypto-mac v0.8.0 + Downloaded num v0.4.0 + Downloaded num-iter v0.1.43 + Downloaded radium v0.3.0 + Downloaded rand_chacha v0.2.2 + Downloaded num-derive v0.3.3 + Downloaded serde_bytes v0.11.9 + Downloaded wasm-bindgen-macro v0.2.84 + Downloaded rand v0.7.3 + Downloaded zeroize_derive v1.4.2 + Downloaded num-rational v0.4.1 + Downloaded wasm-bindgen-macro-support v0.2.84 + Downloaded wasm-bindgen-backend v0.2.84 + Downloaded syn v2.0.15 + Downloaded syn v1.0.109 + Downloaded wee_alloc v0.4.5 + Downloaded libc v0.2.142 + Downloaded casper-contract v1.4.4 + Downloaded hex_fmt v0.3.0 + Downloaded base64 v0.13.1 + Downloaded version_check v0.9.4 + Downloaded sha2 v0.9.9 + Downloaded rand_core v0.5.1 + Downloaded num-bigint v0.4.3 + Downloaded num-integer v0.1.45 + Downloaded base16 v0.2.1 + Downloaded opaque-debug v0.3.0 + Downloaded proc-macro2 v1.0.56 + Downloaded num-traits v0.2.15 + Downloaded bumpalo v3.12.1 + Downloaded byteorder v1.4.3 + Downloaded wasm-bindgen v0.2.84 + Downloaded static_assertions v1.1.0 + Downloaded log v0.4.17 + Downloaded signature v1.2.2 + Downloaded memory_units v0.4.0 + Downloaded k256 v0.7.3 + Downloaded ryu v1.0.13 + Downloaded rand_core v0.6.4 + Downloaded rand v0.8.5 + Downloaded quote v1.0.26 + Downloaded funty v1.1.0 + Downloaded cfg-if v1.0.0 + Downloaded block-buffer v0.9.0 + Downloaded group v0.8.0 + Downloaded elliptic-curve v0.8.5 + Downloaded blake2 v0.9.2 + Downloaded bitvec v0.18.5 + Downloaded getrandom v0.2.9 + Downloaded once_cell v1.17.1 + Downloaded casper-types v1.5.0 + Downloaded 80 crates (4.2 MB) in 0.50s + Compiling proc-macro2 v1.0.56 + Compiling quote v1.0.26 + Compiling unicode-ident v1.0.8 + Compiling version_check v0.9.4 + Compiling autocfg v1.1.0 + Compiling typenum v1.16.0 + Compiling rand_core v0.5.1 + Compiling subtle v2.4.1 + Compiling syn v1.0.109 + Compiling cfg-if v1.0.0 + Compiling radium v0.3.0 + Compiling funty v1.1.0 + Compiling wyz v0.2.0 + Compiling wasm-bindgen-shared v0.2.84 + Compiling serde_derive v1.0.160 + Compiling log v0.4.17 + Compiling serde v1.0.160 + Compiling byteorder v1.4.3 + Compiling crunchy v0.2.2 + Compiling opaque-debug v0.3.0 + Compiling ppv-lite86 v0.2.17 + Compiling once_cell v1.17.1 + Compiling bumpalo v3.12.1 + Compiling generic-array v0.14.7 + Compiling serde_json v1.0.96 + Compiling wee_alloc v0.4.5 + Compiling num-traits v0.2.15 + Compiling num-integer v0.1.45 + Compiling bitvec v0.18.5 + Compiling num-bigint v0.4.3 + Compiling num-iter v0.1.43 + Compiling num-rational v0.4.1 + Compiling rand_chacha v0.2.2 + Compiling itoa v1.0.6 + Compiling hex v0.4.3 + Compiling rand_core v0.6.4 + Compiling static_assertions v1.1.0 + Compiling ryu v1.0.13 + Compiling rand v0.7.3 + Compiling wasm-bindgen v0.2.84 + Compiling memory_units v0.4.0 + Compiling base16 v0.2.1 + Compiling uint v0.9.5 + Compiling rand v0.8.5 + Compiling cfg-if v0.1.10 + Compiling hex_fmt v0.3.0 + Compiling bitflags v1.3.2 + Compiling base64 v0.13.1 + Compiling syn v2.0.15 + Compiling digest v0.9.0 + Compiling crypto-mac v0.10.1 + Compiling block-buffer v0.9.0 + Compiling crypto-mac v0.8.0 + Compiling signature v1.2.2 + Compiling hmac v0.10.1 + Compiling blake2 v0.9.2 + Compiling sha2 v0.9.9 + Compiling ed25519 v1.2.0 + Compiling num-complex v0.4.3 + Compiling ff v0.8.0 + Compiling group v0.8.0 + Compiling num v0.4.0 + Compiling wasm-bindgen-backend v0.2.84 + Compiling num-derive v0.3.3 + Compiling wasm-bindgen-macro-support v0.2.84 + Compiling zeroize_derive v1.4.2 + Compiling wasm-bindgen-macro v0.2.84 + Compiling zeroize v1.3.0 + Compiling elliptic-curve v0.8.5 + Compiling curve25519-dalek v3.2.1 + Compiling ecdsa v0.10.2 + Compiling ed25519-dalek v1.0.1 + Compiling k256 v0.7.3 + Compiling js-sys v0.3.61 + Compiling getrandom v0.2.9 + Compiling serde_bytes v0.11.9 + Compiling casper-types v1.5.0 + Compiling casper-contract v1.4.4 + Compiling contract-utils v0.1.0 (/workspace/casper_staking/staking_contract/utils/contract-utils) + Compiling staking_contract v0.1.0 (/workspace/casper_staking/staking_contract/staking_contract) + Finished release [optimized] target(s) in 1m 59s +wasm-strip staking_contract/target/wasm32-unknown-unknown/release/staking_contract.wasm 2>/dev/null | true +cd ../erc20/erc20-token && cargo build --release --target wasm32-unknown-unknown + Downloaded itoa v1.0.3 + Downloaded generic-array v0.14.6 + Downloaded serde_bytes v0.11.7 + Downloaded cpufeatures v0.2.2 + Downloaded num-complex v0.4.2 + Downloaded ppv-lite86 v0.2.16 + Downloaded once_cell v1.13.0 + Downloaded zeroize_derive v1.3.2 + Downloaded synstructure v0.12.6 + Downloaded unicode-xid v0.2.3 + Downloaded quote v1.0.21 + Downloaded typenum v1.15.0 + Downloaded unicode-ident v1.0.3 + Downloaded uint v0.9.3 + Downloaded rand_core v0.6.3 + Downloaded base64 v0.13.0 + Downloaded ryu v1.0.11 + Downloaded serde_derive v1.0.143 + Downloaded proc-macro2 v1.0.43 + Downloaded serde v1.0.143 + Downloaded serde_json v1.0.83 + Downloaded syn v1.0.99 + Downloaded libc v0.2.131 + Downloaded 23 crates (1.6 MB) in 0.39s + Compiling autocfg v1.1.0 + Compiling version_check v0.9.4 + Compiling proc-macro2 v1.0.43 + Compiling typenum v1.15.0 + Compiling unicode-ident v1.0.3 + Compiling quote v1.0.21 + Compiling syn v1.0.99 + Compiling subtle v2.4.1 + Compiling rand_core v0.5.1 + Compiling unicode-xid v0.2.3 + Compiling wyz v0.2.0 + Compiling funty v1.1.0 + Compiling radium v0.3.0 + Compiling serde_derive v1.0.143 + Compiling serde v1.0.143 + Compiling byteorder v1.4.3 + Compiling crunchy v0.2.2 + Compiling cfg-if v1.0.0 + Compiling ppv-lite86 v0.2.16 + Compiling opaque-debug v0.3.0 + Compiling serde_json v1.0.83 + Compiling itoa v1.0.3 + Compiling ryu v1.0.11 + Compiling wee_alloc v0.4.5 + Compiling rand_core v0.6.3 + Compiling hex v0.4.3 + Compiling generic-array v0.14.6 + Compiling num-traits v0.2.15 + Compiling num-integer v0.1.45 + Compiling num-bigint v0.4.3 + Compiling num-iter v0.1.43 + Compiling num-rational v0.4.1 + Compiling rand_chacha v0.2.2 + Compiling bitvec v0.18.5 + Compiling static_assertions v1.1.0 + Compiling rand v0.8.5 + Compiling bitflags v1.3.2 + Compiling rand v0.7.3 + Compiling uint v0.9.3 + Compiling base64 v0.13.0 + Compiling base16 v0.2.1 + Compiling cfg-if v0.1.10 + Compiling memory_units v0.4.0 + Compiling hex_fmt v0.3.0 + Compiling once_cell v1.13.0 + Compiling digest v0.9.0 + Compiling crypto-mac v0.10.1 + Compiling block-buffer v0.9.0 + Compiling crypto-mac v0.8.0 + Compiling signature v1.2.2 + Compiling hmac v0.10.1 + Compiling blake2 v0.9.2 + Compiling sha2 v0.9.9 + Compiling ed25519 v1.2.0 + Compiling num-complex v0.4.2 + Compiling ff v0.8.0 + Compiling group v0.8.0 + Compiling synstructure v0.12.6 + Compiling zeroize_derive v1.3.2 + Compiling num-derive v0.3.3 + Compiling num v0.4.0 + Compiling zeroize v1.3.0 + Compiling elliptic-curve v0.8.5 + Compiling curve25519-dalek v3.2.1 + Compiling ecdsa v0.10.2 + Compiling ed25519-dalek v1.0.1 + Compiling k256 v0.7.3 + Compiling serde_bytes v0.11.7 + Compiling casper-types v1.5.0 + Compiling casper-contract v1.4.4 + Compiling casper-erc20 v0.2.1 (/workspace/casper_staking/erc20/erc20) +warning: unused import: `alloc::string::String` + --> erc20/src/error.rs:2:5 + | +2 | use alloc::string::String; + | ^^^^^^^^^^^^^^^^^^^^^ + | + = note: `#[warn(unused_imports)]` on by default + +warning: missing documentation for a function + --> erc20/src/entry_points.rs:145:1 + | +145 | pub fn mint() -> EntryPoint { + | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ + | +note: the lint level is defined here + --> erc20/src/lib.rs:13:9 + | +13 | #![warn(missing_docs)] + | ^^^^^^^^^^^^ + +warning: missing documentation for a variant + --> erc20/src/error.rs:28:5 + | +28 | UserAllowance(u16), + | ^^^^^^^^^^^^^ + +warning: `casper-erc20` (lib) generated 3 warnings (run `cargo fix --lib -p casper-erc20` to apply 1 suggestion) + Compiling erc20-token v0.1.0 (/workspace/casper_staking/erc20/erc20-token) +warning: unused import: `core::panic::PanicInfo` + --> erc20-token/src/main.rs:10:5 + | +10 | use core::panic::PanicInfo; + | ^^^^^^^^^^^^^^^^^^^^^^ + | + = note: `#[warn(unused_imports)]` on by default + +warning: `erc20-token` (bin "erc20_token") generated 1 warning (run `cargo fix --bin "erc20_token"` to apply 1 suggestion) + Finished release [optimized] target(s) in 25.61s +wasm-strip ../erc20/erc20-token/target/wasm32-unknown-unknown/release/staking_contract.wasm 2>/dev/null | true +``` \ No newline at end of file diff --git a/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/staking_contract_deploy_command_error.md b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/staking_contract_deploy_command_error.md new file mode 100644 index 00000000..4ae41d84 --- /dev/null +++ b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/staking_contract_deploy_command_error.md @@ -0,0 +1,25 @@ +```bash +$ casper-client put-deploy \ + --chain-name casper-test \ + --node-address http://44.208.234.65:7777 \ + --secret-key ./contract/keys/secret_key.pem \ + --session-path ./contract/target/wasm32-unknown-unknown/release/bridge_pool.wasm \ + --payment-amount 220000000000 +Cryptographic error: secret_key: secret key load failed: could not read './contract/keys/secret_key.pem': No such file or directory (os error 2) + +$ casper-client put-deploy \ + --chain-name casper-test \ + --node-address http://44.208.234.65:7777 \ + --secret-key ./staking_contract/keys/secret_key.pem \ + --session-path ./contract/target/wasm32-unknown-unknown/release/bridge_pool.wasm \ + --payment-amount 220000000000 +IO error: unable to read session file at './contract/target/wasm32-unknown-unknown/release/bridge_pool.wasm': No such file or directory (os error 2) + +$ casper-client put-deploy \ + --chain-name casper-test \ + --node-address http://44.208.234.65:7777 \ + --secret-key ./staking_contract/keys/secret_key.pem \ + --session-path ./staking_contract/target/wasm32-unknown-unknown/release/staking_contract.wasm \ + --payment-amount 220000000000 +Deploy size too large: deploy size of 2000576 bytes exceeds limit of 1048576 +``` \ No newline at end of file diff --git a/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/staking_contract_test.md b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/staking_contract_test.md new file mode 100644 index 00000000..7bfae330 --- /dev/null +++ b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/staking_contract_test.md @@ -0,0 +1,318 @@ +```bash +gitpod /workspace/casper_staking/staking_contract ((v0.3.1)) $ make test +cd staking_contract && cargo build --release --target wasm32-unknown-unknown + Finished release [optimized] target(s) in 0.05s +wasm-strip staking_contract/target/wasm32-unknown-unknown/release/staking_contract.wasm 2>/dev/null | true +cd ../erc20/erc20-token && cargo build --release --target wasm32-unknown-unknown +warning: unused import: `alloc::string::String` + --> erc20/src/error.rs:2:5 + | +2 | use alloc::string::String; + | ^^^^^^^^^^^^^^^^^^^^^ + | + = note: `#[warn(unused_imports)]` on by default + +warning: missing documentation for a function + --> erc20/src/entry_points.rs:145:1 + | +145 | pub fn mint() -> EntryPoint { + | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ + | +note: the lint level is defined here + --> erc20/src/lib.rs:13:9 + | +13 | #![warn(missing_docs)] + | ^^^^^^^^^^^^ + +warning: missing documentation for a variant + --> erc20/src/error.rs:28:5 + | +28 | UserAllowance(u16), + | ^^^^^^^^^^^^^ + +warning: `casper-erc20` (lib) generated 3 warnings (run `cargo fix --lib -p casper-erc20` to apply 1 suggestion) +warning: unused import: `core::panic::PanicInfo` + --> erc20-token/src/main.rs:10:5 + | +10 | use core::panic::PanicInfo; + | ^^^^^^^^^^^^^^^^^^^^^^ + | + = note: `#[warn(unused_imports)]` on by default + +warning: `erc20-token` (bin "erc20_token") generated 1 warning (run `cargo fix --bin "erc20_token"` to apply 1 suggestion) + Finished release [optimized] target(s) in 0.04s +wasm-strip ../erc20/erc20-token/target/wasm32-unknown-unknown/release/staking_contract.wasm 2>/dev/null | true +cp staking_contract/target/wasm32-unknown-unknown/release/*.wasm staking_contract_tests/wasm +cp ../erc20/target/wasm32-unknown-unknown/release/erc20_token.wasm staking_contract_tests/wasm/erc20.wasm +cd staking_contract_tests && cargo test + Downloaded hostname v0.3.1 + Downloaded downcast-rs v1.2.0 + Downloaded bit-vec v0.6.3 + Downloaded chrono v0.4.19 + Downloaded proc-macro2 v1.0.40 + Downloaded proptest v1.0.0 + Downloaded serde_derive v1.0.138 + Downloaded serde_json v1.0.82 + Downloaded unicode-ident v1.0.1 + Downloaded num_cpus v1.13.1 + Downloaded match_cfg v0.1.0 + Downloaded remove_dir_all v0.5.3 + Downloaded rand_xorshift v0.3.0 + Downloaded wait-timeout v0.2.0 + Downloaded rand_chacha v0.3.1 + Downloaded tempfile v3.3.0 + Downloaded syn v1.0.98 + Downloaded schemars_derive v0.8.5 + Downloaded wasmi v0.8.0 + Downloaded filesize v0.2.0 + Downloaded quick-error v1.2.3 + Downloaded tracing-attributes v0.1.22 + Downloaded uuid v0.8.2 + Downloaded value-bag v1.0.0-alpha.9 + Downloaded time v0.1.44 + Downloaded tracing-core v0.1.28 + Downloaded ryu v1.0.10 + Downloaded wasmi-validation v0.3.0 + Downloaded libc v0.2.126 + Downloaded linked-hash-map v0.5.6 + Downloaded num-rational v0.2.4 + Downloaded num-bigint v0.2.6 + Downloaded memory_units v0.3.0 + Downloaded lmdb-sys v0.8.0 + Downloaded thiserror v1.0.31 + Downloaded parity-wasm v0.41.0 + Downloaded itoa v1.0.2 + Downloaded indexmap v1.9.1 + Downloaded hashbrown v0.12.1 + Downloaded fastrand v1.7.0 + Downloaded getrandom v0.2.7 + Downloaded generic-array v0.14.5 + Downloaded hex-buffer-serde v0.3.0 + Downloaded datasize v0.2.10 + Downloaded casper-engine-test-support v2.2.0 + Downloaded bincode v1.3.3 + Downloaded serde v1.0.138 + Downloaded regex-syntax v0.6.27 + Downloaded tracing v0.1.35 + Downloaded casper-hashing v1.4.3 + Downloaded rusty-fork v0.3.0 + Downloaded hex-buffer-serde v0.2.2 + Downloaded itertools v0.10.3 + Downloaded either v1.7.0 + Downloaded datasize_derive v0.2.10 + Downloaded ctor v0.1.22 + Downloaded bit-set v0.5.2 + Downloaded anyhow v1.0.58 + Downloaded pwasm-utils v0.16.0 + Downloaded lmdb v0.8.0 + Downloaded lazy_static v1.4.0 + Downloaded cc v1.0.73 + Downloaded quick-error v2.0.1 + Downloaded thiserror-impl v1.0.31 + Downloaded serde_derive_internals v0.25.0 + Downloaded serde_bytes v0.11.6 + Downloaded schemars v0.8.5 + Downloaded quote v1.0.20 + Downloaded pkg-config v0.3.25 + Downloaded pin-project-lite v0.2.9 + Downloaded fnv v1.0.7 + Downloaded dyn-clone v1.0.6 + Downloaded casper-execution-engine v2.0.0 + Downloaded 73 crates (3.9 MB) in 0.65s + Compiling proc-macro2 v1.0.40 + Compiling quote v1.0.20 + Compiling unicode-ident v1.0.1 + Compiling syn v1.0.98 + Compiling autocfg v1.1.0 + Compiling version_check v0.9.4 + Compiling libc v0.2.126 + Compiling serde_derive v1.0.138 + Compiling typenum v1.15.0 + Compiling serde v1.0.138 + Compiling cfg-if v1.0.0 + Compiling subtle v2.4.1 + Compiling rand_core v0.5.1 + Compiling unicode-xid v0.2.3 + Compiling ppv-lite86 v0.2.16 + Compiling funty v1.1.0 + Compiling wyz v0.2.0 + Compiling radium v0.3.0 + Compiling byteorder v1.4.3 + Compiling opaque-debug v0.3.0 + Compiling fastrand v1.7.0 + Compiling serde_json v1.0.82 + Compiling crunchy v0.2.2 + Compiling remove_dir_all v0.5.3 + Compiling rand_chacha v0.2.2 + Compiling ryu v1.0.10 + Compiling bitvec v0.18.5 + Compiling cpufeatures v0.2.2 + Compiling bitflags v1.3.2 + Compiling hashbrown v0.12.1 + Compiling fnv v1.0.7 + Compiling schemars v0.8.5 + Compiling generic-array v0.14.5 + Compiling once_cell v1.13.0 + Compiling itoa v1.0.2 + Compiling quick-error v1.2.3 + Compiling num-traits v0.2.15 + Compiling num-integer v0.1.45 + Compiling num-bigint v0.4.3 + Compiling num-iter v0.1.43 + Compiling indexmap v1.9.1 + Compiling num-rational v0.4.1 + Compiling bit-vec v0.6.3 + Compiling rand v0.7.3 + Compiling static_assertions v1.1.0 + Compiling lazy_static v1.4.0 + Compiling quick-error v2.0.1 + Compiling dyn-clone v1.0.6 + Compiling getrandom v0.2.7 + Compiling wait-timeout v0.2.0 + Compiling tempfile v3.3.0 + Compiling bit-set v0.5.2 + Compiling regex-syntax v0.6.27 + Compiling num-bigint v0.2.6 + Compiling value-bag v1.0.0-alpha.9 + Compiling rand_core v0.6.3 + Compiling hex_fmt v0.3.0 + Compiling base64 v0.13.0 + Compiling rusty-fork v0.3.0 + Compiling rand_chacha v0.3.1 + Compiling rand_xorshift v0.3.0 + Compiling base16 v0.2.1 + Compiling pkg-config v0.3.25 + Compiling cc v1.0.73 + Compiling num-rational v0.2.4 + Compiling log v0.4.17 + Compiling rand v0.8.5 + Compiling parity-wasm v0.41.0 + Compiling either v1.7.0 + Compiling wee_alloc v0.4.5 + Compiling anyhow v1.0.58 + Compiling digest v0.9.0 + Compiling crypto-mac v0.10.1 + Compiling block-buffer v0.9.0 + Compiling crypto-mac v0.8.0 + Compiling itertools v0.10.3 + Compiling time v0.1.44 + Compiling signature v1.2.2 + Compiling hmac v0.10.1 + Compiling sha2 v0.9.9 + Compiling ed25519 v1.2.0 + Compiling blake2 v0.9.2 + Compiling tracing-core v0.1.28 + Compiling downcast-rs v1.2.0 + Compiling ff v0.8.0 + Compiling num-complex v0.4.2 + Compiling memory_units v0.3.0 + Compiling group v0.8.0 + Compiling pin-project-lite v0.2.9 + Compiling memory_units v0.4.0 + Compiling match_cfg v0.1.0 + Compiling cfg-if v0.1.10 + Compiling num_cpus v1.13.1 + Compiling hostname v0.3.1 + Compiling linked-hash-map v0.5.6 + Compiling chrono v0.4.19 + Compiling filesize v0.2.0 + Compiling wasmi-validation v0.3.0 + Compiling proptest v1.0.0 + Compiling lmdb-sys v0.8.0 + Compiling wasmi v0.8.0 + Compiling synstructure v0.12.6 + Compiling serde_derive_internals v0.25.0 + Compiling lmdb v0.8.0 + Compiling zeroize_derive v1.3.2 + Compiling schemars_derive v0.8.5 + Compiling datasize_derive v0.2.10 + Compiling num-derive v0.3.3 + Compiling ctor v0.1.22 + Compiling thiserror-impl v1.0.31 + Compiling tracing-attributes v0.1.22 + Compiling zeroize v1.3.0 + Compiling datasize v0.2.10 + Compiling elliptic-curve v0.8.5 + Compiling curve25519-dalek v3.2.1 + Compiling ecdsa v0.10.2 + Compiling ed25519-dalek v1.0.1 + Compiling k256 v0.7.3 + Compiling tracing v0.1.35 + Compiling thiserror v1.0.31 + Compiling hex v0.4.3 + Compiling serde_bytes v0.11.6 + Compiling uuid v0.8.2 + Compiling bincode v1.3.3 + Compiling uint v0.9.3 + Compiling hex-buffer-serde v0.3.0 + Compiling hex-buffer-serde v0.2.2 + Compiling pwasm-utils v0.16.0 + Compiling num v0.4.0 + Compiling casper-types v1.5.0 + Compiling casper-hashing v1.4.3 + Compiling casper-contract v1.4.4 + Compiling casper-execution-engine v2.0.0 + Compiling contract-utils v0.1.0 (/workspace/casper_staking/staking_contract/utils/contract-utils) + Compiling casper-engine-test-support v2.2.0 + Compiling test-env v0.1.0 (/workspace/casper_staking/staking_contract/utils/test-env) + Compiling staking_contract_tests v0.1.0 (/workspace/casper_staking/staking_contract/staking_contract_tests) +warning: unused variable: `staking_contract_package_hash` + --> src/staking_contract_tests.rs:211:9 + | +211 | let staking_contract_package_hash = get_stacking_contract_package_hash(&builder); + | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_staking_contract_package_hash` + | + = note: `#[warn(unused_variables)]` on by default + +warning: unused variable: `staking_contract_package_hash` + --> src/staking_contract_tests.rs:346:9 + | +346 | let staking_contract_package_hash = get_stacking_contract_package_hash(&builder); + | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_staking_contract_package_hash` + +warning: unused variable: `staking_contract_package_hash` + --> src/staking_contract_tests.rs:499:9 + | +499 | let staking_contract_package_hash = get_stacking_contract_package_hash(&builder); + | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_staking_contract_package_hash` + +warning: unused variable: `staking_contract_package_hash` + --> src/staking_contract_tests.rs:648:9 + | +648 | let staking_contract_package_hash = get_stacking_contract_package_hash(&builder); + | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_staking_contract_package_hash` + +warning: unused variable: `staking_contract_package_hash` + --> src/staking_contract_tests.rs:780:9 + | +780 | let staking_contract_package_hash = get_stacking_contract_package_hash(&builder); + | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_staking_contract_package_hash` + +warning: constant `ADDRESS` is never used + --> src/staking_contract_tests.rs:15:7 + | +15 | const ADDRESS: &str = "9e7283533626d0c7d43fa9ca745af20d8dac7fc3bfe03cdfe50d523a2a0f498d"; + | ^^^^^^^ + | + = note: `#[warn(dead_code)]` on by default + +warning: `staking_contract_tests` (lib test) generated 6 warnings (run `cargo fix --lib -p staking_contract_tests --tests` to apply 5 suggestions) + Finished test [unoptimized + debuginfo] target(s) in 1m 07s + Running unittests src/lib.rs (target/debug/deps/staking_contract_tests-3339c36f060eeb27) + +running 6 tests +test staking_contract_tests::test_approve_and_stake ... ok +test staking_contract_tests::test_approve_and_stake_and_staker_reward ... ok +test staking_contract_tests::test_approve_and_stake_and_amount_staked ... ok +test staking_contract_tests::test_approve_and_stake_and_get_current_reward ... ok +test staking_contract_tests::test_approve_and_stake_and_withdraw ... ok +test staking_contract_tests::test_approve_and_stake_and_add_reward ... ok + +test result: ok. 6 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 1.47s + + Doc-tests staking_contract_tests + +running 0 tests + +test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s +``` \ No newline at end of file diff --git a/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/usage_review_blank_screen.png b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/usage_review_blank_screen.png new file mode 100644 index 00000000..2acabb40 Binary files /dev/null and b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/usage_review_blank_screen.png differ diff --git a/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/usage_review_error_on_connect_wallet.png b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/usage_review_error_on_connect_wallet.png new file mode 100644 index 00000000..41af8d30 Binary files /dev/null and b/Reviews/318 - Cross Chain Token Bridge-Staking as a Service Solutions/6 - Build, Deploy, and Internally QA Staking Shell App/assets/usage_review_error_on_connect_wallet.png differ