Added logging

pnp · Jul 18, 2023 · ff6ad48 · ff6ad48
1 parent f7ba773
commit ff6ad48
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/src/m365/spo/commands/file/file-move.ts b/src/m365/spo/commands/file/file-move.ts
@@ -179,7 +179,7 @@ class SpoFileMoveCommand extends SpoCommand {
     const targetFileServerRelativeUrl: string = `${urlUtil.getServerRelativePath(contextResponse.WebFullUrl, targetUrl)}/${filename}`;
 
     try {
-      await spo.removeFile(contextResponse.WebFullUrl, targetFileServerRelativeUrl, true);
+      await spo.removeFile(contextResponse.WebFullUrl, targetFileServerRelativeUrl, true, logger, this.verbose);
     }
     catch (err: any) {
       if (err !== undefined && err.message !== undefined && err.message.includes('does not exist')) {

diff --git a/src/utils/spo.spec.ts b/src/utils/spo.spec.ts
@@ -1132,7 +1132,7 @@ describe('utils/spo', () => {
     });
 
 
-    await spo.removeFile('https://contoso.sharepoint.com', 'SharedDocuments/Document.docx', true);
+    await spo.removeFile('https://contoso.sharepoint.com', 'SharedDocuments/Document.docx', true, logger, true);
     assert(postStub.called);
   });
 });
diff --git a/src/utils/spo.ts b/src/utils/spo.ts
@@ -837,7 +837,19 @@ export const spo = {
     return roledefinition;
   },
 
-  async removeFile(webUrl: string, url: string, recycle?: boolean): Promise<void> {
+  /**
+  * removes a file by url
+  * @param webUrl The web url
+  * @param url The url of the file
+  * @param url If file should be recycled or not
+  * @param logger The logger object
+  * @param verbose Set for verbose logging
+  */
+  async removeFile(webUrl: string, url: string, recycle?: boolean, logger?: Logger, verbose?: boolean): Promise<void> {
+    if (verbose && logger) {
+      logger.logToStderr(`Removing file with url ${url}`);
+    }
+
     const serverRelativePath = urlUtil.getServerRelativePath(webUrl, url);
     let requestUrl = `${webUrl}/_api/web/GetFileByServerRelativeUrl('${formatting.encodeQueryParameter(serverRelativePath)}')`;