-
Notifications
You must be signed in to change notification settings - Fork 187
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[docs-only] Change Testing section in the doc #5430
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the testing docs is outdated. Thank you for this PR. Just a few suggestions otherwise looks good to me. I'll look into the docker section and make the necessary changes for testing.
Note, when I was looking at section This IDM_ADMIN_PASSWORD=admin ocis/bin/ocis init --insecure true
OCIS_INSECURE=true PROXY_ENABLE_BASIC_AUTH=true ocis/bin/ocis server should imho be changed to IDM_ADMIN_PASSWORD=admin \
ocis/bin/ocis init --insecure true
OCIS_INSECURE=true \
PROXY_ENABLE_BASIC_AUTH=true \
ocis/bin/ocis server |
Kudos, SonarCloud Quality Gate passed! |
Author: Viktor Scharf <[email protected]> Date: Fri Jan 20 11:54:25 2023 +0100 [docs-only] Change Testing section in the doc (#5430) * change docs * fix after review
* change docs * fix after review
[docs-only] Change Testing section in the doc (#5430)
when @Excds tried to run api test based on the documentation, it turned out that the docs is not up to date
I updated docs tests agains ocis binary
@phil-davis @individual-it could you please have look on the https://owncloud.dev/ocis/development/testing/#testing-with-test-suite-in-docker part. it practically doesn't work. does it need to be fixed?