From 15f645e6faa2b74f297262ae809fb9a18d730af6 Mon Sep 17 00:00:00 2001 From: Christian Kellner Date: Wed, 16 Sep 2020 18:20:15 +0200 Subject: [PATCH] test/builder: more ssl_cert config check Check we can handle a single cert properly, i.e. pass it as a plain string to requests. Also check that if three components are specified, an ValueError is thrown. --- test/unit/test_builder.py | 29 +++++++++++++++++++++++++++++ 1 file changed, 29 insertions(+) diff --git a/test/unit/test_builder.py b/test/unit/test_builder.py index a5ae045..b36e784 100644 --- a/test/unit/test_builder.py +++ b/test/unit/test_builder.py @@ -205,6 +205,35 @@ class TestBuilderPlugin(PluginTest): session = handler.client.http self.assertEqual(session.verify, ssl_verify) + # check we can handle a plain ssl_cert string + ssl_cert = "/a/path/to/a/cert" + cfg["composer"]["ssl_cert"] = ssl_cert + cfgfile = os.path.abspath(os.path.join(tmp, "ko.cfg")) + with open(cfgfile, 'w') as f: + cfg.write(f) + + handler = self.plugin.OSBuildImage(1, + "osbuildImage", + "params", + session, + options) + session = handler.client.http + self.assertEqual(session.cert, ssl_cert) + + # check we handle detect wrong cert configs, i.e. + # three certificate compoments + cfg["composer"]["ssl_cert"] = "1, 2, 3" + cfgfile = os.path.abspath(os.path.join(tmp, "ko.cfg")) + with open(cfgfile, 'w') as f: + cfg.write(f) + + with self.assertRaises(ValueError): + handler = self.plugin.OSBuildImage(1, + "osbuildImage", + "params", + session, + options) + def test_unknown_build_target(self): session = flexmock()