Skip to content

tests: update smoke testing#1011

Open
nabim777 wants to merge 6 commits into
masterfrom
docs/smoke-testing-update
Open

tests: update smoke testing#1011
nabim777 wants to merge 6 commits into
masterfrom
docs/smoke-testing-update

Conversation

@nabim777
Copy link
Copy Markdown
Collaborator

@nabim777 nabim777 commented Apr 21, 2026

Description

This PR updates the smoke testing documents.

Related Issue or Workpackage

  • Fixes <issue_link>

Screenshots (if appropriate):

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)
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Updated CHANGELOG.md file

Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
- [ ] In `Nextcloud`, navigate to `Personal Settings > Openproject` and click on `Connect to OpenProject` button.
- [ ] `Nextcloud` admin should be connected as an `OpenProject` admin.
- [ ] Also, create a user in both `Nextcloud` as well as `OpenProject`.
- [ ] From the personal section of the created user in `Nextcloud`, connect to `OpenProject`.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

login as created user and from the settings -> connect...

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Why is it needed?

Comment thread docs/testing/smoke_testing.md
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
@nabim777 nabim777 force-pushed the docs/smoke-testing-update branch 2 times, most recently from 69f594b to 04254b3 Compare May 19, 2026 05:17
@nabim777 nabim777 requested a review from Ashim-Stha May 19, 2026 05:17
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
@Ashim-Stha
Copy link
Copy Markdown
Collaborator

could you please add missing fullstops(.) and also making consistent either as an admin or as a user admin?

@nabim777 nabim777 force-pushed the docs/smoke-testing-update branch from 04254b3 to 2bb1e85 Compare May 22, 2026 10:38
@nabim777 nabim777 requested a review from Ashim-Stha May 22, 2026 10:39
@nabim777
Copy link
Copy Markdown
Collaborator Author

could you please add missing fullstops(.) and also making consistent either as an admin or as a user admin?

Done ✅

Copy link
Copy Markdown

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 updates the smoke testing documentation for the OpenProject ↔ Nextcloud integration, expanding and clarifying manual test steps for OAuth and OIDC/SSO setups and common smoke tests.

Changes:

  • Expanded/restructured OAuth and OIDC/SSO smoke test instructions (including project-folder related checks).
  • Updated Common Smoke Test Steps and App Upgrade Testing sections (new/renumbered steps and wording improvements).
  • Revised the Table of Contents to reflect the updated smoke test flow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md
@nabim777 nabim777 requested a review from Copilot May 25, 2026 09:25
@nabim777 nabim777 force-pushed the docs/smoke-testing-update branch 2 times, most recently from c35fdd9 to da1a228 Compare May 25, 2026 09:26
Copy link
Copy Markdown

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 1 out of 1 changed files in this pull request and generated 14 comments.

Comments suppressed due to low confidence (1)

docs/testing/smoke_testing.md:100

  • This cross-reference uses an uppercase fragment (#A2-...). GitHub-generated heading IDs are lowercase, so this link won’t resolve. Update the fragment to match the generated ID for the “### A2. Connect Nextcloud with OpenProject” heading.
- [ ] Upon success, try step [Test No A2](#A2-Connect-Nextcloud-with-OpenProject) (Skip first check).
- [ ] Upon success, try step [Test No A4](#a4-connect-openproject-with-nextcloud) (Skip first check).

Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md
Comment thread docs/testing/smoke_testing.md
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md
Comment thread docs/testing/smoke_testing.md
Comment thread docs/testing/smoke_testing.md Outdated
nabim777 added 5 commits May 25, 2026 15:32
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Copy link
Copy Markdown

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 1 out of 1 changed files in this pull request and generated 13 comments.

Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
@nabim777 nabim777 force-pushed the docs/smoke-testing-update branch from 364d176 to 9fb6e35 Compare May 25, 2026 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants