Skip to content

tests: update smoke testing#1011

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

tests: update smoke testing#1011
nabim777 wants to merge 5 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
Comment thread docs/testing/smoke_testing.md Outdated
Comment thread docs/testing/smoke_testing.md Outdated
- [ ] Create a new user( with username, display name, password, and email)
- [ ] Run following command:
- `php occ config:system:set user_oidc --type boolean --value="true" oidc_provider_bearer_validation`
- Note: This enables bearer token validation for `user_oidc`. Without it, login or connection may fail. This may also be missing during local setups.
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.

still unclear that its required for local setup only

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.

I ahave done the changes there. Could you check if it is clear and easy to understand?

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
- Discovery endpoint: `<keycloak_instance_url>/realms/<realm-name>/.well-known/openid-configuration` (for example realm name can be `opnc`)
- Scope: `openid email profile api_v3`
- submit
- [ ] Login as keycloak-created user in `Nextcloud`.
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.

it would be good to add link to the documentation where we can see the keycloak users credentials

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.

I don’t think there is any dedicated documentation for the user credentials.
In the Docker setup, a script automatically creates a user and sets the password, so we use that existing user. In other setups, you can simply create a new user manually and set the password yourself.

Docker setup password is set here:

Copy link
Copy Markdown
Collaborator Author

@nabim777 nabim777 May 19, 2026

Choose a reason for hiding this comment

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

I made some changes there. Please take a look and let me know if it looks good or not.

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.

I don’t think there is any dedicated documentation for the user credentials. In the Docker setup, a script automatically creates a user and sets the password, so we use that existing user. In other setups, you can simply create a new user manually and set the password yourself.

Docker setup password is set here:

integration_openproject/dev/keycloak/keycloak.sh

Line 64 in 65f7742

     "value": "1234",

will I or someone remember or where to look in the future?

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.

I added it. Could you check if it is clear and easy to understand?

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
@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 added 5 commits May 22, 2026 15:54
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>
@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 ✅

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