Port bindings-doctests.sh into CI (#5155)

# Description of Changes

Part of #4970.

Doing this part in a separate PR because it turns out that this test was
failing, so this PR also includes the fixes.

# API and ABI breaking changes

None

# Expected complexity level and risk

1

# Testing

CI passes (but only with the "code" changes).

---------

Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>

Author: bfops

crates/bindings/README.md

@@ -618,7 +618,7 @@ fn players_for_level(ctx: &AnonymousViewContext) -> Vec<PlayerAndLevel> {
         .player_level()
         .level()
         .filter(2u64)
-        .map(|player| {
+        .filter_map(|player| {
             ctx.db
                 .player()
                 .id()

crates/bindings/bindings-doctests.sh

@@ -365,9 +365,9 @@ pub use spacetimedb_bindings_macro::settings;
 ///     // The following line would panic, since we use `insert` rather than `try_insert`.
 ///     // let result = ctx.db.country().insert(Country { code: "CN".into(), national_bird: "Blue Magpie".into() });
 ///
-///     // If we wanted to *update* the row for Australia, we can use the `update` method of `UniqueIndex`.
-///     // The following line will succeed:
-///     ctx.db.country().code().update(Country {
+///     // If we wanted to replace the row for Australia, we can delete it and insert the new row.
+///     assert!(ctx.db.country().code().delete("AU".to_string()));
+///     ctx.db.country().insert(Country {
 ///         code: "AU".into(), national_bird: "Australian Emu".into()
 ///     });
 /// }
@@ -1118,7 +1118,7 @@ impl ReducerContext {
     /// #[reducer]
     /// fn generate_uuid_v4(ctx: &ReducerContext) -> Result<(), Box<dyn std::error::Error>> {
     ///     let uuid = ctx.new_uuid_v4()?;
-    ///     log::info!(uuid);
+    ///     log::info!("{uuid}");
     ///     Ok(())
     /// }
     /// # }
@@ -1140,7 +1140,7 @@ impl ReducerContext {
     /// #[reducer]
     /// fn generate_uuid_v7(ctx: &ReducerContext) -> Result<(), Box<dyn std::error::Error>> {
     ///     let uuid = ctx.new_uuid_v7()?;
-    ///     log::info!(uuid);
+    ///     log::info!("{uuid}");
     ///     Ok(())
     /// }
     /// # }

crates/bindings/src/lib.rs

@@ -30,8 +30,7 @@ impl ReducerContext {
     ///
     /// ```no_run
     /// # #[cfg(target_arch = "wasm32")] mod demo {
-    /// use spacetimedb::{reducer, ReducerContext};
-    /// use rand::Rng;
+    /// use spacetimedb::{rand::Rng, reducer, ReducerContext};
     ///
     /// #[spacetimedb::reducer]
     /// fn rng_demo(ctx: &spacetimedb::ReducerContext) {
@@ -40,7 +39,7 @@ impl ReducerContext {
     ///
     ///     // Or, cache locally for reuse:
     ///     let mut rng = ctx.rng();
-    ///     let floats: Vec<f32> = rng.sample_iter(rand::distributions::Standard).collect();
+    ///     let floats: Vec<f32> = rng.sample_iter(spacetimedb::rand::distributions::Standard).collect();
     /// }
     /// # }
     /// ```

crates/bindings/src/rng.rs

@@ -516,7 +516,7 @@ pub trait IndexIsPointed: Index {}
 ///
 /// ```no_run
 /// # #[cfg(target_arch = "wasm32")] mod demo {
-/// use spacetimedb::{table, PointIndex, ReducerContext, DbContext};
+/// use spacetimedb::{table, DbContext, PointIndex, ReducerContext};
 ///
 /// #[table(accessor = user,
 ///     index(accessor = dogs_and_name, hash(columns = [dogs, name])))]
@@ -537,7 +537,7 @@ pub trait IndexIsPointed: Index {}
 ///
 /// ```no_run
 /// # #[cfg(target_arch = "wasm32")] mod demo {
-/// use spacetimedb::{table, PointIndex, ReducerContext, DbContext};
+/// use spacetimedb::{table, DbContext, RangedIndex, ReducerContext};
 ///
 /// #[table(accessor = user)]
 /// struct User {
@@ -548,7 +548,7 @@ pub trait IndexIsPointed: Index {}
 /// }
 ///
 /// fn demo(ctx: &ReducerContext) {
-///     let by_dogs: PointIndex<_, (u64,), _> = ctx.db().user().dogs();
+///     let by_dogs: RangedIndex<_, (u64,), _> = ctx.db().user().dogs();
 /// }
 /// # }
 /// ```

crates/bindings/src/table.rs

@@ -616,6 +616,10 @@ fn main() -> Result<()> {
                 .dir("crates/bindings-csharp")
                 .run()?;
             cmd!("pnpm", "lint").run()?;
+            cmd!("cargo", "test", "--doc", "--target", "wasm32-unknown-unknown")
+                .dir("crates/bindings")
+                .run()?;
+            cmd!("cargo", "test", "--doc").dir("crates/bindings").run()?;
             // `bindings` is the only crate we care strongly about documenting,
             // since we link to its docs.rs from our website.
             // We won't pass `--no-deps`, though,