feat: add Then and Catch methods with Function overloads

Author: miguelmarcondesf

napi-inl.h

@@ -2879,6 +2879,18 @@ inline MaybeOrValue<Promise> Promise::Catch(napi_value onRejected) const {
 #endif
 }
 
+inline MaybeOrValue<Promise> Promise::Then(const Function& onFulfilled) const {
+  return Then(static_cast<napi_value>(onFulfilled));
+}
+
+inline MaybeOrValue<Promise> Promise::Then(const Function& onFulfilled, const Function& onRejected) const {
+  return Then(static_cast<napi_value>(onFulfilled), static_cast<napi_value>(onRejected));
+}
+
+inline MaybeOrValue<Promise> Promise::Catch(const Function& onRejected) const {
+  return Catch(static_cast<napi_value>(onRejected));
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // Buffer<T> class
 ////////////////////////////////////////////////////////////////////////////////

napi.h

@@ -1580,6 +1580,10 @@ class Promise : public Object {
   MaybeOrValue<Promise> Then(napi_value onFulfilled) const;
   MaybeOrValue<Promise> Then(napi_value onFulfilled, napi_value onRejected) const;
   MaybeOrValue<Promise> Catch(napi_value onRejected) const;
+
+  MaybeOrValue<Promise> Then(const Function& onFulfilled) const;
+  MaybeOrValue<Promise> Then(const Function& onFulfilled, const Function& onRejected) const;
+  MaybeOrValue<Promise> Catch(const Function& onRejected) const;
 };
 
 template <typename T>

test/promise.js

@@ -35,4 +35,9 @@ async function test (binding) {
   assert.strictEqual(thenRejected.isPromise, true);
   const rejectedValue = await thenRejected.promise;
   assert.strictEqual(rejectedValue, 'Rejected!');
+
+  const catchMethod = binding.promise.catchMethod(onRejected);
+  assert.strictEqual(catchMethod.isPromise, true);
+  const catchValue = await catchMethod.promise;
+  assert.strictEqual(catchValue, 'Rejected!');
 }