Skip to content

Added first test case documentation#9321

Merged
mgallien merged 1 commit intomasterfrom
i2h3/doc/test-cases
Jan 28, 2026
Merged

Added first test case documentation#9321
mgallien merged 1 commit intomasterfrom
i2h3/doc/test-cases

Conversation

@i2h3
Copy link
Copy Markdown
Collaborator

@i2h3 i2h3 commented Jan 15, 2026

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.

@i2h3 i2h3 added this to the 4.1.0 milestone Jan 15, 2026
@i2h3 i2h3 self-assigned this Jan 15, 2026
Copilot AI review requested due to automatic review settings January 15, 2026 12:44
@i2h3 i2h3 requested a review from Aiiaiiio as a code owner January 15, 2026 12:44
@i2h3 i2h3 added the CI build/checks continuous integration, Drone, testing label Jan 15, 2026
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 💻 Desktop Clients team Jan 15, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.

Comment thread doc/test-cases/integration/TC-1003-delete-a-file-on-the-server.md Outdated
Comment thread doc/test-cases/integration/macos-file-provider/TC-1000-delete-a-file.md Outdated
Comment thread doc/test-cases/integration/macos-file-provider/TC-1001-restore-a-file.md Outdated
Comment thread doc/test-cases/integration/macos-file-provider/TC-1002-empty-the-trash.md Outdated
Comment thread doc/test-cases/integration/TC-1004-empty-the-trash-on-the-server.md Outdated
@i2h3 i2h3 force-pushed the i2h3/doc/test-cases branch from 01b48cf to e373d11 Compare January 19, 2026 14:31
Copy link
Copy Markdown
Collaborator

@mgallien mgallien left a comment

Choose a reason for hiding this comment

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

can you separate the test case that are platform specific into different folders ?

Comment thread doc/test-cases/integration/TC-1000-delete-a-file.md Outdated
Comment thread doc/test-cases/integration/TC-1001-restore-a-file.md Outdated
Comment thread doc/test-cases/integration/TC-1003-delete-a-file-on-the-server.md Outdated
@i2h3 i2h3 force-pushed the i2h3/doc/test-cases branch from e373d11 to c534c97 Compare January 21, 2026 15:19
@i2h3
Copy link
Copy Markdown
Collaborator Author

i2h3 commented Jan 21, 2026

@mgallien Updated according to your change request.

@i2h3 i2h3 requested review from Copilot and mgallien January 21, 2026 15:19
@i2h3 i2h3 force-pushed the i2h3/doc/test-cases branch 2 times, most recently from 441c102 to 10032c3 Compare January 21, 2026 15:26
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.

Comment thread doc/test-cases/integration/macos-file-provider/TC-1001-restore-a-file.md Outdated
Comment thread doc/test-cases/smoke/macos/TC-0002-installation-on-macos.md
Comment thread doc/test-cases/smoke/windows-virtual-files/TC-0004-installation-on-windows-vfs.md Outdated
Comment thread doc/test-cases/smoke/linux/TC-0003-installation-on-linux.md Outdated
@i2h3 i2h3 force-pushed the i2h3/doc/test-cases branch from 10032c3 to 0dc4881 Compare January 21, 2026 15:41
@i2h3 i2h3 enabled auto-merge January 21, 2026 15:41
@sonarqubecloud
Copy link
Copy Markdown

Please retry analysis of this Pull-Request directly on SonarQube Cloud

@i2h3 i2h3 force-pushed the i2h3/doc/test-cases branch from 0dc4881 to 87696a6 Compare January 23, 2026 13:03
@i2h3 i2h3 moved this from 🧭 Planning evaluation (don't pick) to 🏗️ In progress in 💻 Desktop Clients team Jan 23, 2026
@github-actions
Copy link
Copy Markdown

Artifact containing the AppImage: nextcloud-appimage-pr-9321.zip

Digest: sha256:026a590142fc301f03d2f6befd6f88c1607586c653da93c04494ef2754a998b1

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.

@mgallien mgallien disabled auto-merge January 28, 2026 14:12
Signed-off-by: Iva Horn <iva.horn@nextcloud.com>
@mgallien mgallien force-pushed the i2h3/doc/test-cases branch from 87696a6 to 7c4478e Compare January 28, 2026 14:12
@mgallien mgallien merged commit 4130c00 into master Jan 28, 2026
15 of 18 checks passed
@mgallien mgallien deleted the i2h3/doc/test-cases branch January 28, 2026 14:12
@github-project-automation github-project-automation Bot moved this from 🏗️ In progress to ☑️ Done in 💻 Desktop Clients team Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review CI build/checks continuous integration, Drone, testing documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants