Conversation
|
Original pull request: |
|
This is depending on |
|
@pkamps just adding this here to keep track With the dependency of the ezpublish PR in place, the unit tests work well, with the exception of the HTTP request timeout test, which fails with the following message: I'm not convinced this is an issue in general and may be related to the test VM setup somehow, but I haven't looked into it further yet. |
|
Have you had a chance to see if you can replicate that one test issue on your local @pkamps ? |
|
That test checks if the timeout and retry configuration for a solr connection is working correctly. The test is actually running fine for me: The $connectionTimeout and $maxRetries is set in That does not seem to work in your environment, because $connectionTimeout * $maxRetries is 40 for you. If you like we do a shared screen session and debug the issue. |
Unittest were pretty broken/not maintained. So even if this pull request is not perfectly fixing the unit tests, it's huge step forward.