FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update Python MapScript test suite for Python3 by geographika · Pull Request #5677 · MapServer/MapServer · GitHub

Update Python MapScript test suite for Python3 - #5677

Merged
geographika merged 23 commits into
MapServer:masterfrom
geographika:py3-fixes
Oct 13, 2018
Merged

Update Python MapScript test suite for Python3#5677
geographika merged 23 commits into
MapServer:masterfrom
geographika:py3-fixes

Conversation

Copy link
Copy Markdown
Member

This pull request has changes to several files. The overall purpose is to get the entire MapScript Python test suite passing again, for both Python 2 and 3. There are now 293 tests, covering most of the MapScript API - up from 195 tests prior to this pull request.

Changes can be summarised as follows:

  • Appveyor CI has been updated to build Python wheels and run the tests for both Python 2 and 3
  • Fixing deprecation warnings for regex expressions in Python3
  • Removing broken GD related imageObj creation code in Python MapScript and reverting to the generic functions to allow image creation
  • Add a py2/3 fix for writing images, as several tests were failing due to writing unicode strings rather than bytes to files
  • Moved the new dict hashTableObj to the pyextend.i file (where it should belong)
  • Reintroduced previously failing test cases, and linted all Python test code (including MapScript thread and reference tests)
  • Fixed various failing tests where old MapScript APIs were used (particularly with querying features)
  • Fixed some whitespace issues in SWIG interface files causing warnings

geographika merged commit 913d68e into MapServer:master Oct 13, 2018
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL