This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/1147/ |
On October 30th, 2013, 3:44 p.m. CET, Radek Novacek wrote:
src/software/test/testSoftware.py (Diff revision 1) 255 self.assertEqual(sorted(package_list), sorted(rpm_packages))use "assertListEqual", it produces better output when the test failsOn October 30th, 2013, 4:29 p.m. CET, Michal Minar wrote:
Unfortunately this is available since python 2.7.On November 5th, 2013, 1:09 p.m. CET, Jan Grec wrote:
I'm afraid there are more pieces of tests code that won't work on 2.6 Shall we strictly support python 2.6 and more, or is it ok to run with 2.7 and eventually rewrite tests for older python versions when it's needed?On November 6th, 2013, 1:11 p.m. CET, Michal Minar wrote:
I'd rather stick with code compatible with 2.6. True, not all of the current code works with 2.6 but this is not the reason to use new features from 2.7 since we know these tests need to pass on 2.6 sooner or later.
Okay, I'll change it back to assertEqual. Also docs for assertEqual says: "In addition, if first and second are the exact same type and one of list, tuple, dict, set, frozenset or unicode or any type that a subclass registers with addTypeEqualityFunc() the type-specific equality function will be called in order to generate a more useful default error message (see also the list of type-specific methods)." so I guess assertEqual doesn't matter.
- Jan
On October 30th, 2013, 3:03 p.m. CET, Jan Grec wrote:
Review request for OpenLMI Developers.
By Jan Grec.
Updated Oct. 30, 2013, 3:03 p.m.
Repository:
openlmi-providers
Description
Diffs
|