Skip to content

tests: update smoke testing#1011

Open
nabim777 wants to merge 2 commits intomasterfrom
docs/smoke-testing-update
Open

tests: update smoke testing#1011
nabim777 wants to merge 2 commits intomasterfrom
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
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think it would be good to mention realm-name example as opnc

Comment thread docs/testing/smoke_testing.md Outdated
Copy link
Copy Markdown

@Ashim-Stha Ashim-Stha Apr 24, 2026

Choose a reason for hiding this comment

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

should we mention to check I have a discovery endpoint URL as I don't have this information is checked by default and it might be confusing where to put discovery endpoint

Comment thread docs/testing/smoke_testing.md Outdated
Comment on lines 203 to 206
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

we are following the sequential actions , so do we need to tell to complete those sequential steps?

Comment thread docs/testing/smoke_testing.md Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

as an admin go to

- Client secret: Client secret of openproject from keycloak
- [ ] Login as keycloak-created user in `Openproject`.
- [ ] Log out, then Login as admin in `Openproject`.
- [ ] As an `OpenProject` admin, add keycloak-created user as a member in one of the project.
Copy link
Copy Markdown

@Ashim-Stha Ashim-Stha Apr 24, 2026

Choose a reason for hiding this comment

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

i think we should also add a storage in the project settings -> files in later step

Comment thread docs/testing/smoke_testing.md Outdated
Copy link
Copy Markdown

@Ashim-Stha Ashim-Stha Apr 28, 2026

Choose a reason for hiding this comment

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

what about going to the nextcloud, navigate to Administration settings> OpenProject , then put openproject url, select oauth (selected by default) first before copying the credentials

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

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 on lines 54 to 58
### A3. Add File storage (Nextcloud) to an OpenProject project
- [ ] Complete step [Test No A1](#a1-oauth-configuration).
- [ ] Select an `OpenProject` Project (for example, `Demo Project`) in `OpenProject`.
- [ ] Navigate to `Project settings > Files` of `Demo Project`.
- [ ] Add a file storage name `Nextcloud`( choose `No specific Folder` option ) for `Demo Project`.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

this action is done by admin, its confusing as in the A2 section, created user is logged in, and this A3 section requires to be logged in as an admin

Comment thread docs/testing/smoke_testing.md Outdated
@@ -53,5 +54,5 @@
### A3. Add File storage (Nextcloud) to an OpenProject project
- [ ] Complete step [Test No A1](#a1-oauth-configuration).
- [ ] Select an `OpenProject` Project (for example, `Demo Project`) in `OpenProject`.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Select any project

Copy link
Copy Markdown
Collaborator Author

@nabim777 nabim777 Apr 29, 2026

Choose a reason for hiding this comment

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

what is wrong here i don't understand?

Comment thread docs/testing/smoke_testing.md Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

its good to mention in openproject and as an admin

Comment thread docs/testing/smoke_testing.md Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

we have already created user created in above steps

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.

removed it

@@ -67,23 +68,8 @@
- [ ] Try to connect the created `OpenProject` user as created `Nextcloud` user.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

this action is done by created user, so we need to login as created user in both openproject and nextcloud first

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.

The reader will understand that logging in is required to perform this test. So, I think it is clear

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

would be good to clarify we have to go to the nextcloud as an openproject user -> settings -> openproject -> connect to openproject (we need to be logged in as created user in openproject too)

Comment thread docs/testing/smoke_testing.md Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

we can click on the respective ID rather than double click as double clicking around the text area selects the text which is annoying

Comment thread docs/testing/smoke_testing.md Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

In nextcloud, not in openproject

Comment thread docs/testing/smoke_testing.md Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

in openproject

Comment thread docs/testing/smoke_testing.md Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

in openproject

Comment thread docs/testing/smoke_testing.md Outdated
Copy link
Copy Markdown

@Ashim-Stha Ashim-Stha Apr 28, 2026

Choose a reason for hiding this comment

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

rather than saying open the form, search for work package -> create and link a new work package , in nextcloud

Comment thread docs/testing/smoke_testing.md Outdated

### 6. Check New folder with automatically managed permissions in OpenProject
#### 6. Setup and check project folder in Nextcloud (with project folder setup)
- [ ] Enable `groupfolders` application in `Nextcloud`.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

would be good to mention how to enable it in short

Comment thread docs/testing/smoke_testing.md Outdated
Comment on lines +382 to +383
- [ ] Enable `Automatically managed folders` switch in admin setting and set project folder.
- [ ] Application password should be generated.
Copy link
Copy Markdown

@Ashim-Stha Ashim-Stha Apr 28, 2026

Choose a reason for hiding this comment

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

Then copy and paste the generated application password from nextcloud to openproject

Comment thread docs/testing/smoke_testing.md Outdated

### 6. Check New folder with automatically managed permissions in OpenProject
#### 6. Setup and check project folder in Nextcloud (with project folder setup)
- [ ] Enable `groupfolders` application in `Nextcloud`.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

groupfolders or teamfolders?

### 6. Check New folder with automatically managed permissions in OpenProject
#### 6. Setup and check project folder in Nextcloud (with project folder setup)
- [ ] Enable `groupfolders` application in `Nextcloud`.
- [ ] Enable `Automatically managed folders` switch in admin setting and set project folder.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

in both nextcloud and openproject

Comment thread docs/testing/smoke_testing.md Outdated
- [ ] Enable `groupfolders` application in `Nextcloud`.
- [ ] Enable `Automatically managed folders` switch in admin setting and set project folder.
- [ ] Application password should be generated.
- [ ] Verify that `OpenProject` user and group are created with user `OpenProject` as sub-admin of the group.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

OpenProject user and OpenProject group

Comment thread docs/testing/smoke_testing.md Outdated
- [ ] Application password should be generated.
- [ ] Verify that `OpenProject` user and group are created with user `OpenProject` as sub-admin of the group.
- [ ] Verify that `OpenProjectNoAutomaticProjectFolders` group is also created with user `OpenProject` as sub-admin.
- [ ] Try deleting `OpenProject` user and group, those should not be deleted.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Comment thread docs/testing/smoke_testing.md Outdated
- [ ] Enable `groupfolders` application in `Nextcloud`.
- [ ] Enable `Automatically managed folders` switch in admin setting and set project folder.
- [ ] Application password should be generated.
- [ ] Verify that `OpenProject` user and group are created with user `OpenProject` as sub-admin of the group.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

in nextcloud

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.

#### 6. Setup and check project folder in Nextcloud (with project folder setup)

this step heading itself mentioned in nextcloud. so, i don't think we need to again mention here again.

- [ ] Enable `Automatically managed folders` switch in admin setting and set project folder.
- [ ] Application password should be generated.
- [ ] Verify that `OpenProject` user and group are created with user `OpenProject` as sub-admin of the group.
- [ ] Verify that `OpenProjectNoAutomaticProjectFolders` group is also created with user `OpenProject` as sub-admin.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

in nextcloud

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.

same comment

#1011 (comment)

Comment thread docs/testing/smoke_testing.md Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

these names OIDC Identity Provider and OpenID Connect user backend would be helpful

Comment thread docs/testing/smoke_testing.md Outdated
- [ ] Go to `Administation > OpenID Connect Provider`
- Click the button `+ Add client`:
- Add a client name (not an identifier) such as `openproject`
- Add a redirect URL (<openproject_host>/auth/oidc-<idp-displayname-from-OP>/callback)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
- Add a redirect URL (<openproject_host>/auth/oidc-<idp-displayname-from-OP>/callback)
- Add a redirect URL (<openproject_host>/auth/oidc-<idp-displayname-from-openproject>/callback)

Comment on lines 102 to 103
- [ ] Run following command:
- `php occ config:system:set user_oidc --type boolean --value="true" oidc_provider_bearer_validation`
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

good to mention its needed for only local setup

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.

may be sometime missing in other setup also

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.

best would be to explain why this command is needed in the first place and what it does

Copy link
Copy Markdown
Collaborator Author

@nabim777 nabim777 May 5, 2026

Choose a reason for hiding this comment

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

added this:

- Note: This enables bearer token validation for `user_oidc`. Without it, login or connection may fail. This may also be missing during local setups.


#### B.1.2. Add Nextcloud IDP in OpenProject
#### B.1.2. Add Nextcloud IDP in OpenProject (Without project folder setup)
- [ ] In OpenProject, go to `Administration > Authentication > OpenID providers`
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

we need to add an Enterprise token before this step

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.

Done! I have mentioned it before starting the sso tests

Comment thread docs/testing/smoke_testing.md Outdated
- [ ] In nextcloud, go to `Administration > OpenProject`.
- [ ] Under `Authentication Method`, select `Single-Sign-On through OpenID Connect Identity Provider`.
- [ ] In `Authentication settings`, select `provider Type` as `Nextcloud Hub`.
- [ ] Set Openproject `client ID` by Client ID copied earlier in **Test No B1**.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

also uncheck project folder (automatically managed folder).

- [ ] `OpenProject` admin is connected to `Nextcloud` as a `Nextcloud` admin.
- [ ] Also, create a user in both `Nextcloud` as well as `OpenProject`.
- [ ] Add the created `OpenProject` user as the member of `Demo Project` project (admin can add members to a project).
- [ ] Try to connect the created `OpenProject` user as created `Nextcloud` user.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
- [ ] Try to connect the created `OpenProject` user as created `Nextcloud` user.
- [ ] Login as created user in both openproject and nextcloud
- [ ] Try to connect the created `OpenProject` user as created `Nextcloud` user.

@nabim777 nabim777 self-assigned this May 5, 2026
@nabim777 nabim777 force-pushed the docs/smoke-testing-update branch from cec9972 to 16c3300 Compare May 5, 2026 04:20
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
@nabim777 nabim777 force-pushed the docs/smoke-testing-update branch from 16c3300 to eb52330 Compare May 5, 2026 04:22
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
@nabim777 nabim777 force-pushed the docs/smoke-testing-update branch from eb52330 to c9b81f4 Compare May 5, 2026 04:28
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

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.

3 participants