From e39d2aec22da2116a9a4bbc74a58420d8d54ca7f Mon Sep 17 00:00:00 2001 From: "Brian C. Lane" Date: Fri, 8 Mar 2019 10:09:42 -0800 Subject: [PATCH] tests: Make it easier to update version globs Use constants so we won't have to edit a dozen places in the test when the package versions are bumped. Also switch to using Fedora 31 GPG key now that it has branched for Fedora 30. --- .../pylorax/blueprints/example-glusterfs.toml | 4 +- tests/pylorax/test_projects.py | 2 +- tests/pylorax/test_server.py | 125 ++++++++++-------- 3 files changed, 73 insertions(+), 58 deletions(-) diff --git a/tests/pylorax/blueprints/example-glusterfs.toml b/tests/pylorax/blueprints/example-glusterfs.toml index 36d8919e..54d0d383 100644 --- a/tests/pylorax/blueprints/example-glusterfs.toml +++ b/tests/pylorax/blueprints/example-glusterfs.toml @@ -3,11 +3,11 @@ description = "An example GlusterFS server with samba" [[modules]] name = "glusterfs" -version = "5.*" +version = "6.*" [[modules]] name = "glusterfs-cli" -version = "5.*" +version = "6.*" [[packages]] name = "samba" diff --git a/tests/pylorax/test_projects.py b/tests/pylorax/test_projects.py index d6b0a9aa..a372f788 100644 --- a/tests/pylorax/test_projects.py +++ b/tests/pylorax/test_projects.py @@ -291,7 +291,7 @@ def singlerepo(): "check_gpg": True, "check_ssl": True, "gpgkey_urls": [ - "file:///etc/pki/rpm-gpg/RPM-GPG-KEY-fedora-30-x86_64" + "file:///etc/pki/rpm-gpg/RPM-GPG-KEY-fedora-31-x86_64" ], "name": "single-repo", "system": False, diff --git a/tests/pylorax/test_server.py b/tests/pylorax/test_server.py index 79f6a15a..60c49b09 100644 --- a/tests/pylorax/test_server.py +++ b/tests/pylorax/test_server.py @@ -39,6 +39,21 @@ from pylorax.sysutils import joinpaths # Used for testing UTF-8 input support UTF8_TEST_STRING = "I ο½—π’Šll πŸ‰ΞΏπ˜ π› ο½π”°κœ± π˜π’‰πΈπšœ" + +# HELPER CONSTANTS +HTTP_GLOB = {"name":"httpd", "version":"2.4.*"} +OPENID_GLOB = {"name":"mod_auth_openid", "version":"0.8"} +MODSSL_GLOB = {"name":"mod_ssl", "version":"2.4.*"} +PHP_GLOB = {"name":"php", "version":"7.*"} +PHPMYSQL_GLOB = {"name": "php-mysqlnd", "version":"7.*"} +OPENSSH_GLOB = {"name":"openssh-server", "version": "7.*"} +RSYNC_GLOB = {"name": "rsync", "version": "3.1.*"} +SAMBA_GLOB = {"name": "samba", "version": "4.*.*"} +TMUX_GLOB = {"name": "tmux", "version": "2.8"} +GLUSTERFS_GLOB = {"name": "glusterfs", "version": "6.*"} +GLUSTERFSCLI_GLOB = {"name": "glusterfs-cli", "version": "6.*"} + + def get_system_repo(): """Get an enabled system repo from /etc/yum.repos.d/*repo @@ -160,15 +175,15 @@ class ServerTestCase(unittest.TestCase): info_dict_1 = {"changes":[{"changed":False, "name":"example-http-server"}], "errors":[], "blueprints":[{"description":"An example http server with PHP and MySQL support.", - "modules":[{"name":"httpd", "version":"2.4.*"}, - {"name":"mod_auth_openid", "version":"0.8"}, - {"name":"mod_ssl", "version":"2.4.*"}, - {"name":"php", "version":"7.*"}, - {"name": "php-mysqlnd", "version":"7.*"}], + "modules":[HTTP_GLOB, + OPENID_GLOB, + MODSSL_GLOB, + PHP_GLOB, + PHPMYSQL_GLOB], "name":"example-http-server", - "packages": [{"name":"openssh-server", "version": "7.*"}, - {"name": "rsync", "version": "3.1.*"}, - {"name": "tmux", "version": "2.8"}], + "packages": [OPENSSH_GLOB, + RSYNC_GLOB, + TMUX_GLOB], "groups": [], "version": "0.0.1"}]} resp = self.server.get("/api/v0/blueprints/info/example-http-server") @@ -181,22 +196,22 @@ class ServerTestCase(unittest.TestCase): {"changed":False, "name":"example-http-server"}], "errors":[], "blueprints":[{"description": "An example GlusterFS server with samba", - "modules":[{"name":"glusterfs", "version":"5.*"}, - {"name":"glusterfs-cli", "version":"5.*"}], + "modules":[GLUSTERFS_GLOB, + GLUSTERFSCLI_GLOB], "name":"example-glusterfs", - "packages":[{"name":"samba", "version":"4.*.*"}], + "packages":[SAMBA_GLOB], "groups": [], "version": "0.0.1"}, {"description":"An example http server with PHP and MySQL support.", - "modules":[{"name":"httpd", "version":"2.4.*"}, - {"name":"mod_auth_openid", "version":"0.8"}, - {"name":"mod_ssl", "version":"2.4.*"}, - {"name":"php", "version":"7.*"}, - {"name": "php-mysqlnd", "version":"7.*"}], + "modules":[HTTP_GLOB, + OPENID_GLOB, + MODSSL_GLOB, + PHP_GLOB, + PHPMYSQL_GLOB], "name":"example-http-server", - "packages": [{"name":"openssh-server", "version": "7.*"}, - {"name": "rsync", "version": "3.1.*"}, - {"name": "tmux", "version": "2.8"}], + "packages": [OPENSSH_GLOB, + RSYNC_GLOB, + TMUX_GLOB], "groups": [], "version": "0.0.1"}, ]} @@ -245,10 +260,10 @@ class ServerTestCase(unittest.TestCase): test_blueprint = {"description": "An example GlusterFS server with samba", "name":"example-glusterfs", "version": "0.2.0", - "modules":[{"name":"glusterfs", "version":"5.*"}, - {"name":"glusterfs-cli", "version":"5.*"}], - "packages":[{"name":"samba", "version":"4.*.*"}, - {"name":"tmux", "version":"2.8"}], + "modules":[GLUSTERFS_GLOB, + GLUSTERFSCLI_GLOB], + "packages":[SAMBA_GLOB, + TMUX_GLOB], "groups": []} resp = self.server.post("/api/v0/blueprints/new", @@ -296,10 +311,10 @@ class ServerTestCase(unittest.TestCase): test_blueprint = {"description": "An example GlusterFS server with samba, ws version", "name":"example-glusterfs", "version": "0.3.0", - "modules":[{"name":"glusterfs", "version":"5.*"}, - {"name":"glusterfs-cli", "version":"5.*"}], - "packages":[{"name":"samba", "version":"4.*.*"}, - {"name":"tmux", "version":"2.8"}], + "modules":[GLUSTERFS_GLOB, + GLUSTERFSCLI_GLOB], + "packages":[SAMBA_GLOB, + TMUX_GLOB], "groups": []} resp = self.server.post("/api/v0/blueprints/workspace", @@ -323,10 +338,10 @@ class ServerTestCase(unittest.TestCase): test_blueprint = {"description": "An example GlusterFS server with samba, ws version", "name":"example-glusterfs", "version": "0.4.0", - "modules":[{"name":"glusterfs", "version":"5.*"}, - {"name":"glusterfs-cli", "version":"5.*"}], - "packages":[{"name":"samba", "version":"4.*.*"}, - {"name":"tmux", "version":"2.8"}], + "modules":[GLUSTERFS_GLOB, + GLUSTERFSCLI_GLOB], + "packages":[SAMBA_GLOB, + TMUX_GLOB], "groups": []} resp = self.server.post("/api/v0/blueprints/workspace", @@ -373,10 +388,10 @@ class ServerTestCase(unittest.TestCase): test_blueprint = {"description": "An example GlusterFS server with samba, ws version", "name":"example-glusterfs", "version": "1.4.0", - "modules":[{"name":"glusterfs", "version":"5.*"}, - {"name":"glusterfs-cli", "version":"5.*"}], - "packages":[{"name":"samba", "version":"4.*.*"}, - {"name":"tmux", "version":"2.8"}], + "modules":[GLUSTERFS_GLOB, + GLUSTERFSCLI_GLOB], + "packages":[SAMBA_GLOB, + TMUX_GLOB], "groups": []} resp = self.server.post("/api/v0/blueprints/workspace", data=json.dumps(test_blueprint), @@ -492,10 +507,10 @@ class ServerTestCase(unittest.TestCase): test_blueprint = {"description": "An example GlusterFS server with samba, ws version", "name":"example-glusterfs", "version": "0.3.0", - "modules":[{"name":"glusterfs", "version":"5.*"}, - {"name":"glusterfs-cli", "version":"5.*"}], - "packages":[{"name":"samba", "version":"4.*.*"}, - {"name":"tmux", "version":"2.8"}]} + "modules":[GLUSTERFS_GLOB, + GLUSTERFSCLI_GLOB], + "packages":[SAMBA_GLOB, + TMUX_GLOB]} resp = self.server.post("/api/v0/blueprints/workspace", data=json.dumps(test_blueprint), @@ -774,10 +789,10 @@ class ServerTestCase(unittest.TestCase): test_blueprint = {"description": "An example GlusterFS server with samba", "name":"example-glusterfs", "version": "0.2.0", - "modules":[{"name":"glusterfs", "version":"5.*"}, - {"name":"glusterfs-cli", "version":"5.*"}], - "packages":[{"name":"samba", "version":"4.*.*"}, - {"name":"tmux", "version":"2.8"}], + "modules":[GLUSTERFS_GLOB, + GLUSTERFSCLI_GLOB], + "packages":[SAMBA_GLOB, + TMUX_GLOB], "groups": []} resp = self.server.post("/api/v0/blueprints/new?branch=test", @@ -1238,10 +1253,10 @@ class ServerTestCase(unittest.TestCase): test_blueprint = {"description": "An example GlusterFS server with samba", "name":UTF8_TEST_STRING, "version": "0.2.0", - "modules":[{"name":"glusterfs", "version":"3.*.*"}, - {"name":"glusterfs-cli", "version":"3.*.*"}], - "packages":[{"name":"samba", "version":"4.*.*"}, - {"name":"tmux", "version":"1.8"}], + "modules":[GLUSTERFS_GLOB, + GLUSTERFSCLI_GLOB], + "packages":[SAMBA_GLOB, + TMUX_GLOB], "groups": []} resp = self.server.post("/api/v0/blueprints/new", @@ -1268,10 +1283,10 @@ class ServerTestCase(unittest.TestCase): test_blueprint = {"description": "An example GlusterFS server with samba, ws version", "name":UTF8_TEST_STRING, "version": "0.3.0", - "modules":[{"name":"glusterfs", "version":"3.*.*"}, - {"name":"glusterfs-cli", "version":"3.*.*"}], - "packages":[{"name":"samba", "version":"4.*.*"}, - {"name":"tmux", "version":"1.8"}], + "modules":[GLUSTERFS_GLOB, + GLUSTERFSCLI_GLOB], + "packages":[SAMBA_GLOB, + TMUX_GLOB], "groups": []} resp = self.server.post("/api/v0/blueprints/workspace", @@ -1420,10 +1435,10 @@ class ServerTestCase(unittest.TestCase): test_blueprint = {"description": "A blueprint that has been deleted", "name":"deleted-blueprint", "version": "0.0.1", - "modules":[{"name":"glusterfs", "version":"5.*"}, - {"name":"glusterfs-cli", "version":"5.*"}], - "packages":[{"name":"samba", "version":"4.*.*"}, - {"name":"tmux", "version":"2.8"}], + "modules":[GLUSTERFS_GLOB, + GLUSTERFSCLI_GLOB], + "packages":[SAMBA_GLOB, + TMUX_GLOB], "groups": []} resp = self.server.post("/api/v0/blueprints/new",