feat: implement smart unfollow with resonance evaluation and close friends guard
This commit is contained in:
@@ -16,12 +16,15 @@ class TestUnfollowEngine:
|
||||
self.mock_dopamine = MagicMock()
|
||||
self.mock_dopamine.is_app_session_over.return_value = False
|
||||
self.mock_dopamine.wants_to_change_feed.return_value = False
|
||||
# default boredom
|
||||
self.mock_dopamine.boredom = 0.0
|
||||
|
||||
self.mock_resonance = MagicMock()
|
||||
self.mock_resonance.calculate_resonance.return_value = 0.2
|
||||
|
||||
self.cognitive_stack = {
|
||||
"telepathic": self.mock_telepathic,
|
||||
"dopamine": self.mock_dopamine,
|
||||
"resonance": self.mock_resonance,
|
||||
}
|
||||
|
||||
self.mock_configs = MagicMock()
|
||||
@@ -33,27 +36,39 @@ class TestUnfollowEngine:
|
||||
|
||||
self.logger = logging.getLogger("test")
|
||||
|
||||
def test_unfollow_loop_success(self, monkeypatch):
|
||||
def test_unfollow_loop_unfollows_low_resonance(self, monkeypatch):
|
||||
"""
|
||||
Happy path: Finds 'Following' button -> Clicks it -> Finds 'Unfollow' confirmation -> Clicks it -> Increments totalUnfollowed.
|
||||
Then dopamine signals change of feed.
|
||||
Happy path (Smart Unfollow):
|
||||
1. Clicks profile row.
|
||||
2. Dumps profile UI.
|
||||
3. Resonance is low (< 0.4).
|
||||
4. Clicks 'Following' button on profile.
|
||||
5. Clicks 'Unfollow' confirm.
|
||||
6. Clicks back.
|
||||
"""
|
||||
def fake_extract_semantic_nodes(xml, intent, **kwargs):
|
||||
if "Unfollow" in intent:
|
||||
if "profile rows" in intent:
|
||||
return [{"semantic_string": "Profile Row", "x": 100, "y": 200, "bounds": "[50,150]", "skip": False}]
|
||||
elif "Unfollow" in intent:
|
||||
return [{"semantic_string": "Unfollow Confirmation", "x": 500, "y": 1500, "bounds": "[100,200]", "skip": False}]
|
||||
else:
|
||||
return [{"semantic_string": "Following Button", "x": 900, "y": 600, "bounds": "[100,200]", "skip": False}]
|
||||
|
||||
self.mock_telepathic._extract_semantic_nodes.side_effect = fake_extract_semantic_nodes
|
||||
self.mock_dopamine.wants_to_change_feed.side_effect = [True]
|
||||
self.mock_device.dump_hierarchy.return_value = '<node text="Some Bio"/>'
|
||||
|
||||
# Patch local imports inside the bot_flow namespace since they are locally imported
|
||||
# Patch local imports inside the bot_flow / unfollow_engine namespace
|
||||
mock_sleep = MagicMock()
|
||||
mock_click = MagicMock()
|
||||
import GramAddict.core.bot_flow
|
||||
monkeypatch.setattr(GramAddict.core.bot_flow, "sleep", mock_sleep)
|
||||
monkeypatch.setattr(GramAddict.core.bot_flow, "random_sleep", mock_sleep)
|
||||
monkeypatch.setattr(GramAddict.core.bot_flow, "_humanized_click", mock_click)
|
||||
|
||||
import GramAddict.core.utils
|
||||
monkeypatch.setattr(GramAddict.core.utils, "random_sleep", mock_sleep)
|
||||
|
||||
result = _run_zero_latency_unfollow_loop(
|
||||
self.mock_device, None, None, self.mock_configs, self.mock_session_state, "FollowingList", self.cognitive_stack
|
||||
)
|
||||
@@ -62,12 +77,88 @@ class TestUnfollowEngine:
|
||||
assert result == "BOREDOM_CHANGE_FEED"
|
||||
assert self.mock_session_state.totalUnfollowed == 1
|
||||
|
||||
# Assert humanized clicks were logged correctly
|
||||
assert mock_click.call_count == 2
|
||||
# Assert humanized clicks were logged correctly (Profile Row -> Following Button -> Unfollow Confirm)
|
||||
assert mock_click.call_count == 3
|
||||
mock_click.assert_has_calls([
|
||||
call(self.mock_device, 100, 200),
|
||||
call(self.mock_device, 900, 600),
|
||||
call(self.mock_device, 500, 1500)
|
||||
])
|
||||
assert self.mock_device.back.call_count == 1
|
||||
|
||||
def test_unfollow_loop_keeps_high_resonance_profile(self, monkeypatch):
|
||||
"""
|
||||
If resonance is high, we keep following the account and go back to the list.
|
||||
"""
|
||||
self.mock_resonance.calculate_resonance.return_value = 0.8
|
||||
|
||||
def fake_extract_semantic_nodes(xml, intent, **kwargs):
|
||||
if "profile rows" in intent:
|
||||
return [{"semantic_string": "Profile Row", "x": 100, "y": 200, "bounds": "[50,150]", "skip": False}]
|
||||
return []
|
||||
|
||||
self.mock_telepathic._extract_semantic_nodes.side_effect = fake_extract_semantic_nodes
|
||||
self.mock_dopamine.wants_to_change_feed.side_effect = [True]
|
||||
self.mock_device.dump_hierarchy.return_value = '<node text="Awesome Bio"/>'
|
||||
|
||||
mock_sleep = MagicMock()
|
||||
mock_click = MagicMock()
|
||||
import GramAddict.core.bot_flow
|
||||
monkeypatch.setattr(GramAddict.core.bot_flow, "sleep", mock_sleep)
|
||||
monkeypatch.setattr(GramAddict.core.bot_flow, "random_sleep", mock_sleep)
|
||||
monkeypatch.setattr(GramAddict.core.bot_flow, "_humanized_click", mock_click)
|
||||
import GramAddict.core.utils
|
||||
monkeypatch.setattr(GramAddict.core.utils, "random_sleep", mock_sleep)
|
||||
|
||||
result = _run_zero_latency_unfollow_loop(
|
||||
self.mock_device, None, None, self.mock_configs, self.mock_session_state, "FollowingList", self.cognitive_stack
|
||||
)
|
||||
|
||||
assert result == "BOREDOM_CHANGE_FEED"
|
||||
assert self.mock_session_state.totalUnfollowed == 0
|
||||
|
||||
# Clicked profile, then went back (no unfollow clicks)
|
||||
assert mock_click.call_count == 1
|
||||
mock_click.assert_has_calls([call(self.mock_device, 100, 200)])
|
||||
assert self.mock_device.back.call_count == 1
|
||||
|
||||
def test_unfollow_loop_skips_close_friend(self, monkeypatch):
|
||||
"""
|
||||
If 'enge freunde' is in the XML, skip resonance eval, skip unfollowing, just go back.
|
||||
"""
|
||||
def fake_extract_semantic_nodes(xml, intent, **kwargs):
|
||||
if "profile rows" in intent:
|
||||
return [{"semantic_string": "Profile Row", "x": 100, "y": 200, "bounds": "[50,150]", "skip": False}]
|
||||
return []
|
||||
|
||||
self.mock_telepathic._extract_semantic_nodes.side_effect = fake_extract_semantic_nodes
|
||||
self.mock_dopamine.wants_to_change_feed.side_effect = [True]
|
||||
|
||||
# Mock XML with Enge Freunde
|
||||
self.mock_device.dump_hierarchy.side_effect = [
|
||||
'<node text="FollowingList"/>', # First dump on list
|
||||
'<node text="Enge Freunde"/>' # Second dump on profile
|
||||
]
|
||||
|
||||
mock_sleep = MagicMock()
|
||||
mock_click = MagicMock()
|
||||
import GramAddict.core.bot_flow
|
||||
monkeypatch.setattr(GramAddict.core.bot_flow, "sleep", mock_sleep)
|
||||
monkeypatch.setattr(GramAddict.core.bot_flow, "random_sleep", mock_sleep)
|
||||
monkeypatch.setattr(GramAddict.core.bot_flow, "_humanized_click", mock_click)
|
||||
import GramAddict.core.utils
|
||||
monkeypatch.setattr(GramAddict.core.utils, "random_sleep", mock_sleep)
|
||||
|
||||
result = _run_zero_latency_unfollow_loop(
|
||||
self.mock_device, None, None, self.mock_configs, self.mock_session_state, "FollowingList", self.cognitive_stack
|
||||
)
|
||||
|
||||
assert result == "BOREDOM_CHANGE_FEED"
|
||||
assert self.mock_session_state.totalUnfollowed == 0
|
||||
assert mock_click.call_count == 1
|
||||
assert self.mock_device.back.call_count == 1
|
||||
# Resonance shouldn't even be called for close friends
|
||||
self.mock_resonance.calculate_resonance.assert_not_called()
|
||||
|
||||
def test_unfollow_loop_scrolls_if_empty(self, monkeypatch):
|
||||
"""
|
||||
|
||||
Reference in New Issue
Block a user