Skip to content

Commit

Permalink
Merge pull request #20 from Progi1984/178xInstall
Browse files Browse the repository at this point in the history
1.7.8 : Changed the behavior for install modules
  • Loading branch information
Progi1984 authored Mar 20, 2024
2 parents 323b5a5 + 385be68 commit f8a3159
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,5 +5,5 @@ export interface ModuleManagerUninstalledModulesPageInterface extends BOBasePage
readonly installMessageSuccessful: (moduleTag: string) => string;

goToTabUninstalledModules(page: Page): Promise<void>;
installModule(page: Page, moduleTag: string): Promise<string|null>;
installModule(page: Page, moduleTag: string): Promise<boolean>;
}
Original file line number Diff line number Diff line change
Expand Up @@ -42,15 +42,15 @@ class UninstalledModules extends BOBasePage implements ModuleManagerUninstalledM
}

/**
* Install the module and return the growl message
* Install the module and return if installed
* @param {Page} page
* @param {string} moduleTag
* @returns {Promise<string|null>}
* @returns {Promise<boolean>}
*/
async installModule(page: Page, moduleTag: string): Promise<string|null> {
async installModule(page: Page, moduleTag: string): Promise<boolean> {
await page.locator(this.installModuleButton(moduleTag)).click();

return this.getGrowlMessageContent(page);
return !(await this.elementNotVisible(page, this.installModuleButton(moduleTag)));
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,11 +31,11 @@ class UninstalledModules extends BOBasePage implements ModuleManagerUninstalledM
}

/**
* Install the module and return the growl message
* @returns {Promise<string|null>}
* Install the module and return if installed
* @returns {Promise<boolean>}
*/
async installModule(): Promise<string|null> {
return '';
async installModule(): Promise<boolean> {
return true;
}
}

Expand Down

0 comments on commit f8a3159

Please sign in to comment.