-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/1147/#review1652
-----------------------------------------------------------
Due to changes in lmishell (v0.8 -> v0.9) some usage patterns won't work.
I've picked few of them and opened an issue. Please try the test against
the software provider with update openlmi-tools to ensure validity of this code.
src/software/test/testSoftware.py
<
http://reviewboard-openlmi.rhcloud.com/r/1147/#comment1002>
This won't work, use:
iname.InstalledSoftware.InstanceID
src/software/test/testSoftware.py
<
http://reviewboard-openlmi.rhcloud.com/r/1147/#comment1003>
Please use:
it.InstanceID
src/software/test/testSoftware.py
<
http://reviewboard-openlmi.rhcloud.com/r/1147/#comment1004>
nevra =
iname.InstalledSoftware.InstanceID[len("LMI:LMI_SoftwareIdentity:"):]
package_list.append(nevra)
src/software/test/testSoftware.py
<
http://reviewboard-openlmi.rhcloud.com/r/1147/#comment1005>
name = it.InstanceID[len("LMI:LMI_SoftwareIdentity:"):]
- Michal Minar
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