sqlite,doc: change backup arg name from destination to path

Author: geeksilva97

doc/api/sqlite.md

@@ -538,18 +538,18 @@ exception.
 | `TEXT`    | {string}                   |
 | `BLOB`    | {TypedArray} or {DataView} |
 
-## `sqlite.backup(sourceDb, destination[, options])`
+## `sqlite.backup(sourceDb, path[, options])`
 
 <!-- YAML
 added: v23.8.0
 changes:
   - version: REPLACEME
     pr-url: https://github.com/nodejs/node/pull/56991
-    description: The `destination` argument now supports Buffer and URL objects.
+    description: The `path` argument now supports Buffer and URL objects.
 -->
 
 * `sourceDb` {DatabaseSync} The database to backup. The source database must be open.
-* `destination` {string | Buffer | URL} The path where the backup will be created. If the file already exists,
+* `path` {string | Buffer | URL} The path where the backup will be created. If the file already exists,
   the contents will be overwritten.
 * `options` {Object} Optional configuration for the backup. The
   following properties are supported:

src/node_sqlite.cc

@@ -1048,7 +1048,7 @@ void Backup(const FunctionCallbackInfo<Value>& args) {
   ASSIGN_OR_RETURN_UNWRAP(&db, args[0].As<Object>());
   THROW_AND_RETURN_ON_BAD_STATE(env, !db->IsOpen(), "database is not open");
   std::optional<std::string> dest_path =
-      ValidateDatabasePath(env, args[1], "destination");
+      ValidateDatabasePath(env, args[1], "path");
   if (!dest_path.has_value()) {
     return;
   }

test/parallel/test-sqlite-backup.mjs

@@ -50,32 +50,32 @@ describe('backup()', () => {
       backup(database);
     }, {
       code: 'ERR_INVALID_ARG_TYPE',
-      message: 'The "destination" argument must be a string, Uint8Array, or URL without null bytes.'
+      message: 'The "path" argument must be a string, Uint8Array, or URL without null bytes.'
     });
 
     t.assert.throws(() => {
       backup(database, {});
     }, {
       code: 'ERR_INVALID_ARG_TYPE',
-      message: 'The "destination" argument must be a string, Uint8Array, or URL without null bytes.'
+      message: 'The "path" argument must be a string, Uint8Array, or URL without null bytes.'
     });
   });
 
-  test('throws if the database destination contains null bytes', (t) => {
+  test('throws if the database path contains null bytes', (t) => {
     const database = makeSourceDb();
 
     t.assert.throws(() => {
       backup(database, Buffer.from('l\0cation'));
     }, {
       code: 'ERR_INVALID_ARG_TYPE',
-      message: 'The "destination" argument must be a string, Uint8Array, or URL without null bytes.'
+      message: 'The "path" argument must be a string, Uint8Array, or URL without null bytes.'
     });
 
     t.assert.throws(() => {
       backup(database, 'l\0cation');
     }, {
       code: 'ERR_INVALID_ARG_TYPE',
-      message: 'The "destination" argument must be a string, Uint8Array, or URL without null bytes.'
+      message: 'The "path" argument must be a string, Uint8Array, or URL without null bytes.'
     });
   });
 
@@ -297,7 +297,7 @@ test('backup fails when source db is invalid', async (t) => {
   });
 });
 
-test('backup fails when destination cannot be opened', async (t) => {
+test('backup fails when path cannot be opened', async (t) => {
   const database = makeSourceDb();
 
   await t.assert.rejects(async () => {

test/parallel/test-sqlite.js

@@ -143,7 +143,6 @@ test('URL is supported as the database path', (t) => {
   );
 });
 
-
 test('URL query params are supported', (t) => {
   const url = pathToFileURL(nextDb());
   const db = new DatabaseSync(url);