Skip to content

Commit b6551e2

Browse files
committed
Fix player tests
1 parent 3f628af commit b6551e2

2 files changed

Lines changed: 23 additions & 0 deletions

File tree

tests/core/test_player_grouping.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -234,6 +234,10 @@ def test_no_circular_dependency_in_synced_to(self, mock_mass: MagicMock) -> None
234234
controller._players = {"leader": leader, "member": member}
235235
mock_mass.players = controller
236236

237+
# Mark players as initialized so they are returned by all_players()
238+
leader.set_initialized()
239+
member.set_initialized()
240+
237241
# Trigger synced_to calculation via update_state
238242
leader.update_state(signal_event=False)
239243
member.update_state(signal_event=False)
@@ -305,6 +309,11 @@ def test_provider_instance_id_expands_to_all_players(self, mock_mass: MagicMock)
305309
# Set up get_provider to return the provider for instance ID
306310
mock_mass.get_provider = MagicMock(return_value=provider)
307311

312+
# Mark players as initialized so they are returned by all_players()
313+
player_a.set_initialized()
314+
player_b.set_initialized()
315+
player_c.set_initialized()
316+
308317
# Trigger state calculation
309318
player_a.update_state(signal_event=False)
310319
player_b.update_state(signal_event=False)

tests/core/test_protocol_linking.py

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -338,6 +338,10 @@ def test_find_matching_by_mac(self, mock_mass: MagicMock) -> None:
338338
"cc_aabbccddee": Throttler(1, 0.05),
339339
}
340340

341+
# Mark players as initialized so they are returned by all_players()
342+
airplay_player.set_initialized()
343+
chromecast_player.set_initialized()
344+
341345
# Find matching players for AirPlay player
342346
matches = controller._find_matching_protocol_players(airplay_player)
343347

@@ -1727,6 +1731,11 @@ def test_native_airplay_player_sees_protocol_players_as_visible_parents(
17271731
"airplay_sonos_1": Throttler(1, 0.05),
17281732
}
17291733

1734+
# Mark players as initialized so they are returned by all_players()
1735+
homepod.set_initialized()
1736+
sonos_player.set_initialized()
1737+
sonos_airplay.set_initialized()
1738+
17301739
# Update protocol players first, then parents
17311740
sonos_airplay.update_state(signal_event=False)
17321741
sonos_player.update_state(signal_event=False)
@@ -1796,6 +1805,11 @@ def test_protocol_linked_player_sees_native_airplay_player(self, mock_mass: Magi
17961805
"airplay_sonos_1": Throttler(1, 0.05),
17971806
}
17981807

1808+
# Mark players as initialized so they are returned by all_players()
1809+
homepod.set_initialized()
1810+
sonos_player.set_initialized()
1811+
sonos_airplay.set_initialized()
1812+
17991813
# Update protocol players first, then parents
18001814
sonos_airplay.update_state(signal_event=False)
18011815
homepod.update_state(signal_event=False)

0 commit comments

Comments
 (0)