test_runner fixtures: feedback from review

I forgot to add these fixes before merging:
- typo in Postgres doc-comment
- add 'wal_acceptor' to the list of conflicting processes.
This commit is contained in:
Eric Seppanen
2021-05-03 18:44:13 -07:00
parent ce646ea845
commit 3e15a5c325

View File

@@ -53,7 +53,7 @@ def zenfixture(func):
@pytest.fixture(autouse=True, scope='session')
def safety_check():
""" Ensure that no unwanted daemons are running before we start testing. """
cmd = ['pgrep', '-c', 'pageserver|postgres']
cmd = ['pgrep', '-c', 'pageserver|postgres|wal_acceptor']
result = subprocess.run(cmd)
if result.returncode == 0:
# returncode of 0 means it found something.
@@ -128,7 +128,7 @@ def pageserver(zenith_cli):
class Postgres:
""" An object representing a running pageserver. """
""" An object representing a running postgres daemon. """
def __init__(self, zenith_cli):
self.zenith_cli = zenith_cli