bc95026147
All checks were successful
Zapret source guards / validate (push) Successful in 4s
В settings.sqlite3 появились настоящие таблицы: presets (uid pid:<uuid4>, UNIQUE(scope, file_name)), preset_remote_sources (FK с каскадом) и profile_identities (порядок реестра фиксируется колонкой position — на него опирается резолвер полных дублей). Привязка пресета к источнику держится за uid: переименование меняет только file_name, удаление убирает привязку каскадом. Старые JSON-секции мигрируются лениво. Попутно исправлен баг: оценки стратегий и мета папок профилей ключевались только legacy-ключами name:/sig: и молча отбрасывали uid-ключи, которыми рантайм давно прошивает профили; uid теперь принимается везде, а legacy- записи переносятся на uid при resolve идентичности. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
222 lines
9.6 KiB
Python
222 lines
9.6 KiB
Python
"""CRUD привязок удалённых пресетов поверх settings store."""
|
||
|
||
from __future__ import annotations
|
||
|
||
import unittest
|
||
from pathlib import Path
|
||
from tempfile import TemporaryDirectory
|
||
from unittest.mock import patch
|
||
|
||
|
||
class RemotePresetBindingsTests(unittest.TestCase):
|
||
def setUp(self):
|
||
self._temp = TemporaryDirectory()
|
||
self.addCleanup(self._temp.cleanup)
|
||
patcher = patch("settings.store.MAIN_DIRECTORY", str(Path(self._temp.name)))
|
||
patcher.start()
|
||
self.addCleanup(patcher.stop)
|
||
# Флаг одноразовой миграции секции — сбрасываем между тестовыми БД.
|
||
from presets import remote_bindings as rb
|
||
|
||
rb._LEGACY_SECTION_MIGRATED = False
|
||
|
||
def test_defaults_and_missing(self):
|
||
from presets import remote_bindings as rb
|
||
|
||
self.assertEqual(rb.load_remote_preset_bindings("winws2"), {})
|
||
self.assertIsNone(rb.get_remote_preset_binding("winws2", "nope.txt"))
|
||
self.assertFalse(rb.delete_remote_preset_binding("winws2", "nope.txt"))
|
||
self.assertFalse(rb.rename_remote_preset_binding("winws2", "a.txt", "b.txt"))
|
||
self.assertIsNone(rb.find_remote_preset_by_url("https://example.com/x.txt"))
|
||
|
||
def test_set_get_roundtrip_with_normalization(self):
|
||
from presets import remote_bindings as rb
|
||
|
||
binding = rb.make_remote_preset_binding(
|
||
"https://example.com/p.txt", synced_hash="abc", now_iso="2026-08-14T00:00:00Z"
|
||
)
|
||
saved = rb.set_remote_preset_binding("winws2", "My.txt", binding)
|
||
self.assertIsNotNone(saved)
|
||
loaded = rb.get_remote_preset_binding("winws2", "My.txt")
|
||
self.assertEqual(loaded["url"], "https://example.com/p.txt")
|
||
self.assertEqual(loaded["synced_hash"], "abc")
|
||
self.assertTrue(loaded["auto"])
|
||
self.assertFalse(loaded["detached"])
|
||
|
||
def test_binding_without_url_rejected(self):
|
||
from presets import remote_bindings as rb
|
||
|
||
self.assertIsNone(rb.set_remote_preset_binding("winws2", "My.txt", {"url": ""}))
|
||
self.assertIsNone(rb.get_remote_preset_binding("winws2", "My.txt"))
|
||
|
||
def test_scope_isolation(self):
|
||
from presets import remote_bindings as rb
|
||
|
||
rb.set_remote_preset_binding(
|
||
"winws2", "A.txt", rb.make_remote_preset_binding("https://example.com/a.txt")
|
||
)
|
||
self.assertIsNone(rb.get_remote_preset_binding("winws1", "A.txt"))
|
||
self.assertIsNotNone(rb.get_remote_preset_binding("winws2", "A.txt"))
|
||
|
||
def test_update_fields(self):
|
||
from presets import remote_bindings as rb
|
||
|
||
rb.set_remote_preset_binding(
|
||
"winws1", "B.txt", rb.make_remote_preset_binding("https://example.com/b.txt")
|
||
)
|
||
updated = rb.update_remote_preset_binding("winws1", "B.txt", detached=True, error="oops")
|
||
self.assertTrue(updated["detached"])
|
||
self.assertEqual(updated["error"], "oops")
|
||
self.assertIsNone(rb.update_remote_preset_binding("winws1", "missing.txt", error="x"))
|
||
|
||
def test_rename_migrates_binding(self):
|
||
from presets import remote_bindings as rb
|
||
|
||
rb.set_remote_preset_binding(
|
||
"winws2", "Old.txt", rb.make_remote_preset_binding("https://example.com/o.txt")
|
||
)
|
||
self.assertTrue(rb.rename_remote_preset_binding("winws2", "Old.txt", "New.txt"))
|
||
self.assertIsNone(rb.get_remote_preset_binding("winws2", "Old.txt"))
|
||
self.assertEqual(
|
||
rb.get_remote_preset_binding("winws2", "New.txt")["url"],
|
||
"https://example.com/o.txt",
|
||
)
|
||
|
||
def test_delete(self):
|
||
from presets import remote_bindings as rb
|
||
|
||
rb.set_remote_preset_binding(
|
||
"winws2", "Del.txt", rb.make_remote_preset_binding("https://example.com/d.txt")
|
||
)
|
||
self.assertTrue(rb.delete_remote_preset_binding("winws2", "Del.txt"))
|
||
self.assertIsNone(rb.get_remote_preset_binding("winws2", "Del.txt"))
|
||
|
||
def test_find_by_url_across_scopes(self):
|
||
from presets import remote_bindings as rb
|
||
|
||
rb.set_remote_preset_binding(
|
||
"winws1", "C.txt", rb.make_remote_preset_binding("https://example.com/c.txt")
|
||
)
|
||
found = rb.find_remote_preset_by_url("https://example.com/c.txt")
|
||
self.assertIsNotNone(found)
|
||
scope, file_name, binding = found
|
||
self.assertEqual(scope, "winws1")
|
||
self.assertEqual(file_name, "C.txt")
|
||
self.assertEqual(binding["url"], "https://example.com/c.txt")
|
||
|
||
def test_preset_uid_is_stable_across_rename(self):
|
||
from presets import remote_bindings as rb
|
||
|
||
uid = rb.get_or_create_preset_uid("winws2", "Дом.txt")
|
||
self.assertTrue(uid.startswith("pid:"))
|
||
self.assertEqual(rb.get_or_create_preset_uid("winws2", "Дом.txt"), uid)
|
||
rb.set_remote_preset_binding(
|
||
"winws2", "Дом.txt", rb.make_remote_preset_binding("https://example.com/d.txt")
|
||
)
|
||
self.assertTrue(rb.rename_remote_preset_binding("winws2", "Дом.txt", "Дача.txt"))
|
||
# uid не изменился, привязка переехала вместе с ним.
|
||
self.assertEqual(rb.get_preset_uid("winws2", "Дача.txt"), uid)
|
||
self.assertIsNone(rb.get_preset_uid("winws2", "Дом.txt"))
|
||
self.assertEqual(
|
||
rb.get_remote_preset_binding("winws2", "Дача.txt")["url"],
|
||
"https://example.com/d.txt",
|
||
)
|
||
|
||
def test_delete_identity_cascades_to_binding(self):
|
||
from presets import remote_bindings as rb
|
||
|
||
rb.set_remote_preset_binding(
|
||
"winws2", "Гость.txt", rb.make_remote_preset_binding("https://example.com/g.txt")
|
||
)
|
||
self.assertTrue(rb.delete_preset_identity("winws2", "Гость.txt"))
|
||
self.assertIsNone(rb.get_remote_preset_binding("winws2", "Гость.txt"))
|
||
self.assertIsNone(rb.get_preset_uid("winws2", "Гость.txt"))
|
||
|
||
def test_same_file_in_two_scopes_gets_two_uids(self):
|
||
from presets import remote_bindings as rb
|
||
|
||
uid2 = rb.get_or_create_preset_uid("winws2", "Общий.txt")
|
||
uid1 = rb.get_or_create_preset_uid("winws1", "Общий.txt")
|
||
self.assertNotEqual(uid1, uid2)
|
||
|
||
def test_rename_to_occupied_name_is_rejected(self):
|
||
from presets import remote_bindings as rb
|
||
|
||
rb.get_or_create_preset_uid("winws2", "A.txt")
|
||
rb.get_or_create_preset_uid("winws2", "B.txt")
|
||
# UNIQUE(scope, file_name) не даёт двум uid схлопнуться на одно имя.
|
||
self.assertFalse(rb.rename_remote_preset_binding("winws2", "A.txt", "B.txt"))
|
||
|
||
def test_legacy_json_section_migrates_into_tables(self):
|
||
from presets import remote_bindings as rb
|
||
from settings import store as settings_store
|
||
|
||
settings_store.set_remote_presets_settings(
|
||
{
|
||
"winws2": {
|
||
"Старый.txt": {
|
||
"url": "https://example.com/old.txt",
|
||
"synced_hash": "abc",
|
||
"auto": False,
|
||
}
|
||
},
|
||
"winws1": {},
|
||
}
|
||
)
|
||
binding = rb.get_remote_preset_binding("winws2", "Старый.txt")
|
||
self.assertIsNotNone(binding)
|
||
self.assertEqual(binding["url"], "https://example.com/old.txt")
|
||
self.assertEqual(binding["synced_hash"], "abc")
|
||
self.assertFalse(binding["auto"])
|
||
# Секция после переноса очищена.
|
||
section = settings_store.get_remote_presets_settings()
|
||
self.assertEqual(section.get("winws2"), {})
|
||
|
||
def test_profile_identity_registry_uses_table_with_lazy_migration(self):
|
||
from settings import store as settings_store
|
||
|
||
# Легаси: реестр в JSON-секции.
|
||
settings_store._update_settings(
|
||
lambda data: settings_store._set_path_value(
|
||
data,
|
||
("profile_identity", "winws2"),
|
||
{"uid:abc": {"name": "YouTube", "sig": "list=youtube"}},
|
||
)
|
||
)
|
||
registry = settings_store.get_profile_identity_registry("winws2")
|
||
self.assertEqual(registry, {"uid:abc": {"name": "YouTube", "sig": "list=youtube"}})
|
||
# Запись теперь в таблице: перезапишем и перечитаем.
|
||
saved = settings_store.set_profile_identity_registry(
|
||
"winws2", {"uid:def": {"name": "Discord", "sig": ""}}
|
||
)
|
||
self.assertEqual(saved, {"uid:def": {"name": "Discord", "sig": ""}})
|
||
self.assertEqual(
|
||
settings_store.get_profile_identity_registry("winws2"),
|
||
{"uid:def": {"name": "Discord", "sig": ""}},
|
||
)
|
||
self.assertEqual(settings_store.get_profile_identity_registry("winws1"), {})
|
||
|
||
def test_normalization_survives_settings_roundtrip(self):
|
||
from settings.normalize import normalize_settings
|
||
|
||
normalized = normalize_settings(
|
||
{
|
||
"remote_presets": {
|
||
"winws2": {
|
||
"Ok.txt": {"url": "https://example.com/ok.txt", "auto": False},
|
||
"NoUrl.txt": {"etag": "x"},
|
||
"": {"url": "https://example.com/empty-key.txt"},
|
||
},
|
||
"junk": {"A.txt": {"url": "https://example.com/a.txt"}},
|
||
}
|
||
}
|
||
)
|
||
section = normalized["remote_presets"]
|
||
self.assertEqual(set(section.keys()), {"winws2", "winws1"})
|
||
self.assertEqual(set(section["winws2"].keys()), {"Ok.txt"})
|
||
self.assertFalse(section["winws2"]["Ok.txt"]["auto"])
|
||
self.assertTrue(section["winws2"]["Ok.txt"]["detached"] is False)
|
||
|
||
|
||
if __name__ == "__main__":
|
||
unittest.main()
|