Skip to content

Commit

Permalink
Merge branch 'dev' into km_WOR-132_addtional_testing
Browse files Browse the repository at this point in the history
  • Loading branch information
kevinmarete committed Nov 27, 2024
2 parents eebf7c6 + d95b49d commit 6b43400
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
2 changes: 0 additions & 2 deletions integration-tests/tests/preview-drs-uri.js
Original file line number Diff line number Diff line change
Expand Up @@ -54,8 +54,6 @@ const testPreviewDrsUriFn = _.flow(
await click(page, clickable({ textContains: testEntity.entityType }));
console.log('opening preview for entity...');
await click(page, elementInDataTableRow(testEntity.name, testEntity.attributes.file_uri));
console.log('waiting for no spinners in preview modal...');
await waitForNoSpinners(page);
console.log('verifying text in preview modal...');
await findText(page, 'Filename');
await findText(page, 'File size');
Expand Down
7 changes: 6 additions & 1 deletion integration-tests/utils/integration-helpers.js
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,12 @@ const withWorkspace = (test) => async (options) => {
await test({ ...options, workspaceName });
} finally {
console.log('withWorkspace cleanup ...');
const didDelete = await withSignedInPage(deleteWorkspaceInUi)({ ...options, workspaceName });
let didDelete = false;
try {
didDelete = await withSignedInPage(deleteWorkspaceInUi)({ ...options, workspaceName });
} catch (err) {
console.error(`Error during workspace cleanup: ${err}`);
}
if (!didDelete) {
// Pass test on a failed cleanup - expect leaked resources to be cleaned up by the test `delete-orphaned-workspaces`
console.error(`Unable to delete workspace ${workspaceName} via the UI. The resource will be leaked!`);
Expand Down

0 comments on commit 6b43400

Please sign in to comment.