Skip to content
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

More tests for CI/CD #2751

Open
Diman0 opened this issue Apr 7, 2023 · 0 comments
Open

More tests for CI/CD #2751

Diman0 opened this issue Apr 7, 2023 · 0 comments
Labels
priority/p1 (Critical) bug with workaround / Should have type/enhancement Enhances existing functionality

Comments

@Diman0
Copy link
Member

Diman0 commented Apr 7, 2023

We have seen in the last couple of weeks multiple bugs that could have been prevented with better test coverage.

Let's gather in this issue the functionality that should have (better) tests.

  • Config-export
  • Config-import
  • Webdav connectivity
  • webmail
  • pop3

config-export/import

For Config-export/import we could just do an export and then a replacing import.

webdav

For testing webdav, we could initialize the webdav container with an existing database for a user.
Then we use curl (?) to submit a webdav request to the webdav endpoint.
If the response contains the expected response code, then the test succeeds

webmail

We need two tests for both roundcube/snappymail.
enable header authentication
curl to submit a request for /webmail that is automatically logged in.
I think the result of the above curl should be the main page of roundcube/snappymail?

@Diman0 Diman0 added priority/p1 (Critical) bug with workaround / Should have type/enhancement Enhances existing functionality labels Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority/p1 (Critical) bug with workaround / Should have type/enhancement Enhances existing functionality
Projects
None yet
Development

No branches or pull requests

1 participant