fix: make tests check no_std, fix clippy/fmt

.github/workflows/rust.yml

@@ -24,7 +24,7 @@ jobs:
         run: MIRIFLAGS=-Zmiri-strict-provenance cargo miri test
       - name: Test (gecko-ffi) with Miri
         run: MIRIFLAGS=-Zmiri-strict-provenance cargo miri test --features=gecko-ffi
-      
+
   build:
     runs-on: ubuntu-latest
     steps:
@@ -39,7 +39,9 @@ jobs:
       run: cargo test --features=serde --verbose
     - name: Run tests (gecko-ffi)
       run: cargo test --tests --features=gecko-ffi --verbose
-
+    - name: Run tests (no_std)
+      run: cargo test --tests --no-default-features --verbose
+
   fmt:
     runs-on: ubuntu-latest
     steps:
@@ -54,8 +56,8 @@ jobs:
         with:
           command: fmt
           args: --all -- --check
-          
-          
+
+
   clippy:
     runs-on: ubuntu-latest
     steps:
@@ -72,8 +74,8 @@ jobs:
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
           args: --workspace --tests --examples
-          
-          
+
+
   docs:
     runs-on: ubuntu-latest
     env:

src/lib.rs

@@ -142,13 +142,12 @@
 //! [pinned]: https://doc.rust-lang.org/std/pin/index.html
 
 #![cfg_attr(not(feature = "std"), no_std)]
-
 #![allow(clippy::comparison_chain, clippy::missing_safety_doc)]
 
 extern crate alloc;
 
-use alloc::{vec::Vec, boxed::Box};
 use alloc::alloc::*;
+use alloc::{boxed::Box, vec::Vec};
 use core::borrow::*;
 use core::cmp::*;
 use core::convert::TryFrom;
@@ -861,7 +860,7 @@ impl<T> ThinVec<T> {
     /// Note: Because this shifts over the remaining elements, it has a
     /// worst-case performance of *O*(*n*). If you don't need the order of elements
     /// to be preserved, use [`swap_remove`] instead. If you'd like to remove
-    /// elements from the beginning of the `ThinVec`, consider using `std::collections::VecDeque`.
+    /// elements from the beginning of the `ThinVec`, consider using `core::collections::VecDeque`.
     ///
     /// [`swap_remove`]: ThinVec::swap_remove
     ///
@@ -1971,7 +1970,7 @@ impl<T> FromIterator<T> for ThinVec<T> {
     #[inline]
     fn from_iter<I: IntoIterator<Item = T>>(iter: I) -> ThinVec<T> {
         let mut vec = ThinVec::new();
-        vec.extend(iter.into_iter());
+        vec.extend(iter);
         vec
     }
 }
@@ -2043,7 +2042,7 @@ impl<T> From<Box<[T]>> for ThinVec<T> {
 }
 
 impl<T> From<Vec<T>> for ThinVec<T> {
-    /// Convert a `std::Vec` into a `ThinVec`.
+    /// Convert a `core::Vec` into a `ThinVec`.
     ///
     /// **NOTE:** this must reallocate to change the layout!
     ///
@@ -2061,7 +2060,7 @@ impl<T> From<Vec<T>> for ThinVec<T> {
 }
 
 impl<T> From<ThinVec<T>> for Vec<T> {
-    /// Convert a `ThinVec` into a `std::Vec`.
+    /// Convert a `ThinVec` into a `core::Vec`.
     ///
     /// **NOTE:** this must reallocate to change the layout!
     ///
@@ -2640,10 +2639,11 @@ impl std::io::Write for ThinVec<u8> {
 #[cfg(test)]
 mod tests {
     use super::{ThinVec, MAX_CAP};
+    use crate::alloc::{string::ToString, vec};
 
     #[test]
     fn test_size_of() {
-        use std::mem::size_of;
+        use core::mem::size_of;
         assert_eq!(size_of::<ThinVec<u8>>(), size_of::<&u8>());
 
         assert_eq!(size_of::<Option<ThinVec<u8>>>(), size_of::<&u8>());
@@ -3016,8 +3016,12 @@ mod std_tests {
     #![allow(clippy::reversed_empty_ranges)]
 
     use super::*;
-    use std::mem::size_of;
-    use std::usize;
+    use crate::alloc::{
+        format,
+        string::{String, ToString},
+    };
+    use core::mem::size_of;
+    use core::usize;
 
     struct DropCounter<'a> {
         count: &'a mut u32,
@@ -3675,7 +3679,7 @@ mod std_tests {
     fn test_splice_forget() {
         let mut v = thin_vec![1, 2, 3, 4, 5];
         let a = [10, 11, 12];
-        ::std::mem::forget(v.splice(2..4, a.iter().cloned()));
+        ::core::mem::forget(v.splice(2..4, a.iter().cloned()));
         assert_eq!(v, &[1, 2]);
     }
 
@@ -4194,16 +4198,16 @@ mod std_tests {
                 let v: ThinVec<$typename> = ThinVec::with_capacity(1 /* ensure allocation */);
                 let head_ptr: *mut $typename = v.data_raw();
                 assert_eq!(
-                    head_ptr as usize % std::mem::align_of::<$typename>(),
+                    head_ptr as usize % core::mem::align_of::<$typename>(),
                     0,
                     "expected Header::data<{}> to be aligned",
                     stringify!($typename)
                 );
             }};
         }
 
-        const HEADER_SIZE: usize = std::mem::size_of::<Header>();
-        assert_eq!(2 * std::mem::size_of::<usize>(), HEADER_SIZE);
+        const HEADER_SIZE: usize = core::mem::size_of::<Header>();
+        assert_eq!(2 * core::mem::size_of::<usize>(), HEADER_SIZE);
 
         #[repr(C, align(128))]
         struct Funky<T>(T);