Improve error handling in updateDocuments to ensure exceptions are properly thrown or handled

Author: ArnabChatterjee20k

src/Database/Database.php

@@ -4456,7 +4456,7 @@ public function updateDocuments(
                 try {
                     $onNext && $onNext($doc);
                 } catch (Exception $e) {
-                    $onError && $onError($e);
+                    $onError ? $onError($e) : throw $e;
                 }
                 $modified++;
             }

tests/e2e/Adapter/Scopes/DocumentTests.php

@@ -3550,6 +3550,23 @@ public function testUpdateDocuments(): void
 
         // Test Update half of the documents
         $results = [];
+        try {
+            $count = $database->updateDocuments($collection, new Document([
+                'string' => 'text📝 updated',
+            ]), [
+                Query::lessThan('integer', -1),
+            ], onNext: function ($doc) use (&$results) {
+                $results[] = $doc;
+                throw new Exception("Error thrown to test update doesn't stopped and error is caught");
+            });
+        } catch (Exception $e) {
+            if ($e instanceof Exception) {
+                $this->assertInstanceOf(Exception::class, $e);
+                $this->assertEquals("Error thrown to test that deletion doesn't stop and error is caught", $e->getMessage());
+            } else {
+                $this->fail("Caught value is not an Exception.");
+            }
+        }
         $count = $database->updateDocuments($collection, new Document([
             'string' => 'text📝 updated',
         ]), [