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] Acceptance test changes pending for master #39612

Merged
merged 17 commits into from
Dec 22, 2021

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented Dec 21, 2021

Description

These are the pending acceptance test changes that have been accumulated in PR #39514 waiting for release-10.9.0 to be merged. This PR is to the new tip of master. Each commit has already been reviewed in the original PR and the code has been cherry-picked here - hopefully it is easy to review!

Now these test changes are ready to merge:

  1. Improve checks in tests for disabling a user - related to issues no enable/disable user provisioning API endpoint ocis#1420 and Improve Then steps in API acceptance tests #39512 - this provides extra checks in test scenarios that will help the oCIS test suite to fail for the right reason on these test scenarios.

  2. Reduce use of "download range" in acceptance tests - part of Reduce use of "with range" in acceptance tests #39513

  3. Fix SetupHelper runOcc to use strings only in exception messages

  4. Add tests to demonstrate group names with leading and trailing spaces

  5. Adjust group tests for issue Creating a group with space at end does not work #39533

  6. Refactor getGroups tests to not depend on the starting groups - this removes the need to list the pre-existing groups in a system-under-test.

  7. Fix missing HTTP status code in createPublicLinkShare test scenarios

  8. Use when step for the test scenario with only Then. (small refactoring of webUIAdminSettings/adminGeneralSettings.feature )

  9. Fixed order dependent flaky test detected in files_classifier app (cherry-picked from PR [tests-only]Fixed flaky test detected in files_classifiers #39548 )

  10. add tests for db-convert occ command (cherry-picked and squashed from PR [full-ci][tests-only] Add tests for db-convert occ command #39393 )

  11. Do not allow white space at the start or end of a group name (cherry-picked and squashed from PR Do not allow white space at the start or end of a group name #39540 ) Having this here will allow the API tests to run in oCIS, and group name validation rules can be corrected there as needed.

  12. Add bug demonstration test for sending PUT request to other user's webdav endpoint

  13. refactor test helpers and contexts for parallel deployment tests (code from [full-ci][tests-only] Refactor test codes for parallel deployment tests #39603 )

  14. skipOnEncryption dbConversion.feature as it needs special CI setup - skips the dbConversion scenario, because that is a very special scenario that requires starting with sqlite installed.

  15. added tests for files_external occ command - cherry-picked from PR [tests-only] [full-ci]tests for occ command files_external #39511

  16. Add spaces as a new webdav url type - merged from [tests-only] Add spaces as a new WebDAV url type #39599

  17. Add extra checks to getPersonalSpaceIdForUser - to better report when a space can't be found

How Has This Been Tested?

CI

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE

@sonarcloud
Copy link

sonarcloud bot commented Dec 21, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@phil-davis phil-davis merged commit 35fa936 into master Dec 22, 2021
@delete-merged-branch delete-merged-branch bot deleted the acceptance-test-changes-waiting-2021-12 branch December 22, 2021 04:13
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.

7 participants