Skip to content

Commit

Permalink
Fix broken server tests
Browse files Browse the repository at this point in the history
These tests had lots of issues, including some inappropriate mask images
which nullified the tests!
  • Loading branch information
nyalldawson committed May 3, 2020
1 parent 00df371 commit a934f0f
Show file tree
Hide file tree
Showing 12 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions tests/src/python/test_qgsserver.py
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@


from io import StringIO from io import StringIO
from qgis.server import QgsServer, QgsServerRequest, QgsBufferServerRequest, QgsBufferServerResponse from qgis.server import QgsServer, QgsServerRequest, QgsBufferServerRequest, QgsBufferServerResponse
from qgis.core import QgsRenderChecker, QgsApplication, QgsFontUtils from qgis.core import QgsRenderChecker, QgsApplication, QgsFontUtils, QgsMultiRenderChecker
from qgis.testing import unittest from qgis.testing import unittest
from qgis.PyQt.QtCore import QSize from qgis.PyQt.QtCore import QSize
from utilities import unitTestDataPath from utilities import unitTestDataPath
Expand Down Expand Up @@ -188,13 +188,13 @@ def _img_diff(self, image, control_image, max_diff, max_size_diff=QSize(), outpu
if outputJpg: if outputJpg:
return (True, "QgsRenderChecker can't be used for JPG images") return (True, "QgsRenderChecker can't be used for JPG images")


control = QgsRenderChecker() control = QgsMultiRenderChecker()
control.setControlPathPrefix("qgis_server") control.setControlPathPrefix("qgis_server")
control.setControlName(control_image) control.setControlName(control_image)
control.setRenderedImage(temp_image) control.setRenderedImage(temp_image)
if max_size_diff.isValid(): if max_size_diff.isValid():
control.setSizeTolerance(max_size_diff.width(), max_size_diff.height()) control.setSizeTolerance(max_size_diff.width(), max_size_diff.height())
return control.compareImages(control_image, max_diff), control.report() return control.runTest(control_image, max_diff), control.report()


def _img_diff_error(self, response, headers, image, max_diff=100, max_size_diff=QSize(), unittest_data_path='control_images', outputJpg=False): def _img_diff_error(self, response, headers, image, max_diff=100, max_size_diff=QSize(), unittest_data_path='control_images', outputJpg=False):


Expand Down
6 changes: 3 additions & 3 deletions tests/src/python/test_qgsserver_wms_getprint.py
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
import subprocess import subprocess


from test_qgsserver import QgsServerTestBase from test_qgsserver import QgsServerTestBase
from qgis.core import QgsProject, QgsRenderChecker from qgis.core import QgsProject, QgsRenderChecker, QgsMultiRenderChecker
from qgis.server import QgsServerRequest from qgis.server import QgsServerRequest
from utilities import getExecutablePath, unitTestDataPath from utilities import getExecutablePath, unitTestDataPath


Expand Down Expand Up @@ -105,13 +105,13 @@ def _pdf_diff(self, pdf, control_image, max_diff, max_size_diff=QSize(), dpi=96)
temp_image = os.path.join(tempfile.gettempdir(), "%s_result.png" % control_image) temp_image = os.path.join(tempfile.gettempdir(), "%s_result.png" % control_image)
self._pdf_to_png(temp_pdf, temp_image, dpi=dpi, page=1) self._pdf_to_png(temp_pdf, temp_image, dpi=dpi, page=1)


control = QgsRenderChecker() control = QgsMultiRenderChecker()
control.setControlPathPrefix("qgis_server") control.setControlPathPrefix("qgis_server")
control.setControlName(control_image) control.setControlName(control_image)
control.setRenderedImage(temp_image) control.setRenderedImage(temp_image)
if max_size_diff.isValid(): if max_size_diff.isValid():
control.setSizeTolerance(max_size_diff.width(), max_size_diff.height()) control.setSizeTolerance(max_size_diff.width(), max_size_diff.height())
return control.compareImages(control_image, max_diff), control.report() return control.runTest(control_image, max_diff), control.report()


def _pdf_diff_error(self, response, headers, image, max_diff=100, max_size_diff=QSize(), unittest_data_path='control_images', dpi=96): def _pdf_diff_error(self, response, headers, image, max_diff=100, max_size_diff=QSize(), unittest_data_path='control_images', dpi=96):


Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file not shown.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit a934f0f

Please sign in to comment.