Revert "chore: unconcluent tests with SafeArrayIterator"

This reverts commit 916d489.

Author: tpoisseau

lib/sqlite.js

@@ -1,6 +1,5 @@
 'use strict';
 const {
-  SafeArrayIterator,
   globalThis,
 } = primordials;
 
@@ -9,13 +8,8 @@ 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 new SafeArrayIterator(statementIterate.apply(this, arguments));
-  return statementIterate.apply(this, arguments);
-  return Iterator.from(
-    statementIterate.apply(this, arguments)
-    // new SafeArrayIterator(statementIterate.apply(this, arguments))
-  );
+  return Iterator.from(statementIterate.apply(this, arguments));
 };

src/node_sqlite.cc

@@ -515,10 +515,6 @@ 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);
@@ -629,15 +625,11 @@ 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();