Skip to content

Commit

Permalink
Rename test functions to specify that they are testing FSClient
Browse files Browse the repository at this point in the history
The original draft of these tests was class-based, with the class name
describing that the tests were for FSClient. With the class removed,
this should be in the test name.
  • Loading branch information
terminalmage authored and s0undt3ch committed Feb 9, 2024
1 parent 063adf0 commit b223b13
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/pytests/unit/test_fileclient.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
from tests.support.mock import MagicMock, patch


def test_master_no_fs_update(master_opts):
def test_fsclient_master_no_fs_update(master_opts):
"""
Test that an FSClient spawned from the master does not cause fileserver
backends to be refreshed on instantiation. The master already has the
Expand All @@ -18,7 +18,7 @@ def test_master_no_fs_update(master_opts):
assert client.channel.fs.update.call_count == 0


def test_masterless_fs_update(minion_opts):
def test_fsclient_masterless_fs_update(minion_opts):
"""
Test that an FSClient spawned from a masterless run refreshes the
fileserver backends. This is necessary to ensure that a masterless run
Expand Down

0 comments on commit b223b13

Please sign in to comment.