-
-
Notifications
You must be signed in to change notification settings - Fork 153
Use QJazz in tests #5168
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
base: master
Are you sure you want to change the base?
Use QJazz in tests #5168
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.
I tried it locally, it works.
Still an issue on CI for now.
Will tags such as 3.34
, 3.40
will be available on Docker hubs later ?
23d1e4b
to
ca13211
Compare
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.
- Needed QGIS Server plugins are not installed in the current state of this PR. That's why the CI is not working. It was working locally, because my plugins were already existing before.
- Can you remove the script
tests/add_server_plugins.sh
? - In
tests/add_server_plugins.sh
, we need to use to the custom repository for QGIS plugins. Please do the change - Will tags such as
3.34
,3.40
will be available on Docker hubs ? https://hub.docker.com/r/3liz/qjazz/tags
Is-it not missing the mount for QGIS plugins in |
For testing purpose, I have pressed the rebased button, to have latest commits from master. |
Rules for plugins are the following:
|
00fba5e
to
42aefa7
Compare
There is tests checking The qjazz proxy handle standard The path returned in the service response may be changed with the appropriate |
Found that QGIS does not stick to standard and handle headers in case sensitive way, at least for 'X-Qgis-*' headers. |
I cannot check what's wrong on failed test, i need at least the request that failed if this is is a request to the QGIS service. |
@Gustry : please check the timeout error on Playwright test, with proper rights there is no issue locally with the url I can guess from the details. |
4bbaab3
to
51d7be8
Compare
Something is very slow in this PR. |
@Gustry LZMQGSRVVERSION is wrong in test: 3.34, should |
Fixed qgis-server value in matrix configuration. |
@Gustry tests failed for reasons not related to Qjazz |
Yes, we know the status of the current CI on the master branch, there are some ongoing work. |
8d8450a
to
990cff4
Compare
7494821
to
ddf2022
Compare
a9fe19e
to
f956a0d
Compare
599596e
to
04b9361
Compare
Fixed |
@Gustry is there a way to stop the tests at first failure and generate logs ? All tests are failing with a 30s timeout, that means we must wait for hours before getting the logs. |
e4ff4e1
to
f34b6f4
Compare
@Gustry BLEEDING EDGE tests are completely failing because the plugin source url is invalid:
And thus the lizmap server plugin is not installed.... AMHA tests should not run at all if the Lizmap server plugin is not installed. |
QJazz QGIS server in tests.