Merge lp:~gary-lasker/software-center/tech-items-to-launcher-fix-lp1006483 into lp:software-center/5.2

[Gary Lasker]

* lp:~gary-lasker/software-center/tech-items-to-launcher-fix-lp1006483:
   - don't add an item to the launcher if its desktop file specifies
     NoDisplay=true (LP: #1006483)

This branch fixes bug 1006483 by checking an application's desktop file for NoDisplay=true, and if this value is set, the application is not added to the Unity launcher.

A convenient test case is the Deja Dup package, which sets NoDisplay=true in its desktop file.

[Michael Vogt (mvo)]

Thanks, this change looks fine.

It would be nice to have a test for this too, _add_application_to_unity_launcher() appears to be not covered
currently and it would be nice to test it with additional test files in data/app-install/desktop/ using a
mock launcher_info to test that send_application_to_launcher() is called for the right desktop files
(should be really straightforward to add I hope).

While looking at this a bit more (trying to figure out if it was covered for not) I noticed that it seems
like the "test_unity_launcher_integration.py" file needs updating to cover the new behavior that the launcher
item is added on "transaction-finished" instead of "transaction-started".

It currently sends a fake "transaction-started" signal in the test, but in order to test
if the signal is actually delivered to unity it also needs to send a "transaction-finished" signal too.
So I would suggest (probably in the original branch or as a seperate branch not here in this branch as this
branch is nice and short) to update the test to send the "transaction-finished" signal and also to add a mock
for available_pane._add_application_to_unity_launcher so that after
self._navigate_to_appdetails_and_install(test_pkgname)" in the test the test can check that
the call was made as expected.

This got a bit long, so the summary is: this branch is fine and approve once the test situation is
sorted out.

[Gary Lasker (gary-lasker)]

Hi Michael, please check the separate MP for the enhanced set of unit tests here:

  https://code.launchpad.net/~gary-lasker/software-center/launcher-integration-unit-tests/+merge/115236

Thanks!

=== modified file 'softwarecenter/ui/gtk3/panes/availablepane.py'
--- softwarecenter/ui/gtk3/panes/availablepane.py	2012-06-20 02:12:19 +0000
+++ softwarecenter/ui/gtk3/panes/availablepane.py	2012-06-20 02:12:19 +0000
@@ -37,6 +37,7 @@
 from softwarecenter.paths import APP_INSTALL_PATH
 from softwarecenter.utils import (wait_for_apt_cache_ready,
                                   get_exec_line_from_desktop,
+                                  is_no_display_desktop_file,
                                   get_file_path_from_iconname,
                                   convert_desktop_file_to_installed_location)
 from softwarecenter.db.appfilter import AppFilter
@@ -428,9 +429,11 @@
             return
         # do not add apps that have no Exec entry in their desktop file
         # (e.g. wine, see LP: #848437 or ubuntu-restricted-extras,
-        # see LP: #913756)
+        # see LP: #913756), also, don't add the item if NoDisplay is
+        # specified (see LP: #1006483)
         if (os.path.exists(installed_desktop_file_path) and
-            not get_exec_line_from_desktop(installed_desktop_file_path)):
+            (not get_exec_line_from_desktop(installed_desktop_file_path) or
+            is_no_display_desktop_file(installed_desktop_file_path))):
             return
 
         # now gather up the unity launcher info items and send the app to the