Iterating Android tests

CHANGELOG.md

@@ -13,6 +13,7 @@
 * Fix a data race that could cause a reading thread to read from a no-longer-valid memory mapping ([realm/realm-core#6411](https://github.com/realm/realm-core/pull/6411), since v11.3.0-rc.0).
 * Fixed an issue that could cause a crash when performing count() on an undefined query. ([realm/realm-core#6443](https://github.com/realm/realm-core/issues/6443), since v12.0.0-alpha.2)
 * Added missing implementation of `User.state` and changed the `UserState` enum values to use pascal case to conform to the v11 implementation (except for `UserState.Active` that we now deprecate in favor of `UserState.LoggedIn`). [#5686](https://github.com/realm/realm-js/issues/5686)
+* Getting the `indexOf` a missing value will no longer return `4294967295` instead of `-1` and the `Set#has` will no longer return `true` when missing. Caused by an incorrect conversion of `size_t` to `Number` on x86 (32bit) architectures. ([#5746](https://github.com/realm/realm-js/pull/5746), since 12.0.0-alpha.0)
 
 ### Compatibility
 * React Native >= v0.71.0

packages/bindgen/src/realm_js_helpers.h

@@ -368,5 +368,14 @@ auto schedulerWrapBlockingFunction(F&& f)
     };
 }
 
+/**
+ * Helps with correct handling of -1/npos for count_t
+ */
+template <typename T>
+auto asSigned(T num)
+{
+    return std::make_signed_t<T>(num);
+}
+
 } // namespace
 } // namespace realm::js

packages/bindgen/src/templates/jsi.ts

@@ -227,8 +227,8 @@ function convertPrimToJsi(addon: JsiAddon, type: string, expr: string): string {
       return `jsi::Value(double(${expr}))`;
 
     case "count_t":
-      // NOTE: using int64_t cast here to get -1.0 for size_t(-1), aka npos.
-      return `jsi::Value(double(int64_t(${expr})))`;
+      // NOTE: using asSigned() here to get -1.0 for size_t(-1), aka npos.
+      return `jsi::Value(double(asSigned(${expr})))`;
 
     case "int64_t":
       return `bigIntFromI64(_env, ${expr})`;
@@ -317,8 +317,8 @@ function convertPrimFromJsi(addon: JsiAddon, type: string, expr: string): string
       return `int32_t((${expr}).asNumber())`;
 
     case "count_t":
-      // NOTE: using int64_t here is important to correctly handle -1.0 aka npos.
-      return `size_t(int64_t((${expr}).asNumber()))`;
+      // NOTE: using size_t here is important to correctly handle -1.0 aka npos.
+      return `size_t(ptrdiff_t((${expr}).asNumber()))`;
 
     case "int64_t":
       return `bigIntToI64(_env, jsi::Value(_env, ${expr}))`;

packages/bindgen/src/templates/node.ts

@@ -180,8 +180,8 @@ function convertPrimToNode(addon: NodeAddon, type: string, expr: string): string
       return `Napi::Number::New(${env}, ${expr})`;
 
     case "count_t":
-      // NOTE: using int64_t cast here to get -1.0 for size_t(-1), aka npos.
-      return `Napi::Number::New(${env}, int64_t(${expr}))`;
+      // NOTE: using asSigned() here to get -1.0 for size_t(-1), aka npos.
+      return `Napi::Number::New(${env}, asSigned(${expr}))`;
 
     case "int64_t":
     case "uint64_t":