1 files changed, 267 insertions, 57 deletions

diff --git a/tests/sound-menu.cc b/tests/sound-menu.cc
index 10c0cb9..6fe2c90 100644
--- a/tests/sound-menu.cc
+++ b/tests/sound-menu.cc
@@ -27,87 +27,297 @@ extern "C" {
 
 class SoundMenuTest : public ::testing::Test
 {
-	protected:
-		GTestDBus * bus = nullptr;
+    protected:
+        GTestDBus * bus = nullptr;
 
-		virtual void SetUp() {
-			bus = g_test_dbus_new(G_TEST_DBUS_NONE);
-			g_test_dbus_up(bus);
-		}
+        virtual void SetUp() {
+            bus = g_test_dbus_new(G_TEST_DBUS_NONE);
+            g_test_dbus_up(bus);
+        }
 
-		virtual void TearDown() {
-			g_test_dbus_down(bus);
-			g_clear_object(&bus);
-		}
+        virtual void TearDown() {
+            g_test_dbus_down(bus);
+            g_clear_object(&bus);
+        }
 
-		void verify_item_attribute (GMenuModel * mm, guint index, const gchar * name, GVariant * value) {
-			g_variant_ref_sink(value);
+        void verify_item_attribute (GMenuModel * mm, guint index, const gchar * name, GVariant * value) {
+            g_variant_ref_sink(value);
 
-			gchar * variantstr = g_variant_print(value, TRUE);
-			g_debug("Expecting item %d to have a '%s' attribute: %s", index, name, variantstr);
+            gchar * variantstr = g_variant_print(value, TRUE);
+            g_debug("Expecting item %d to have a '%s' attribute: %s", index, name, variantstr);
 
-			const GVariantType * type = g_variant_get_type(value);
-			GVariant * itemval = g_menu_model_get_item_attribute_value(mm, index, name, type);
+            const GVariantType * type = g_variant_get_type(value);
+            GVariant * itemval = g_menu_model_get_item_attribute_value(mm, index, name, type);
 
-			ASSERT_NE(nullptr, itemval);
-			EXPECT_TRUE(g_variant_equal(itemval, value));
+            ASSERT_NE(nullptr, itemval);
+            EXPECT_TRUE(g_variant_equal(itemval, value));
 
-			g_variant_unref(value);
-		}
+            g_variant_unref(value);
+        }
+
+        void verify_item_attribute_is_not_set(GMenuModel * mm, guint index, const gchar * name, const GVariantType * type) {
+            GVariant * itemval = g_menu_model_get_item_attribute_value(mm, index, name, type);
+            EXPECT_EQ(itemval, nullptr);
+        }
 };
 
 TEST_F(SoundMenuTest, BasicObject) {
-	SoundMenu * menu = sound_menu_new (nullptr, SOUND_MENU_DISPLAY_FLAGS_NONE);
+    SoundMenu * menu = sound_menu_new (nullptr, SOUND_MENU_DISPLAY_FLAGS_NONE);
 
-	ASSERT_NE(nullptr, menu);
+    ASSERT_NE(nullptr, menu);
 
-	g_clear_object(&menu);
-	return;
+    g_clear_object(&menu);
+    return;
 }
 
 TEST_F(SoundMenuTest, AddRemovePlayer) {
-	SoundMenu * menu = sound_menu_new (nullptr, SOUND_MENU_DISPLAY_FLAGS_NONE);
+    SoundMenu * menu = sound_menu_new (nullptr, SOUND_MENU_DISPLAY_FLAGS_NONE);
+
+    MediaPlayerTrack * track = media_player_track_new("Artist", "Title", "Album", "http://art.url");
+
+    MediaPlayerMock * media = MEDIA_PLAYER_MOCK(
+        g_object_new(TYPE_MEDIA_PLAYER_MOCK,
+            "mock-id", "player-id",
+            "mock-name", "Test Player",
+            "mock-state", "Playing",
+            "mock-is-running", TRUE,
+            "mock-can-raise", FALSE,
+            "mock-current-track", track,
+            "mock-can-do-play", TRUE,
+            "mock-can-do-next", TRUE,
+            "mock-can-do-prev", TRUE,
+            NULL)
+    );
+    g_clear_object(&track);
+
+    sound_menu_add_player(menu, MEDIA_PLAYER(media));
+
+    ASSERT_NE(nullptr, menu->menu);
+    EXPECT_EQ(2, g_menu_model_get_n_items(G_MENU_MODEL(menu->menu)));
+
+    GMenuModel * section = g_menu_model_get_item_link(G_MENU_MODEL(menu->menu), 1, G_MENU_LINK_SECTION);
+    ASSERT_NE(nullptr, section);
+    EXPECT_EQ(2, g_menu_model_get_n_items(section)); /* No playlists, so two items */
+
+    /* Player display */
+    verify_item_attribute(section, 0, "action", g_variant_new_string("indicator.player-id"));
+    verify_item_attribute(section, 0, "x-canonical-type", g_variant_new_string("com.canonical.unity.media-player"));
+
+    /* Player control */
+    verify_item_attribute(section, 1, "x-canonical-type", g_variant_new_string("com.canonical.unity.playback-item"));
+    verify_item_attribute(section, 1, "x-canonical-play-action", g_variant_new_string("indicator.play.player-id"));
+    verify_item_attribute(section, 1, "x-canonical-next-action", g_variant_new_string("indicator.next.player-id"));
+    verify_item_attribute(section, 1, "x-canonical-previous-action", g_variant_new_string("indicator.previous.player-id"));
+
+    g_clear_object(&section);
+
+    sound_menu_remove_player(menu, MEDIA_PLAYER(media));
+
+    EXPECT_EQ(1, g_menu_model_get_n_items(G_MENU_MODEL(menu->menu)));
+
+    g_clear_object(&media);
+    g_clear_object(&menu);
+    return;
+}
+
+TEST_F(SoundMenuTest, AddRemovePlayerNoPlayNextPrev) {
+    SoundMenu * menu = sound_menu_new (nullptr, SOUND_MENU_DISPLAY_FLAGS_NONE);
+
+    MediaPlayerTrack * track = media_player_track_new("Artist", "Title", "Album", "http://art.url");
+
+    MediaPlayerMock * media = MEDIA_PLAYER_MOCK(
+        g_object_new(TYPE_MEDIA_PLAYER_MOCK,
+            "mock-id", "player-id",
+            "mock-name", "Test Player",
+            "mock-state", "Playing",
+            "mock-is-running", TRUE,
+            "mock-can-raise", FALSE,
+            "mock-current-track", track,
+            "mock-can-do-play", FALSE,
+            "mock-can-do-next", FALSE,
+            "mock-can-do-prev", FALSE,
+            NULL)
+    );
+    g_clear_object(&track);
+
+    sound_menu_add_player(menu, MEDIA_PLAYER(media));
+
+    ASSERT_NE(nullptr, menu->menu);
+    EXPECT_EQ(2, g_menu_model_get_n_items(G_MENU_MODEL(menu->menu)));
+
+    GMenuModel * section = g_menu_model_get_item_link(G_MENU_MODEL(menu->menu), 1, G_MENU_LINK_SECTION);
+    ASSERT_NE(nullptr, section);
+    EXPECT_EQ(2, g_menu_model_get_n_items(section)); /* No playlists, so two items */
+
+    /* Player display */
+    verify_item_attribute(section, 0, "action", g_variant_new_string("indicator.player-id"));
+    verify_item_attribute(section, 0, "x-canonical-type", g_variant_new_string("com.canonical.unity.media-player"));
+
+    /* Player control */
+    verify_item_attribute(section, 1, "x-canonical-type", g_variant_new_string("com.canonical.unity.playback-item"));
+    verify_item_attribute(section, 1, "x-canonical-play-action", g_variant_new_string("indicator.play.player-id.disabled"));
+    verify_item_attribute(section, 1, "x-canonical-next-action", g_variant_new_string("indicator.next.player-id.disabled"));
+    verify_item_attribute(section, 1, "x-canonical-previous-action", g_variant_new_string("indicator.previous.player-id.disabled"));
+
+    g_clear_object(&section);
+
+    sound_menu_remove_player(menu, MEDIA_PLAYER(media));
+
+    EXPECT_EQ(1, g_menu_model_get_n_items(G_MENU_MODEL(menu->menu)));
+
+    g_clear_object(&media);
+    g_clear_object(&menu);
+    return;
+}
+
+TEST_F(SoundMenuTest, AddRemovePlayerNoNext) {
+    SoundMenu * menu = sound_menu_new (nullptr, SOUND_MENU_DISPLAY_FLAGS_NONE);
 
-	MediaPlayerTrack * track = media_player_track_new("Artist", "Title", "Album", "http://art.url");
+    MediaPlayerTrack * track = media_player_track_new("Artist", "Title", "Album", "http://art.url");
 
-	MediaPlayerMock * media = MEDIA_PLAYER_MOCK(
-		g_object_new(TYPE_MEDIA_PLAYER_MOCK,
-			"mock-id", "player-id",
-			"mock-name", "Test Player",
-			"mock-state", "Playing",
-			"mock-is-running", TRUE,
-			"mock-can-raise", FALSE,
-			"mock-current-track", track,
-			NULL)
-	);
-	g_clear_object(&track);
+    MediaPlayerMock * media = MEDIA_PLAYER_MOCK(
+        g_object_new(TYPE_MEDIA_PLAYER_MOCK,
+            "mock-id", "player-id",
+            "mock-name", "Test Player",
+            "mock-state", "Playing",
+            "mock-is-running", TRUE,
+            "mock-can-raise", FALSE,
+            "mock-current-track", track,
+            "mock-can-do-play", TRUE,
+            "mock-can-do-next", FALSE,
+            "mock-can-do-prev", TRUE,
+            NULL)
+    );
+    g_clear_object(&track);
 
-	sound_menu_add_player(menu, MEDIA_PLAYER(media));
+    sound_menu_add_player(menu, MEDIA_PLAYER(media));
 
-	ASSERT_NE(nullptr, menu->menu);
-	EXPECT_EQ(2, g_menu_model_get_n_items(G_MENU_MODEL(menu->menu)));
+    ASSERT_NE(nullptr, menu->menu);
+    EXPECT_EQ(2, g_menu_model_get_n_items(G_MENU_MODEL(menu->menu)));
 
-	GMenuModel * section = g_menu_model_get_item_link(G_MENU_MODEL(menu->menu), 1, G_MENU_LINK_SECTION);
-	ASSERT_NE(nullptr, section);
-	EXPECT_EQ(2, g_menu_model_get_n_items(section)); /* No playlists, so two items */
+    GMenuModel * section = g_menu_model_get_item_link(G_MENU_MODEL(menu->menu), 1, G_MENU_LINK_SECTION);
+    ASSERT_NE(nullptr, section);
+    EXPECT_EQ(2, g_menu_model_get_n_items(section)); /* No playlists, so two items */
 
-	/* Player display */
-	verify_item_attribute(section, 0, "action", g_variant_new_string("indicator.player-id"));
-	verify_item_attribute(section, 0, "x-canonical-type", g_variant_new_string("com.canonical.unity.media-player"));
+    /* Player display */
+    verify_item_attribute(section, 0, "action", g_variant_new_string("indicator.player-id"));
+    verify_item_attribute(section, 0, "x-canonical-type", g_variant_new_string("com.canonical.unity.media-player"));
 
-	/* Player control */
-	verify_item_attribute(section, 1, "x-canonical-type", g_variant_new_string("com.canonical.unity.playback-item"));
-	verify_item_attribute(section, 1, "x-canonical-play-action", g_variant_new_string("indicator.play.player-id"));
-	verify_item_attribute(section, 1, "x-canonical-next-action", g_variant_new_string("indicator.next.player-id"));
-	verify_item_attribute(section, 1, "x-canonical-previous-action", g_variant_new_string("indicator.previous.player-id"));
+    /* Player control */
+    verify_item_attribute(section, 1, "x-canonical-type", g_variant_new_string("com.canonical.unity.playback-item"));
+    verify_item_attribute(section, 1, "x-canonical-play-action", g_variant_new_string("indicator.play.player-id"));
+    verify_item_attribute(section, 1, "x-canonical-next-action", g_variant_new_string("indicator.next.player-id.disabled"));
+    verify_item_attribute(section, 1, "x-canonical-previous-action", g_variant_new_string("indicator.previous.player-id"));
 
-	g_clear_object(&section);
+    g_clear_object(&section);
 
-	sound_menu_remove_player(menu, MEDIA_PLAYER(media));
+    sound_menu_remove_player(menu, MEDIA_PLAYER(media));
 
-	EXPECT_EQ(1, g_menu_model_get_n_items(G_MENU_MODEL(menu->menu)));
+    EXPECT_EQ(1, g_menu_model_get_n_items(G_MENU_MODEL(menu->menu)));
 
-	g_clear_object(&media);
-	g_clear_object(&menu);
-	return;
+    g_clear_object(&media);
+    g_clear_object(&menu);
+    return;
 }
+
+TEST_F(SoundMenuTest, AddRemovePlayerNoPrev) {
+    SoundMenu * menu = sound_menu_new (nullptr, SOUND_MENU_DISPLAY_FLAGS_NONE);
+
+    MediaPlayerTrack * track = media_player_track_new("Artist", "Title", "Album", "http://art.url");
+
+    MediaPlayerMock * media = MEDIA_PLAYER_MOCK(
+        g_object_new(TYPE_MEDIA_PLAYER_MOCK,
+            "mock-id", "player-id",
+            "mock-name", "Test Player",
+            "mock-state", "Playing",
+            "mock-is-running", TRUE,
+            "mock-can-raise", FALSE,
+            "mock-current-track", track,
+            "mock-can-do-play", TRUE,
+            "mock-can-do-next", TRUE,
+            "mock-can-do-prev", FALSE,
+            NULL)
+    );
+    g_clear_object(&track);
+
+    sound_menu_add_player(menu, MEDIA_PLAYER(media));
+
+    ASSERT_NE(nullptr, menu->menu);
+    EXPECT_EQ(2, g_menu_model_get_n_items(G_MENU_MODEL(menu->menu)));
+
+    GMenuModel * section = g_menu_model_get_item_link(G_MENU_MODEL(menu->menu), 1, G_MENU_LINK_SECTION);
+    ASSERT_NE(nullptr, section);
+    EXPECT_EQ(2, g_menu_model_get_n_items(section)); /* No playlists, so two items */
+
+    /* Player display */
+    verify_item_attribute(section, 0, "action", g_variant_new_string("indicator.player-id"));
+    verify_item_attribute(section, 0, "x-canonical-type", g_variant_new_string("com.canonical.unity.media-player"));
+
+    /* Player control */
+    verify_item_attribute(section, 1, "x-canonical-type", g_variant_new_string("com.canonical.unity.playback-item"));
+    verify_item_attribute(section, 1, "x-canonical-play-action", g_variant_new_string("indicator.play.player-id"));
+    verify_item_attribute(section, 1, "x-canonical-next-action", g_variant_new_string("indicator.next.player-id"));
+    verify_item_attribute(section, 1, "x-canonical-previous-action", g_variant_new_string("indicator.previous.player-id.disabled"));
+
+    g_clear_object(&section);
+
+    sound_menu_remove_player(menu, MEDIA_PLAYER(media));
+
+    EXPECT_EQ(1, g_menu_model_get_n_items(G_MENU_MODEL(menu->menu)));
+
+    g_clear_object(&media);
+    g_clear_object(&menu);
+    return;
+}
+
+TEST_F(SoundMenuTest, AddRemovePlayerNoPlay) {
+    SoundMenu * menu = sound_menu_new (nullptr, SOUND_MENU_DISPLAY_FLAGS_NONE);
+
+    MediaPlayerTrack * track = media_player_track_new("Artist", "Title", "Album", "http://art.url");
+
+    MediaPlayerMock * media = MEDIA_PLAYER_MOCK(
+        g_object_new(TYPE_MEDIA_PLAYER_MOCK,
+            "mock-id", "player-id",
+            "mock-name", "Test Player",
+            "mock-state", "Playing",
+            "mock-is-running", TRUE,
+            "mock-can-raise", FALSE,
+            "mock-current-track", track,
+            "mock-can-do-play", FALSE,
+            "mock-can-do-next", TRUE,
+            "mock-can-do-prev", TRUE,
+            NULL)
+    );
+    g_clear_object(&track);
+
+    sound_menu_add_player(menu, MEDIA_PLAYER(media));
+
+    ASSERT_NE(nullptr, menu->menu);
+    EXPECT_EQ(2, g_menu_model_get_n_items(G_MENU_MODEL(menu->menu)));
+
+    GMenuModel * section = g_menu_model_get_item_link(G_MENU_MODEL(menu->menu), 1, G_MENU_LINK_SECTION);
+    ASSERT_NE(nullptr, section);
+    EXPECT_EQ(2, g_menu_model_get_n_items(section)); /* No playlists, so two items */
+
+    /* Player display */
+    verify_item_attribute(section, 0, "action", g_variant_new_string("indicator.player-id"));
+    verify_item_attribute(section, 0, "x-canonical-type", g_variant_new_string("com.canonical.unity.media-player"));
+
+    /* Player control */
+    verify_item_attribute(section, 1, "x-canonical-type", g_variant_new_string("com.canonical.unity.playback-item"));
+    verify_item_attribute(section, 1, "x-canonical-play-action", g_variant_new_string("indicator.play.player-id.disabled"));
+    verify_item_attribute(section, 1, "x-canonical-next-action", g_variant_new_string("indicator.next.player-id"));
+    verify_item_attribute(section, 1, "x-canonical-previous-action", g_variant_new_string("indicator.previous.player-id"));
+
+    g_clear_object(&section);
+
+    sound_menu_remove_player(menu, MEDIA_PLAYER(media));
+
+    EXPECT_EQ(1, g_menu_model_get_n_items(G_MENU_MODEL(menu->menu)));
+
+    g_clear_object(&media);
+    g_clear_object(&menu);
+    return;
+}
+
+//