Merge lp://qastaging/~sevilerow-team/sevilerow/fonts_test into lp://qastaging/~sevilerow-team/sevilerow/tests

Looks good like we discussed +1

Holding off on the merge until I test with fonts and make some additional changes that were brought up in discussion about the MP

Test with fonts on the Mako

As we discussed on IRC the logic of the final loop should be something more like:

for font in fontlist_string:
    assertTrue(font in installed_fonts)

Also, the test for /custom/usr/share/fonts will fail with the current tarball.

Instead why not wrap the whole test in if (os.path.exists(fontspath)) since if we don't put fonts in there we don't need to test them?

This also needs to be fixed so that the test will not fail if no directory is present, perhaps simply warn the user

I pushed a new update, rev 24. This includes the following changes.

1.) Remove test_customized_applications.py: This test has been replaced by test_webbrowser_settings.py

2.) Fixed test_fonts.py by making the changes suggested from sweeney

3.) Tested the script using the /usr/share/fonts on the Mako device, this directory contains a set of "truetype" fonts, all of which are currently in use and are present in "fc-list". After testing, I switched the default path to "/custom/usr/share/fonts" (see line 26) and re-ran the script. The test recognized that there was no directory (/custom/usr/share/fonts), see item #4

4.) Made the test Pass regardless if the fontspath is present or not, it the custom fonts path is NOT present, "No Custom Fonts Found" will be recorded in the test logs.

5.) Line 50 contains "self.assertIn(font, installed_fonts)" I'm using this method rather than "self.assertTrue(font in installed_fonts" because if there is a failure the test will output the missing string. For Example, please see: https://pastebin.canonical.com/99149/

Approved after IRC discussion.

This is much better. Nice work.