Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

T5486: smoketest: adjust to new process_named_running() implementation #2154

Merged
merged 1 commit into from Aug 17, 2023
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
7 changes: 4 additions & 3 deletions smoketest/scripts/cli/test_service_dns_dynamic.py
Expand Up @@ -22,10 +22,11 @@

from vyos.configsession import ConfigSessionError
from vyos.util import cmd
from vyos.util import process_named_running
from vyos.util import process_running

PROCESS_NAME = 'ddclient'
DDCLIENT_CONF = '/run/ddclient/ddclient.conf'
DDCLIENT_PID = '/run/ddclient/ddclient.pid'
base_path = ['service', 'dns', 'dynamic']

def get_config_value(key):
Expand Down Expand Up @@ -89,7 +90,7 @@ def test_dyndns_service(self):
self.assertTrue(pwd == "'" + password + "'")

# Check for running process
self.assertTrue(process_named_running(PROCESS_NAME))
self.assertTrue(process_running(DDCLIENT_PID))

def test_dyndns_rfc2136(self):
# Check if DDNS service can be configured and runs
Expand Down Expand Up @@ -119,7 +120,7 @@ def test_dyndns_rfc2136(self):
# TODO: inspect generated configuration file

# Check for running process
self.assertTrue(process_named_running(PROCESS_NAME))
self.assertTrue(process_running(DDCLIENT_PID))

if __name__ == '__main__':
unittest.main(verbosity=2)