Commit Graph

8 Commits

Author SHA1 Message Date
Clark Boylan 9e2d54bb5a Modernize test suite
This modernizes the lodgeit testsuite to use stestr instead of nose. The
primary reason for this is nose is not python3.10 compatible and we
would like to use newer python with lodgeit.

Change-Id: I8ae480d22bbef0258afc9d6ffd6cd820993430a1
2022-11-10 10:30:15 -08:00
Tristan Cacqueray 9b854e7729 Fix python3 compatibility issue
This change fix python3 compatibility:
* replace except obj, name with except obj as name
* use print function
* replace iteritems by six.iteritems
* replace xrange by range
* convert items() result to list

Change-Id: I70364e47541274d446c4449f820dce3e0b331b27
2020-03-04 13:09:24 -06:00
David Pursehouse b19a482d47 Fix PEP-8 errors
There are several instances of the error:

   E305 expected 2 blank lines after class or function definition

Fix them all by adding the necessary blank lines.

Change-Id: I37e2990772f05a100fe303c1512b7bdfb10a660d
2017-02-25 13:47:11 +09:00
David Pursehouse f79c65815a Fix up the tests and add tox configuration to run them
The alfajor HTTP test wrapper is not working properly.  All the tests
are failing with:

  AttributeError: '_APIClientResponse' object has no attribute '_status_code'

It doesn't look like alfajor is being maintained any more [1, 2], so
remove the dependency on it and run the tests directly through the
werkzeug client instead.

The test_json_get_styles test still failed because the content and
ordering of the styles returned from the API is dependent on the
platform on which the tests run.  Instead of testing against a fixed
expected result, which could be different on another platform,
generate the expected results dynamically.

Update the tox configuration and test-requirements.txt to be able to
run the tests via tox.

[1] https://github.com/idealist/Alfajor
[2] https://github.com/idealistdev/alfajor

Change-Id: I4db6783b9d725ce096575035ea27d28be8e6fed5
2015-11-24 13:33:00 +09:00
David Pursehouse fbb61e9c1d Add tox configuration to enable PEP-8 checks, and fix various failures
- E126 continuation line over-indented for hanging indent
- E127 continuation line over-indented for visual indent
- E128 continuation line under-indented for visual indent
- E131 continuation line unaligned for hanging indent
- E265 block comment should start with '# '
- E272 multiple spaces before keyword
- E302 expected 2 blank lines, found <n>
- E712 comparison to False should be 'if cond is False:' or 'if not cond:'
- F401 '<name>' imported but unused

Change-Id: I2344cf2e8fd7910e72e9d8043423d25e43065ef2
2015-11-17 19:09:23 +09:00
Dan Colish b03af04f78 add more tests and some comments 2011-04-04 09:37:11 -07:00
Dan Colish 3a2ba35e63 add a few more tests, use more url_for, don't be web2.0 with selects 2011-04-03 13:19:38 -07:00
Dan Colish 46db573e3c add a simple test to see if most url endpoints work, fix failures 2011-04-02 16:40:09 -07:00