Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces the first documentation for test cases as part of improving the release process. It establishes a framework for manual testing with three categories (smoke, regression, integration) and provides five initial integration test cases focused on the macOS File Provider extension's trash functionality.
Changes:
- Created directory structure and documentation for test cases
- Added five integration test cases covering file deletion, restoration, and trash operations
- Documented the test process and categorization approach
Reviewed changes
Copilot reviewed 6 out of 8 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| doc/test-cases/README.md | Explains the test process, categories, and format |
| doc/test-cases/smoke/.gitignore | Placeholder for smoke tests directory |
| doc/test-cases/regression/.gitignore | Placeholder for regression tests directory |
| doc/test-cases/integration/TC-1000-delete-a-file.md | Test case for deleting files locally |
| doc/test-cases/integration/TC-1001-restore-a-file.md | Test case for restoring deleted files |
| doc/test-cases/integration/TC-1002-empty-the-trash.md | Test case for emptying trash locally |
| doc/test-cases/integration/TC-1003-delete-a-file-on-the-server.md | Test case for deleting files on server |
| doc/test-cases/integration/TC-1004-empty-the-trash-on-the-server.md | Test case for emptying trash on server |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
01b48cf to
e373d11
Compare
mgallien
left a comment
There was a problem hiding this comment.
can you separate the test case that are platform specific into different folders ?
e373d11 to
c534c97
Compare
|
@mgallien Updated according to your change request. |
441c102 to
10032c3
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 17 changed files in this pull request and generated 10 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
10032c3 to
0dc4881
Compare
|
Please retry analysis of this Pull-Request directly on SonarQube Cloud |
0dc4881 to
87696a6
Compare
|
Artifact containing the AppImage: nextcloud-appimage-pr-9321.zip Digest: To test this change/fix you can download the above artifact file, unzip it, and run it. Please make sure to quit your existing Nextcloud app and backup your data. |
Signed-off-by: Iva Horn <iva.horn@nextcloud.com>
87696a6 to
7c4478e
Compare
This needs further discussion inside the team. It is a start, for now. We do not have a full QA engineering team to work on automated end-to-end tests so we need to figure out a highly efficient middle group to improve our testing and release process for the desktop client.