Skip to content
This repository has been archived by the owner on May 4, 2020. It is now read-only.

Start of a test suite #27

Closed
wants to merge 1 commit into from
Closed

Conversation

hlawrenz
Copy link

I put together some tests to aid in porting to Python 3. Hopefully they're a good starting point. Feedback from those better acquainted with the code would be appreciated. I attempted to make them so that they'll work in both 2 and 3.

@takluyver
Copy link
Contributor

As I've just mentioned in issue #1, you could also integrate the tests which Manfred Moitzi added for his xlwt3 fork. You need to contact him to officially relicense the tests, because they all say 'License: GPLv3', but he was quite happy to do that when I asked him about xlrd tests.

https://bitbucket.org/mozman/xlwt3/src/default/tests

@cjw296
Copy link
Member

cjw296 commented Jan 25, 2014

Would merging this pull request cause problems for either merging in Manfred's tests or #32? I'd like to get some tests in and Travis and Jenkins up and running...

__author__ = 'hrwl'

import unittest
import six
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No reliance on six please. We didn't need it for xlrd and I'm keep to keep it the same for xlwt. We just want our own 'compat' module in xlwt/compat.py...

@takluyver
Copy link
Contributor

I don't think this should really interfere with adding Manfred's tests, though there may be some duplication of testing that it's worth cutting out later. Neither should it be a problem for any effort to support Python 3 - I'd say having some tests is an important step for that.

@cjw296
Copy link
Member

cjw296 commented Apr 7, 2014

Nothing came back from @hlawrenz so closing in favour on Menfred's tests,

@cjw296 cjw296 closed this Apr 7, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants