@alfresco/aca-playwright-shared 7.4.1 → 7.5.0-24770823677

package/fesm2022/alfresco-aca-playwright-shared.mjs

@@ -1,4 +1,4 @@
-import { AlfrescoApi, SitesApi as SitesApi$1, UploadApi, NodesApi as NodesApi$1, PeopleApi, AdminUsersApi, AdminTenantsApi, GroupsApi, QueriesApi as QueriesApi$1, SearchApi as SearchApi$1, SecurityGroupsApi, SecurityMarksApi, SharedlinksApi, FavoritesApi, TrashcanApi as TrashcanApi$1, CommentsApi, CategoriesApi as CategoriesApi$1, TagsApi as TagsApi$1, ResultSetPaging, SharedLinkPaging, FavoritePaging, Site, SiteMemberEntry } from '@alfresco/js-api';
+import { AlfrescoApi, SitesApi as SitesApi$1, UploadApi, NodesApi as NodesApi$1, PeopleApi, AdminUsersApi, AdminTenantsApi, GroupsApi, QueriesApi as QueriesApi$1, SearchApi as SearchApi$1, SecurityGroupsApi, SecurityMarksApi, ContentClient, SharedlinksApi, FavoritesApi, TrashcanApi as TrashcanApi$1, CommentsApi, CategoriesApi as CategoriesApi$1, TagsApi as TagsApi$1, ResultSetPaging, SharedLinkPaging, FavoritePaging, Site, SiteMemberEntry } from '@alfresco/js-api';
 import { expect, chromium, devices, test as test$1 } from '@playwright/test';
 import * as path from 'path';
 import { resolve } from 'path';
@@ -4657,9 +4657,6 @@ const config = {
 class ApiClientFactory {
     constructor() {
         this.alfrescoApi = new AlfrescoApi(config);
-    }
-    async setUpAcaBackend(userName, password) {
-        await this.login(userName, password);
         this.sites = new SitesApi$1(this.alfrescoApi);
         this.upload = new UploadApi(this.alfrescoApi);
         this.nodes = new NodesApi$1(this.alfrescoApi);
@@ -4671,6 +4668,7 @@ class ApiClientFactory {
         this.search = new SearchApi$1(this.alfrescoApi);
         this.securityGroupsApi = new SecurityGroupsApi(this.alfrescoApi);
         this.securityMarksApi = new SecurityMarksApi(this.alfrescoApi);
+        this.contentClient = new ContentClient(config, '/alfresco');
         this.share = new SharedlinksApi(this.alfrescoApi);
         this.favorites = new FavoritesApi(this.alfrescoApi);
         this.trashCan = new TrashcanApi$1(this.alfrescoApi);
@@ -4678,6 +4676,9 @@ class ApiClientFactory {
         this.queriesApi = new QueriesApi$1(this.alfrescoApi);
         this.categoriesApi = new CategoriesApi$1(this.alfrescoApi);
         this.tagsApi = new TagsApi$1(this.alfrescoApi);
+    }
+    async setUpAcaBackend(userName, password) {
+        await this.login(userName, password);
         return this;
     }
     async tearDown() {
@@ -4685,26 +4686,30 @@ class ApiClientFactory {
         return this;
     }
     async login(userName, password) {
-        const predefinedUserKey = Object.keys(users).find((user) => user === userName || users[user].username === userName);
+        const predefinedUserKey = Object.keys(users).find((userAlias) => userAlias === userName || users[userAlias].username === userName);
         const userToLog = predefinedUserKey ? users[predefinedUserKey] : undefined;
-        let e;
         const user = userToLog?.username ?? userName;
         const userPassword = userToLog?.password ?? password;
+        if (!userPassword) {
+            throw new Error(`[API Client Factory] No password provided for user ${user}`);
+        }
         try {
-            e = await this.alfrescoApi.login(user, userPassword);
+            await this.alfrescoApi.login(user, userPassword);
         }
         catch (error) {
-            console.error(`[API Client Factory] Log in user ${user} failed ${e}`);
+            logger.error(`[API Client Factory] Log in user ${user} failed ${error}`);
             throw error;
         }
     }
     async loginUser(user) {
-        let e;
+        if (!user.username || !user.password) {
+            throw new Error(`[API Client Factory] No username or password provided`);
+        }
         try {
-            e = await this.alfrescoApi.login(user.username, user.password);
+            await this.alfrescoApi.login(user.username, user.password);
         }
         catch (error) {
-            console.error(`[API Client Factory] Log in user ${user.username} failed ${e}`);
+            logger.error(`[API Client Factory] Log in user ${user.username} failed ${error}`);
             throw error;
         }
     }
@@ -4712,20 +4717,20 @@ class ApiClientFactory {
         const person = new Person(user);
         const peopleApi = new PeopleApi(this.alfrescoApi);
         try {
-            return peopleApi.createPerson(person);
+            return await peopleApi.createPerson(person);
         }
         catch (error) {
-            console.error('[API Client Factory] createUser failed : ', error);
+            logger.error(`[API Client Factory] createUser failed: ${error}`);
             return null;
         }
     }
     async changePassword(username, newPassword) {
         const peopleApi = new PeopleApi(this.alfrescoApi);
         try {
-            return peopleApi.updatePerson(username, { password: newPassword });
+            return await peopleApi.updatePerson(username, { password: newPassword });
         }
         catch (error) {
-            console.error('[API Client Factory] changePassword failed : ', error);
+            logger.error(`[API Client Factory] changePassword failed: ${error}`);
             return null;
         }
     }
@@ -5119,7 +5124,9 @@ class FileActionsApi {
         }
         catch {
             const actual = await apiCall();
-            throw new Error(`waitForNodes: Timed out waiting for "${searchTerm}" — expected ${data.expect} nodes, found ${actual}`);
+            const message = `waitForNodes: Timed out waiting for "${searchTerm}" — expected ${data.expect} nodes, found ${actual}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async queryNodesSearchHighlight(searchTerm) {
@@ -5164,16 +5171,16 @@ class FileActionsApi {
                 return (await this.queryNodesSearchHighlight(searchTerm)).list.pagination.totalItems;
             }
             catch (error) {
-                console.warn(`queryNodesSearchHighlight failed for "${searchTerm}":`, error);
+                logger.warn(`queryNodesSearchHighlight failed for "${searchTerm}": ${error}`);
                 return 0;
             }
         };
         try {
             await waitForApi(apiCall, predicate, 30, 2500);
-            console.log(`waitForNodesSearchHighlight: Found ${data.expect} nodes with search term "${searchTerm}"`);
+            logger.log(`waitForNodesSearchHighlight: Found ${data.expect} nodes with search term "${searchTerm}"`);
         }
         catch (error) {
-            console.error(`Error: ${error}`);
+            logger.error(`Error: ${error}`);
         }
     }
     async updateNodeContent(nodeId, content, majorVersion = true, comment, newName) {
@@ -5183,10 +5190,10 @@ class FileActionsApi {
                 comment: comment,
                 name: newName
             };
-            return this.apiService.nodes.updateNodeContent(nodeId, content, opts);
+            return await this.apiService.nodes.updateNodeContent(nodeId, content, opts);
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.updateNodeContent.name}`, error);
+            logger.error(`${this.constructor.name} ${this.updateNodeContent.name}: ${error}`);
             return Promise.reject(error);
         }
     }
@@ -5230,7 +5237,7 @@ class SharedLinksApi {
                 nodeId: id,
                 expiresAt: expireDate
             };
-            return this.apiService.share.createSharedLink(data);
+            return await this.apiService.share.createSharedLink(data);
         }
         catch (error) {
             return null;
@@ -5242,12 +5249,14 @@ class SharedLinksApi {
             if (ids && ids.length > 0) {
                 for (const id of ids) {
                     const sharedLink = await this.shareFileById(id, expireDate);
-                    sharedLinks.push(sharedLink);
+                    if (sharedLink) {
+                        sharedLinks.push(sharedLink);
+                    }
                 }
             }
         }
         catch (error) {
-            console.error(`SharedLinksApi shareFilesByIds : catch : `, error);
+            logger.error(`SharedLinksApi shareFilesByIds : catch : ${error}`);
         }
         return sharedLinks;
     }
@@ -5256,71 +5265,64 @@ class SharedLinksApi {
             const opts = {
                 maxItems
             };
-            return this.apiService.share.listSharedLinks(opts);
+            return await this.apiService.share.listSharedLinks(opts);
         }
         catch (error) {
-            console.error(`SharedLinksApi getSharedLinks : catch : `, error);
+            logger.error(`SharedLinksApi getSharedLinks : catch : ${error}`);
             return new SharedLinkPaging();
         }
     }
-    async waitForFilesToBeShared(filesIds) {
+    async waitForFilesToBeShared(fileIds) {
         try {
             const sharedFile = async () => {
-                const sharedFiles = (await this.getSharedLinks()).list.entries.map((link) => link.entry.nodeId);
-                const foundItems = filesIds.every((id) => sharedFiles.includes(id));
-                if (foundItems) {
-                    return Promise.resolve(foundItems);
-                }
-                else {
-                    return Promise.reject(foundItems);
+                const sharedFiles = (await this.getSharedLinks()).list?.entries?.map((link) => link.entry.nodeId) ?? [];
+                const foundItems = fileIds.every((id) => sharedFiles.includes(id));
+                if (!foundItems) {
+                    const message = 'Not all files are shared yet';
+                    logger.error(message);
+                    throw new Error(message);
                 }
             };
-            return await Utils.retryCall(sharedFile);
+            await Utils.retryCall(sharedFile);
         }
         catch (error) {
-            console.error(`SharedLinksApi waitForFilesToBeShared :  catch : ${error}`);
-            console.error(`\tWait timeout reached waiting for files to be shared`);
+            logger.error(`SharedLinksApi waitForFilesToBeShared : catch : ${error} - Wait timeout reached waiting for files to be shared`);
         }
     }
     async getSharedIdOfNode(fileId) {
-        try {
-            const sharedLinksEntries = (await this.getSharedLinks())?.list.entries;
-            const found = sharedLinksEntries.find((sharedLink) => sharedLink.entry.nodeId === fileId);
-            return found?.entry.id;
-        }
-        catch (error) {
-            console.error(`SharedLinksApi getSharedIdOfNode : catch : `, error);
-            return null;
+        const sharedLinksEntries = (await this.getSharedLinks())?.list?.entries ?? [];
+        const found = sharedLinksEntries.find((sharedLink) => sharedLink.entry.nodeId === fileId);
+        if (!found?.entry.id) {
+            const message = `SharedLinksApi getSharedIdOfNode: no shared link found for node ${fileId}`;
+            logger.error(message);
+            throw new Error(message);
         }
+        return found.entry.id;
     }
     async unshareFileById(fileId) {
         try {
             const sharedId = await this.getSharedIdOfNode(fileId);
-            return this.apiService.share.deleteSharedLink(sharedId);
+            await this.apiService.share.deleteSharedLink(sharedId);
         }
         catch (error) {
-            console.error(`SharedLinksApi unshareFileById : catch : `, error);
+            logger.error(`SharedLinksApi unshareFileById : catch : ${error}`);
         }
     }
-    async waitForFilesToNotBeShared(filesIds) {
+    async waitForFilesToNotBeShared(fileIds) {
         try {
             const sharedFile = async () => {
-                const sharedFiles = (await this.getSharedLinks()).list.entries.map((link) => link.entry.nodeId);
-                const foundItems = filesIds.some((id) => {
-                    return sharedFiles.includes(id);
-                });
+                const sharedFiles = (await this.getSharedLinks()).list?.entries?.map((link) => link.entry.nodeId) ?? [];
+                const foundItems = fileIds.some((id) => sharedFiles.includes(id));
                 if (foundItems) {
-                    return Promise.reject(foundItems);
-                }
-                else {
-                    return Promise.resolve(foundItems);
+                    const message = 'Some files are still shared';
+                    logger.error(message);
+                    throw new Error(message);
                 }
             };
-            return await Utils.retryCall(sharedFile);
+            await Utils.retryCall(sharedFile);
         }
         catch (error) {
-            console.error(`SharedLinksApi waitForFilesToNotBeShared :  catch : ${error}`);
-            console.error(`\tWait timeout reached waiting for files to no longer be shared`);
+            logger.error(`SharedLinksApi waitForFilesToNotBeShared : catch : ${error} - Wait timeout reached waiting for files to no longer be shared`);
         }
     }
 }
@@ -5446,7 +5448,7 @@ class FavoritesPageApi {
             }
         }
         catch (error) {
-            console.error('FavoritesApi: removeFavoritesByIds failed ', error);
+            logger.error(`FavoritesApi: removeFavoritesByIds failed: ${error}`);
         }
     }
 }
@@ -5555,7 +5557,9 @@ class NodesApi {
             return await this.createNode('cm:folder', name, parentId, title, description, null, author, true, aspectNames);
         }
         catch (error) {
-            throw new Error(`${this.constructor.name} ${this.createFolder.name}: ${error}`);
+            const message = `${this.constructor.name} ${this.createFolder.name}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async createFile(name, parentId = '-my-', title = '', description = '', author = '', majorVersion = true, aspectNames = []) {
@@ -5563,7 +5567,9 @@ class NodesApi {
             return await this.createNode('cm:content', name, parentId, title, description, null, author, majorVersion, aspectNames);
         }
         catch (error) {
-            throw new Error(`${this.constructor.name} ${this.createFile.name}: ${error}`);
+            const message = `${this.constructor.name} ${this.createFile.name}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async createFiles(names, relativePath = '/') {
@@ -5571,7 +5577,9 @@ class NodesApi {
             return await this.createContent({ files: names }, relativePath);
         }
         catch (error) {
-            throw new Error(`${this.constructor.name} ${this.createFiles.name}: ${error}`);
+            const message = `${this.constructor.name} ${this.createFiles.name}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async createMultipleFiles(count, parentId = '-my-') {
@@ -5582,6 +5590,7 @@ class NodesApi {
                 createdFiles.push(file);
             }
         }
+        logger.info(`${this.constructor.name} ${this.createMultipleFiles.name}: created ${createdFiles.length} of ${count} files in parent "${parentId}"`);
         return createdFiles;
     }
     async createFolders(names, relativePath = '/') {
@@ -5589,7 +5598,9 @@ class NodesApi {
             return await this.createContent({ folders: names }, relativePath);
         }
         catch (error) {
-            throw new Error(`${this.constructor.name} ${this.createFolders.name}: ${error}`);
+            const message = `${this.constructor.name} ${this.createFolders.name}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async deleteDeletedNode(name) {
@@ -5597,7 +5608,7 @@ class NodesApi {
             await this.apiService.trashCan.deleteDeletedNode(name);
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.deleteDeletedNode.name}: ${error}`);
+            logger.error(`${this.constructor.name} ${this.deleteDeletedNode.name}: ${error}`);
         }
     }
     async createNode(nodeType, name, parentId = '-my-', title = '', description = '', imageProps = null, author = '', majorVersion = true, aspectNames = []) {
@@ -5616,12 +5627,14 @@ class NodesApi {
             nodeBody.properties = Object.assign(nodeBody.properties, imageProps);
         }
         try {
-            return this.apiService.nodes.createNode(parentId, nodeBody, {
+            return await this.apiService.nodes.createNode(parentId, nodeBody, {
                 majorVersion
             });
         }
         catch (error) {
-            throw new Error(`${this.constructor.name} ${this.createNode.name}: ${error}`);
+            const message = `${this.constructor.name} ${this.createNode.name}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async renameNode(nodeId, newName) {
@@ -5629,7 +5642,7 @@ class NodesApi {
             return this.apiService.nodes.updateNode(nodeId, { name: newName });
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.renameNode.name}`, error);
+            logger.error(`${this.constructor.name} ${this.renameNode.name}: ${error}`);
             return null;
         }
     }
@@ -5643,7 +5656,7 @@ class NodesApi {
             await this.apiService.nodes.deleteNodes(nodeIds, { permanent });
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.deleteNodes.name}`, error);
+            logger.error(`${this.constructor.name} ${this.deleteNodes.name}: ${error}`);
         }
     }
     async updateNode(nodeId, nodeBodyUpdate, opts) {
@@ -5651,7 +5664,7 @@ class NodesApi {
             return await this.apiService.nodes.updateNode(nodeId, nodeBodyUpdate, opts);
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.updateNode.name}`, error);
+            logger.error(`${this.constructor.name} ${this.updateNode.name}: ${error}`);
             return null;
         }
     }
@@ -5666,7 +5679,7 @@ class NodesApi {
             await this.deleteNodes(userNodesIds);
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.deleteCurrentUserNodes.name}`, error);
+            logger.error(`${this.constructor.name} ${this.deleteCurrentUserNodes.name}: ${error}`);
         }
     }
     async lockNodes(nodeIds, lockType = 'ALLOW_OWNER_CHANGES') {
@@ -5676,7 +5689,7 @@ class NodesApi {
             }
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.lockNodes.name}`, error);
+            logger.error(`${this.constructor.name} ${this.lockNodes.name}: ${error}`);
         }
     }
     async unlockNodes(nodeIds) {
@@ -5686,7 +5699,7 @@ class NodesApi {
             }
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.unlockNodes.name}`, error);
+            logger.error(`${this.constructor.name} ${this.unlockNodes.name}: ${error}`);
         }
     }
     async createContent(content, relativePath = '/') {
@@ -5694,7 +5707,9 @@ class NodesApi {
             return this.apiService.nodes.createNode('-my-', flattenNodeContentTree(content, relativePath));
         }
         catch (error) {
-            throw new Error(`${this.constructor.name} ${this.createContent.name}: ${error}`);
+            const message = `${this.constructor.name} ${this.createContent.name}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async getNodeById(id) {
@@ -5702,7 +5717,9 @@ class NodesApi {
             return this.apiService.nodes.getNode(id);
         }
         catch (error) {
-            throw new Error(`${this.constructor.name} ${this.getNodeById.name}: ${error}`);
+            const message = `${this.constructor.name} ${this.getNodeById.name}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async getNodeIdFromParent(name, parentId) {
@@ -5711,7 +5728,7 @@ class NodesApi {
             return children.find((elem) => elem.entry.name === name)?.entry.id ?? '';
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.getNodeIdFromParent.name}`, error);
+            logger.error(`${this.constructor.name} ${this.getNodeIdFromParent.name}: ${error}`);
             return '';
         }
     }
@@ -5723,7 +5740,7 @@ class NodesApi {
             return this.apiService.nodes.listNodeChildren(nodeId, opts);
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.getNodeChildren.name}`, error);
+            logger.error(`${this.constructor.name} ${this.getNodeChildren.name}: ${error}`);
             return null;
         }
     }
@@ -5732,7 +5749,7 @@ class NodesApi {
             await this.apiService.nodes.deleteNode(id, { permanent });
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.deleteNodeById.name}`, error);
+            logger.error(`${this.constructor.name} ${this.deleteNodeById.name}: ${error}`);
         }
     }
     async cleanupNodeTemplatesItems(nodeNames) {
@@ -5744,7 +5761,7 @@ class NodesApi {
             }
         }
         catch (error) {
-            console.error('Admin Actions - cleanupNodeTemplatesItems failed : ', error);
+            logger.error(`Admin Actions - cleanupNodeTemplatesItems failed: ${error}`);
         }
     }
     async cleanupSpaceTemplatesItems(nodeNames) {
@@ -5756,7 +5773,7 @@ class NodesApi {
             }
         }
         catch (error) {
-            console.error('Admin Actions - cleanupSpaceTemplatesFolder failed : ', error);
+            logger.error(`Admin Actions - cleanupSpaceTemplatesFolder failed: ${error}`);
         }
     }
     async getNodeTemplatesFolderId() {
@@ -5764,7 +5781,7 @@ class NodesApi {
             return this.getNodeIdFromParent('Node Templates', await this.getDataDictionaryId());
         }
         catch (error) {
-            console.error('Admin Actions - getNodeTemplatesFolderId failed : ', error);
+            logger.error(`Admin Actions - getNodeTemplatesFolderId failed: ${error}`);
             return '';
         }
     }
@@ -5773,13 +5790,13 @@ class NodesApi {
             return this.getNodeIdFromParent('Space Templates', await this.getDataDictionaryId());
         }
         catch (error) {
-            console.error('Admin Actions - getSpaceTemplatesFolderId failed : ', error);
+            logger.error(`Admin Actions - getSpaceTemplatesFolderId failed: ${error}`);
             return '';
         }
     }
     async getDataDictionaryId() {
         return this.getNodeIdFromParent('Data Dictionary', '-root-').catch((error) => {
-            console.error('Admin Actions - getDataDictionaryId failed : ', error);
+            logger.error(`Admin Actions - getDataDictionaryId failed: ${error}`);
             return '';
         });
     }
@@ -5799,7 +5816,7 @@ class NodesApi {
             return this.apiService.nodes.updateNode(nodeId, data);
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.setGranularPermission.name}`, error);
+            logger.error(`${this.constructor.name} ${this.setGranularPermission.name}: ${error}`);
             return null;
         }
     }
@@ -5810,7 +5827,7 @@ class NodesApi {
             return this.setInheritPermissions(nodeId, false);
         }
         catch (error) {
-            console.error('Admin Actions - removeUserAccessOnNodeTemplate failed : ', error);
+            logger.error(`Admin Actions - removeUserAccessOnNodeTemplate failed: ${error}`);
             return null;
         }
     }
@@ -5821,7 +5838,7 @@ class NodesApi {
             return this.setInheritPermissions(nodeId, false);
         }
         catch (error) {
-            console.error('Admin Actions - removeUserAccessOnSpaceTemplate failed : ', error);
+            logger.error(`Admin Actions - removeUserAccessOnSpaceTemplate failed: ${error}`);
             return null;
         }
     }
@@ -5835,7 +5852,7 @@ class NodesApi {
             return this.apiService.nodes.updateNode(nodeId, data);
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.setInheritPermissions.name}`, error);
+            logger.error(`${this.constructor.name} ${this.setInheritPermissions.name}: ${error}`);
             return null;
         }
     }
@@ -5844,7 +5861,7 @@ class NodesApi {
             return this.apiService.nodes.updateNode(nodeId, { aspectNames });
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.addAspects.name}`, error);
+            logger.error(`${this.constructor.name} ${this.addAspects.name}: ${error}`);
             return null;
         }
     }
@@ -5863,7 +5880,7 @@ class NodesApi {
             return link;
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.createFileLink.name}`, error);
+            logger.error(`${this.constructor.name} ${this.createFileLink.name}: ${error}`);
             return null;
         }
     }
@@ -5885,7 +5902,7 @@ class NodesApi {
             return link;
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.createFolderLink.name}`, error);
+            logger.error(`${this.constructor.name} ${this.createFolderLink.name}: ${error}`);
             return null;
         }
     }
@@ -5896,7 +5913,9 @@ class NodesApi {
             return this.createFileLink(nodeId, destinationParentId);
         }
         catch (error) {
-            throw new Error(`Admin Actions - createLinkToFileName failed : ${error}`);
+            const message = `Admin Actions - createLinkToFileName failed : ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async createLinkToFolderName(originalFolderName, originalFolderParentId, destinationParentId) {
@@ -5906,7 +5925,9 @@ class NodesApi {
             return this.createFolderLink(nodeId, destinationParentId);
         }
         catch (error) {
-            throw new Error(`Admin Actions - createLinkToFolderName failed : ${error}`);
+            const message = `Admin Actions - createLinkToFolderName failed : ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async getNodeProperty(nodeId, property) {
@@ -5915,7 +5936,9 @@ class NodesApi {
             return node.entry.properties?.[property] ?? '';
         }
         catch (error) {
-            throw new Error(`${this.constructor.name} ${this.getNodeProperty.name}: ${error}`);
+            const message = `${this.constructor.name} ${this.getNodeProperty.name}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async isFileShared(nodeId) {
@@ -5924,7 +5947,9 @@ class NodesApi {
             return sharedId !== '';
         }
         catch (error) {
-            throw new Error(`${this.constructor.name} ${this.isFileShared.name}: ${error}`);
+            const message = `${this.constructor.name} ${this.isFileShared.name}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async getLockType(nodeId) {
@@ -5933,7 +5958,9 @@ class NodesApi {
             return lockType || '';
         }
         catch (error) {
-            throw new Error(`${this.constructor.name} ${this.getLockType.name}: ${error}`);
+            const message = `${this.constructor.name} ${this.getLockType.name}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async isFileLockedWrite(nodeId) {
@@ -5941,7 +5968,9 @@ class NodesApi {
             return (await this.getLockType(nodeId)) === 'WRITE_LOCK';
         }
         catch (error) {
-            throw new Error(`${this.constructor.name} ${this.isFileLockedWrite.name}: ${error}`);
+            const message = `${this.constructor.name} ${this.isFileLockedWrite.name}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
 }
@@ -5986,22 +6015,28 @@ class SitesApi {
             id: siteId || title
         };
         try {
-            return this.apiService.sites.createSite(site);
+            return await this.apiService.sites.createSite(site);
         }
         catch (error) {
-            throw new Error(`SitesApi ${this.createSite.name}: ${error}`);
+            const message = `SitesApi ${this.createSite.name}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async getDocLibId(siteId) {
         try {
             const id = (await this.apiService.sites.listSiteContainers(siteId)).list?.entries?.[0]?.entry?.id;
             if (!id) {
-                throw new Error(`Document library not found for site ${siteId}`);
+                const message = `Document library not found for site ${siteId}`;
+                logger.error(message);
+                throw new Error(message);
             }
             return id;
         }
         catch (error) {
-            throw new Error(`Failed to get document library ID for site ${siteId}: ${error}`);
+            const message = `Failed to get document library ID for site ${siteId}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     /**
@@ -6018,7 +6053,7 @@ class SitesApi {
             }
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.deleteSites.name}`, error);
+            logger.error(`${this.constructor.name} ${this.deleteSites.name}: ${error}`);
         }
     }
     async updateSiteMember(siteId, userId, role) {
@@ -6026,10 +6061,10 @@ class SitesApi {
             role: role
         };
         try {
-            return this.apiService.sites.updateSiteMembership(siteId, userId, siteRole);
+            return await this.apiService.sites.updateSiteMembership(siteId, userId, siteRole);
         }
         catch (error) {
-            console.error(`SitesApi updateSiteMember : catch : `, error);
+            logger.error(`SitesApi updateSiteMember : catch : ${error}`);
             return new SiteMemberEntry();
         }
     }
@@ -6056,7 +6091,9 @@ class SitesApi {
             return this.apiService.sites.createSiteMembershipRequestForPerson(personId, body);
         }
         catch (error) {
-            throw new Error(`Failed to create site membership request for person ${personId} and site ${siteId}: ${error}`);
+            const message = `Failed to create site membership request for person ${personId} and site ${siteId}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async approveSiteMembershipRequest(siteId, inviteeId) {
@@ -6064,7 +6101,9 @@ class SitesApi {
             return this.apiService.sites.approveSiteMembershipRequest(siteId, inviteeId);
         }
         catch (error) {
-            throw new Error(`Failed to approve site membership request for invitee ${inviteeId} and site ${siteId}: ${error}`);
+            const message = `Failed to approve site membership request for invitee ${inviteeId} and site ${siteId}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async hasMembershipRequest(personId, siteId) {
@@ -6074,7 +6113,9 @@ class SitesApi {
             return requests.includes(siteId);
         }
         catch (error) {
-            throw new Error(`Failed to check site membership request for person ${personId} and site ${siteId}: ${error}`);
+            const message = `Failed to check site membership request for person ${personId} and site ${siteId}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async deleteSiteMember(siteId, userId) {
@@ -6082,7 +6123,7 @@ class SitesApi {
             return this.apiService.sites.deleteSiteMembership(siteId, userId);
         }
         catch (error) {
-            console.error(`SitesApi deleteSiteMember : catch : `, error);
+            logger.error(`SitesApi deleteSiteMember : catch : ${error}`);
         }
     }
     async getSite(siteId) {
@@ -6090,7 +6131,9 @@ class SitesApi {
             return this.apiService.sites.getSite(siteId);
         }
         catch (error) {
-            throw new Error(`Failed to get site ${siteId}: ${error}`);
+            const message = `Failed to get site ${siteId}: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
 }
@@ -6147,7 +6190,7 @@ class SearchApi {
         }
     }
     async getTotalItems(username) {
-        return (await this.querySearchFiles(username)).list.pagination.totalItems;
+        return (await this.querySearchFiles(username)).list?.pagination?.totalItems ?? 0;
     }
     async waitForApi(username, data) {
         try {
@@ -6177,23 +6220,25 @@ class SearchApi {
             let result;
             do {
                 result = await this.apiService.search.search(query);
-                const currentCount = result.list.pagination.count;
+                const currentCount = result.list?.pagination?.count ?? 0;
                 if (currentCount !== options.nodesExpected) {
                     retryCount++;
                     if (retryCount % 30 === 0) {
-                        console.info(`waitForFolderPathIndexing: After ${retryCount} seconds, expected ${options.nodesExpected} nodes but found ${currentCount} in folder ${folderId}`);
+                        logger.info(`waitForFolderPathIndexing: After ${retryCount} seconds, expected ${options.nodesExpected} nodes but found ${currentCount} in folder ${folderId}`);
                     }
                     if (retryCount >= retryLimit) {
-                        throw new Error(`Expected ${options.nodesExpected} nodes but found ${currentCount} after ${retryLimit} retries`);
+                        const message = `Expected ${options.nodesExpected} nodes but found ${currentCount} after ${retryLimit} retries`;
+                        logger.error(message);
+                        throw new Error(message);
                     }
                     await Utils.delayInSeconds(1);
                 }
-            } while (result.list.pagination.count !== options.nodesExpected);
-            console.info(`waitForFolderPathIndexing: Found expected ${options.nodesExpected} nodes in folder ${folderId}`);
-            return result.list.pagination.count;
+            } while (result.list?.pagination?.count !== options.nodesExpected);
+            logger.info(`waitForFolderPathIndexing: Found expected ${options.nodesExpected} nodes in folder ${folderId}`);
+            return result.list?.pagination?.count ?? 0;
         }
         catch (error) {
-            console.error(`waitForFolderPathIndexing failed for folderId "${folderId}": ${error}`);
+            logger.error(`waitForFolderPathIndexing failed for folderId "${folderId}": ${error}`);
             throw error;
         }
     }
@@ -6247,7 +6292,7 @@ class TrashcanApi {
             }
         }
         catch (error) {
-            console.error('User Actions - emptyTrashcan failed : ', error);
+            logger.error(`User Actions - emptyTrashcan failed: ${error}`);
         }
     }
 }
@@ -6298,8 +6343,7 @@ class QueriesApi {
             return await Utils.retryCall(sites);
         }
         catch (error) {
-            console.error(`QueriesApi waitForSites : catch : `);
-            console.error(`\tExpected: ${data.expect} items, but found ${error}`);
+            logger.error(`QueriesApi waitForSites : catch : Expected: ${data.expect} items, but found ${error}`);
             return null;
         }
     }
@@ -6313,7 +6357,7 @@ class QueriesApi {
             return sites.list.pagination.totalItems;
         }
         catch (error) {
-            console.error(`QueriesApi findSitesTotalItems : catch :`, error);
+            logger.error(`QueriesApi findSitesTotalItems : catch : ${error}`);
             return null;
         }
     }
@@ -6356,20 +6400,20 @@ class CategoriesApi {
             return this.apiService.categoriesApi.createSubcategories(categoryId, categoryBodyCreate, opts);
         }
         catch (error) {
-            console.error(error);
+            logger.error(`${error}`);
             return null;
         }
     }
     async deleteCategory(categoryId) {
-        if (categoryId === null) {
-            console.error('categoryId is null, skipping deletion');
+        if (!categoryId) {
+            logger.error('categoryId is null, skipping deletion');
             return;
         }
         try {
             await this.apiService.categoriesApi.deleteCategory(categoryId);
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.deleteCategory.name}: ${error}`);
+            logger.error(`${this.constructor.name} ${this.deleteCategory.name}: ${error}`);
         }
     }
     async linkNodeToCategory(nodeId, categoryLinkBodyCreate, opts) {
@@ -6377,7 +6421,7 @@ class CategoriesApi {
             return this.apiService.categoriesApi.linkNodeToCategory(nodeId, categoryLinkBodyCreate, opts);
         }
         catch (error) {
-            console.error(`${this.constructor.name} ${this.linkNodeToCategory.name}: ${error}`);
+            logger.error(`${this.constructor.name} ${this.linkNodeToCategory.name}: ${error}`);
             return null;
         }
     }
@@ -6427,12 +6471,16 @@ class TagsApi {
                 else if ('list' in result) {
                     const firstEntry = result.list?.entries?.[0];
                     if (!firstEntry) {
-                        throw new Error(`createTags returned a paging result with no entries for tag "${tag}"`);
+                        const message = `createTags returned a paging result with no entries for tag "${tag}"`;
+                        logger.error(message);
+                        throw new Error(message);
                     }
                     created = firstEntry;
                 }
                 else {
-                    throw new Error(`createTags returned an unexpected response format for tag "${tag}"`);
+                    const message = `createTags returned an unexpected response format for tag "${tag}"`;
+                    logger.error(message);
+                    throw new Error(message);
                 }
                 logger.info(`Tag created: "${created.entry.tag}" (id: ${created.entry.id})`);
                 results.push(created);
@@ -6440,15 +6488,19 @@ class TagsApi {
             return results;
         }
         catch (error) {
-            throw new Error(`Failed to create tags: ${error}`);
+            const message = `Failed to create tags: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async assignTagToNode(nodeId, tag) {
         try {
-            return this.apiService.tagsApi.assignTagToNode(nodeId, tag);
+            return await this.apiService.tagsApi.assignTagToNode(nodeId, tag);
         }
         catch (error) {
-            throw new Error(`Failed to assign tag to node: ${error}`);
+            const message = `Failed to assign tag to node: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async deleteTags(...tags) {
@@ -6460,7 +6512,9 @@ class TagsApi {
             }
         }
         catch (error) {
-            throw new Error(`Failed to delete tags: ${error}`);
+            const message = `Failed to delete tags: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async listTagsForNode(nodeId) {
@@ -6468,7 +6522,9 @@ class TagsApi {
             return this.apiService.tagsApi.listTagsForNode(nodeId);
         }
         catch (error) {
-            throw new Error(`Failed to list tags for node: ${error}`);
+            const message = `Failed to list tags for node: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async listTags(params) {
@@ -6476,7 +6532,9 @@ class TagsApi {
             return this.apiService.tagsApi.listTags(params);
         }
         catch (error) {
-            throw new Error(`Failed to list tags: ${error}`);
+            const message = `Failed to list tags: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
     async deleteTagByTagName(tagName) {
@@ -6486,7 +6544,9 @@ class TagsApi {
             await this.deleteTags(...tags);
         }
         catch (error) {
-            throw new Error(`Failed to delete tags by tag name: ${error}`);
+            const message = `Failed to delete tags by tag name: ${error}`;
+            logger.error(message);
+            throw new Error(message);
         }
     }
 }