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][full-ci] removed webUIFilesList test suite from old acceptance test #10588

Merged
merged 1 commit into from
Mar 21, 2024

Conversation

PrajwolAmatya
Copy link
Contributor

@PrajwolAmatya PrajwolAmatya commented Mar 12, 2024

Description

In this PR, removed the webUIFilesList test suite, which consists of filesList.feature and sort.feature files.

Removed tests:

  • Scenario: Resources are listed and files list displays quicklink quick action: covered in link.feature L57

  • Scenario: Empty folders display no resources in the list: covered in most scenarios with step creating folder and uploading resources

  • Scenario: files are not selected when the user logs in: mostly covered in copy move tests

  • Scenario: select files: mostly covered in copy move tests

  • tests related to sorting files/folders: covered by unit tests

  • Forward port to master: [tests-only][full-ci]removed webUIFilesList test suite from webUI tests #10655

Related Issue

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)

@PrajwolAmatya PrajwolAmatya force-pushed the test/try-create-file-with-existing-name branch from 361864d to 731a901 Compare March 12, 2024 11:30
@PrajwolAmatya PrajwolAmatya self-assigned this Mar 12, 2024
@PrajwolAmatya PrajwolAmatya force-pushed the test/try-create-file-with-existing-name branch 3 times, most recently from 1322821 to 2c498f6 Compare March 18, 2024 06:15
@PrajwolAmatya PrajwolAmatya changed the title [tests-only][full-ci] added e2e test to create a new file with existing name [tests-only][full-ci] removed webUIFilesList test suite from old acceptance test Mar 18, 2024
@PrajwolAmatya PrajwolAmatya marked this pull request as ready for review March 18, 2024 06:42
@PrajwolAmatya PrajwolAmatya changed the title [tests-only][full-ci] removed webUIFilesList test suite from old acceptance test [tests-only][full-ci] removed webUIFilesList test suite from old acceptance test and E2E test added for sorting files/folders Mar 20, 2024
@PrajwolAmatya PrajwolAmatya force-pushed the test/try-create-file-with-existing-name branch from e963e11 to 43a512a Compare March 20, 2024 08:28
@PrajwolAmatya PrajwolAmatya force-pushed the test/try-create-file-with-existing-name branch from 43a512a to a1a4834 Compare March 21, 2024 06:06
@PrajwolAmatya PrajwolAmatya changed the title [tests-only][full-ci] removed webUIFilesList test suite from old acceptance test and E2E test added for sorting files/folders [tests-only][full-ci] removed webUIFilesList test suite from old acceptance test Mar 21, 2024
Copy link

sonarcloud bot commented Mar 21, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud

@saw-jan saw-jan merged commit 891c7c7 into stable-8.0 Mar 21, 2024
3 checks passed
@delete-merged-branch delete-merged-branch bot deleted the test/try-create-file-with-existing-name branch March 21, 2024 08:17
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