From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from gandalf.ozlabs.org (mail.ozlabs.org [IPv6:2404:9400:2221:ea00::3]) by passt.top (Postfix) with ESMTPS id 91F385A026F for ; Tue, 27 Jun 2023 04:54:44 +0200 (CEST) Received: by gandalf.ozlabs.org (Postfix, from userid 1007) id 4Qqq7J6wDrz4wqd; Tue, 27 Jun 2023 12:54:36 +1000 (AEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gibson.dropbear.id.au; s=201602; t=1687834476; bh=rAH6pm1GNsKc3vHtDTYXBBl8rbZfDszl78vRspoaFeU=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=FiYbEEADHIWV+u3elrI8oTEXq/WMqvPX2EeSIbE8VXKltoYmxdpYNLHAs6hzBCOkG GRqhxw/58FbVpk8tqFLB8JGzRqPrEfmw9iWGPmiRwU+f2aVKyOjkM4D0QU6hew96Yz OIALJyXFZvJlaVe1Qeg58E4zWiprHKd8KtaGpbts= From: David Gibson To: passt-dev@passt.top, Stefano Brivio Subject: [PATCH 10/27] tasst: Helper functions for executing commands in different places Date: Tue, 27 Jun 2023 12:54:11 +1000 Message-ID: <20230627025429.2209702-11-david@gibson.dropbear.id.au> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20230627025429.2209702-1-david@gibson.dropbear.id.au> References: <20230627025429.2209702-1-david@gibson.dropbear.id.au> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Message-ID-Hash: XBYFDKKHLKM3RS5IXUPY7LOAHHI5MESZ X-Message-ID-Hash: XBYFDKKHLKM3RS5IXUPY7LOAHHI5MESZ X-MailFrom: dgibson@gandalf.ozlabs.org X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: crosa@redhat.com, jarichte@redhat.com, David Gibson X-Mailman-Version: 3.3.8 Precedence: list List-Id: Development discussion and patches for passt Archived-At: Archived-At: List-Archive: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: Add a library "tasst" for use in avocado tests of passt & pasta. We start by adding the outline of logic to run commands in various places (e.g. namespaces, VMs). We add some avocado tests for the test library itself, tagged 'meta' to distinguish it from tests for passt/pasta proper. Signed-off-by: David Gibson --- test/Makefile | 8 +++- test/tasst/exesite.py | 99 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 105 insertions(+), 2 deletions(-) create mode 100644 test/tasst/exesite.py diff --git a/test/Makefile b/test/Makefile index c06d63b6..0e641024 100644 --- a/test/Makefile +++ b/test/Makefile @@ -243,10 +243,14 @@ flake8: $(VENV) # # C0116 missing-function-docstring # We have a bunch of trivial functions for which docstrings would be overkill -# +# C0103 invalid-name +# As well as complaining about bad formatting style, this complains +# about short identifiers (< 3 characters?). For many locals and +# some methods, this is stupid. pylint: $(VENV) $(VENV)/bin/pylint \ - --disable=C0116 \ + --disable=C0116 \ + --disable=C0103 \ $(PYPKGS) .PHONY: check diff --git a/test/tasst/exesite.py b/test/tasst/exesite.py new file mode 100644 index 00000000..9f037f77 --- /dev/null +++ b/test/tasst/exesite.py @@ -0,0 +1,99 @@ +#! /usr/bin/env avocado-runner-avocado-classless + +# SPDX-License-Identifier: GPL-2.0-or-later +# +# Copyright Red Hat +# Author: David Gibson + +""" +Test A Simple Socket Transport + +tasst/exesite.py - Manage simulated network sites for testing +""" + + +import contextlib + +import avocado +from avocado.utils.process import CmdError +from avocado_classless.test import assert_eq, assert_raises, test_output + + +class Site(contextlib.AbstractContextManager): + """ + A (usually virtual or simulated) location where we can execute + commands and configure networks. + + """ + + def __init__(self, name): + self.name = name # For debugging + + def hostify(self, cmd, **kwargs): + raise NotImplementedError + + def __enter__(self): + return self + + def __exit__(self, *exc_details): + pass + + def output(self, cmd, strip_trail_nl=False, **kwargs): + kwargs['strip_trail_nl'] = strip_trail_nl + cmd, kwargs = self.hostify(cmd, **kwargs) + return avocado.utils.process.system_output(cmd, **kwargs) + + def fg(self, cmd, **kwargs): + cmd, kwargs = self.hostify(cmd, **kwargs) + return avocado.utils.process.system(cmd, **kwargs) + + def require_cmds(self, *cmds): + missing = [c for c in cmds + if self.fg(f'type {c}', ignore_status=True) != 0] + if missing: + raise avocado.TestCancel( + f"Missing commands {', '.join(missing)} on {self.name}") + + +def test_site(sitefn): + def test_true(s): + with s as site: + site.fg('true') + + def test_false(s): + with s as site: + assert_raises(CmdError, site.fg, 'false') + + def test_echo(s): + with s as site: + msg = 'Hello tasst' + out = site.output(f'echo {msg}') + assert_eq(out, msg.encode('utf-8') + b'\n') + + def test_timeout(s): + with s as site: + site.fg('sleep infinity', timeout=0.1, ignore_status=True) + + return test_output(test_true, test_false, test_echo, test_timeout)(sitefn) + + +class RealHost(Site): + """Represents the host on which the tests are running (as opposed + to some simulated host created by the tests) + + """ + + def __init__(self): + super().__init__('REAL_HOST') + + def hostify(self, cmd, *, sudo=False, **kwargs): + assert not sudo, "BUG: Shouldn't run commands with privilege on host" + return cmd, kwargs + + +REAL_HOST = RealHost() + + +@test_site +def real_host(): + return REAL_HOST -- 2.41.0