50f9f695ba
All checks were successful
Zapret source guards / validate (push) Successful in 10s
Страница «Свой DNS» (dns.ui.custom_server_page) открывается из «Настройки DNS» и показывает путь хлебными крошками. Три поля вместо пяти: адрес DoH, IP-адреса одной строкой (IPv4 и IPv6 вместе) и название, которое можно не писать. Если задан только адрес DoH, программа узнаёт адреса сервера у эталонных серверов по DoH и у DNS системы, а затем проверяет каждый настоящим запросом DoH с проверкой сертификата (dns.doh_lookup). На Windows без DoH адрес должен отвечать и на обычный DNS, иначе сервер не сохраняется. Шаблон DoH своего сервера попадает в настройки адаптера так же, как у серверов программы. Правила своих серверов вынесены из окна в dns.custom_servers; список читается и пишется в фоне одной транзакцией (settings.store.update_custom_dns_servers) и приходит на страницу вместе с её состоянием. Страница DNS обновляет состояние при каждом открытии. Кнопка «Вернуть автоматически» заменена первой плиткой сетки и применяется сразу, без вопроса. Кнопка сброса кэша не выключается на время работы: выключенная теряла фокус, он уходил на сетку плиток, и страница прокручивалась к выбранному серверу. Сетка прокручивает к плитке только при фокусе с клавиатуры. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
112 lines
4.4 KiB
Python
112 lines
4.4 KiB
Python
from __future__ import annotations
|
||
|
||
import unittest
|
||
from tempfile import TemporaryDirectory
|
||
from unittest.mock import patch
|
||
|
||
from settings import store as settings_store
|
||
|
||
|
||
class SettingsReadCostTests(unittest.TestCase):
|
||
"""Чтение настроек не должно копировать весь документ ради одного значения."""
|
||
|
||
def setUp(self) -> None:
|
||
self._temp_dir = TemporaryDirectory()
|
||
self.addCleanup(self._temp_dir.cleanup)
|
||
patcher = patch.object(settings_store, "MAIN_DIRECTORY", self._temp_dir.name)
|
||
patcher.start()
|
||
self.addCleanup(patcher.stop)
|
||
settings_store.reset_settings()
|
||
|
||
def test_scalar_getter_does_not_copy_whole_document(self) -> None:
|
||
with patch.object(settings_store, "read_settings") as read_settings:
|
||
settings_store.get_dpi_autostart()
|
||
|
||
read_settings.assert_not_called()
|
||
|
||
def test_scalar_getter_returns_stored_value(self) -> None:
|
||
settings_store.set_dpi_autostart(False)
|
||
self.assertFalse(settings_store.get_dpi_autostart())
|
||
|
||
settings_store.set_dpi_autostart(True)
|
||
self.assertTrue(settings_store.get_dpi_autostart())
|
||
|
||
def test_container_value_is_detached_from_cache(self) -> None:
|
||
settings_store.set_hosts_selection({"youtube": "default"})
|
||
|
||
selection = settings_store.get_hosts_selection()
|
||
selection["youtube"] = "hacked"
|
||
selection["discord"] = "hacked"
|
||
|
||
self.assertEqual(settings_store.get_hosts_selection(), {"youtube": "default"})
|
||
|
||
def test_section_copy_is_detached_from_cache(self) -> None:
|
||
program = settings_store.get_program_settings()
|
||
program["dpi_autostart"] = "corrupted"
|
||
|
||
self.assertNotEqual(settings_store.get_program_settings().get("dpi_autostart"), "corrupted")
|
||
|
||
def test_custom_dns_servers_are_detached_from_cache(self) -> None:
|
||
stored = settings_store.update_custom_dns_servers(
|
||
lambda _current: [{"id": "one", "name": "One", "ipv4": ["1.1.1.1"]}]
|
||
)
|
||
self.assertEqual([server["name"] for server in stored], ["One"])
|
||
|
||
servers = settings_store.get_custom_dns_servers()
|
||
servers.append({"id": "two", "name": "Hacked", "ipv4": ["8.8.8.8"]})
|
||
servers[0]["ipv4"] = ["9.9.9.9"]
|
||
|
||
self.assertEqual(settings_store.get_custom_dns_servers(), stored)
|
||
|
||
def test_getter_does_not_resolve_database_path_again(self) -> None:
|
||
# Path.resolve() на Windows — ~85 мкс, в двадцать раз дороже запроса
|
||
# ревизии. Страница читает десятки настроек за один показ.
|
||
settings_store.get_dpi_autostart()
|
||
|
||
with patch.object(
|
||
settings_store,
|
||
"get_settings_database_path",
|
||
wraps=settings_store.get_settings_database_path,
|
||
) as database_path:
|
||
for _ in range(20):
|
||
settings_store.get_dpi_autostart()
|
||
|
||
database_path.assert_not_called()
|
||
|
||
def test_changed_settings_root_opens_its_own_database(self) -> None:
|
||
settings_store.set_dpi_autostart(False)
|
||
other_dir = TemporaryDirectory()
|
||
self.addCleanup(other_dir.cleanup)
|
||
|
||
with patch.object(settings_store, "MAIN_DIRECTORY", other_dir.name):
|
||
settings_store.reset_settings()
|
||
settings_store.set_dpi_autostart(True)
|
||
self.assertTrue(settings_store.get_dpi_autostart())
|
||
self.assertTrue(settings_store.get_settings_database_path().is_file())
|
||
|
||
# Вернулись к прежнему корню — читается прежняя база, а не кэш чужой.
|
||
self.assertFalse(settings_store.get_dpi_autostart())
|
||
|
||
def test_closed_database_is_opened_again_for_same_root(self) -> None:
|
||
settings_store.set_dpi_autostart(False)
|
||
|
||
settings_store.close_settings_database()
|
||
|
||
self.assertFalse(settings_store.get_dpi_autostart())
|
||
|
||
def test_read_settings_still_returns_detached_document(self) -> None:
|
||
data = settings_store.read_settings()
|
||
data["program"]["dpi_autostart"] = "corrupted"
|
||
|
||
self.assertNotEqual(
|
||
settings_store.read_settings()["program"]["dpi_autostart"],
|
||
"corrupted",
|
||
)
|
||
|
||
def test_write_is_visible_to_the_next_scalar_read(self) -> None:
|
||
settings_store.set_tray_close_mode("minimize_only")
|
||
self.assertEqual(settings_store.get_tray_close_mode(), "minimize_only")
|
||
|
||
|
||
if __name__ == "__main__":
|
||
unittest.main()
|