Move the software API to a separate service org.opensuse.DInstaller.Software

[imobachgs]

Problem

This PRs moves the software management logic to a separate service, org.opensuse.DInstaller.Software. See #168 for further information.

Solution

This is the first PR to move the software API to a separate org.opensuse.DInstaller.Software. It sets the basic infrastructure:

• Move the software API to a separate SoftwareService.
• Extend bin/d-installer to run all services at once or just one of them. More work is expected to improve this executable soon.
• Support for using a different Y2DIR per service.
• Adapt the UI to use the new software service.

Testing

• Added a new unit test
• Tested manually

Tasks (other PRs)

• Keep track of the status
• API to add resolvables to the packages proposal (if needed)

[coveralls]

Pull Request Test Coverage Report for Build 2514470738

• 66 of 134 (49.25%) changed or added relevant lines in 15 files are covered.
• 41 unchanged lines in 4 files lost coverage.
• Overall coverage decreased (-5.9%) to 72.718%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
service/lib/dinstaller/dbus/clients.rb	0	1	0.0%
service/lib/dinstaller/dbus/manager.rb	0	1	0.0%
service/lib/dinstaller/progress.rb	1	2	50.0%
service/lib/dinstaller/package_callbacks.rb	0	2	0.0%
service/lib/dinstaller/software.rb	12	14	85.71%
service/lib/dinstaller/dbus/clients/software.rb	26	29	89.66%
service/lib/dinstaller/dbus/software.rb	0	23	0.0%
service/lib/dinstaller/dbus/software_service.rb	0	35	0.0%

Files with Coverage Reduction	New Missed Lines	%
service/lib/dinstaller/dbus/clients.rb	1	0%
service/lib/dinstaller/manager.rb	1	92.52%
service/lib/dinstaller/software.rb	1	65.55%
service/lib/dinstaller/dbus/software.rb	38	0%

Totals
Change from base Build 2503418449:	-5.9%
Covered Lines:	1490
Relevant Lines:	2049

---

💛 - Coveralls

[dgdavid]

So far, LGTM. Will you add unit testing in this or a subsequent PR?

[joseivanlopez]

NP: Starts

[joseivanlopez]

This line is not needed. You can simply use dbus_object.Probe(&block), which will perform the D-Bus call without passing any block if the block was not given.

[joseivanlopez]

At some point we will need a DBus::Base class.

[joseivanlopez]

LGTM