On Oct. 30, 2013, 2:44 p.m., Radek Novacek wrote:
> src/software/test/testSoftware.py, line 98
>
<
http://reviewboard-openlmi.rhcloud.com/r/1147/diff/1/?file=6187#file6187l...
>
> What is the purpose of this function? Remove epoch from NVR? If so then note
that epoch could be arbitrary number, this works only for epoch <0..9>.
Yeah, some kind of description "why?" would be helpful. I would use regular
expression:
string = re.sub(r'-\d+:', '-', string)
On Oct. 30, 2013, 2:44 p.m., Radek Novacek wrote:
> src/software/test/testSoftware.py, line 255
>
<
http://reviewboard-openlmi.rhcloud.com/r/1147/diff/1/?file=6187#file6187l...
>
> use "assertListEqual", it produces better output when the test fails
Unfortunately this is available since python 2.7.
On Oct. 30, 2013, 2:44 p.m., Radek Novacek wrote:
> src/software/test/testSoftware.py, line 592
>
<
http://reviewboard-openlmi.rhcloud.com/r/1147/diff/1/?file=6187#file6187l...
>
> self.assertIsNone()
Again, since python 2.7.
On Oct. 30, 2013, 2:44 p.m., Radek Novacek wrote:
> src/software/test/testSoftware.py, line 312
>
<
http://reviewboard-openlmi.rhcloud.com/r/1147/diff/1/?file=6187#file6187l...
>
> You can use
>
> .InstalledSoftware.InstanceID
>
> instead of
>
> .path["InstalledSoftware"]["InstanceID"]
From what I know:
.InstalledSoftware["InstanceID"]
Because InstalledSoftware is an instance of pywbem.CIMInstanceName, which doesn't
allow for smart attribute access.
On Oct. 30, 2013, 2:44 p.m., Radek Novacek wrote:
> src/software/test/testSoftware.py, line 300
>
<
http://reviewboard-openlmi.rhcloud.com/r/1147/diff/1/?file=6187#file6187l...
>
> assertEqual -> assertListEqual
Since python 2.7.
On Oct. 30, 2013, 2:44 p.m., Radek Novacek wrote:
> src/software/test/testSoftware.py, line 284
>
<
http://reviewboard-openlmi.rhcloud.com/r/1147/diff/1/?file=6187#file6187l...
>
> The matching against 'yum list available' is missing. If it is on
purpose, add at least a TODO comment.
This should be achievable. Try to remove installed packages and duplicities from lmi list
and compare them. (I haven't tried).
- Michal
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/1147/#review1566
-----------------------------------------------------------
On Oct. 30, 2013, 2:03 p.m., Jan Grec wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/1147/
-----------------------------------------------------------
(Updated Oct. 30, 2013, 2:03 p.m.)
Review request for OpenLMI Developers.
Repository: openlmi-providers
Description
-------
software - tests: software test suite with basic tests
This is only a "current-status" review. I'll be very thankful for any
comments on what's wrong, how the structure should look like, etc.
My TODO is to
* repair all broken tests
* add all tests mentioned
* bind test suite with new LmiTestCase class
I apologize for inconvenience, but I wasn't able to save previous review notes.
Diffs
-----
src/software/test/lmi-test.repo PRE-CREATION
src/software/test/testSoftware.py PRE-CREATION
Diff:
http://reviewboard-openlmi.rhcloud.com/r/1147/diff/
Testing
-------
Thanks,
Jan Grec