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

[tests-only] Removed scenario related to folder upload from old web UI test #10480

Merged
merged 2 commits into from
Feb 15, 2024

Conversation

PrajwolAmatya
Copy link
Contributor

@PrajwolAmatya PrajwolAmatya commented Feb 14, 2024

Description

Removed a bug demonstration test scenario related to issue #2443 from the web UI test and added a step for uploading folder in public link share. As the folder upload is not available in playwright, the step for E2E test is added and is commented for now.
The test is also done manually and the issue doesn't occur in the latest master branch.

Related Issue

Screenshots (if appropriate):

Result from manual test.

Screencast.from.02-14-2024.11.33.42.AM.webm

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests
  • Documentation
  • Maintenance (e.g. dependency updates or tooling)

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link
Contributor

@amrita-shrestha amrita-shrestha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Member

@SagarGi SagarGi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@PrajwolAmatya PrajwolAmatya merged commit 5966e89 into master Feb 15, 2024
3 checks passed
@delete-merged-branch delete-merged-branch bot deleted the upload-folder-to-link-share branch February 15, 2024 11:10
saw-jan pushed a commit that referenced this pull request Mar 5, 2024
…I test (#10480)

* removed scenario related to upload folder in link share from web UI test

* added a step for folder upload in link share
saw-jan pushed a commit that referenced this pull request Mar 5, 2024
…I test (#10480)

* removed scenario related to upload folder in link share from web UI test

* added a step for folder upload in link share
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants