Skip to content

Commit

Permalink
[ACS-8891] remove e2e test data (#4179)
Browse files Browse the repository at this point in the history
* [ACS-8886] enable excluded test

* [ACS-8886] enable excluded test

* [ACS-8886] test fix and added ticket to resovle

* [ACS-8886] test reexcluded as valid fail

* [ACS-8886] test enalbe

* [ACS-8886] test fix for admin

* ACS-8891-remove-test-data

* ACS-8891-test-fix

* ACS-8891-test-fix

* test fix
  • Loading branch information
akashrathod28 authored Nov 21, 2024
1 parent 8e731c2 commit 6cdd5da
Show file tree
Hide file tree
Showing 23 changed files with 44 additions and 40 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -363,7 +363,7 @@ test.describe('Create file from template', () => {
});

test.describe('File created from template on Personal Files Libraries', () => {
const randomLibraryName = `playwright-library-${Utils.random()}`;
const randomLibraryName = `playwright-library-c2-${Utils.random()}`;
let sitesApi: SitesApi;

test.beforeAll(async () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -394,7 +394,7 @@ test.describe('Create folder from template', () => {
});

test.describe('Folder created from template on Personal Files Libraries', () => {
const randomLibraryName = `playwright-library-${Utils.random()}`;
const randomLibraryName = `playwright-library-c1-${Utils.random()}`;
let sitesApi: SitesApi;

test.beforeAll(async () => {
Expand Down
15 changes: 8 additions & 7 deletions e2e/playwright/create-actions/src/tests/create-library.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,8 @@ test.describe('Create Libraries ', () => {
const errorMessageNotPresent = 'Error message is not displayed';
const tabKeyString = 'Tab';
const username = `user-${Utils.random()}`;
const commonLibraryName = `playwright-library-${Utils.random()}`;
const commonTrashLibraryName = `playwright-library-${Utils.random()}`;
const commonLibraryName = `playwright-library1-${Utils.random()}`;
const commonTrashLibraryName = `playwright-library2-${Utils.random()}`;
const createdLibrariesIds: string[] = [];

test.beforeAll(async () => {
Expand All @@ -69,17 +69,18 @@ test.describe('Create Libraries ', () => {
sitesApi = await SitesApi.initialize(username, username);
nodesApi = await NodesApi.initialize(username, username);
trashcanApi = await TrashcanApi.initialize(username, username);
await sitesApi.createSite(commonLibraryName);
createdLibrariesIds.push(commonLibraryName);
await sitesApi.createSite(commonTrashLibraryName);
await sitesApi.deleteSites([commonTrashLibraryName], false);
const commonLibraryId = (await sitesApi.createSite(commonLibraryName)).entry.id;
createdLibrariesIds.push(commonLibraryId);
const commonTrashLibraryId = (await sitesApi.createSite(commonTrashLibraryName)).entry.id;
createdLibrariesIds.push(commonTrashLibraryId);
await sitesApi.deleteSites([commonTrashLibraryId], false);
} catch (error) {
console.error(`beforeAll failed : ${error}`);
}
});

test.beforeEach(async ({ myLibrariesPage, page }) => {
randomLibraryName = `playwright-library-${Utils.random()}`;
randomLibraryName = `playwright-library3-${Utils.random()}`;
randomLibraryId = `libraryId-${Utils.random()}`;
randomLibraryDescription = `libraryDescription-${Utils.random()}`;
libraryDialog = myLibrariesPage.libraryDialog;
Expand Down
2 changes: 1 addition & 1 deletion e2e/playwright/edit-actions/src/tests/edit-folder.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ test.describe('Edit folder', () => {
let folderNameToEditId: string;

const username = `user-${Utils.random()}`;
const parent = `parent-${Utils.random()}`;
const parent = `parent-edit-${Utils.random()}`;
const folderName = `folder-${Utils.random()}`;
const folderDescription = 'my folder description';

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ test.describe('Mark items as favorites', () => {
let sharedApi: SharedLinksApi;

const username = `user-${Utils.random()}`;
const parent = `parent-${Utils.random()}`;
const parent = `parent-fev-${Utils.random()}`;

const fileNotFavUI = `fileNotFavUI-${Utils.random()}.txt`;
const fileFavUI = `fileFavUI-${Utils.random()}.txt`;
Expand Down
5 changes: 1 addition & 4 deletions e2e/playwright/info-drawer/src/tests/comments.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,7 @@ test.describe('Info Drawer - Comments', () => {
});

test('[C299189] from Shared Files - Comments are displayed ordered by created date in descending order', async ({ sharedPage }) => {
await sharedPage.navigate();
const sharedFileName = `sharedFile-e2e-${Utils.random()}`;
const e2eCommentFirst = `e2e-comment-${Utils.random()}`;
const e2eCommentSecond = `e2e-comment-${Utils.random()}`;
Expand All @@ -96,8 +97,6 @@ test.describe('Info Drawer - Comments', () => {
await apiClientFactory.commentsApi.createComment(sharedFileId, { content: e2eCommentFirst });
await apiClientFactory.commentsApi.createComment(sharedFileId, { content: e2eCommentSecond });
await apiClientFactory.share.createSharedLink({ nodeId: sharedFileId });
await fileActionsApi.waitForNodes(sharedFileName, { expect: 1 });
await sharedPage.navigate();
await Utils.reloadPageIfRowNotVisible(sharedPage, sharedFileName);
await expect(sharedPage.dataTable.getRowByName(sharedFileName)).toBeVisible();
await sharedPage.dataTable.getRowByName(sharedFileName).click();
Expand Down Expand Up @@ -130,9 +129,7 @@ test.describe('Info Drawer - Comments', () => {
const commentInfoFileName = `e2e-commentFile-${Utils.random()}`;
const commentInfoFileId = (await nodesApi.createFile(commentInfoFileName)).entry.id;
await favoritesActions.addFavoritesByIds('file', [commentInfoFileId]);
await fileActionsApi.waitForNodes(commentInfoFileName, { expect: 1 });
await apiClientFactory.commentsApi.createComment(commentInfoFileId, { content: e2eCommentFirst });
await fileActionsApi.waitForNodes(commentInfoFileName, { expect: 1 });
await favoritePage.navigate();
await expect(favoritePage.dataTable.getRowByName(commentInfoFileName)).toBeVisible();
await favoritePage.dataTable.getRowByName(commentInfoFileName).click();
Expand Down
2 changes: 1 addition & 1 deletion e2e/playwright/info-drawer/src/tests/general.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ test.describe('Info Drawer - General', () => {
});

test('[C268999] Info drawer closes on page refresh', async ({ personalFiles }) => {
const parentFolder = `parent-${Utils.random()}`;
const parentFolder = `parent-general-${Utils.random()}`;
const file1 = `file1-${Utils.random()}.txt`;
const folder1 = `folder1-${Utils.random()}`;
const parentId = (await nodesApi.createFolder(parentFolder)).entry.id;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -148,6 +148,7 @@ test.describe('Library actions ', () => {
await adminSitesApi.deleteSites(adminModerateLibraryIds);
await adminSitesApi.deleteSites(user1LibraryIds);
await adminSitesApi.deleteSites(user2LibraryIds);
await user2SitesApi.deleteSites([user2Library5Delete, user2Library6Delete, user2Library7Delete]);
const trashcanApi = await TrashcanApi.initialize(username2, username2);
await trashcanApi.emptyTrashcan();
} catch (error) {
Expand Down
2 changes: 1 addition & 1 deletion e2e/playwright/list-views/src/tests/favorites.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ test.describe('Favorites Files', () => {
const username = `user-${Utils.random()}`;
const siteName = `site-${Utils.random()}`;
const favFolderName = `favFolder-${Utils.random()}`;
const parentFolder = `parent-${Utils.random()}`;
const parentFolder = `fev-parent-${Utils.random()}`;
const fileName1 = `file1-${Utils.random()}.txt`;
const fileName2 = `file2-${Utils.random()}.txt`;
const fileName3 = `file3-${Utils.random()}.txt`;
Expand Down
2 changes: 1 addition & 1 deletion e2e/playwright/list-views/src/tests/permissions.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,7 @@ test.describe('Special permissions', () => {

test('[C213178] on Recent Files', async ({ recentFilesPage }) => {
await recentFilesPage.navigate();
expect(await recentFilesPage.dataTable.getRowsCount(), 'Incorrect number of items').toBe(1);
expect(await recentFilesPage.dataTable.isItemPresent(fileName)).toBe(true);
expect(await recentFilesPage.dataTable.getItemLocationText(fileName)).toEqual('Unknown');
});

Expand Down
4 changes: 2 additions & 2 deletions e2e/playwright/navigation/src/tests/breadcrumb.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -31,15 +31,15 @@ test.describe('viewer action file', () => {
let siteActions: SitesApi;
let trashcanApi: TrashcanApi;
const username = `user-${Utils.random()}`;
const parent = `parent-${Utils.random()}`;
const parent = `parent-viewer-${Utils.random()}`;
let parentId: string;
const subFolder1 = `subFolder1-${Utils.random()}`;
let subFolder1Id: string;
const subFolder2 = `subFolder2-${Utils.random()}`;
let subFolder2Id: string;
const fileName1 = `file1-${Utils.random()}.txt`;

const siteName = `site-${Utils.random()}`;
const siteName = `site-breadcrumb-${Utils.random()}`;
let docLibId: string;
const parentFromSite = `parent-in-site-${Utils.random()}`;
let parentFromSiteId: string;
Expand Down
2 changes: 1 addition & 1 deletion e2e/playwright/navigation/src/tests/single-click.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ test.describe('Single click on item name', () => {
const deletedFile1 = `file1-${Utils.random()}.txt`;
const deletedFolder1 = `folder1-${Utils.random()}`;

const siteName = `site-${Utils.random()}`;
const siteName = `site-single-${Utils.random()}`;
const fileSite = `fileSite-${Utils.random()}.txt`;

test.beforeAll(async () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ test.describe('Pagination on multiple pages : ', () => {
let trashcanApi: TrashcanApi;
let favoritesApi: FavoritesPageApi;

const parent = `parent-${random}`;
const parent = `parent-multi-${random}`;
let initialFavoritesTotalItems: number;

const apiClientFactory = new ApiClientFactory();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,12 @@ test.describe('Search - Filters - Location', () => {

const random = Utils.random();
const username = `user1-${random}`;
const site = `site-${random}`;
const site = `site-search-${random}`;
const fileJpg = `${random}-file.jpg`;
const filePdf = `${random}-file.pdf`;
const userFolder = `${random}-userFolder`;
const siteFolder = `${random}-folderSite`;
let siteId: string;

test.beforeEach(async ({ loginPage }) => {
await Utils.tryLoginUser(loginPage, username, username, 'beforeEach failed');
Expand All @@ -52,7 +53,7 @@ test.describe('Search - Filters - Location', () => {
siteAdminApi = await SitesApi.initialize('admin');
await siteAdminApi.createSite(site, SITE_VISIBILITY.PUBLIC);
await siteAdminApi.addSiteMember(site, username, SITE_ROLES.SITE_MANAGER.ROLE);
const siteId = await siteAdminApi.getDocLibId(site);
siteId = await siteAdminApi.getDocLibId(site);
await nodesApi.createFile(fileJpg, siteId);
await nodesApi.createFile(filePdf);
await nodesApi.createFolder(siteFolder, siteId);
Expand All @@ -63,7 +64,7 @@ test.describe('Search - Filters - Location', () => {
});

test.afterAll(async () => {
await Utils.deleteNodesSitesEmptyTrashcan(nodesApi, trashcanApi, 'afterAll failed', siteAdminApi);
await Utils.deleteNodesSitesEmptyTrashcan(nodesApi, trashcanApi, 'afterAll failed', siteAdminApi, [siteId]);
});

test('[C279231] Filter by location - files', async ({ searchPage }) => {
Expand Down
2 changes: 1 addition & 1 deletion e2e/playwright/search/src/tests/search-sorting.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ test.describe('Search sorting', () => {
const user1 = `user1-${random}`;
const user2 = `user2-${random}`;

const parent = `parent-${random}`;
const parent = `parent-search-${random}`;
let parentId: string;

const fileJpg = {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ test.describe('Share a file', () => {
const random = Utils.random();

const username = `user-${random}`;
const parent = `parent-${random}`;
const parent = `parent-share-${random}`;
let parentId: string;

const file3 = `file3-${random}.txt`;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ test.describe('Unshare a file from Search Results', () => {
let trashcanApi: TrashcanApi;

const username = `user-${random}`;
const parent = `parent-${random}`;
const parent = `parent-unshare-${random}`;
let parentId: string;
const searchRandom = random;
let sitesApi: SitesApi;
Expand All @@ -56,7 +56,8 @@ test.describe('Unshare a file from Search Results', () => {
const file3 = `search-file-${searchRandom}-3.txt`;
const file4 = `search-file-${searchRandom}-4.txt`;

const sitePrivate = `site-private-${random}`;
const sitePrivate = `site-private1-${random}`;
let docLibId: string;

let fileSite2Id: string;
const fileSite1 = `search-file-${searchRandom}-Site1.txt`;
Expand Down Expand Up @@ -84,7 +85,7 @@ test.describe('Unshare a file from Search Results', () => {
file4Id = (await nodesApi.createFile(file4, parentId)).entry.id;

await sitesApi.createSite(sitePrivate, Site.VisibilityEnum.PRIVATE);
const docLibId = await sitesApi.getDocLibId(sitePrivate);
docLibId = await sitesApi.getDocLibId(sitePrivate);

const fileSite1Id = (await nodesApiAdmin.createFile(fileSite1, docLibId)).entry.id;
fileSite2Id = (await nodesApiAdmin.createFile(fileSite2, docLibId)).entry.id;
Expand All @@ -109,7 +110,7 @@ test.describe('Unshare a file from Search Results', () => {
});

test.afterAll(async () => {
await Utils.deleteNodesSitesEmptyTrashcan(nodesApi, trashcanApi, 'afterAll failed', sitesApi, [sitePrivate]);
await Utils.deleteNodesSitesEmptyTrashcan(nodesApi, trashcanApi, 'afterAll failed', sitesApi, [docLibId]);
});

test('[C306995] Unshare dialog UI', async ({ personalFiles, searchPage }) => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ test.describe('Folders - available actions : ', () => {
test.beforeAll(async () => {
test.setTimeout(timeouts.extendedTest);
const apiClientFactory = new ApiClientFactory();
const parentName = `parent-${Utils.random()}`;
const parentName = `parent-folder-${Utils.random()}`;
await apiClientFactory.setUpAcaBackend('admin');
await apiClientFactory.createUser({ username });
nodesApi = await NodesApi.initialize(username, username);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ test.describe('Special permissions : ', () => {
const random = testData.random;
let docLibId: string;

const sitePrivate = `site-private-${random}`;
const sitePrivate = `site-private2-${random}`;

const userManager = `manager-${random}`;
const userConsumer = `consumer-${random}`;
Expand Down Expand Up @@ -181,7 +181,7 @@ test.describe('Special permissions : ', () => {
});

test.afterAll(async () => {
await Utils.deleteNodesSitesEmptyTrashcan(undefined, undefined, 'afterAll failed', managerSiteActions, [sitePrivate]);
await Utils.deleteNodesSitesEmptyTrashcan(undefined, undefined, 'afterAll failed', managerSiteActions, [docLibId]);
});

test.describe('Consumer', () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ test.describe('Download from Personal Files', () => {

const random = Utils.random();
const username = `user-${random}`;
const parent = `parent-${random}`;
const parent = `parent-download-${random}`;
const childFile = `childFile-${random}.txt`;
const childFolder = `childFolder-${random}`;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ test.describe('Version actions', () => {
const filenameBeforeUpdate = `${filesToUpload[0].name}-${random}`;
const filenameAfterUpdate = `${filesToUpload[1].name}-${random}`;
const username = `user-${random}`;
const parentFolder = `parent-${random}`;
const parentFolder = `parent-version-${random}`;
let parentFolderId: string;
let fileId: string;

Expand Down
2 changes: 1 addition & 1 deletion e2e/playwright/viewer/src/tests/viewer-useraction.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ import { SiteBodyCreate } from '@alfresco/js-api';
test.describe('from File Libraries', () => {
const apiClientFactory = new ApiClientFactory();
const username = `user-${Utils.random()}`;
const siteName = `site-${Utils.random()}`;
const siteName = `site-from-lib-${Utils.random()}`;
const destination = `destFL-${Utils.random()}`;
let destinationId: string;
const xlsxLibraries = `xlsxFL-${Utils.random()}`;
Expand Down
9 changes: 6 additions & 3 deletions e2e/playwright/viewer/src/tests/viewer.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,8 @@ test.describe('viewer file', () => {
let fileDocxId: string;
let nodesApi: NodesApi;
let trashcanApi: TrashcanApi;
let siteActionsAdmin: SitesApi;
let siteActionsUser: SitesApi;

test.beforeAll(async () => {
test.setTimeout(timeouts.extendedTest);
Expand All @@ -64,8 +66,8 @@ test.describe('viewer file', () => {
trashcanApi = await TrashcanApi.initialize(username, username);
const shareActions = await SharedLinksApi.initialize(username, username);
const favoritesActions = await FavoritesPageApi.initialize(username, username);
const siteActionsUser = await SitesApi.initialize(username, username);
const siteActionsAdmin = await SitesApi.initialize('admin');
siteActionsUser = await SitesApi.initialize(username, username);
siteActionsAdmin = await SitesApi.initialize('admin');
const fileActionApiAdmin = await FileActionsApi.initialize('admin');
const node = await nodesApi.createFolder(randomFolderName);
folderId = node.entry.id;
Expand Down Expand Up @@ -97,7 +99,8 @@ test.describe('viewer file', () => {
});

test.afterAll(async () => {
await Utils.deleteNodesSitesEmptyTrashcan(nodesApi, trashcanApi, 'afterAll failed');
await Utils.deleteNodesSitesEmptyTrashcan(nodesApi, trashcanApi, 'afterAll failed', siteActionsUser, [docLibSiteUserId]);
await Utils.deleteNodesSitesEmptyTrashcan(nodesApi, trashcanApi, 'afterAll failed', siteActionsAdmin, [docLibId]);
});

test('[C279269] Viewer opens on double clicking on a file from Personal Files', async ({ personalFiles }) => {
Expand Down

0 comments on commit 6cdd5da

Please sign in to comment.