Comment 2 for bug 1589174

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

I finally found your skipIf (not in bzr, but in the last upload).
+ @t_help.skipIf(True, "LP: #1589174")
Is a bit too generic :-)

That is actually no dependency to the local Debian/Ubuntu filesystem layout as we first assumed - but instead happening in generate_sources_list.

That should actually always happen except:
- apt_configure_enabled = False
- apt_preserve_sources_list = True
- the specific path depends on apt_custom_sources_list OR is template_fn = cloud.get_template_filename('sources.list.%s' % (cloud.distro.name))

None of the first two apply, distro name is Debian.
But what I didn't expect can be seen in the log:
tests.unittests.test_handler.test_handler_apt_configure_sources_list: DEBUG: Skipping module named notimportant, no package 'mirror' located

That means in this particular case find_apt_mirror_info failed which I didn't expect - looking into it.

That said we might still (if you want) guard quite a lot of the tests with soemthing like "if apt is here" as the feature and thereby the tests only make sens in that case.