From e75b6808b8c40733ac01146455038ceebb2cb72a Mon Sep 17 00:00:00 2001 From: "T.Rzepka" Date: Thu, 3 Oct 2024 22:19:01 +0200 Subject: [PATCH] Fixed style issues and complains about exceptions with description. --- tests/test_single_file_watched.py | 132 ++++++++++++++---------------- 1 file changed, 62 insertions(+), 70 deletions(-) diff --git a/tests/test_single_file_watched.py b/tests/test_single_file_watched.py index 4329da94..e1d8c1da 100644 --- a/tests/test_single_file_watched.py +++ b/tests/test_single_file_watched.py @@ -1,9 +1,7 @@ -# coding: utf-8 +"""Test cases when a single file is subscribed.""" -""" Test cases when a single file is subscribed. """ - -from os import mkdir import time +from os import mkdir from .utils import P, StartWatching, TestEventQueue @@ -11,18 +9,18 @@ def test_selftest(p: P, start_watching: StartWatching, event_queue: TestEventQueue) -> None: - """ Check if all events are catched in SLEEP time. """ + """Check if all events are catched in SLEEP time.""" emitter = start_watching(path=p()) # Pretty sure this should work - with open(p('file1.bak'), 'wt') as fp: - fp.write('test1') - - with open(p('file2.bak'), 'wt') as fp: - fp.write('test2') + with open(p("file1.bak"), "w") as fp: + fp.write("test1") + + with open(p("file2.bak"), "w") as fp: + fp.write("test2") time.sleep(SLEEP) - + found_files = set() try: while event_queue.qsize(): @@ -33,101 +31,95 @@ def test_selftest(p: P, start_watching: StartWatching, event_queue: TestEventQue found_files.add(event.src_path) finally: emitter.stop() - - assert len(found_files) == 2, 'Number of expected files differ. Increase sleep time.' + + assert len(found_files) == 2, "Number of expected files differ. Increase sleep time." def test_file_access(p: P, start_watching: StartWatching, event_queue: TestEventQueue) -> None: - """ Check if file fires events. """ + """Check if file fires events.""" - file1 = 'file1.bak' + file1 = "file1.bak" tmpfile = p(file1) - with open(tmpfile, 'wt') as fp: - fp.write('init1') - + with open(tmpfile, "w") as fp: + fp.write("init1") + emitter = start_watching(path=tmpfile) # This is what we want to see - with open(tmpfile, 'wt') as fp: - fp.write('test1') + with open(tmpfile, "w") as fp: + fp.write("test1") time.sleep(SLEEP) - + try: while event_queue.qsize(): event, _ = event_queue.get() if event.is_directory: assert event.src_path == p() continue - else: - assert event.src_path.endswith(file1) + assert event.src_path.endswith(file1) break else: - assert False, 'No event catched.' + raise AssertionError # No event catched finally: emitter.stop() -def test_file_access_multiple( - p: P, start_watching: StartWatching, event_queue: TestEventQueue -) -> None: - """ Check if file fires events multiple times. """ +def test_file_access_multiple(p: P, start_watching: StartWatching, event_queue: TestEventQueue) -> None: + """Check if file fires events multiple times.""" - file1 = 'file1.bak' + file1 = "file1.bak" tmpfile = p(file1) - with open(tmpfile, 'wt') as fp: - fp.write('init1') - + with open(tmpfile, "w") as fp: + fp.write("init1") + emitter = start_watching(path=tmpfile) try: - for i in range(5): + for _i in range(5): # This is what we want to see multiple times - with open(tmpfile, 'wt') as fp: - fp.write('test1') + with open(tmpfile, "w") as fp: + fp.write("test1") time.sleep(SLEEP) - + while event_queue.qsize(): event, _ = event_queue.get() if event.is_directory: assert event.src_path == p() continue - else: - assert event.src_path.endswith(file1) + assert event.src_path.endswith(file1) break else: - assert False, 'No event catched.' - + raise AssertionError # No event catched + finally: emitter.stop() -def test_file_access_other_file( - p: P, start_watching: StartWatching, event_queue: TestEventQueue -) -> None: - """ Check if other files doesn't fires events. """ +def test_file_access_other_file(p: P, start_watching: StartWatching, event_queue: TestEventQueue) -> None: + """Check if other files doesn't fires events.""" - file1 = 'file1.bak' + file1 = "file1.bak" tmpfile = p(file1) - with open(tmpfile, 'wt') as fp: - fp.write('init1') - + with open(tmpfile, "w") as fp: + fp.write("init1") + emitter = start_watching(path=tmpfile) # Don't wanted - with open(p('file2.bak'), 'wt') as fp: - fp.write('test2') + with open(p("file2.bak"), "w") as fp: + fp.write("test2") # but this - with open(tmpfile, 'wt') as fp: - fp.write('test1') - + with open(tmpfile, "w") as fp: + fp.write("test1") + time.sleep(SLEEP) - + found_files = set() try: while event_queue.qsize(): @@ -139,32 +131,32 @@ def test_file_access_other_file( assert event.src_path.endswith(file1) finally: emitter.stop() - - assert len(found_files) == 1, 'Number of expected files differ. Wrong file catched.' + + assert len(found_files) == 1, "Number of expected files differ. Wrong file catched." def test_create_folder(p: P, start_watching: StartWatching, event_queue: TestEventQueue) -> None: - """ Check if creation of a directory and inside files doesn't fires events. """ + """Check if creation of a directory and inside files doesn't fires events.""" - file1 = 'file1.bak' + file1 = "file1.bak" tmpfile = p(file1) - with open(tmpfile, 'wt') as fp: - fp.write('init1') - + with open(tmpfile, "w") as fp: + fp.write("init1") + emitter = start_watching(path=tmpfile) # Don't wanted - mkdir(p('myfolder')) - with open(p('myfolder/file2.bak'), 'wt') as fp: - fp.write('test2') + mkdir(p("myfolder")) + with open(p("myfolder/file2.bak"), "w") as fp: + fp.write("test2") # but this - with open(tmpfile, 'wt') as fp: - fp.write('test1') - + with open(tmpfile, "w") as fp: + fp.write("test1") + time.sleep(SLEEP) - + found_files = set() try: while event_queue.qsize(): @@ -176,5 +168,5 @@ def test_create_folder(p: P, start_watching: StartWatching, event_queue: TestEve assert event.src_path.endswith(file1) finally: emitter.stop() - - assert len(found_files) == 1, 'Number of expected files differ. Wrong file catched.' + + assert len(found_files) == 1, "Number of expected files differ. Wrong file catched."