From f9a0d85192d159da3fe9948a8f90528bef606f8b Mon Sep 17 00:00:00 2001 From: vivodi <103735539+vivodi@users.noreply.github.com> Date: Sat, 25 Jan 2025 00:05:13 +0800 Subject: [PATCH] fix --- flexget/tests/test_misc.py | 6 ++++-- flexget/tests/test_seen.py | 3 ++- pyproject.toml | 2 +- 3 files changed, 7 insertions(+), 4 deletions(-) diff --git a/flexget/tests/test_misc.py b/flexget/tests/test_misc.py index 0c99f45848..a8a379031f 100644 --- a/flexget/tests/test_misc.py +++ b/flexget/tests/test_misc.py @@ -40,11 +40,13 @@ def test_disable_builtins(self, execute_task): # Execute the task once, then we'll make sure seen plugin isn't rejecting on future executions execute_task('test') task = execute_task('test') - assert task.find_entry('accepted', title='dupe1') and task.find_entry( + assert task.find_entry('accepted', title='dupe1') + assert task.find_entry( 'accepted', title='dupe2' ), 'disable is not working?' task = execute_task('test2') - assert task.find_entry(title='dupe1').accepted and task.find_entry( + assert task.find_entry(title='dupe1').accepted + assert task.find_entry( 'accepted', title='dupe2' ), 'disable is not working?' diff --git a/flexget/tests/test_seen.py b/flexget/tests/test_seen.py index e7e42c3722..92ef277f75 100644 --- a/flexget/tests/test_seen.py +++ b/flexget/tests/test_seen.py @@ -49,7 +49,8 @@ def test_seen(self, execute_task): # test that we don't filter reject on non-string fields (ie, seen same imdb_score) task = execute_task('test_number') - assert task.find_entry(title='New title 1') and task.find_entry(title='New title 2'), ( + assert task.find_entry(title='New title 1') + assert task.find_entry(title='New title 2'), ( 'Item should not have been rejected because of number field' ) diff --git a/pyproject.toml b/pyproject.toml index 04d4182bfa..027306042b 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -139,7 +139,7 @@ line-length = 99 extend-exclude = ["flexget/ui"] [tool.ruff.lint] -select = ["C4", "E", "F", "I", "ISC", "PGH", "PLE", "RET", "RUF", "SIM", "TCH", "UP", "W"] +select = ["C4", "E", "F", "I", "ISC", "PGH", "PLE", "PT", "RET", "RUF", "SIM", "TCH", "UP", "W"] ignore = [ "E501", # TODO: enable this rule (requires a lot of manual work) "PLE1205", # Maybe can re-enable after https://github.com/astral-sh/ruff/issues/13390