fix(NODE-6328): pass through optional options in txn APIs

Author: nbbeeken

src/legacy_wrappers/session.js

@@ -7,12 +7,28 @@ Object.defineProperty(module.exports, '__esModule', { value: true });
 
 module.exports.makeLegacyClientSession = function (baseClass) {
   class LegacyClientSession extends baseClass {
-    abortTransaction(callback) {
-      return maybeCallback(super.abortTransaction(), callback);
+    abortTransaction(options, callback) {
+      callback =
+        typeof callback === 'function'
+          ? callback
+          : typeof options === 'function'
+            ? options
+            : undefined;
+      options = typeof options !== 'function' ? options : undefined;
+
+      return maybeCallback(super.abortTransaction(options), callback);
     }
 
-    commitTransaction(callback) {
-      return maybeCallback(super.commitTransaction(), callback);
+    commitTransaction(options, callback) {
+      callback =
+        typeof callback === 'function'
+          ? callback
+          : typeof options === 'function'
+            ? options
+            : undefined;
+      options = typeof options !== 'function' ? options : undefined;
+
+      return maybeCallback(super.commitTransaction(options), callback);
     }
 
     endSession(options, callback) {

test/tools/api.js

@@ -57,8 +57,8 @@ const api = [
   { className: 'ChangeStream', method: 'next', returnType: 'Promise<TChange>' },
   { className: 'ChangeStream', method: 'tryNext', returnType: 'Promise<Document | null>' },
 
-  { className: 'ClientSession', method: 'abortTransaction', returnType: 'Promise<Document>' },
-  { className: 'ClientSession', method: 'commitTransaction', returnType: 'Promise<Document>' },
+  { className: 'ClientSession', method: 'abortTransaction', returnType: 'Promise<Document>', possibleCallbackPositions: [1, 2] },
+  { className: 'ClientSession', method: 'commitTransaction', returnType: 'Promise<Document>', possibleCallbackPositions: [1, 2] },
   { className: 'ClientSession', method: 'endSession', returnType: 'Promise<void>' },
   { className: 'ClientSession', method: 'withTransaction', returnType: 'Promise<void>', notAsync: true },