tests: Add further testing

Co-authored-by: Jacob Rothstein <[email protected]>
Signed-off-by: John Nunley <[email protected]>

Author: notgull

Diff for: .github/workflows/ci.yml

@@ -13,6 +13,7 @@ on:
 
 env:
   CARGO_INCREMENTAL: 0
+  CARGO_NET_GIT_FETCH_WITH_CLI: true
   CARGO_NET_RETRY: 10
   CARGO_TERM_COLOR: always
   RUST_BACKTRACE: 1
@@ -26,42 +27,80 @@ defaults:
 
 jobs:
   test:
-    runs-on: ${{ matrix.os }}
+    runs-on: ubuntu-latest
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-latest]
-        rust: [nightly, beta, stable]
+        include:
+          - rust: stable
+          - rust: beta
+          - rust: nightly
+          - rust: nightly
+            target: i686-unknown-linux-gnu
     steps:
       - uses: actions/checkout@v4
       - name: Install Rust
         run: rustup update ${{ matrix.rust }} && rustup default ${{ matrix.rust }}
+      - name: Install cross-compilation tools
+        uses: taiki-e/setup-cross-toolchain-action@v1
+        with:
+          target: ${{ matrix.target }}
+        if: matrix.target != ''
       - run: cargo build --all --all-features --all-targets
+      - run: cargo test --all
+      - run: cargo test --all --release
+      - run: cargo test --no-default-features --tests
+      - run: cargo test --no-default-features --tests --release
+      - name: Install cargo-hack
+        uses: taiki-e/install-action@cargo-hack
+      - run: rustup target add thumbv7m-none-eabi
       - name: Run cargo check (without dev-dependencies to catch missing feature flags)
-        if: startsWith(matrix.rust, 'nightly')
-        run: cargo check -Z features=dev_dep
-      - run: cargo test
+        run: cargo hack build --all --no-dev-deps
+      - run: cargo hack build --all --target thumbv7m-none-eabi --no-default-features --no-dev-deps
+      - run: cargo hack build --target thumbv7m-none-eabi --no-default-features --no-dev-deps --features portable-atomic
+      - name: Install wasm-pack
+        uses: taiki-e/install-action@wasm-pack
+      - run: wasm-pack test --node
+      - run: wasm-pack test --node --no-default-features
+      - run: wasm-pack test --node --no-default-features --features portable-atomic
+      - name: Clone some dependent crates
+        run: |
+          git clone https://github.com/smol-rs/event-listener-strategy.git
+          git clone https://github.com/smol-rs/async-channel.git
+          git clone https://github.com/smol-rs/async-lock.git
+      - name: Patch dependent crates
+        run: |
+          echo '[patch.crates-io]' >> event-listener-strategy/Cargo.toml
+          echo 'event-listener = { path = ".." }' >> event-listener-strategy/Cargo.toml
+          echo '[patch.crates-io]' >> async-channel/Cargo.toml
+          echo 'event-listener = { path = ".." }' >> async-channel/Cargo.toml
+          echo 'event-listener-strategy = { path = "../event-listener-strategy" }' >> async-channel/Cargo.toml
+          echo '[patch.crates-io]' >> async-lock/Cargo.toml
+          echo 'event-listener = { path = ".." }' >> async-lock/Cargo.toml
+          echo 'event-listener-strategy = { path = "../event-listener-strategy" }' >> async-lock/Cargo.toml
+          echo 'async-channel = { path = "../async-channel" }' >> async-lock/Cargo.toml
+      - name: Test dependent crates
+        run: |
+           cargo test --manifest-path=event-listener-strategy/Cargo.toml
+           cargo test --manifest-path=async-channel/Cargo.toml
+           cargo test --manifest-path=async-lock/Cargo.toml
 
   msrv:
     runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        # When updating this, the reminder to update the minimum supported
-        # Rust version in Cargo.toml.
-        rust: ['1.39']
     steps:
       - uses: actions/checkout@v4
-      - name: Install Rust
-        run: rustup update ${{ matrix.rust }} && rustup default ${{ matrix.rust }}
-      - run: cargo build
+      - name: Install cargo-hack
+        uses: taiki-e/install-action@cargo-hack
+      - run: cargo hack build --all --rust-version
+      - run: cargo hack build --all --no-default-features --rust-version
 
   clippy:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
       - name: Install Rust
         run: rustup update stable
-      - run: cargo clippy --all-features --all-targets
+      - run: cargo clippy --all --all-features --all-targets
 
   fmt:
     runs-on: ubuntu-latest
@@ -77,10 +116,12 @@ jobs:
       - uses: actions/checkout@v4
       - name: Install Rust
         run: rustup toolchain install nightly --component miri && rustup default nightly
-      - run: cargo miri test
-        env:
-          MIRIFLAGS: -Zmiri-strict-provenance -Zmiri-symbolic-alignment-check -Zmiri-disable-isolation
-          RUSTFLAGS: ${{ env.RUSTFLAGS }} -Z randomize-layout
+      - run: |
+          echo "MIRIFLAGS=-Zmiri-strict-provenance -Zmiri-symbolic-alignment-check -Zmiri-disable-isolation" >>"${GITHUB_ENV}"
+          echo "RUSTFLAGS=${RUSTFLAGS} -Z randomize-layout" >>"${GITHUB_ENV}"
+      - run: cargo miri test --all
+      - run: cargo miri test --no-default-features --tests
+      - run: cargo miri test --no-default-features --features portable-atomic --tests
 
   security_audit:
     permissions:
@@ -89,8 +130,17 @@ jobs:
       issues: write
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       # https://github.com/rustsec/audit-check/issues/2
       - uses: rustsec/audit-check@master
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
+
+  loom:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Install Rust
+        run: rustup update stable
+      - name: Loom tests
+        run: RUSTFLAGS="--cfg=loom" cargo test --release --test loom --features loom  

Diff for: Cargo.toml

@@ -6,7 +6,7 @@ name = "event-listener"
 version = "5.3.0"
 authors = ["Stjepan Glavina <[email protected]>"]
 edition = "2021"
-rust-version = "1.56"
+rust-version = "1.63"
 description = "Notify async tasks or threads"
 license = "Apache-2.0 OR MIT"
 repository = "https://github.com/smol-rs/event-listener"
@@ -21,7 +21,7 @@ portable-atomic = ["portable-atomic-crate", "portable-atomic-util"]
 
 [dependencies]
 portable-atomic-crate = { version = "1.6.0", package = "portable-atomic", optional = true }
-portable-atomic-util = { version = "0.1.5", optional = true }
+portable-atomic-util = { version = "0.1.5", optional = true, features = ["alloc"] }
 
 [dev-dependencies]
 futures = { version = "0.3", default-features = false, features = ["std"] }
@@ -32,6 +32,7 @@ waker-fn = "1"
 [[bench]]
 name = "bench"
 harness = false
+required-features = ["std"]
 
 [lib]
 bench = false

Diff for: benches/bench.rs

@@ -1,21 +1,18 @@
 use criterion::{criterion_group, criterion_main, Criterion};
 use event_listener::{Event, Listener};
+use std::iter;
 
 const COUNT: usize = 8000;
 
 fn bench_events(c: &mut Criterion) {
     c.bench_function("notify_and_wait", |b| {
         let ev = Event::new();
+        let mut handles = Vec::with_capacity(COUNT);
         b.iter(|| {
-            let mut handles = Vec::with_capacity(COUNT);
-
-            for _ in 0..COUNT {
-                handles.push(ev.listen());
-            }
-
+            handles.extend(iter::repeat_with(|| ev.listen()).take(COUNT));
             ev.notify(COUNT);
 
-            for handle in handles {
+            for handle in handles.drain(..) {
                 handle.wait();
             }
         });

Diff for: examples/mutex.rs

@@ -90,9 +90,7 @@ impl<T> Mutex<T> {
                 }
                 Some(l) => {
                     // Wait until a notification is received.
-                    if l.wait_deadline(deadline).is_none() {
-                        return None;
-                    }
+                    l.wait_deadline(deadline)?;
                 }
             }
         }

Diff for: src/lib.rs

@@ -75,8 +75,8 @@
 )]
 #![warn(missing_docs, missing_debug_implementations, rust_2018_idioms)]
 
-use loom::atomic::{self, AtomicPtr, AtomicUsize, Ordering};
-use loom::Arc;
+use crate::loom::atomic::{self, AtomicPtr, AtomicUsize, Ordering};
+use crate::loom::Arc;
 use notify::{GenericNotify, Internal, NotificationPrivate};
 
 use std::cell::{Cell, UnsafeCell};
@@ -91,7 +91,6 @@ use std::sync::{Mutex, MutexGuard, TryLockError};
 use std::task::{Context, Poll, Waker};
 use std::thread::{self, Thread};
 use std::time::{Duration, Instant};
-use std::usize;
 
 mod notify;
 pub use notify::{IntoNotification, Notification};
@@ -385,24 +384,6 @@ impl Event {
     pub fn notify_additional_relaxed(&self, n: usize) -> usize {
         self.notify(n.additional().relaxed())
     }
-}
-
-impl<T> Event<T> {
-    /// Creates a new [`Event`] with a tag.
-    ///
-    /// # Examples
-    ///
-    /// ```
-    /// use event_listener::Event;
-    ///
-    /// let event = Event::<usize>::with_tag();
-    /// ```
-    #[inline]
-    pub const fn with_tag() -> Event<T> {
-        Event {
-            inner: AtomicPtr::new(ptr::null_mut()),
-        }
-    }
 
     /// Return the listener count by acquiring a lock.
     ///
@@ -430,15 +411,32 @@ impl<T> Event<T> {
     /// drop(listener2);
     /// assert_eq!(event.total_listeners(), 0);        
     /// ```
-    #[cfg(feature = "std")]
     #[inline]
     pub fn total_listeners(&self) -> usize {
         if let Some(inner) = self.try_inner() {
-            inner.total_listeners()
+            inner.lock().len
         } else {
             0
         }
     }
+}
+
+impl<T> Event<T> {
+    /// Creates a new [`Event`] with a tag.
+    ///
+    /// # Examples
+    ///
+    /// ```
+    /// use event_listener::Event;
+    ///
+    /// let event = Event::<usize>::with_tag();
+    /// ```
+    #[inline]
+    pub const fn with_tag() -> Event<T> {
+        Event {
+            inner: AtomicPtr::new(ptr::null_mut()),
+        }
+    }
 
     /// Returns a guard listening for a notification.
     ///
@@ -504,7 +502,7 @@ impl<T> Event<T> {
         if let Some(inner) = self.try_inner() {
             let limit = if notify.is_additional(Internal::new()) {
                 // Notify if there is at least one unnotified listener.
-                std::usize::MAX
+                usize::MAX
             } else {
                 // Notify if there is at least one unnotified listener and the number of notified
                 // listeners is less than `n`.
@@ -535,7 +533,8 @@ impl<T> Event<T> {
     /// ```
     #[inline]
     pub fn is_notified(&self) -> bool {
-        self.try_inner().map_or(false, |inner| inner.notified.load(Ordering::Acquire) > 0)
+        self.try_inner()
+            .map_or(false, |inner| inner.notified.load(Ordering::Acquire) > 0)
     }
 
     /// Returns a reference to the inner state if it was initialized.
@@ -614,11 +613,13 @@ impl<T> Drop for Event<T> {
 impl fmt::Debug for Event {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         let inner = match self.try_inner() {
-            None => return f
-                .debug_tuple("Event")
-                .field(&format_args!("<uninitialized>"))
-                .finish(),
-            Some(inner) => inner
+            None => {
+                return f
+                    .debug_tuple("Event")
+                    .field(&format_args!("<uninitialized>"))
+                    .finish()
+            }
+            Some(inner) => inner,
         };
 
         let guard = match inner.list.try_lock() {