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

feat: local integration tests #70

Merged
merged 93 commits into from
May 23, 2024

++

403f904
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

feat: local integration tests #70

++
403f904
Select commit
Loading
Failed to load commit list.
GitHub Actions / clippy succeeded May 23, 2024 in 0s

reviewdog [clippy] report

reported by reviewdog 🐶

Findings (0)
Filtered Findings (15)

README.md|1 col 3| warning: item in documentation is missing backticks
--> contracts/src/../../README.md:1:3
|
1 | # OpenZeppelin Contracts for Stylus
| ^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown
note: the lint level is defined here
--> contracts/src/lib.rs:3:22
|
3 | #![warn(clippy::all, clippy::pedantic)]
| ^^^^^^^^^^^^^^^^
= note: #[warn(clippy::doc_markdown)] implied by #[warn(clippy::pedantic)]
help: try
|
1 | # OpenZeppelin Contracts for Stylus
| ~~~~~~~~~~~~~~
README.md|14 col 1| warning: item in documentation is missing backticks
--> contracts/src/../../README.md:14:1
|
14 | OpenZeppelin Contracts for Stylus exists thanks to its contributors. There are
| ^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown
help: try
|
14 | OpenZeppelin Contracts for Stylus exists thanks to its contributors. There are
| ~~~~~~~~~~~~~~
README.md|29 col 1| warning: item in documentation is missing backticks
--> contracts/src/../../README.md:29:1
|
29 | OpenZeppelin Contracts for Stylus is released under the MIT License.
| ^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown
help: try
|
29 | OpenZeppelin Contracts for Stylus is released under the MIT License.
| ~~~~~~~~~~~~~~
contracts/src/erc20/mod.rs|3 col 30| warning: item in documentation is missing backticks
--> contracts/src/erc20/mod.rs:3:30
|
3 | //! We have followed general OpenZeppelin Contracts guidelines: functions
| ^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown
help: try
|
3 | //! We have followed general OpenZeppelin Contracts guidelines: functions
| ~~~~~~~~~~~~~~
contracts/src/erc721/mod.rs|307 col 15| warning: this argument is passed by value, but not consumed in the function body
--> contracts/src/erc721/mod.rs:307:15
|
307 | data: Bytes,
| ^^^^^ help: consider taking a reference instead: &Bytes
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_value
= note: #[warn(clippy::needless_pass_by_value)] implied by #[warn(clippy::pedantic)]
contracts/src/erc721/mod.rs|473 col 5| warning: this method could have a #[must_use] attribute
--> contracts/src/erc721/mod.rs:473:5
|
473 | / pub fn is_approved_for_all(
474 | | &self,
475 | | owner: Address,
476 | | operator: Address,
477 | | ) -> bool {
| |_^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#must_use_candidate
= note: #[warn(clippy::must_use_candidate)] implied by #[warn(clippy::pedantic)]
help: add the attribute
|
473 ~ #[must_use] pub fn is_approved_for_all(
474 + &self,
475 + owner: Address,
476 + operator: Address,
477 ~ ) -> bool {
|
contracts/src/erc721/mod.rs|497 col 5| warning: this method could have a #[must_use] attribute
--> contracts/src/erc721/mod.rs:497:5
|
497 | pub fn owner_of_inner(&self, token_id: U256) -> Address {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: add the attribute: #[must_use] pub fn _owner_of_inner(&self, token_id: U256) -> Address
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#must_use_candidate
contracts/src/erc721/mod.rs|508 col 5| warning: this method could have a #[must_use] attribute
--> contracts/src/erc721/mod.rs:508:5
|
508 | pub fn get_approved_inner(&self, token_id: U256) -> Address {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: add the attribute: #[must_use] pub fn _get_approved_inner(&self, token_id: U256) -> Address
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#must_use_candidate
contracts/src/erc721/mod.rs|524 col 5| warning: this method could have a #[must_use] attribute
--> contracts/src/erc721/mod.rs:524:5
|
524 | / pub fn is_authorized(
525 | | &self,
526 | | owner: Address,
527 | | spender: Address,
528 | | token_id: U256,
529 | | ) -> bool {
| |
^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#must_use_candidate
help: add the attribute
|
524 ~ #[must_use] pub fn is_authorized(
525 + &self,
526 + owner: Address,
527 + spender: Address,
528 + token_id: U256,
529 ~ ) -> bool {
|
contracts/src/erc721/mod.rs|737 col 15| warning: this argument is passed by value, but not consumed in the function body
--> contracts/src/erc721/mod.rs:737:15
|
737 | data: Bytes,
| ^^^^^ help: consider taking a reference instead: &Bytes
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_value
contracts/src/erc721/mod.rs|883 col 15| warning: this argument is passed by value, but not consumed in the function body
--> contracts/src/erc721/mod.rs:883:15
|
883 | data: Bytes,
| ^^^^^ help: consider taking a reference instead: &Bytes
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_value
contracts/src/erc721/mod.rs|131 col 1| warning: missing documentation for a struct
--> contracts/src/erc721/mod.rs:131:1
|
131 | / sol_interface! {
132 | | /// ERC-721 token receiver interface.
133 | | ///
134 | | /// Interface for any contract that wants to support safeTransfers
135 | | /// from ERC-721 asset contracts.
136 | | interface IERC721Receiver {
| |_____________________________^
|
note: the lint level is defined here
--> contracts/src/lib.rs:2:9
|
2 | #![warn(missing_docs, unreachable_pub, rust_2021_compatibility)]
| ^^^^^^^^^^^^
= note: this warning originates in the macro sol_interface (in Nightly builds, run with -Z macro-backtrace for more info)
contracts/src/erc721/mod.rs|131 col 1| warning: missing documentation for a struct field
--> contracts/src/erc721/mod.rs:131:1
|
131 | / sol_interface! {
132 | | /// ERC-721 token receiver interface.
133 | | ///
134 | | /// Interface for any contract that wants to support safeTransfers
... |
151 | | }
152 | | }
| |
^
|
= note: this warning originates in the macro sol_interface (in Nightly builds, run with -Z macro-backtrace for more info)
contracts/src/erc721/mod.rs|131 col 1| warning: missing documentation for an associated function
--> contracts/src/erc721/mod.rs:131:1
|
131 | / sol_interface! {
132 | | /// ERC-721 token receiver interface.
133 | | ///
134 | | /// Interface for any contract that wants to support safeTransfers
... |
151 | | }
152 | | }
| |
^
|
= note: this warning originates in the macro sol_interface (in Nightly builds, run with -Z macro-backtrace for more info)
contracts/src/erc721/mod.rs|131 col 1| warning: missing documentation for a method
--> contracts/src/erc721/mod.rs:131:1
|
131 | / sol_interface! {
132 | | /// ERC-721 token receiver interface.
133 | | ///
134 | | /// Interface for any contract that wants to support safeTransfers
... |
151 | | }
152 | | }
| |
^
|
= note: this warning originates in the macro sol_interface (in Nightly builds, run with -Z macro-backtrace for more info)