fix: address code review comments on imagemagick sample

Author: angelcaamal

functions/imagemagick/index.js

@@ -63,7 +63,7 @@ exports.blurOffensiveImages = async event => {
 // Blurs the given file using ImageMagick, and uploads it to another bucket.
 const blurImage = async (file, blurredBucketName) => {
   const tempLocalPath = `/tmp/${path.parse(file.name).base}`;
-  const tempLocalBlurredPath = `/tmp/blurred-${path.parse(file.name).base}`; // 1. Declarar la variable
+  const tempLocalBlurredPath = `/tmp/blurred-${path.parse(file.name).base}`;
 
   // Download file from bucket.
   try {
@@ -98,6 +98,6 @@ const blurImage = async (file, blurredBucketName) => {
 
   // Delete the temporary file.
   await fs.unlink(tempLocalPath);
-  return fs.unlink(tempLocalBlurredPath);
+  return await fs.unlink(tempLocalBlurredPath);
 };
 // [END functions_imagemagick_blur]

functions/imagemagick/test/index.test.js

@@ -15,7 +15,7 @@
 'use strict';
 
 const assert = require('assert');
-const {execSync, spawn} = require('child_process');
+const {spawn} = require('child_process');
 const {Storage} = require('@google-cloud/storage');
 const sinon = require('sinon');
 const {request} = require('gaxios');
@@ -62,11 +62,6 @@ async function startFF(port) {
   return {ffProc, ffProcHandler};
 }
 
-// ImageMagick is available by default in Cloud Run Functions environments
-// https://cloud.google.com/functions/1stgendocs/tutorials/imagemagick-1st-gen.md#importing_dependencies
-// Manually install it for testing only.
-//execSync('sudo apt-get install imagemagick -y');
-
 describe('functions/imagemagick tests', () => {
   before(async () => {
     let exists;
@@ -178,7 +173,7 @@ describe('functions/imagemagick tests', () => {
 
   after(async () => {
     try {
-      // await blurredBucket.file(testFiles.offensive).delete();
+      await blurredBucket.file(testFiles.offensive).delete();
     } catch (err) {
       console.log('Error deleting uploaded file:', err.message);
     }