diff options
author | Calum Lind <calumlind+deluge@gmail.com> | 2021-07-24 14:11:58 +0100 |
---|---|---|
committer | Calum Lind <calumlind+deluge@gmail.com> | 2021-08-01 08:48:27 +0100 |
commit | b89b2c45b145c33c41115a9aac6e46b085ce668a (patch) | |
tree | 8c373cdb925284b6915061659efb2444bfa0b89c /deluge/tests | |
parent | e38f1173cf470f2f8373f2fa4edbb726c932c980 (diff) | |
download | deluge-b89b2c45b145c33c41115a9aac6e46b085ce668a.tar.gz deluge-b89b2c45b145c33c41115a9aac6e46b085ce668a.tar.bz2 deluge-b89b2c45b145c33c41115a9aac6e46b085ce668a.zip |
[Console] Fix using windows-curses on Windows
The console tests are still failing on Windows due to an issue where the
sys args are not being correctly replaced in the tests so the pytest
args are being passed to console.
Diffstat (limited to 'deluge/tests')
-rw-r--r-- | deluge/tests/common.py | 2 | ||||
-rw-r--r-- | deluge/tests/test_client.py | 6 | ||||
-rw-r--r-- | deluge/tests/test_common.py | 6 | ||||
-rw-r--r-- | deluge/tests/test_ui_console.py | 4 | ||||
-rw-r--r-- | deluge/tests/test_ui_entry.py | 18 |
5 files changed, 15 insertions, 21 deletions
diff --git a/deluge/tests/common.py b/deluge/tests/common.py index f0ddb2b99..be33f8c58 100644 --- a/deluge/tests/common.py +++ b/deluge/tests/common.py @@ -69,7 +69,7 @@ def add_watchdog(deferred, timeout=0.05, message=None): return value deferred.addBoth(callback) - watchdog = reactor.callLater(timeout, defer.timeout, deferred) + watchdog = reactor.callLater(timeout, defer.Deferred.addTimeout, deferred) return watchdog diff --git a/deluge/tests/test_client.py b/deluge/tests/test_client.py index c89ad5309..ae1e95a71 100644 --- a/deluge/tests/test_client.py +++ b/deluge/tests/test_client.py @@ -11,7 +11,7 @@ from twisted.internet import defer import deluge.component as component from deluge import error -from deluge.common import AUTH_LEVEL_NORMAL, get_localhost_auth, windows_check +from deluge.common import AUTH_LEVEL_NORMAL, get_localhost_auth from deluge.core.authmanager import AUTH_LEVEL_ADMIN from deluge.ui.client import Client, DaemonSSLProxy, client @@ -79,10 +79,6 @@ class NoVersionSendingClient(Client): class ClientTestCase(BaseTestCase, DaemonBase): - - if windows_check(): - skip = 'windows cant start_core not enough arguments for format string' - def set_up(self): d = self.common_set_up() d.addCallback(self.start_core) diff --git a/deluge/tests/test_common.py b/deluge/tests/test_common.py index 3cecb6467..4f6aa2fd4 100644 --- a/deluge/tests/test_common.py +++ b/deluge/tests/test_common.py @@ -8,6 +8,7 @@ from __future__ import unicode_literals import os +import sys import tarfile from twisted.trial import unittest @@ -98,8 +99,9 @@ class CommonTestCase(unittest.TestCase): self.assertTrue(is_infohash('2dc5d0e71a66fe69649a640d39cb00a259704973')) def test_get_path_size(self): - if windows_check(): - raise unittest.SkipTest('os devnull is different on windows') + if windows_check() and sys.version_info < (3, 8): + # https://bugs.python.org/issue1311 + raise unittest.SkipTest('os.devnull returns False on Windows') self.assertTrue(get_path_size(os.devnull) == 0) self.assertTrue(get_path_size('non-existant.file') == -1) diff --git a/deluge/tests/test_ui_console.py b/deluge/tests/test_ui_console.py index da97f5c15..3667c608e 100644 --- a/deluge/tests/test_ui_console.py +++ b/deluge/tests/test_ui_console.py @@ -9,7 +9,6 @@ from __future__ import unicode_literals import argparse -from deluge.common import windows_check from deluge.ui.console.cmdline.commands.add import Command from deluge.ui.console.cmdline.commands.config import json_eval from deluge.ui.console.widgets.fields import TextInput @@ -46,8 +45,7 @@ class UIConsoleFieldTestCase(BaseTestCase): complete=False, ) self.assertTrue(t) - if not windows_check(): - self.assertTrue(t.handle_read(33)) + self.assertTrue(t.handle_read(33)) class UIConsoleCommandsTestCase(BaseTestCase): diff --git a/deluge/tests/test_ui_entry.py b/deluge/tests/test_ui_entry.py index bf4cff5f3..f85bc7d7d 100644 --- a/deluge/tests/test_ui_entry.py +++ b/deluge/tests/test_ui_entry.py @@ -19,6 +19,9 @@ from twisted.internet import defer import deluge import deluge.component as component +import deluge.ui.console +import deluge.ui.console.cmdline.commands.quit +import deluge.ui.console.main import deluge.ui.web.server from deluge.common import PY2, get_localhost_auth, windows_check from deluge.ui import ui_entry @@ -28,11 +31,6 @@ from . import common from .basetest import BaseTestCase from .daemon_base import DaemonBase -if not windows_check(): - import deluge.ui.console - import deluge.ui.console.cmdline.commands.quit - import deluge.ui.console.main - DEBUG_COMMAND = False sys_stdout = sys.stdout @@ -102,7 +100,7 @@ class UIWithDaemonBaseTestCase(UIBaseTestCase, DaemonBase): class DelugeEntryTestCase(BaseTestCase): if windows_check(): - skip = 'cannot test console ui on windows' + skip = 'Console ui test on Windows broken due to sys args issue' def set_up(self): common.set_tmp_config_dir() @@ -250,7 +248,7 @@ class WebUIBaseTestCase(UIBaseTestCase): class WebUIScriptEntryTestCase(BaseTestCase, WebUIBaseTestCase): if windows_check(): - skip = 'cannot test console ui on windows' + skip = 'Console ui test on Windows broken due to sys args issue' def __init__(self, testname): super(WebUIScriptEntryTestCase, self).__init__(testname) @@ -269,7 +267,7 @@ class WebUIScriptEntryTestCase(BaseTestCase, WebUIBaseTestCase): class WebUIDelugeScriptEntryTestCase(BaseTestCase, WebUIBaseTestCase): if windows_check(): - skip = 'cannot test console ui on windows' + skip = 'Console ui test on Windows broken due to sys args issue' def __init__(self, testname): super(WebUIDelugeScriptEntryTestCase, self).__init__(testname) @@ -466,7 +464,7 @@ class ConsoleScriptEntryWithDaemonTestCase( ): if windows_check(): - skip = 'cannot test console ui on windows' + skip = 'Console ui test on Windows broken due to sys args issue' def __init__(self, testname): super(ConsoleScriptEntryWithDaemonTestCase, self).__init__(testname) @@ -492,7 +490,7 @@ class ConsoleScriptEntryWithDaemonTestCase( class ConsoleScriptEntryTestCase(BaseTestCase, ConsoleUIBaseTestCase): if windows_check(): - skip = 'cannot test console ui on windows' + skip = 'Console ui test on Windows broken due to sys args issue' def __init__(self, testname): super(ConsoleScriptEntryTestCase, self).__init__(testname) |