[ACS-8924] Failing Sonar job (#4219)

Alfresco · Nov 13, 2024 · 27e318a · 27e318a
1 parent ad0ec78
commit 27e318a
Show file tree

Hide file tree

Showing 8 changed files with 13 additions and 11 deletions.
diff --git a/...aca-content/src/lib/components/bulk-actions-dropdown/bulk-actions-dropdown.component.html b/...aca-content/src/lib/components/bulk-actions-dropdown/bulk-actions-dropdown.component.html
@@ -32,6 +32,7 @@
       [value]="option"
       [title]="option.tooltip | translate"
       [attr.data-automation-id]="option.id"
+      (keyup.enter)="runAction(option)"
       (click)="runAction(option)"
     >
       <div class="aca-bulk-actions-option-content">

diff --git a/projects/aca-content/src/lib/components/search/search-input/search-input.component.html b/projects/aca-content/src/lib/components/search/search-input/search-input.component.html
@@ -44,6 +44,7 @@
           [(ngModel)]="option.value"
           [disabled]="option.shouldDisable()"
           (change)="searchByOption()"
+          (keyup.enter)="$event.stopPropagation()"
           (click)="$event.stopPropagation()"
         >
           {{ option.key | translate }}

diff --git a/projects/aca-playwright-shared/src/api/favorites-api.ts b/projects/aca-playwright-shared/src/api/favorites-api.ts
@@ -89,7 +89,7 @@ export class FavoritesPageApi {
           return Promise.resolve(isFavorite);
         }
       };
-      return Utils.retryCall(favorite);
+      return await Utils.retryCall(favorite);
     } catch (error) {}
     return isFavorite;
   }
@@ -112,7 +112,7 @@ export class FavoritesPageApi {
           return Promise.resolve(totalItems);
         }
       };
-      return Utils.retryCall(favoriteFiles);
+      return await Utils.retryCall(favoriteFiles);
     } catch {}
   }
 

diff --git a/projects/aca-playwright-shared/src/api/file-actions.ts b/projects/aca-playwright-shared/src/api/file-actions.ts
@@ -125,7 +125,7 @@ export class FileActionsApi {
           return Promise.resolve(isLocked);
         }
       };
-      return Utils.retryCall(locked, data.retry);
+      return await Utils.retryCall(locked, data.retry);
     } catch {}
     return isLocked;
   }

diff --git a/projects/aca-playwright-shared/src/api/nodes-api.ts b/projects/aca-playwright-shared/src/api/nodes-api.ts
@@ -48,7 +48,7 @@ export class NodesApi {
     aspectNames: string[] = null
   ): Promise<NodeEntry | null> {
     try {
-      return this.createNode('cm:folder', name, parentId, title, description, null, author, null, aspectNames);
+      return await this.createNode('cm:folder', name, parentId, title, description, null, author, null, aspectNames);
     } catch (error) {
       console.error(`${this.constructor.name} ${this.createFolder.name}`, error);
       return null;
@@ -65,7 +65,7 @@ export class NodesApi {
     aspectNames: string[] = null
   ): Promise<NodeEntry> {
     try {
-      return this.createNode('cm:content', name, parentId, title, description, null, author, majorVersion, aspectNames);
+      return await this.createNode('cm:content', name, parentId, title, description, null, author, majorVersion, aspectNames);
     } catch (error) {
       console.error(`${this.constructor.name} ${this.createFile.name}`, error);
       return null;
@@ -74,7 +74,7 @@ export class NodesApi {
 
   async createFiles(names: string[], relativePath = '/'): Promise<NodePaging> {
     try {
-      return this.createContent({ files: names }, relativePath);
+      return await this.createContent({ files: names }, relativePath);
     } catch (error) {
       console.error(`${this.constructor.name} ${this.createFiles.name}: ${error}`);
       return null;
@@ -83,7 +83,7 @@ export class NodesApi {
 
   async createFolders(names: string[], relativePath = '/'): Promise<NodePaging> {
     try {
-      return this.createContent({ folders: names }, relativePath);
+      return await this.createContent({ folders: names }, relativePath);
     } catch (error) {
       console.error(`${this.constructor.name} ${this.createFolders.name}: ${error}`);
       return null;

diff --git a/projects/aca-playwright-shared/src/api/queries-api.ts b/projects/aca-playwright-shared/src/api/queries-api.ts
@@ -50,7 +50,7 @@ export class QueriesApi {
         }
       };
 
-      return Utils.retryCall(sites);
+      return await Utils.retryCall(sites);
     } catch (error) {
       console.error(`QueriesApi waitForSites : catch : `);
       console.error(`\tExpected: ${data.expect} items, but found ${error}`);

diff --git a/projects/aca-playwright-shared/src/api/search-api.ts b/projects/aca-playwright-shared/src/api/search-api.ts
@@ -73,7 +73,7 @@ export class SearchPageApi {
         }
       };
 
-      return Utils.retryCall(searchFiles);
+      return await Utils.retryCall(searchFiles);
     } catch {}
   }
 }
diff --git a/projects/aca-playwright-shared/src/api/shared-links-api.ts b/projects/aca-playwright-shared/src/api/shared-links-api.ts
@@ -89,7 +89,7 @@ export class SharedLinksApi {
         }
       };
 
-      return Utils.retryCall(sharedFile);
+      return await Utils.retryCall(sharedFile);
     } catch (error) {
       console.error(`SharedLinksApi waitForFilesToBeShared :  catch : ${error}`);
       console.error(`\tWait timeout reached waiting for files to be shared`);
@@ -132,7 +132,7 @@ export class SharedLinksApi {
         }
       };
 
-      return Utils.retryCall(sharedFile);
+      return 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`);