From f44691ee6315f82342ed165405b5d26a13cce044 Mon Sep 17 00:00:00 2001 From: Ben Nemec Date: Thu, 27 Mar 2014 21:30:27 +0000 Subject: [PATCH] Add test case for config generator hostname check When I fixed the referenced bug I failed to add a unit test to cover the error case, so it's been up to the reviewers to make sure it doesn't regress. Let's give the reviewers a break and add a proper test case. Change-Id: I46c0f082acd9da6e84c7ac4d4cba387ec97c23c8 Related-Bug: #1205411 --- tests/unit/config/test_generator.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/tests/unit/config/test_generator.py b/tests/unit/config/test_generator.py index 290fc6d2e..845c6947c 100644 --- a/tests/unit/config/test_generator.py +++ b/tests/unit/config/test_generator.py @@ -104,6 +104,13 @@ class GeneratorTestcase(test.BaseTestCase): result = generator._sanitize_default('host', fake_fqdn) self.assertEqual('oslo', result) + def test_sanitize_value_matches_hostname(self): + fake_hostname = 'foo' + self.useFixture(fixtures.MonkeyPatch('socket.gethostname', + lambda: fake_hostname)) + result = generator._sanitize_default('something', fake_hostname) + self.assertEqual(fake_hostname, result) + def test_bad_lib(self): self.assertRaises(ImportError, generator.generate, ['-l', 'broken_opts_lib'])