react-native-image-stitcher 0.15.2 → 0.16.1

package/cpp/tests/CMakeLists.txt

@@ -1,104 +0,0 @@
-# SPDX-License-Identifier: Apache-2.0
-#
-# cpp/tests/CMakeLists.txt — v0.10.0 audit #9A
-#
-# Standalone Google Test runner for the shared C++ port under `cpp/`.
-# Build + run via:
-#
-#     cmake -S cpp/tests -B build/cpp-tests
-#     cmake --build build/cpp-tests
-#     (cd build/cpp-tests && ctest --output-on-failure)
-#
-# Or, from the repo root: `scripts/run-cpp-tests.sh`.
-#
-# What this DOES test:
-#   - Pure-C++ types in cpp/: `Pose`, `PlaneTransform`,
-#     `StitcherFrameData`, `PixelBufferReader` interface contract.
-#   - `timeBudgetCrossed` — the keyframe gate's OpenCV-free time-budget
-#     force-accept predicate (keyframe_timebudget_test.cpp).
-#   - `warpRoiExceedsGuard` — the OpenCV-free warp-canvas size guard that
-#     triggers the cylindrical-fallback pre-pass (warp_guard_test.cpp).
-#   (The StitcherWorkletRegistry test was removed when that source was
-#    archived in the Phase-3 cleanup — its CMake entries had gone stale
-#    and broke the whole suite's configure step.)
-#
-# What this DOES NOT test yet (deferred to v0.11.0+):
-#   - `KeyframeGate` (the full gate) — depends on OpenCV.  Will need an
-#     OpenCV-aware CMake config (link the same opencv_world the prod
-#     build uses).  NOTE: the OpenCV-free `timeBudgetCrossed` predicate
-#     (the time-budget force-accept boundary logic) IS covered, via
-#     keyframe_timebudget_test.cpp — it's an inline header-only function.
-#   - JSI host-object dispatch — needs a real Hermes runtime.
-#   - Anything in `stitcher.cpp` (uses OpenCV stitching pipeline).
-
-cmake_minimum_required(VERSION 3.20)
-project(stitcher_cpp_tests CXX)
-
-set(CMAKE_CXX_STANDARD 17)
-set(CMAKE_CXX_STANDARD_REQUIRED ON)
-set(CMAKE_CXX_EXTENSIONS OFF)
-
-# Must precede `gtest_discover_tests` — without this the discovered
-# cases aren't registered into a CTestTestfile.cmake at this directory
-# level, and `ctest` reports "No tests were found".
-enable_testing()
-
-# Fetch GoogleTest pinned to v1.14.0.  Pin matches what the AOSP /
-# Android NDK test ecosystem uses today; bumps should be deliberate.
-include(FetchContent)
-FetchContent_Declare(
-  googletest
-  GIT_REPOSITORY https://github.com/google/googletest.git
-  GIT_TAG        v1.14.0
-)
-# Prevent GoogleTest from overriding our compiler/linker options
-# (Windows-only quirk; harmless on macOS/Linux).
-set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)
-FetchContent_MakeAvailable(googletest)
-
-# ─────────────────────────────────────────────────────────────────────
-# Include paths
-# ─────────────────────────────────────────────────────────────────────
-# Order matters: stubs/ comes FIRST so `#include <jsi/jsi.h>` and
-# `#include <react-native-worklets-core/WKTJsiWorklet.h>` resolve to
-# the test-only stubs before any system header.  The production cpp/
-# directory comes after so retailens:: headers (e.g. ar_frame_pose.h,
-# stitcher_frame_data.hpp) are found as the production code expects.
-include_directories(
-  ${CMAKE_CURRENT_SOURCE_DIR}/stubs
-  ${CMAKE_CURRENT_SOURCE_DIR}/..
-)
-
-# ─────────────────────────────────────────────────────────────────────
-# Test executable
-# ─────────────────────────────────────────────────────────────────────
-add_executable(stitcher_cpp_tests
-  # Test sources.  All subjects are header-only (Pose / PlaneTransform,
-  # StitcherFrameData, and the gate's inline `timeBudgetCrossed`
-  # predicate), so no production .cpp needs linking here.
-  ${CMAKE_CURRENT_SOURCE_DIR}/pose_test.cpp
-  ${CMAKE_CURRENT_SOURCE_DIR}/stitcher_frame_data_test.cpp
-  ${CMAKE_CURRENT_SOURCE_DIR}/keyframe_timebudget_test.cpp
-  ${CMAKE_CURRENT_SOURCE_DIR}/warp_guard_test.cpp
-)
-
-target_link_libraries(stitcher_cpp_tests
-  PRIVATE
-    GTest::gtest_main
-)
-
-# Treat warnings as errors in the test build to catch the kind of
-# silent regressions (unused variables, signed/unsigned comparisons,
-# narrowing conversions) that production cross-compilation flags would
-# otherwise suppress.
-if(CMAKE_CXX_COMPILER_ID MATCHES "Clang|GNU")
-  target_compile_options(stitcher_cpp_tests PRIVATE
-    -Wall -Wextra -Wpedantic -Werror
-  )
-endif()
-
-# Register with CTest so `ctest --output-on-failure` picks up the
-# individual TEST() cases (gtest_discover_tests scans the binary at
-# build time, no manual ADD_TEST per case).
-include(GoogleTest)
-gtest_discover_tests(stitcher_cpp_tests)

package/cpp/tests/README.md

@@ -1,86 +0,0 @@
-# `cpp/tests/` — shared C++ unit test suite
-
-v0.10.0 audit `#9A` introduced this directory to give the cross-platform
-shared C++ code under `cpp/` a Google Test harness that runs on the
-developer's host machine (not on a device or emulator).  Pairs with the
-Android-side JUnit suite added in `#11A` (see
-`android/src/test/java/io/imagestitcher/rn/`).
-
-## Run
-
-```sh
-scripts/run-cpp-tests.sh           # configure + build + ctest
-scripts/run-cpp-tests.sh --clean   # nuke build/cpp-tests/ first
-```
-
-Requires `cmake ≥ 3.20` and a C++17 toolchain (the macOS AppleClang
-shipped with Xcode 14+ is fine; Linux GCC 9+ / Clang 10+ also work).
-
-Build artefacts land under `build/cpp-tests/` (gitignored).  Google
-Test is fetched at configure time via CMake `FetchContent` pinned to
-`v1.14.0`; no system-wide install required.
-
-## Scope (v0.10.0)
-
-Covered:
-
-- `Pose`, `PlaneTransform` (POD layout / size / field offsets — pinned
-  to the cross-platform marshalling contract documented in
-  `cpp/ar_frame_pose.h`).
-- `StitcherFrameData` (default-construction invariants the JSI host
-  object's `get()` dispatch relies on).
-- `PixelBufferReader` interface contract (clipping behaviour of
-  `copyTo` — validated via the `FakePixelBufferReader` test helper).
-- `StitcherWorkletRegistry` storage lifecycle: shared-instance,
-  install/uninstall/count/snapshot, snapshot independence, concurrent
-  installs yield unique IDs (16 threads × 32 installs).
-
-Not yet covered (intentional deferrals):
-
-- `KeyframeGate` (`cpp/keyframe_gate.cpp`) — depends on OpenCV
-  (`opencv2/imgproc.hpp`, `opencv2/video.hpp` for `calcOpticalFlowPyrLK`).
-  Linking the production OpenCV xcframework / Android SDK into the
-  host-side test target would balloon CI time and disk usage; the
-  alternative is to land a stripped-down `libopencv-core` host build
-  just for tests.  Deferred — comes with the v0.11.0 cross-platform
-  parity suite (`#2C`).
-- `stitcher.cpp` — uses the full OpenCV stitching pipeline; same
-  reason as above.
-- JSI host-object dispatch (`stitcher_frame_jsi.cpp`,
-  `stitcher_proxy_jsi.cpp`, `stitcher_worklet_dispatch.cpp`) — needs
-  a real Hermes runtime.  The `StitcherWorkletRegistry` tests sidestep
-  this via the `_installEntryForTests` seam + JSI stubs under
-  `stubs/`; the JSI dispatch paths can't be similarly stubbed because
-  they actively call into the runtime.
-
-## How the JSI-dependent registry tests work without a real JSI
-
-`stitcher_worklet_registry.cpp` `#include`s
-`<jsi/jsi.h>` and `<react-native-worklets-core/WKTJsiWorklet.h>` to
-construct `WorkletInvoker` instances from a real JS runtime.  The test
-target sidesteps both by:
-
-1. Putting `cpp/tests/stubs/` first on the compiler's include path so
-   `#include <jsi/jsi.h>` resolves to `stubs/jsi/jsi.h` (which declares
-   `facebook::jsi::Runtime` / `Value` as empty classes — enough for
-   the registry's reference-only usage), and
-   `#include <react-native-worklets-core/WKTJsiWorklet.h>` resolves to
-   `stubs/react-native-worklets-core/WKTJsiWorklet.h` (which declares
-   `RNWorklet::WorkletInvoker` with a no-op constructor).
-2. Calling `_installEntryForTests(nullptr)` instead of the production
-   `install(runtime, value)` path.  The registry stores the
-   `shared_ptr<WorkletInvoker>` but never dereferences it (it only
-   hands it back via `snapshot`), so `nullptr` is safe.
-
-The stubs live exclusively under `cpp/tests/stubs/`; production
-builds never see them.  See `stubs/jsi/jsi.h`'s docstring for the
-guard-rails.
-
-## When NOT to add a test here
-
-- If the test needs a real JSI runtime, real OpenCV operations, or
-  real-device sensor data, it belongs in `android/src/androidTest/`
-  (instrumented), the iOS Swift test target, or the v0.11.0 parity
-  harness — NOT here.
-- If the test verifies TypeScript/JS-side behaviour, it belongs under
-  `src/**/__tests__/` (Jest).

package/cpp/tests/keyframe_timebudget_test.cpp

@@ -1,65 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-//
-// keyframe_timebudget_test.cpp — host unit tests for the pure
-// `retailens::timeBudgetCrossed` predicate (the keyframe gate's
-// time-budget force-accept decision).
-//
-// The full KeyframeGate depends on OpenCV and cannot run in this
-// harness (see CMakeLists.txt).  The time-budget boundary logic was
-// therefore deliberately extracted into an inline, OpenCV-free
-// predicate in keyframe_gate.hpp precisely so it CAN be unit-tested
-// here without linking the gate's OpenCV-dependent .cpp.
-
-#include <gtest/gtest.h>
-
-#include "keyframe_gate.hpp"
-
-using retailens::timeBudgetCrossed;
-
-// intervalMs <= 0 disables the budget entirely (opt-out path).
-TEST(TimeBudgetCrossed, DisabledWhenIntervalNonPositive) {
-  EXPECT_FALSE(timeBudgetCrossed(0.0, 1000, 999999));
-  EXPECT_FALSE(timeBudgetCrossed(-5.0, 1000, 999999));
-}
-
-// Never fires before the first accept (lastAcceptMs sentinel -1),
-// regardless of how large `nowMs` is.
-TEST(TimeBudgetCrossed, NeverFiresBeforeFirstAccept) {
-  EXPECT_FALSE(timeBudgetCrossed(2000.0, -1, 1000000));
-}
-
-// Fires exactly at the boundary (elapsed == interval) and beyond.
-TEST(TimeBudgetCrossed, FiresAtAndAfterBoundary) {
-  EXPECT_TRUE(timeBudgetCrossed(2000.0, 1000, 3000));  // elapsed == 2000
-  EXPECT_TRUE(timeBudgetCrossed(2000.0, 1000, 5000));  // elapsed  > 2000
-}
-
-// Does NOT fire just under the boundary.
-TEST(TimeBudgetCrossed, DoesNotFireJustUnderBoundary) {
-  EXPECT_FALSE(timeBudgetCrossed(2000.0, 1000, 2999));  // elapsed == 1999
-}
-
-// A backwards or equal clock must never fire.  A monotonic source
-// should prevent now < lastAccept, but the predicate must be robust.
-TEST(TimeBudgetCrossed, BackwardsOrEqualClockDoesNotFire) {
-  EXPECT_FALSE(timeBudgetCrossed(2000.0, 5000, 4000));  // now < lastAccept
-  EXPECT_FALSE(timeBudgetCrossed(2000.0, 5000, 5000));  // elapsed 0 < 2000
-}
-
-// Sub-millisecond budget must NOT collapse to "accept every frame":
-// the predicate compares elapsed in double, so a 0.5 ms budget needs
-// ~1 ms elapsed (not 0).  Guards the truncation regression.
-TEST(TimeBudgetCrossed, SubMillisecondBudgetDoesNotAcceptEveryFrame) {
-  EXPECT_FALSE(timeBudgetCrossed(0.5, 1000, 1000));  // elapsed 0.0 < 0.5
-  EXPECT_TRUE(timeBudgetCrossed(0.5, 1000, 1001));   // elapsed 1.0 >= 0.5
-}
-
-// Realistic 2 s budget across a slow pan: a keyframe accepted at t,
-// the next force-accept lands at t + 2000 ms, not before.
-TEST(TimeBudgetCrossed, TwoSecondBudgetTypicalUse) {
-  const int64_t lastAccept = 10000;
-  EXPECT_FALSE(timeBudgetCrossed(2000.0, lastAccept, lastAccept + 1500));
-  EXPECT_FALSE(timeBudgetCrossed(2000.0, lastAccept, lastAccept + 1999));
-  EXPECT_TRUE(timeBudgetCrossed(2000.0, lastAccept, lastAccept + 2000));
-  EXPECT_TRUE(timeBudgetCrossed(2000.0, lastAccept, lastAccept + 2001));
-}

package/cpp/tests/pose_test.cpp

@@ -1,74 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-//
-// pose_test.cpp — v0.10.0 audit #9A
-//
-// Layout / size invariants for the cross-platform POD structs that
-// marshal AR-frame pose data between Swift/Kotlin and shared C++.
-// The Pose / PlaneTransform structs MUST stay binary-compatible
-// across iOS (Swift → C++) and Android (Kotlin → JNI → C++) — any
-// silent field reorder, padding shift, or size change would diverge
-// gate decisions between platforms.
-//
-// These are pinned to the contract in `cpp/ar_frame_pose.h`'s
-// docstring; if the struct shape evolves intentionally, update both
-// the docstring and these tests in the same commit.
-
-#include "ar_frame_pose.h"
-
-#include <gtest/gtest.h>
-
-#include <cstddef>
-#include <type_traits>
-
-using retailens::Pose;
-using retailens::PlaneTransform;
-
-TEST(PoseLayoutTest, IsStandardLayoutPod) {
-  // Required for `memcpy` marshalling and for the iOS Obj-C++ /
-  // Android JNI bridges to write into the struct directly.
-  EXPECT_TRUE(std::is_standard_layout<Pose>::value);
-  EXPECT_TRUE(std::is_trivially_copyable<Pose>::value);
-}
-
-TEST(PoseLayoutTest, SizeMatchesExpectedFields) {
-  // 11 floats (tx, ty, tz, qx, qy, qz, qw, fx, fy, cx, cy) + 2 int32_t
-  // (imageWidth, imageHeight) = 11*4 + 2*4 = 52 bytes.  No padding
-  // expected: every field is 4-byte aligned and the struct contains
-  // only 4-byte primitives.
-  EXPECT_EQ(sizeof(Pose), static_cast<std::size_t>(11 * 4 + 2 * 4));
-}
-
-TEST(PoseLayoutTest, FieldOrderMatchesContract) {
-  // Translation comes before rotation; rotation before intrinsics;
-  // intrinsics before image dimensions.  Swift / Kotlin marshallers
-  // assume this order — flipping any pair silently breaks the
-  // memcpy-based bridge.
-  EXPECT_EQ(offsetof(Pose, tx), 0u);
-  EXPECT_EQ(offsetof(Pose, ty), sizeof(float) * 1);
-  EXPECT_EQ(offsetof(Pose, tz), sizeof(float) * 2);
-  EXPECT_EQ(offsetof(Pose, qx), sizeof(float) * 3);
-  EXPECT_EQ(offsetof(Pose, qy), sizeof(float) * 4);
-  EXPECT_EQ(offsetof(Pose, qz), sizeof(float) * 5);
-  EXPECT_EQ(offsetof(Pose, qw), sizeof(float) * 6);
-  EXPECT_EQ(offsetof(Pose, fx), sizeof(float) * 7);
-  EXPECT_EQ(offsetof(Pose, fy), sizeof(float) * 8);
-  EXPECT_EQ(offsetof(Pose, cx), sizeof(float) * 9);
-  EXPECT_EQ(offsetof(Pose, cy), sizeof(float) * 10);
-  EXPECT_EQ(offsetof(Pose, imageWidth), sizeof(float) * 11);
-  EXPECT_EQ(offsetof(Pose, imageHeight),
-            sizeof(float) * 11 + sizeof(int32_t));
-}
-
-TEST(PlaneTransformLayoutTest, IsStandardLayoutPod) {
-  EXPECT_TRUE(std::is_standard_layout<PlaneTransform>::value);
-  EXPECT_TRUE(std::is_trivially_copyable<PlaneTransform>::value);
-}
-
-TEST(PlaneTransformLayoutTest, SixteenFloatsContiguous) {
-  // The `m[16]` array MUST be a contiguous 64-byte block — both
-  // bridges call `memcpy(planeTransform.m, source, 64)`.  No leading
-  // padding, no field reorder (there's only one field, but pinning the
-  // size catches any accidental wrapper/struct change).
-  EXPECT_EQ(sizeof(PlaneTransform), static_cast<std::size_t>(16 * 4));
-  EXPECT_EQ(offsetof(PlaneTransform, m), 0u);
-}

package/cpp/tests/stitcher_frame_data_test.cpp

@@ -1,132 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-//
-// stitcher_frame_data_test.cpp — v0.10.0 audit #9A
-//
-// Sanity coverage for the `StitcherFrameData` POD payload + the
-// `PixelBufferReader` interface contract.  The shared C++
-// `StitcherFrameData` is constructed by both the iOS Obj-C++ side
-// (`StitcherFrameHostObject.mm`) and the Android JNI side
-// (`stitcher_frame_jni.cpp`); these tests pin the default-construction
-// invariants both sides depend on (e.g. `hasTranslation=false`,
-// `qw=1.0`).
-//
-// The `PixelBufferReader` tests use a fake-buffer implementation
-// (`FakePixelBufferReader`) to validate the `copyTo` clipping
-// behaviour the docstring promises.
-
-#include "stitcher_frame_data.hpp"
-
-#include <gtest/gtest.h>
-
-#include <cstdint>
-#include <cstring>
-#include <memory>
-#include <vector>
-
-using retailens::PixelBufferReader;
-using retailens::StitcherFrameData;
-
-namespace {
-
-/// Minimal fake reader — backs a fixed byte vector.  Used by the
-/// `PixelBufferReader` contract tests below to verify
-/// `copyTo` clips at the smaller of (maxBytes, byteSize).
-class FakePixelBufferReader : public PixelBufferReader {
- public:
-  explicit FakePixelBufferReader(std::vector<uint8_t> bytes)
-      : _bytes(std::move(bytes)) {}
-
-  std::size_t byteSize() const override { return _bytes.size(); }
-
-  std::size_t copyTo(uint8_t* dst, std::size_t maxBytes) override {
-    const std::size_t n =
-        maxBytes < _bytes.size() ? maxBytes : _bytes.size();
-    std::memcpy(dst, _bytes.data(), n);
-    return n;
-  }
-
- private:
-  std::vector<uint8_t> _bytes;
-};
-
-}  // namespace
-
-// ─── StitcherFrameData default-construction invariants ─────────────
-
-TEST(StitcherFrameDataTest, DefaultsAreSafeForJSIDispatch) {
-  // The JSI host object's `get()` dispatch keys off these defaults to
-  // expose `undefined` for unset fields.  In particular:
-  //   - `hasTranslation == false`  →  pose.translation === undefined
-  //   - `arTrackingState.empty()`  →  arTrackingState === undefined
-  //   - `qw == 1.0` with rest zero →  identity rotation (safe default
-  //     for non-AR mode where rotation is unknown)
-  StitcherFrameData d;
-  EXPECT_EQ(d.width, 0);
-  EXPECT_EQ(d.height, 0);
-  EXPECT_TRUE(d.source.empty());
-  EXPECT_TRUE(d.pixelFormat.empty());
-  EXPECT_TRUE(d.orientation.empty());
-  EXPECT_DOUBLE_EQ(d.timestampNs, 0.0);
-  EXPECT_DOUBLE_EQ(d.qx, 0.0);
-  EXPECT_DOUBLE_EQ(d.qy, 0.0);
-  EXPECT_DOUBLE_EQ(d.qz, 0.0);
-  EXPECT_DOUBLE_EQ(d.qw, 1.0);  // identity rotation
-  EXPECT_DOUBLE_EQ(d.tx, 0.0);
-  EXPECT_DOUBLE_EQ(d.ty, 0.0);
-  EXPECT_DOUBLE_EQ(d.tz, 0.0);
-  EXPECT_FALSE(d.hasTranslation);
-  EXPECT_TRUE(d.arTrackingState.empty());
-  EXPECT_EQ(d.pixelReader, nullptr);
-}
-
-TEST(StitcherFrameDataTest, IsCopyable) {
-  // `StitcherFrameData` is documented as "value-typed (cheap to copy;
-  // ~100 bytes)".  Copy needs to deep-copy the strings + bump the
-  // pixelReader shared_ptr refcount.
-  StitcherFrameData a;
-  a.source = "ar";
-  a.width = 1920;
-  a.height = 1080;
-  a.pixelReader = std::make_shared<FakePixelBufferReader>(
-      std::vector<uint8_t>{1, 2, 3});
-
-  StitcherFrameData b = a;
-  EXPECT_EQ(b.source, "ar");
-  EXPECT_EQ(b.width, 1920);
-  EXPECT_EQ(b.height, 1080);
-  ASSERT_NE(b.pixelReader, nullptr);
-  EXPECT_EQ(b.pixelReader.use_count(), 2);  // both a and b hold a ref
-  EXPECT_EQ(b.pixelReader->byteSize(), 3u);
-}
-
-// ─── PixelBufferReader contract ────────────────────────────────────
-
-TEST(PixelBufferReaderTest, CopyToReturnsAllBytesWhenMaxBytesExceedsSize) {
-  FakePixelBufferReader reader({0x11, 0x22, 0x33});
-  uint8_t buf[8] = {0};
-  const std::size_t written = reader.copyTo(buf, sizeof(buf));
-  EXPECT_EQ(written, 3u);
-  EXPECT_EQ(buf[0], 0x11);
-  EXPECT_EQ(buf[1], 0x22);
-  EXPECT_EQ(buf[2], 0x33);
-  EXPECT_EQ(buf[3], 0u);  // untouched tail
-}
-
-TEST(PixelBufferReaderTest, CopyToClipsWhenMaxBytesIsSmaller) {
-  // Contract per stitcher_frame_data.hpp: "Implementations MUST handle
-  // the case where maxBytes < byteSize() (clip silently)."
-  FakePixelBufferReader reader({0xAA, 0xBB, 0xCC, 0xDD});
-  uint8_t buf[2] = {0};
-  const std::size_t written = reader.copyTo(buf, sizeof(buf));
-  EXPECT_EQ(written, 2u);
-  EXPECT_EQ(buf[0], 0xAA);
-  EXPECT_EQ(buf[1], 0xBB);
-}
-
-TEST(PixelBufferReaderTest, CopyToWithZeroMaxBytesReturnsZero) {
-  FakePixelBufferReader reader({0x01, 0x02, 0x03});
-  uint8_t dummy = 0xFF;
-  const std::size_t written = reader.copyTo(&dummy, 0);
-  EXPECT_EQ(written, 0u);
-  EXPECT_EQ(dummy, 0xFF);  // dst untouched when maxBytes == 0
-}

package/cpp/tests/stubs/jsi/jsi.h

@@ -1,33 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-//
-// jsi.h — TEST-ONLY stub of facebook::jsi types.
-//
-// The real jsi.h ships with React Native and pulls in a large surface
-// area (Runtime, Value, Object, Function, HostObject, HostFunction,
-// Array, ArrayBuffer, PropNameID, …) along with the build infra to
-// link them.  For pure-C++ unit tests that exercise data-structure
-// invariants of code that REFERENCES jsi types but never CALLS into
-// them (e.g. `StitcherWorkletRegistry` storing a `shared_ptr` and
-// forwarding `Runtime&` to a constructor stub), we only need the
-// types to be NAMED so headers compile.
-//
-// Pattern: this stub is placed first on the test target's include
-// path so `#include <jsi/jsi.h>` resolves here instead of to RN's
-// real header.  Production builds NEVER see this file — it lives
-// only under `cpp/tests/stubs/`, which is referenced exclusively by
-// `cpp/tests/CMakeLists.txt`.
-//
-// Tests that need to actually CONSTRUCT or CALL into JSI types should
-// not use this stub — they should run against a real JSI runtime (a
-// future v0.11.0+ test target that links Hermes).
-
-#pragma once
-
-namespace facebook {
-namespace jsi {
-
-class Runtime {};
-class Value {};
-
-}  // namespace jsi
-}  // namespace facebook

package/cpp/tests/stubs/react-native-worklets-core/WKTJsiWorklet.h

@@ -1,34 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-//
-// WKTJsiWorklet.h — TEST-ONLY stub of RNWorklet::WorkletInvoker.
-//
-// `cpp/stitcher_worklet_registry.cpp` constructs a
-// `std::make_shared<RNWorklet::WorkletInvoker>(runtime, value)` inside
-// `install`.  The real WorkletInvoker (from react-native-worklets-core)
-// captures the worklet's source / closure / runtime affinity and is
-// non-trivial to stand up in a unit-test context.
-//
-// This stub provides JUST the symbols needed for the registry to
-// compile and link.  The constructor and destructor are no-ops; calling
-// methods on a stub invoker is undefined behaviour, but the registry
-// itself never does (it only stores the shared_ptr and hands it out
-// via `snapshot`).  Tests construct entries directly via
-// `_installEntryForTests(nullptr)` to avoid even the trivial
-// allocation.
-//
-// See cpp/tests/stubs/jsi/jsi.h for the parallel stub of facebook::jsi.
-
-#pragma once
-
-#include <jsi/jsi.h>
-
-namespace RNWorklet {
-
-class WorkletInvoker {
- public:
-  WorkletInvoker(facebook::jsi::Runtime& /*runtime*/,
-                 const facebook::jsi::Value& /*workletValue*/) {}
-  ~WorkletInvoker() = default;
-};
-
-}  // namespace RNWorklet

package/cpp/tests/warp_guard_test.cpp

@@ -1,48 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-/**
- * Unit tests for the warp-canvas size guard (cpp/warp_guard.hpp).
- *
- * The guard decides when a warp ROI is "degenerate" — the trigger for
- * both the cylindrical-fallback pre-pass and the in-loop final safety net.
- * The cases that matter: normal ROIs pass, non-positive dims fail, the
- * 100 MP boundary is inclusive, the real observed divergence (8171×12336)
- * is caught, and a ROI whose int32 area would overflow is still caught.
- */
-#include "warp_guard.hpp"
-
-#include <gtest/gtest.h>
-
-using retailens::warpRoiExceedsGuard;
-
-TEST(WarpGuard, AcceptsNormalRoi) {
-  EXPECT_FALSE(warpRoiExceedsGuard(4000, 2000));  // 8 MP
-  EXPECT_FALSE(warpRoiExceedsGuard(1, 1));
-}
-
-TEST(WarpGuard, RejectsNonPositiveDims) {
-  EXPECT_TRUE(warpRoiExceedsGuard(0, 1000));
-  EXPECT_TRUE(warpRoiExceedsGuard(1000, 0));
-  EXPECT_TRUE(warpRoiExceedsGuard(-5, 1000));
-  EXPECT_TRUE(warpRoiExceedsGuard(1000, -5));
-}
-
-TEST(WarpGuard, BoundaryIsInclusive) {
-  EXPECT_FALSE(warpRoiExceedsGuard(100000, 1000));  // exactly 100 MP — allowed
-  EXPECT_TRUE(warpRoiExceedsGuard(100000, 1001));   // 100.1 MP — over
-}
-
-TEST(WarpGuard, RejectsTheObservedDivergence) {
-  // 8171×12336 = 100.8 MP — the exact STITCH_CAMERA_PARAMS_FAIL canvas.
-  EXPECT_TRUE(warpRoiExceedsGuard(8171, 12336));
-}
-
-TEST(WarpGuard, RejectsInt32OverflowingRoi) {
-  // 65536×65536 = 2^32; an int32 area would wrap to 0 and slip past the
-  // guard. The int64 area math catches it.
-  EXPECT_TRUE(warpRoiExceedsGuard(65536, 65536));
-}
-
-TEST(WarpGuard, HonoursCustomThreshold) {
-  EXPECT_FALSE(warpRoiExceedsGuard(1000, 1000, 2'000'000));  // 1 MP < 2 MP
-  EXPECT_TRUE(warpRoiExceedsGuard(2000, 1000, 1'000'000));   // 2 MP > 1 MP
-}