Commit 8f1021c
1 parent af6b4cc commit 8f1021c
1 file changed
+1
-0
lines changedLines changed: 1 addition & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
29 | 29 |
| |
30 | 30 |
| |
31 | 31 |
| |
| 32 | + | |
32 | 33 |
| |
33 | 34 |
| |
34 | 35 |
| |
|
1 parent af6b4cc commit 8f1021c
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
29 | 29 |
| |
30 | 30 |
| |
31 | 31 |
| |
| 32 | + | |
32 | 33 |
| |
33 | 34 |
| |
34 | 35 |
| |
|
4 commit comments
m-kuhn commentedon Nov 19, 2017
I had to disable this test as it was becoming really unstable recently. I'm sorry, I'm not able to fix it myself and it's currently giving many false alarms.
Can someone take over to re-enable (parts of) it?
It looks like it has been failing repeatedly since #5664 (See https://travis-ci.org/qgis/QGIS/builds) but since the PR itself was green and it was already failing quite often before, I'm not convinced, there's a connection between PR and test results.
Thanks a lot
@elpaso @pblottiere @rldhont
pblottiere commentedon Nov 20, 2017
Argh... I thought that the commit of Even on Spatialite provider (581d0d3) had stabilized these tests...
BTW, according to Travis, it seems that it's a real error this time (and not some flakyness):
I take a look to fix it. Moreover, I'm gonna split these WMS tests for
getmap
,getprint
, and so on. This way, we'll be able to deactivate only some tests (and not all of them).elpaso commentedon Nov 20, 2017
@pblottiere no: unfortunately the test was faling almost all the times. I had a look yesterday and I found mainly tiny rendering differences, can we increase the acceptable difference threshold ?
pblottiere commentedon Nov 20, 2017
:(. Can I take a look to your travis builds somewhere?
OK. I'm gonna do that at the same time.
We really have to tackle these issues, it's annoying for everybody. But I didn't succeed in reproducing these issues locally so far...