Exclude tests/.* for ruff linting

.pre-commit-config.yaml

@@ -20,7 +20,7 @@ repos:
       - id: ruff-format
         exclude: &exclude_ruff >
           (?x)^(
-
+            tests/.*|
           )$
 
       - id: ruff

tests/base/test_statemachine.py

@@ -25,7 +25,7 @@ def __init__(self, player, track):
         super().__init__(player)
         self.track = track
         self._last_time = None
-        self._played = 0.
+        self._played = 0.0
 
     def __str__(self):
         if self.in_state:
@@ -55,8 +55,7 @@ class Paused(state_machine.State):
     TRANSITIONS = {STOP: STOPPED}
 
     def __init__(self, player, playing_state):
-        assert isinstance(playing_state, Playing), \
-            'Must provide the playing state to pause'
+        assert isinstance(playing_state, Playing), 'Must provide the playing state to pause'
         super().__init__(player)
         self.playing_state = playing_state
 
@@ -117,14 +116,13 @@ def stop(self):
 
 
 class TestStateMachine(unittest.TestCase):
-
     def test_basic(self):
         cd_player = CdPlayer()
         self.assertEqual(cd_player.state, STOPPED)
 
         cd_player.play('Eminem - The Real Slim Shady')
         self.assertEqual(cd_player.state, PLAYING)
-        time.sleep(1.)
+        time.sleep(1.0)
 
         cd_player.pause()
         self.assertEqual(cd_player.state, PAUSED)

tests/persistence/test_inmemory.py

@@ -5,6 +5,8 @@
 
 import plumpy
 
+import plumpy
+
 
 class TestInMemoryPersister(unittest.TestCase):
     def test_save_load_roundtrip(self):

tests/persistence/test_pickle.py

@@ -10,6 +10,8 @@
 import plumpy
 from test.utils import ProcessWithCheckpoint
 
+import plumpy
+
 
 class TestPicklePersister(unittest.TestCase):
     def test_save_load_roundtrip(self):

tests/rmq/test_process_comms.py

@@ -3,13 +3,13 @@
 import copy
 
 import kiwipy
-from kiwipy import rmq
 import pytest
 import shortuuid
+from kiwipy import rmq
 
 import plumpy
-from plumpy import process_comms
 import plumpy.communications
+from plumpy import process_comms
 
 from .. import utils
 
@@ -44,7 +44,6 @@ def sync_controller(thread_communicator: rmq.RmqThreadCommunicator):
 
 
 class TestRemoteProcessController:
-
     @pytest.mark.asyncio
     async def test_pause(self, thread_communicator, async_controller):
         proc = utils.WaitForSignalProcess(communicator=thread_communicator)
@@ -123,7 +122,6 @@ def on_broadcast_receive(**msg):
 
 
 class TestRemoteProcessThreadController:
-
     @pytest.mark.asyncio
     async def test_pause(self, thread_communicator, sync_controller):
         proc = utils.WaitForSignalProcess(communicator=thread_communicator)

tests/test_expose.py

@@ -6,7 +6,6 @@
 from plumpy.ports import PortNamespace
 from plumpy.process_spec import ProcessSpec
 from plumpy.processes import Process
-from test.utils import NewLoopProcess
 
 
 def validator_function(input, port):

tests/test_process_comms.py

@@ -3,17 +3,15 @@
 from tests import utils
 
 import plumpy
-from plumpy import communications, process_comms
+from plumpy import process_comms
 
 
 class Process(plumpy.Process):
-
     def run(self):
         pass
 
 
 class CustomObjectLoader(plumpy.DefaultObjectLoader):
-
     def load_object(self, identifier):
         if identifier == 'jimmy':
             return Process
@@ -45,7 +43,7 @@ async def test_continue():
 
 @pytest.mark.asyncio
 async def test_loader_is_used():
-    """ Make sure that the provided class loader is used by the process launcher """
+    """Make sure that the provided class loader is used by the process launcher"""
     loader = CustomObjectLoader()
     proc = Process()
     persister = plumpy.InMemoryPersister(loader=loader)