enable share action on favorites (#731)

This commit is contained in:
Cilibiu Bogdan 2018-10-18 16:29:26 +03:00 committed by Denys Vuika
parent 30b445dde9
commit 4b8dbe81d4
4 changed files with 5 additions and 14 deletions

View File

@ -282,8 +282,7 @@ describe('Context menu actions - single selection : ', () => {
expect(await contextMenu.isMenuItemPresent('Copy')).toBe(true, `Copy is not displayed for ${fileUser}`);
expect(await contextMenu.isMenuItemPresent('Move')).toBe(true, `Move is not displayed for ${fileUser}`);
expect(await contextMenu.isMenuItemPresent('Delete')).toBe(true, `Delete is not displayed for ${fileUser}`);
// todo enable when ACA-1886 is fixed
// expect(await contextMenu.isMenuItemPresent('Share')).toBe(true, `Share is not displayed for ${fileUser}`);
expect(await contextMenu.isMenuItemPresent('Share')).toBe(true, `Share is not displayed for ${fileUser}`);
expect(await contextMenu.isMenuItemPresent('Manage Versions')).toBe(true, `Manage Versions is not displayed for ${fileUser}`);
// TODO: enable when ACA-1794 is fixed
// expect(await contextMenu.isMenuItemPresent('Permissions')).toBe(true, `Permissions is not displayed for ${fileUser}`);

View File

@ -456,8 +456,7 @@ describe('Granular permissions available actions : ', () => {
// TODO: enable when ACA-1737 is done
// expect(await contextMenu.isMenuItemPresent('Move')).toBe(false, `Move is displayed for ${file1}`);
// expect(await contextMenu.isMenuItemPresent('Delete')).toBe(false, `Delete is displayed for ${file1}`);
// todo enable when ACA-1886 is fixed
// expect(await contextMenu.isMenuItemPresent('Share')).toBe(true, `Share is not displayed for ${file1}`);
expect(await contextMenu.isMenuItemPresent('Share')).toBe(true, `Share is not displayed for ${file1}`);
expect(await contextMenu.isMenuItemPresent('Manage Versions')).toBe(true, `Manage Versions is not displayed for ${file1}`);
// TODO: enable when ACA-1794 is fixed
// expect(await contextMenu.isMenuItemPresent('Permissions')).toBe(true, `Permissions is not displayed for ${file1}`);
@ -708,8 +707,7 @@ describe('Granular permissions available actions : ', () => {
expect(await viewerToolbar.isButtonPresent('View details')).toBe(true, `View details is not displayed`);
await viewerToolbar.openMoreMenu();
expect(await viewerToolbar.menu.isMenuItemPresent('Favorite')).toBe(true, `Favorite is not displayed`);
// todo enable when ACA-1886 is fixed
// expect(await viewerToolbar.menu.isMenuItemPresent('Share')).toBe(true, `Share is not displayed`);
expect(await viewerToolbar.menu.isMenuItemPresent('Share')).toBe(true, `Share is not displayed`);
expect(await viewerToolbar.menu.isMenuItemPresent('Copy')).toBe(true, `Copy is not displayed`);
// TODO: enable when ACA-1737 is done
// expect(await viewerToolbar.menu.isMenuItemPresent('Move')).toBe(false, `Move is displayed`);

View File

@ -786,8 +786,7 @@ describe('Viewer actions', () => {
expect(await dataTable.getRowByName(pdfFavorites).isPresent()).toBe(true, 'Item is not present in Trash');
});
// todo enable when ACA-1886 is fixed
xit('Share action - [C286395]', async () => {
it('Share action - [C286395]', async () => {
await dataTable.doubleClickOnRowByName(docxFavorites);
expect(await viewer.isViewerOpened()).toBe(true, 'Viewer is not opened');

View File

@ -26,7 +26,6 @@
import { RuleContext, RuleParameter } from '@alfresco/adf-extensions';
import {
isNotTrashcan,
isNotFavorites,
isNotLibraries,
isFavorites,
isLibraries,
@ -70,11 +69,7 @@ export function canShareFile(
context: RuleContext,
...args: RuleParameter[]
): boolean {
if (
isNotTrashcan(context, ...args) &&
isNotFavorites(context, ...args) &&
context.selection.file
) {
if (isNotTrashcan(context, ...args) && context.selection.file) {
return true;
}
return false;