chore: unconcluent tests with SafeArrayIterator

Author: tpoisseau

lib/sqlite.js

@@ -1,5 +1,6 @@
 'use strict';
 const {
+  SafeArrayIterator,
   globalThis,
 } = primordials;
 
@@ -8,8 +9,13 @@ const { emitExperimentalWarning } = require('internal/util');
 emitExperimentalWarning('SQLite');
 module.exports = internalBinding('sqlite');
 
-const { Iterator } = globalThis;
+const {Iterator} = globalThis;
 const statementIterate = module.exports.StatementSync.prototype.iterate;
 module.exports.StatementSync.prototype.iterate = function iterate() {
-  return Iterator.from(statementIterate.apply(this, arguments));
+  return new SafeArrayIterator(statementIterate.apply(this, arguments));
+  return statementIterate.apply(this, arguments);
+  return Iterator.from(
+    statementIterate.apply(this, arguments)
+    // new SafeArrayIterator(statementIterate.apply(this, arguments))
+  );
 };

src/node_sqlite.cc

@@ -474,6 +474,10 @@ struct IterateCaptureContext {
   StatementSync* stmt;
 };
 
+void IteratorFunc(const v8::FunctionCallbackInfo<v8::Value>& info) {
+  info.GetReturnValue().Set(info.Holder());
+}
+
 void StatementSync::IterateReturnCallback(
     const FunctionCallbackInfo<Value>& args) {
   Environment* env = Environment::GetCurrent(args);
@@ -584,11 +588,15 @@ void StatementSync::Iterate(const FunctionCallbackInfo<Value>& args) {
       v8::FunctionTemplate::New(isolate,
                                 StatementSync::IterateReturnCallback,
                                 External::New(isolate, captureContext));
+  v8::Local<v8::FunctionTemplate> iteratorFuncTemplate =
+      v8::FunctionTemplate::New(isolate, IteratorFunc);
 
   iterableIteratorTemplate->Set(String::NewFromUtf8Literal(isolate, "next"),
                                 nextFuncTemplate);
   iterableIteratorTemplate->Set(String::NewFromUtf8Literal(isolate, "return"),
                                 returnFuncTemplate);
+  iterableIteratorTemplate->Set(v8::Symbol::GetIterator(isolate),
+                                iteratorFuncTemplate);
 
   auto iterableIterator =
       iterableIteratorTemplate->NewInstance(context).ToLocalChecked();