refactor(samples): Fix documentation, typos, and function names

samples/manifest-request.js

@@ -40,12 +40,12 @@ async function main(
   // const sourceAgentPoolName = 'projects/my-project/agentPools/transfer_service_default'
 
   // The root directory path on the source filesystem
-  // const rootDirectory = '/directory/to/transfer/source',
+  // const rootDirectory = '/directory/to/transfer/source'
 
   // The ID of the GCS bucket to transfer data to
   // const gcsSinkBucket = 'my-sink-bucket'
 
-  // Transfer manifest location. Must be a
+  // Transfer manifest location. Must be a `gs:` URL
   // const manifestLocation = 'gs://my-bucket/sample_manifest.csv'
 
   // Creates a client
@@ -54,7 +54,7 @@ async function main(
   /**
    * Creates a request to transfer from the local file system to the sink bucket
    */
-  async function transferDirectory() {
+  async function transferViaManifest() {
     const createRequest = {
       transferJob: {
         projectId,
@@ -87,7 +87,7 @@ async function main(
     );
   }
 
-  transferDirectory();
+  transferViaManifest();
   // [END storagetransfer_manifest_request]
 }
 

samples/posix-download.js

@@ -45,16 +45,16 @@ async function main(
   // An optional path on the Google Cloud Storage bucket to download from
   // const gcsSourcePath = 'foo/bar/'
 
-  // The root directory path on the source filesystem
-  // const rootDirectory = '/directory/to/transfer/source',
+  // The root directory path on the destination filesystem
+  // const rootDirectory = '/directory/to/transfer/sink'
 
   // Creates a client
   const client = new StorageTransferServiceClient();
 
   /**
    * Creates a request to transfer from the local file system to the sink bucket
    */
-  async function transferDirectory() {
+  async function downloadFromGCS() {
     const createRequest = {
       transferJob: {
         projectId,
@@ -87,7 +87,7 @@ async function main(
     );
   }
 
-  transferDirectory();
+  downloadFromGCS();
   // [END storagetransfer_download_to_posix]
 }
 

samples/posix-request.js

@@ -39,7 +39,7 @@ async function main(
   // const sourceAgentPoolName = 'projects/my-project/agentPools/transfer_service_default'
 
   // The root directory path on the source filesystem
-  // const rootDirectory = '/directory/to/transfer/source',
+  // const rootDirectory = '/directory/to/transfer/source'
 
   // The ID of the GCS bucket to transfer data to
   // const gcsSinkBucket = 'my-sink-bucket'

samples/posix-to-posix-request.js

@@ -44,7 +44,7 @@ async function main(
   // const sinkAgentPoolName = 'projects/my-project/agentPools/transfer_service_default'
 
   // The root directory path on the source filesystem
-  // const rootDirectory = '/directory/to/transfer/source',
+  // const rootDirectory = '/directory/to/transfer/source'
 
   // The root directory path on the sink filesystem
   // const destinationDirectory = '/directory/to/transfer/sink'

samples/test/manifest-request.test.js

@@ -72,7 +72,7 @@ describe('manifest-request', () => {
     await fs.rmdir(rootDirectory);
   });
 
-  it('should create a transfer job with a manifest from POSIX to GCS', async () => {
+  it('should create a transfer job using a manifest from POSIX to GCS', async () => {
     const output = await runSample('manifest-request', [
       projectId,
       sourceAgentPoolName,