From f9bab9fa29bc294849bafe7f3961bdf660e13420 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=9F=D1=80=D0=B5=D0=B4=D1=80=D0=B0=D0=B3=20=D0=9D=D0=B8?= =?UTF-8?q?=D0=BA=D0=BE=D0=BB=D0=B8=D1=9B?= Date: Sun, 7 Apr 2024 13:40:44 +0200 Subject: [PATCH] sometimes things dont need to make sense --- tests/test_configurations.py | 51 ++++++++++++++++-------------------- 1 file changed, 22 insertions(+), 29 deletions(-) diff --git a/tests/test_configurations.py b/tests/test_configurations.py index 93af0a0ef..533c12229 100644 --- a/tests/test_configurations.py +++ b/tests/test_configurations.py @@ -1,12 +1,13 @@ from LSP.plugin.core.configurations import WindowConfigManager from test_mocks import DISABLED_CONFIG from test_mocks import TEST_CONFIG +from unittest import TestCase from unittest.mock import MagicMock +from unittesting import ViewTestCase import sublime -import unittest -class GlobalConfigManagerTests(unittest.TestCase): +class GlobalConfigManagerTests(TestCase): def test_empty_configs(self): window_mgr = WindowConfigManager(sublime.active_window(), {}) @@ -24,35 +25,29 @@ def test_override_config(self): self.assertFalse(list(window_mgr.all.values())[0].enabled) -class WindowConfigManagerTests(unittest.TestCase): +class WindowConfigManagerTests(ViewTestCase): def test_no_configs(self): - view = sublime.active_window().active_view() - self.assertIsNotNone(view) - assert view - manager = WindowConfigManager(sublime.active_window(), {}) - self.assertEqual(list(manager.match_view(view)), []) + self.assertIsNotNone(self.view) + self.assertIsNotNone(self.window) + manager = WindowConfigManager(self.window, {}) + self.assertEqual(list(manager.match_view(self.view)), []) def test_with_single_config(self): - window = sublime.active_window() - view = window.active_view() - self.assertIsNotNone(view) - assert view - manager = WindowConfigManager(window, {TEST_CONFIG.name: TEST_CONFIG}) - view.syntax = MagicMock(return_value=sublime.Syntax( + self.assertIsNotNone(self.view) + self.assertIsNotNone(self.window) + manager = WindowConfigManager(self.window, {TEST_CONFIG.name: TEST_CONFIG}) + self.view.syntax = MagicMock(return_value=sublime.Syntax( path="Packages/Text/Plain text.tmLanguage", name="Plain Text", scope="text.plain", hidden=False )) - view.settings().set("lsp_uri", "file:///foo/bar.txt") - self.assertEqual(list(manager.match_view(view)), [TEST_CONFIG]) + self.view.settings().set("lsp_uri", "file:///foo/bar.txt") + self.assertEqual(list(manager.match_view(self.view)), [TEST_CONFIG]) def test_applies_project_settings(self): - window = sublime.active_window() - view = window.active_view() - assert view - window.project_data = MagicMock(return_value={ + self.window.project_data = MagicMock(return_value={ "settings": { "LSP": { "test": { @@ -61,24 +56,22 @@ def test_applies_project_settings(self): } } }) - manager = WindowConfigManager(window, {DISABLED_CONFIG.name: DISABLED_CONFIG}) - view.syntax = MagicMock(return_value=sublime.Syntax( + manager = WindowConfigManager(self.window, {DISABLED_CONFIG.name: DISABLED_CONFIG}) + self.view.syntax = MagicMock(return_value=sublime.Syntax( path="Packages/Text/Plain text.tmLanguage", name="Plain Text", scope="text.plain", hidden=False )) - view.settings().set("lsp_uri", "file:///foo/bar.txt") - configs = list(manager.match_view(view)) + self.view.settings().set("lsp_uri", "file:///foo/bar.txt") + configs = list(manager.match_view(self.view)) self.assertEqual(len(configs), 1) config = configs[0] self.assertEqual(DISABLED_CONFIG.name, config.name) self.assertTrue(config.enabled) def test_disables_temporarily(self): - window = sublime.active_window() - view = window.active_view() - window.project_data = MagicMock(return_value={ + self.window.project_data = MagicMock(return_value={ "settings": { "LSP": { "test": { @@ -88,7 +81,7 @@ def test_disables_temporarily(self): } }) - manager = WindowConfigManager(window, {DISABLED_CONFIG.name: DISABLED_CONFIG}) + manager = WindowConfigManager(self.window, {DISABLED_CONFIG.name: DISABLED_CONFIG}) # disables config in-memory manager.disable_config(DISABLED_CONFIG.name, only_for_session=True) - self.assertFalse(any(manager.match_view(view))) + self.assertFalse(any(manager.match_view(self.view)))