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

Feature: Addition of E2E script #182

Merged
merged 1 commit into from
May 29, 2024
Merged

Feature: Addition of E2E script #182

merged 1 commit into from
May 29, 2024

Conversation

dipankardas011
Copy link
Collaborator

@dipankardas011 dipankardas011 commented May 23, 2024

Task Description

It adds a bash script for checking performing the e2e test cases

Action items

  • --yes / -y option for automating the e2e script for delete and creation cleanup steps
  • ssl testing
  • Building from source and utilizing that is needed
  • migration testing
  • Start and Stop

@dipankardas011 dipankardas011 requested a review from Xieyt May 23, 2024 13:06
@dipankardas011 dipankardas011 force-pushed the feat-add-e2e-test branch 2 times, most recently from e5025d8 to d49ccbe Compare May 24, 2024 10:03
@dipankardas011 dipankardas011 marked this pull request as ready for review May 24, 2024 10:08
@dipankardas011 dipankardas011 force-pushed the feat-add-e2e-test branch 9 times, most recently from 00c15bd to 26bbd3f Compare May 27, 2024 12:15
@dipankardas011
Copy link
Collaborator Author

dipankardas011 commented May 27, 2024

Problem in migration e2e tests

  • when a site is created in v0.9.0 and without any wait you try to perform curl operation it returns 5XX status code, but after some time it returns 2xx
    Status: Fixed
    Fix: Updated the curl command with retry flag

  • when a site is running in v0.9.0 and migrated to latest version there is problem deleting the site becuase there is some migration setup where the folder perm changed to 1000 uid,guid and thus removing access to the folder for the site from the current user
    Status: Fixed
    Fix: Updated the removeal of ~/frappe dir with sudo

cc @Xieyt

@dipankardas011 dipankardas011 force-pushed the feat-add-e2e-test branch 3 times, most recently from 85c608b to 0de3cdd Compare May 28, 2024 10:18
@dipankardas011 dipankardas011 force-pushed the feat-add-e2e-test branch 3 times, most recently from 3690034 to 04dbd87 Compare May 29, 2024 09:34
Signed-off-by: Dipankar Das <65275144+dipankardas011@users.noreply.github.com>
@Xieyt Xieyt merged commit 46a76c7 into develop May 29, 2024
1 check passed
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.

2 participants