Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ACS-9154] Fix unstable tests in ACA after migration to local ACS #4341

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

datguychen
Copy link
Contributor

JIRA ticket link or changeset's description
https://hyland.atlassian.net/browse/ACS-9154

@datguychen
Copy link
Contributor Author

datguychen commented Jan 17, 2025

Waiting for config.json file to be deleted from saved search to rebase the branch and see how the tests perform then.

EDIT: The changes for config.json will be done in the near future, because the changes were reverted in this release train.

@datguychen datguychen force-pushed the ACS-9154-fix-unstable-e2es-local-acs branch from 1dc915c to 4931dd1 Compare January 24, 2025 14:08
@datguychen datguychen force-pushed the ACS-9154-fix-unstable-e2es-local-acs branch from 15f80bf to d20d4ac Compare January 27, 2025 10:42
@datguychen datguychen marked this pull request as ready for review January 27, 2025 10:43
@AleksanderSklorz
Copy link
Contributor

I have one question - there is still excluded one test in e2e/playwright/search/exclude.tests.json: C277736, what with this one?

@datguychen
Copy link
Contributor Author

datguychen commented Jan 28, 2025

I have one question - there is still excluded one test in e2e/playwright/search/exclude.tests.json: C277736, what with this one? @AleksanderSklorz

This test checks the sorting of the dataTable by Modified date. Currently, the files uploaded during the test run have the same Modified date because they are uploaded simultaneously, resulting in inconsistent test outcomes. To re-enable this test, we need to consider a different approach.

At present, files are uploaded when the environment is created, which could be used for the Modified date comparison. However, we plan to switch the tests from the acadev environment to locally built environments with local ACS. This transition could be challenging due to the time difference between environment build (when environment files are created) and test run (when test files are created).

We need more time to evaluate how the tests perform with the locally built environments to determine if this approach would work for this e2e test.

I will create a ticket for this e2e test and update it in the exclude file in my next commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants