Skip to content

Commit

Permalink
Usage of the crate includes defined there events (#1842)
Browse files Browse the repository at this point in the history
* Usage of the crate includes defined there events

* Inclusion of the event from `event_def_unused` is expected
  • Loading branch information
xgreenx committed Jul 14, 2023
1 parent dd8d861 commit 1a8b157
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 2 deletions.
6 changes: 6 additions & 0 deletions integration-tests/events/event-def-unused/src/lib.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
#![cfg_attr(not(feature = "std"), no_std, no_main)]

#[ink::trait_definition]
pub trait FlipperTrait {
#[ink(message)]
fn flip(&mut self);
}

#[ink::event]
pub struct EventDefUnused {
#[ink(topic)]
Expand Down
15 changes: 13 additions & 2 deletions integration-tests/events/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,15 @@ pub mod events {
}
}

/// Implementing the trait from the `event_def_unused` crate includes all defined
/// events there.
impl event_def_unused::FlipperTrait for Events {
#[ink(message)]
fn flip(&mut self) {
self.value = !self.value;
}
}

#[cfg(test)]
mod tests {
use super::*;
Expand All @@ -85,7 +94,7 @@ pub mod events {
#[test]
fn collects_specs_for_all_linked_and_used_events() {
let event_specs = ink::metadata::collect_events();
assert_eq!(6, event_specs.len());
assert_eq!(7, event_specs.len());

assert!(event_specs
.iter()
Expand All @@ -106,7 +115,9 @@ pub mod events {
.iter()
.any(|evt| evt.label() == &"InlineAnonymousEvent"));

assert!(!event_specs
// The event is not used in the code by being included in the metadata
// because we implement trait form `event_def_unused` crate.
assert!(event_specs
.iter()
.any(|evt| evt.label() == &"EventDefUnused"));
}
Expand Down

0 comments on commit 1a8b157

Please sign in to comment.