X-Git-Url: https://code.grnet.gr/git/ganeti-local/blobdiff_plain/45cc491328e32ab77eeabbc628693470f975b931..43c16a8a1adfd543751fcaf60ad4c8e04cf83688:/test/ganeti.utils_unittest.py diff --git a/test/ganeti.utils_unittest.py b/test/ganeti.utils_unittest.py index bad5a9a..80e737c 100755 --- a/test/ganeti.utils_unittest.py +++ b/test/ganeti.utils_unittest.py @@ -1,7 +1,7 @@ #!/usr/bin/python # -# Copyright (C) 2006, 2007 Google Inc. +# Copyright (C) 2006, 2007, 2010, 2011 Google Inc. # # This program is free software; you can redistribute it and/or modify # it under the terms of the GNU General Public License as published by @@ -21,1043 +21,81 @@ """Script for unittesting the utils module""" -import unittest +import errno +import fcntl +import glob import os -import time -import tempfile import os.path -import os -import stat +import re +import shutil import signal import socket -import shutil -import re -import select -import string -import OpenSSL +import stat +import tempfile +import time +import unittest import warnings -import distutils.version -import glob +import random +import operator -import ganeti import testutils from ganeti import constants +from ganeti import compat from ganeti import utils from ganeti import errors -from ganeti import serializer -from ganeti.utils import IsProcessAlive, RunCmd, \ - RemoveFile, MatchNameComponent, FormatUnit, \ - ParseUnit, AddAuthorizedKey, RemoveAuthorizedKey, \ - ShellQuote, ShellQuoteArgs, TcpPing, ListVisibleFiles, \ - SetEtcHostsEntry, RemoveEtcHostsEntry, FirstFree, OwnIpAddress, \ - TailFile, ForceDictType, SafeEncode, IsNormAbsPath, FormatTime, \ - UnescapeAndSplit, RunParts, PathJoin, HostInfo - -from ganeti.errors import LockError, UnitParseError, GenericError, \ - ProgrammerError, OpPrereqError - - -class TestIsProcessAlive(unittest.TestCase): - """Testing case for IsProcessAlive""" - - def testExists(self): - mypid = os.getpid() - self.assert_(IsProcessAlive(mypid), - "can't find myself running") - - def testNotExisting(self): - pid_non_existing = os.fork() - if pid_non_existing == 0: - os._exit(0) - elif pid_non_existing < 0: - raise SystemError("can't fork") - os.waitpid(pid_non_existing, 0) - self.assert_(not IsProcessAlive(pid_non_existing), - "nonexisting process detected") - - -class TestPidFileFunctions(unittest.TestCase): - """Tests for WritePidFile, RemovePidFile and ReadPidFile""" - - def setUp(self): - self.dir = tempfile.mkdtemp() - self.f_dpn = lambda name: os.path.join(self.dir, "%s.pid" % name) - utils.DaemonPidFileName = self.f_dpn - - def testPidFileFunctions(self): - pid_file = self.f_dpn('test') - utils.WritePidFile('test') - self.failUnless(os.path.exists(pid_file), - "PID file should have been created") - read_pid = utils.ReadPidFile(pid_file) - self.failUnlessEqual(read_pid, os.getpid()) - self.failUnless(utils.IsProcessAlive(read_pid)) - self.failUnlessRaises(GenericError, utils.WritePidFile, 'test') - utils.RemovePidFile('test') - self.failIf(os.path.exists(pid_file), - "PID file should not exist anymore") - self.failUnlessEqual(utils.ReadPidFile(pid_file), 0, - "ReadPidFile should return 0 for missing pid file") - fh = open(pid_file, "w") - fh.write("blah\n") - fh.close() - self.failUnlessEqual(utils.ReadPidFile(pid_file), 0, - "ReadPidFile should return 0 for invalid pid file") - utils.RemovePidFile('test') - self.failIf(os.path.exists(pid_file), - "PID file should not exist anymore") - - def testKill(self): - pid_file = self.f_dpn('child') - r_fd, w_fd = os.pipe() - new_pid = os.fork() - if new_pid == 0: #child - utils.WritePidFile('child') - os.write(w_fd, 'a') - signal.pause() - os._exit(0) - return - # else we are in the parent - # wait until the child has written the pid file - os.read(r_fd, 1) - read_pid = utils.ReadPidFile(pid_file) - self.failUnlessEqual(read_pid, new_pid) - self.failUnless(utils.IsProcessAlive(new_pid)) - utils.KillProcess(new_pid, waitpid=True) - self.failIf(utils.IsProcessAlive(new_pid)) - utils.RemovePidFile('child') - self.failUnlessRaises(ProgrammerError, utils.KillProcess, 0) - - def tearDown(self): - for name in os.listdir(self.dir): - os.unlink(os.path.join(self.dir, name)) - os.rmdir(self.dir) - - -class TestRunCmd(testutils.GanetiTestCase): - """Testing case for the RunCmd function""" - - def setUp(self): - testutils.GanetiTestCase.setUp(self) - self.magic = time.ctime() + " ganeti test" - self.fname = self._CreateTempFile() - - def testOk(self): - """Test successful exit code""" - result = RunCmd("/bin/sh -c 'exit 0'") - self.assertEqual(result.exit_code, 0) - self.assertEqual(result.output, "") - - def testFail(self): - """Test fail exit code""" - result = RunCmd("/bin/sh -c 'exit 1'") - self.assertEqual(result.exit_code, 1) - self.assertEqual(result.output, "") - - def testStdout(self): - """Test standard output""" - cmd = 'echo -n "%s"' % self.magic - result = RunCmd("/bin/sh -c '%s'" % cmd) - self.assertEqual(result.stdout, self.magic) - result = RunCmd("/bin/sh -c '%s'" % cmd, output=self.fname) - self.assertEqual(result.output, "") - self.assertFileContent(self.fname, self.magic) - - def testStderr(self): - """Test standard error""" - cmd = 'echo -n "%s"' % self.magic - result = RunCmd("/bin/sh -c '%s' 1>&2" % cmd) - self.assertEqual(result.stderr, self.magic) - result = RunCmd("/bin/sh -c '%s' 1>&2" % cmd, output=self.fname) - self.assertEqual(result.output, "") - self.assertFileContent(self.fname, self.magic) - - def testCombined(self): - """Test combined output""" - cmd = 'echo -n "A%s"; echo -n "B%s" 1>&2' % (self.magic, self.magic) - expected = "A" + self.magic + "B" + self.magic - result = RunCmd("/bin/sh -c '%s'" % cmd) - self.assertEqual(result.output, expected) - result = RunCmd("/bin/sh -c '%s'" % cmd, output=self.fname) - self.assertEqual(result.output, "") - self.assertFileContent(self.fname, expected) - - def testSignal(self): - """Test signal""" - result = RunCmd(["python", "-c", "import os; os.kill(os.getpid(), 15)"]) - self.assertEqual(result.signal, 15) - self.assertEqual(result.output, "") - - def testListRun(self): - """Test list runs""" - result = RunCmd(["true"]) - self.assertEqual(result.signal, None) - self.assertEqual(result.exit_code, 0) - result = RunCmd(["/bin/sh", "-c", "exit 1"]) - self.assertEqual(result.signal, None) - self.assertEqual(result.exit_code, 1) - result = RunCmd(["echo", "-n", self.magic]) - self.assertEqual(result.signal, None) - self.assertEqual(result.exit_code, 0) - self.assertEqual(result.stdout, self.magic) - - def testFileEmptyOutput(self): - """Test file output""" - result = RunCmd(["true"], output=self.fname) - self.assertEqual(result.signal, None) - self.assertEqual(result.exit_code, 0) - self.assertFileContent(self.fname, "") - - def testLang(self): - """Test locale environment""" - old_env = os.environ.copy() - try: - os.environ["LANG"] = "en_US.UTF-8" - os.environ["LC_ALL"] = "en_US.UTF-8" - result = RunCmd(["locale"]) - for line in result.output.splitlines(): - key, value = line.split("=", 1) - # Ignore these variables, they're overridden by LC_ALL - if key == "LANG" or key == "LANGUAGE": - continue - self.failIf(value and value != "C" and value != '"C"', - "Variable %s is set to the invalid value '%s'" % (key, value)) - finally: - os.environ = old_env - - def testDefaultCwd(self): - """Test default working directory""" - self.failUnlessEqual(RunCmd(["pwd"]).stdout.strip(), "/") - - def testCwd(self): - """Test default working directory""" - self.failUnlessEqual(RunCmd(["pwd"], cwd="/").stdout.strip(), "/") - self.failUnlessEqual(RunCmd(["pwd"], cwd="/tmp").stdout.strip(), "/tmp") - cwd = os.getcwd() - self.failUnlessEqual(RunCmd(["pwd"], cwd=cwd).stdout.strip(), cwd) - - def testResetEnv(self): - """Test environment reset functionality""" - self.failUnlessEqual(RunCmd(["env"], reset_env=True).stdout.strip(), "") - self.failUnlessEqual(RunCmd(["env"], reset_env=True, - env={"FOO": "bar",}).stdout.strip(), "FOO=bar") - - -class TestRunParts(unittest.TestCase): - """Testing case for the RunParts function""" - - def setUp(self): - self.rundir = tempfile.mkdtemp(prefix="ganeti-test", suffix=".tmp") - - def tearDown(self): - shutil.rmtree(self.rundir) - - def testEmpty(self): - """Test on an empty dir""" - self.failUnlessEqual(RunParts(self.rundir, reset_env=True), []) - - def testSkipWrongName(self): - """Test that wrong files are skipped""" - fname = os.path.join(self.rundir, "00test.dot") - utils.WriteFile(fname, data="") - os.chmod(fname, stat.S_IREAD | stat.S_IEXEC) - relname = os.path.basename(fname) - self.failUnlessEqual(RunParts(self.rundir, reset_env=True), - [(relname, constants.RUNPARTS_SKIP, None)]) - - def testSkipNonExec(self): - """Test that non executable files are skipped""" - fname = os.path.join(self.rundir, "00test") - utils.WriteFile(fname, data="") - relname = os.path.basename(fname) - self.failUnlessEqual(RunParts(self.rundir, reset_env=True), - [(relname, constants.RUNPARTS_SKIP, None)]) - - def testError(self): - """Test error on a broken executable""" - fname = os.path.join(self.rundir, "00test") - utils.WriteFile(fname, data="") - os.chmod(fname, stat.S_IREAD | stat.S_IEXEC) - (relname, status, error) = RunParts(self.rundir, reset_env=True)[0] - self.failUnlessEqual(relname, os.path.basename(fname)) - self.failUnlessEqual(status, constants.RUNPARTS_ERR) - self.failUnless(error) - - def testSorted(self): - """Test executions are sorted""" - files = [] - files.append(os.path.join(self.rundir, "64test")) - files.append(os.path.join(self.rundir, "00test")) - files.append(os.path.join(self.rundir, "42test")) - - for fname in files: - utils.WriteFile(fname, data="") - - results = RunParts(self.rundir, reset_env=True) - - for fname in sorted(files): - self.failUnlessEqual(os.path.basename(fname), results.pop(0)[0]) - - def testOk(self): - """Test correct execution""" - fname = os.path.join(self.rundir, "00test") - utils.WriteFile(fname, data="#!/bin/sh\n\necho -n ciao") - os.chmod(fname, stat.S_IREAD | stat.S_IEXEC) - (relname, status, runresult) = RunParts(self.rundir, reset_env=True)[0] - self.failUnlessEqual(relname, os.path.basename(fname)) - self.failUnlessEqual(status, constants.RUNPARTS_RUN) - self.failUnlessEqual(runresult.stdout, "ciao") - - def testRunFail(self): - """Test correct execution, with run failure""" - fname = os.path.join(self.rundir, "00test") - utils.WriteFile(fname, data="#!/bin/sh\n\nexit 1") - os.chmod(fname, stat.S_IREAD | stat.S_IEXEC) - (relname, status, runresult) = RunParts(self.rundir, reset_env=True)[0] - self.failUnlessEqual(relname, os.path.basename(fname)) - self.failUnlessEqual(status, constants.RUNPARTS_RUN) - self.failUnlessEqual(runresult.exit_code, 1) - self.failUnless(runresult.failed) - - def testRunMix(self): - files = [] - files.append(os.path.join(self.rundir, "00test")) - files.append(os.path.join(self.rundir, "42test")) - files.append(os.path.join(self.rundir, "64test")) - files.append(os.path.join(self.rundir, "99test")) - - files.sort() - - # 1st has errors in execution - utils.WriteFile(files[0], data="#!/bin/sh\n\nexit 1") - os.chmod(files[0], stat.S_IREAD | stat.S_IEXEC) - - # 2nd is skipped - utils.WriteFile(files[1], data="") - - # 3rd cannot execute properly - utils.WriteFile(files[2], data="") - os.chmod(files[2], stat.S_IREAD | stat.S_IEXEC) - - # 4th execs - utils.WriteFile(files[3], data="#!/bin/sh\n\necho -n ciao") - os.chmod(files[3], stat.S_IREAD | stat.S_IEXEC) - - results = RunParts(self.rundir, reset_env=True) - - (relname, status, runresult) = results[0] - self.failUnlessEqual(relname, os.path.basename(files[0])) - self.failUnlessEqual(status, constants.RUNPARTS_RUN) - self.failUnlessEqual(runresult.exit_code, 1) - self.failUnless(runresult.failed) - - (relname, status, runresult) = results[1] - self.failUnlessEqual(relname, os.path.basename(files[1])) - self.failUnlessEqual(status, constants.RUNPARTS_SKIP) - self.failUnlessEqual(runresult, None) - - (relname, status, runresult) = results[2] - self.failUnlessEqual(relname, os.path.basename(files[2])) - self.failUnlessEqual(status, constants.RUNPARTS_ERR) - self.failUnless(runresult) - - (relname, status, runresult) = results[3] - self.failUnlessEqual(relname, os.path.basename(files[3])) - self.failUnlessEqual(status, constants.RUNPARTS_RUN) - self.failUnlessEqual(runresult.output, "ciao") - self.failUnlessEqual(runresult.exit_code, 0) - self.failUnless(not runresult.failed) - - -class TestRemoveFile(unittest.TestCase): - """Test case for the RemoveFile function""" - - def setUp(self): - """Create a temp dir and file for each case""" - self.tmpdir = tempfile.mkdtemp('', 'ganeti-unittest-') - fd, self.tmpfile = tempfile.mkstemp('', '', self.tmpdir) - os.close(fd) - - def tearDown(self): - if os.path.exists(self.tmpfile): - os.unlink(self.tmpfile) - os.rmdir(self.tmpdir) - - - def testIgnoreDirs(self): - """Test that RemoveFile() ignores directories""" - self.assertEqual(None, RemoveFile(self.tmpdir)) - - - def testIgnoreNotExisting(self): - """Test that RemoveFile() ignores non-existing files""" - RemoveFile(self.tmpfile) - RemoveFile(self.tmpfile) - - - def testRemoveFile(self): - """Test that RemoveFile does remove a file""" - RemoveFile(self.tmpfile) - if os.path.exists(self.tmpfile): - self.fail("File '%s' not removed" % self.tmpfile) - - - def testRemoveSymlink(self): - """Test that RemoveFile does remove symlinks""" - symlink = self.tmpdir + "/symlink" - os.symlink("no-such-file", symlink) - RemoveFile(symlink) - if os.path.exists(symlink): - self.fail("File '%s' not removed" % symlink) - os.symlink(self.tmpfile, symlink) - RemoveFile(symlink) - if os.path.exists(symlink): - self.fail("File '%s' not removed" % symlink) - - -class TestRename(unittest.TestCase): - """Test case for RenameFile""" - - def setUp(self): - """Create a temporary directory""" - self.tmpdir = tempfile.mkdtemp() - self.tmpfile = os.path.join(self.tmpdir, "test1") - - # Touch the file - open(self.tmpfile, "w").close() +from ganeti.utils import RunCmd, \ + FirstFree, \ + RunParts - def tearDown(self): - """Remove temporary directory""" - shutil.rmtree(self.tmpdir) - - def testSimpleRename1(self): - """Simple rename 1""" - utils.RenameFile(self.tmpfile, os.path.join(self.tmpdir, "xyz")) - self.assert_(os.path.isfile(os.path.join(self.tmpdir, "xyz"))) - - def testSimpleRename2(self): - """Simple rename 2""" - utils.RenameFile(self.tmpfile, os.path.join(self.tmpdir, "xyz"), - mkdir=True) - self.assert_(os.path.isfile(os.path.join(self.tmpdir, "xyz"))) - - def testRenameMkdir(self): - """Rename with mkdir""" - utils.RenameFile(self.tmpfile, os.path.join(self.tmpdir, "test/xyz"), - mkdir=True) - self.assert_(os.path.isdir(os.path.join(self.tmpdir, "test"))) - self.assert_(os.path.isfile(os.path.join(self.tmpdir, "test/xyz"))) - - utils.RenameFile(os.path.join(self.tmpdir, "test/xyz"), - os.path.join(self.tmpdir, "test/foo/bar/baz"), - mkdir=True) - self.assert_(os.path.isdir(os.path.join(self.tmpdir, "test"))) - self.assert_(os.path.isdir(os.path.join(self.tmpdir, "test/foo/bar"))) - self.assert_(os.path.isfile(os.path.join(self.tmpdir, "test/foo/bar/baz"))) - - -class TestMatchNameComponent(unittest.TestCase): - """Test case for the MatchNameComponent function""" - - def testEmptyList(self): - """Test that there is no match against an empty list""" - - self.failUnlessEqual(MatchNameComponent("", []), None) - self.failUnlessEqual(MatchNameComponent("test", []), None) - - def testSingleMatch(self): - """Test that a single match is performed correctly""" - mlist = ["test1.example.com", "test2.example.com", "test3.example.com"] - for key in "test2", "test2.example", "test2.example.com": - self.failUnlessEqual(MatchNameComponent(key, mlist), mlist[1]) - - def testMultipleMatches(self): - """Test that a multiple match is returned as None""" - mlist = ["test1.example.com", "test1.example.org", "test1.example.net"] - for key in "test1", "test1.example": - self.failUnlessEqual(MatchNameComponent(key, mlist), None) - - def testFullMatch(self): - """Test that a full match is returned correctly""" - key1 = "test1" - key2 = "test1.example" - mlist = [key2, key2 + ".com"] - self.failUnlessEqual(MatchNameComponent(key1, mlist), None) - self.failUnlessEqual(MatchNameComponent(key2, mlist), key2) - - def testCaseInsensitivePartialMatch(self): - """Test for the case_insensitive keyword""" - mlist = ["test1.example.com", "test2.example.net"] - self.assertEqual(MatchNameComponent("test2", mlist, case_sensitive=False), - "test2.example.net") - self.assertEqual(MatchNameComponent("Test2", mlist, case_sensitive=False), - "test2.example.net") - self.assertEqual(MatchNameComponent("teSt2", mlist, case_sensitive=False), - "test2.example.net") - self.assertEqual(MatchNameComponent("TeSt2", mlist, case_sensitive=False), - "test2.example.net") - - - def testCaseInsensitiveFullMatch(self): - mlist = ["ts1.ex", "ts1.ex.org", "ts2.ex", "Ts2.ex"] - # Between the two ts1 a full string match non-case insensitive should work - self.assertEqual(MatchNameComponent("Ts1", mlist, case_sensitive=False), - None) - self.assertEqual(MatchNameComponent("Ts1.ex", mlist, case_sensitive=False), - "ts1.ex") - self.assertEqual(MatchNameComponent("ts1.ex", mlist, case_sensitive=False), - "ts1.ex") - # Between the two ts2 only case differs, so only case-match works - self.assertEqual(MatchNameComponent("ts2.ex", mlist, case_sensitive=False), - "ts2.ex") - self.assertEqual(MatchNameComponent("Ts2.ex", mlist, case_sensitive=False), - "Ts2.ex") - self.assertEqual(MatchNameComponent("TS2.ex", mlist, case_sensitive=False), - None) - - -class TestTimestampForFilename(unittest.TestCase): - def test(self): - self.assert_("." not in utils.TimestampForFilename()) - self.assert_(":" not in utils.TimestampForFilename()) - - -class TestCreateBackup(testutils.GanetiTestCase): - def setUp(self): - testutils.GanetiTestCase.setUp(self) - - self.tmpdir = tempfile.mkdtemp() - - def tearDown(self): - testutils.GanetiTestCase.tearDown(self) - - shutil.rmtree(self.tmpdir) - - def testEmpty(self): - filename = utils.PathJoin(self.tmpdir, "config.data") - utils.WriteFile(filename, data="") - bname = utils.CreateBackup(filename) - self.assertFileContent(bname, "") - self.assertEqual(len(glob.glob("%s*" % filename)), 2) - utils.CreateBackup(filename) - self.assertEqual(len(glob.glob("%s*" % filename)), 3) - utils.CreateBackup(filename) - self.assertEqual(len(glob.glob("%s*" % filename)), 4) - - fifoname = utils.PathJoin(self.tmpdir, "fifo") - os.mkfifo(fifoname) - self.assertRaises(errors.ProgrammerError, utils.CreateBackup, fifoname) - - def testContent(self): - bkpcount = 0 - for data in ["", "X", "Hello World!\n" * 100, "Binary data\0\x01\x02\n"]: - for rep in [1, 2, 10, 127]: - testdata = data * rep - - filename = utils.PathJoin(self.tmpdir, "test.data_") - utils.WriteFile(filename, data=testdata) - self.assertFileContent(filename, testdata) - - for _ in range(3): - bname = utils.CreateBackup(filename) - bkpcount += 1 - self.assertFileContent(bname, testdata) - self.assertEqual(len(glob.glob("%s*" % filename)), 1 + bkpcount) - - -class TestFormatUnit(unittest.TestCase): - """Test case for the FormatUnit function""" - - def testMiB(self): - self.assertEqual(FormatUnit(1, 'h'), '1M') - self.assertEqual(FormatUnit(100, 'h'), '100M') - self.assertEqual(FormatUnit(1023, 'h'), '1023M') - - self.assertEqual(FormatUnit(1, 'm'), '1') - self.assertEqual(FormatUnit(100, 'm'), '100') - self.assertEqual(FormatUnit(1023, 'm'), '1023') - - self.assertEqual(FormatUnit(1024, 'm'), '1024') - self.assertEqual(FormatUnit(1536, 'm'), '1536') - self.assertEqual(FormatUnit(17133, 'm'), '17133') - self.assertEqual(FormatUnit(1024 * 1024 - 1, 'm'), '1048575') - - def testGiB(self): - self.assertEqual(FormatUnit(1024, 'h'), '1.0G') - self.assertEqual(FormatUnit(1536, 'h'), '1.5G') - self.assertEqual(FormatUnit(17133, 'h'), '16.7G') - self.assertEqual(FormatUnit(1024 * 1024 - 1, 'h'), '1024.0G') - - self.assertEqual(FormatUnit(1024, 'g'), '1.0') - self.assertEqual(FormatUnit(1536, 'g'), '1.5') - self.assertEqual(FormatUnit(17133, 'g'), '16.7') - self.assertEqual(FormatUnit(1024 * 1024 - 1, 'g'), '1024.0') - - self.assertEqual(FormatUnit(1024 * 1024, 'g'), '1024.0') - self.assertEqual(FormatUnit(5120 * 1024, 'g'), '5120.0') - self.assertEqual(FormatUnit(29829 * 1024, 'g'), '29829.0') - - def testTiB(self): - self.assertEqual(FormatUnit(1024 * 1024, 'h'), '1.0T') - self.assertEqual(FormatUnit(5120 * 1024, 'h'), '5.0T') - self.assertEqual(FormatUnit(29829 * 1024, 'h'), '29.1T') - - self.assertEqual(FormatUnit(1024 * 1024, 't'), '1.0') - self.assertEqual(FormatUnit(5120 * 1024, 't'), '5.0') - self.assertEqual(FormatUnit(29829 * 1024, 't'), '29.1') - -class TestParseUnit(unittest.TestCase): - """Test case for the ParseUnit function""" - - SCALES = (('', 1), - ('M', 1), ('G', 1024), ('T', 1024 * 1024), - ('MB', 1), ('GB', 1024), ('TB', 1024 * 1024), - ('MiB', 1), ('GiB', 1024), ('TiB', 1024 * 1024)) - - def testRounding(self): - self.assertEqual(ParseUnit('0'), 0) - self.assertEqual(ParseUnit('1'), 4) - self.assertEqual(ParseUnit('2'), 4) - self.assertEqual(ParseUnit('3'), 4) - - self.assertEqual(ParseUnit('124'), 124) - self.assertEqual(ParseUnit('125'), 128) - self.assertEqual(ParseUnit('126'), 128) - self.assertEqual(ParseUnit('127'), 128) - self.assertEqual(ParseUnit('128'), 128) - self.assertEqual(ParseUnit('129'), 132) - self.assertEqual(ParseUnit('130'), 132) - def testFloating(self): - self.assertEqual(ParseUnit('0'), 0) - self.assertEqual(ParseUnit('0.5'), 4) - self.assertEqual(ParseUnit('1.75'), 4) - self.assertEqual(ParseUnit('1.99'), 4) - self.assertEqual(ParseUnit('2.00'), 4) - self.assertEqual(ParseUnit('2.01'), 4) - self.assertEqual(ParseUnit('3.99'), 4) - self.assertEqual(ParseUnit('4.00'), 4) - self.assertEqual(ParseUnit('4.01'), 8) - self.assertEqual(ParseUnit('1.5G'), 1536) - self.assertEqual(ParseUnit('1.8G'), 1844) - self.assertEqual(ParseUnit('8.28T'), 8682212) +class TestParseCpuMask(unittest.TestCase): + """Test case for the ParseCpuMask function.""" - def testSuffixes(self): - for sep in ('', ' ', ' ', "\t", "\t "): - for suffix, scale in TestParseUnit.SCALES: - for func in (lambda x: x, str.lower, str.upper): - self.assertEqual(ParseUnit('1024' + sep + func(suffix)), - 1024 * scale) + def testWellFormed(self): + self.assertEqual(utils.ParseCpuMask(""), []) + self.assertEqual(utils.ParseCpuMask("1"), [1]) + self.assertEqual(utils.ParseCpuMask("0-2,4,5-5"), [0,1,2,4,5]) def testInvalidInput(self): - for sep in ('-', '_', ',', 'a'): - for suffix, _ in TestParseUnit.SCALES: - self.assertRaises(UnitParseError, ParseUnit, '1' + sep + suffix) - - for suffix, _ in TestParseUnit.SCALES: - self.assertRaises(UnitParseError, ParseUnit, '1,3' + suffix) - - -class TestSshKeys(testutils.GanetiTestCase): - """Test case for the AddAuthorizedKey function""" - - KEY_A = 'ssh-dss AAAAB3NzaC1w5256closdj32mZaQU root@key-a' - KEY_B = ('command="/usr/bin/fooserver -t --verbose",from="1.2.3.4" ' - 'ssh-dss AAAAB3NzaC1w520smc01ms0jfJs22 root@key-b') - - def setUp(self): - testutils.GanetiTestCase.setUp(self) - self.tmpname = self._CreateTempFile() - handle = open(self.tmpname, 'w') - try: - handle.write("%s\n" % TestSshKeys.KEY_A) - handle.write("%s\n" % TestSshKeys.KEY_B) - finally: - handle.close() - - def testAddingNewKey(self): - AddAuthorizedKey(self.tmpname, 'ssh-dss AAAAB3NzaC1kc3MAAACB root@test') - - self.assertFileContent(self.tmpname, - "ssh-dss AAAAB3NzaC1w5256closdj32mZaQU root@key-a\n" - 'command="/usr/bin/fooserver -t --verbose",from="1.2.3.4"' - " ssh-dss AAAAB3NzaC1w520smc01ms0jfJs22 root@key-b\n" - "ssh-dss AAAAB3NzaC1kc3MAAACB root@test\n") - - def testAddingAlmostButNotCompletelyTheSameKey(self): - AddAuthorizedKey(self.tmpname, - 'ssh-dss AAAAB3NzaC1w5256closdj32mZaQU root@test') - - self.assertFileContent(self.tmpname, - "ssh-dss AAAAB3NzaC1w5256closdj32mZaQU root@key-a\n" - 'command="/usr/bin/fooserver -t --verbose",from="1.2.3.4"' - " ssh-dss AAAAB3NzaC1w520smc01ms0jfJs22 root@key-b\n" - "ssh-dss AAAAB3NzaC1w5256closdj32mZaQU root@test\n") - - def testAddingExistingKeyWithSomeMoreSpaces(self): - AddAuthorizedKey(self.tmpname, - 'ssh-dss AAAAB3NzaC1w5256closdj32mZaQU root@key-a') - - self.assertFileContent(self.tmpname, - "ssh-dss AAAAB3NzaC1w5256closdj32mZaQU root@key-a\n" - 'command="/usr/bin/fooserver -t --verbose",from="1.2.3.4"' - " ssh-dss AAAAB3NzaC1w520smc01ms0jfJs22 root@key-b\n") - - def testRemovingExistingKeyWithSomeMoreSpaces(self): - RemoveAuthorizedKey(self.tmpname, - 'ssh-dss AAAAB3NzaC1w5256closdj32mZaQU root@key-a') - - self.assertFileContent(self.tmpname, - 'command="/usr/bin/fooserver -t --verbose",from="1.2.3.4"' - " ssh-dss AAAAB3NzaC1w520smc01ms0jfJs22 root@key-b\n") - - def testRemovingNonExistingKey(self): - RemoveAuthorizedKey(self.tmpname, - 'ssh-dss AAAAB3Nsdfj230xxjxJjsjwjsjdjU root@test') - - self.assertFileContent(self.tmpname, - "ssh-dss AAAAB3NzaC1w5256closdj32mZaQU root@key-a\n" - 'command="/usr/bin/fooserver -t --verbose",from="1.2.3.4"' - " ssh-dss AAAAB3NzaC1w520smc01ms0jfJs22 root@key-b\n") - - -class TestEtcHosts(testutils.GanetiTestCase): - """Test functions modifying /etc/hosts""" - - def setUp(self): - testutils.GanetiTestCase.setUp(self) - self.tmpname = self._CreateTempFile() - handle = open(self.tmpname, 'w') - try: - handle.write('# This is a test file for /etc/hosts\n') - handle.write('127.0.0.1\tlocalhost\n') - handle.write('192.168.1.1 router gw\n') - finally: - handle.close() - - def testSettingNewIp(self): - SetEtcHostsEntry(self.tmpname, '1.2.3.4', 'myhost.domain.tld', ['myhost']) - - self.assertFileContent(self.tmpname, - "# This is a test file for /etc/hosts\n" - "127.0.0.1\tlocalhost\n" - "192.168.1.1 router gw\n" - "1.2.3.4\tmyhost.domain.tld myhost\n") - self.assertFileMode(self.tmpname, 0644) - - def testSettingExistingIp(self): - SetEtcHostsEntry(self.tmpname, '192.168.1.1', 'myhost.domain.tld', - ['myhost']) - - self.assertFileContent(self.tmpname, - "# This is a test file for /etc/hosts\n" - "127.0.0.1\tlocalhost\n" - "192.168.1.1\tmyhost.domain.tld myhost\n") - self.assertFileMode(self.tmpname, 0644) - - def testSettingDuplicateName(self): - SetEtcHostsEntry(self.tmpname, '1.2.3.4', 'myhost', ['myhost']) - - self.assertFileContent(self.tmpname, - "# This is a test file for /etc/hosts\n" - "127.0.0.1\tlocalhost\n" - "192.168.1.1 router gw\n" - "1.2.3.4\tmyhost\n") - self.assertFileMode(self.tmpname, 0644) - - def testRemovingExistingHost(self): - RemoveEtcHostsEntry(self.tmpname, 'router') - - self.assertFileContent(self.tmpname, - "# This is a test file for /etc/hosts\n" - "127.0.0.1\tlocalhost\n" - "192.168.1.1 gw\n") - self.assertFileMode(self.tmpname, 0644) - - def testRemovingSingleExistingHost(self): - RemoveEtcHostsEntry(self.tmpname, 'localhost') - - self.assertFileContent(self.tmpname, - "# This is a test file for /etc/hosts\n" - "192.168.1.1 router gw\n") - self.assertFileMode(self.tmpname, 0644) - - def testRemovingNonExistingHost(self): - RemoveEtcHostsEntry(self.tmpname, 'myhost') - - self.assertFileContent(self.tmpname, - "# This is a test file for /etc/hosts\n" - "127.0.0.1\tlocalhost\n" - "192.168.1.1 router gw\n") - self.assertFileMode(self.tmpname, 0644) - - def testRemovingAlias(self): - RemoveEtcHostsEntry(self.tmpname, 'gw') - - self.assertFileContent(self.tmpname, - "# This is a test file for /etc/hosts\n" - "127.0.0.1\tlocalhost\n" - "192.168.1.1 router\n") - self.assertFileMode(self.tmpname, 0644) - - -class TestShellQuoting(unittest.TestCase): - """Test case for shell quoting functions""" - - def testShellQuote(self): - self.assertEqual(ShellQuote('abc'), "abc") - self.assertEqual(ShellQuote('ab"c'), "'ab\"c'") - self.assertEqual(ShellQuote("a'bc"), "'a'\\''bc'") - self.assertEqual(ShellQuote("a b c"), "'a b c'") - self.assertEqual(ShellQuote("a b\\ c"), "'a b\\ c'") - - def testShellQuoteArgs(self): - self.assertEqual(ShellQuoteArgs(['a', 'b', 'c']), "a b c") - self.assertEqual(ShellQuoteArgs(['a', 'b"', 'c']), "a 'b\"' c") - self.assertEqual(ShellQuoteArgs(['a', 'b\'', 'c']), "a 'b'\\\''' c") - - -class TestTcpPing(unittest.TestCase): - """Testcase for TCP version of ping - against listen(2)ing port""" - - def setUp(self): - self.listener = socket.socket(socket.AF_INET, socket.SOCK_STREAM) - self.listener.bind((constants.LOCALHOST_IP_ADDRESS, 0)) - self.listenerport = self.listener.getsockname()[1] - self.listener.listen(1) - - def tearDown(self): - self.listener.shutdown(socket.SHUT_RDWR) - del self.listener - del self.listenerport - - def testTcpPingToLocalHostAccept(self): - self.assert_(TcpPing(constants.LOCALHOST_IP_ADDRESS, - self.listenerport, - timeout=10, - live_port_needed=True, - source=constants.LOCALHOST_IP_ADDRESS, - ), - "failed to connect to test listener") - - self.assert_(TcpPing(constants.LOCALHOST_IP_ADDRESS, - self.listenerport, - timeout=10, - live_port_needed=True, - ), - "failed to connect to test listener (no source)") - - -class TestTcpPingDeaf(unittest.TestCase): - """Testcase for TCP version of ping - against non listen(2)ing port""" - - def setUp(self): - self.deaflistener = socket.socket(socket.AF_INET, socket.SOCK_STREAM) - self.deaflistener.bind((constants.LOCALHOST_IP_ADDRESS, 0)) - self.deaflistenerport = self.deaflistener.getsockname()[1] - - def tearDown(self): - del self.deaflistener - del self.deaflistenerport - - def testTcpPingToLocalHostAcceptDeaf(self): - self.failIf(TcpPing(constants.LOCALHOST_IP_ADDRESS, - self.deaflistenerport, - timeout=constants.TCP_PING_TIMEOUT, - live_port_needed=True, - source=constants.LOCALHOST_IP_ADDRESS, - ), # need successful connect(2) - "successfully connected to deaf listener") - - self.failIf(TcpPing(constants.LOCALHOST_IP_ADDRESS, - self.deaflistenerport, - timeout=constants.TCP_PING_TIMEOUT, - live_port_needed=True, - ), # need successful connect(2) - "successfully connected to deaf listener (no source addr)") - - def testTcpPingToLocalHostNoAccept(self): - self.assert_(TcpPing(constants.LOCALHOST_IP_ADDRESS, - self.deaflistenerport, - timeout=constants.TCP_PING_TIMEOUT, - live_port_needed=False, - source=constants.LOCALHOST_IP_ADDRESS, - ), # ECONNREFUSED is OK - "failed to ping alive host on deaf port") + for data in ["garbage", "0,", "0-1-2", "2-1", "1-a"]: + self.assertRaises(errors.ParseError, utils.ParseCpuMask, data) - self.assert_(TcpPing(constants.LOCALHOST_IP_ADDRESS, - self.deaflistenerport, - timeout=constants.TCP_PING_TIMEOUT, - live_port_needed=False, - ), # ECONNREFUSED is OK - "failed to ping alive host on deaf port (no source addr)") +class TestParseMultiCpuMask(unittest.TestCase): + """Test case for the ParseMultiCpuMask function.""" -class TestOwnIpAddress(unittest.TestCase): - """Testcase for OwnIpAddress""" + def testWellFormed(self): + self.assertEqual(utils.ParseMultiCpuMask(""), []) + self.assertEqual(utils.ParseMultiCpuMask("1"), [[1]]) + self.assertEqual(utils.ParseMultiCpuMask("0-2,4,5-5"), [[0, 1, 2, 4, 5]]) + self.assertEqual(utils.ParseMultiCpuMask("all"), [[-1]]) + self.assertEqual(utils.ParseMultiCpuMask("0-2:all:4,6-8"), + [[0, 1, 2], [-1], [4, 6, 7, 8]]) - def testOwnLoopback(self): - """check having the loopback ip""" - self.failUnless(OwnIpAddress(constants.LOCALHOST_IP_ADDRESS), - "Should own the loopback address") - - def testNowOwnAddress(self): - """check that I don't own an address""" - - # network 192.0.2.0/24 is reserved for test/documentation as per - # rfc 3330, so we *should* not have an address of this range... if - # this fails, we should extend the test to multiple addresses - DST_IP = "192.0.2.1" - self.failIf(OwnIpAddress(DST_IP), "Should not own IP address %s" % DST_IP) - - -def _GetSocketCredentials(path): - """Connect to a Unix socket and return remote credentials. - - """ - sock = socket.socket(socket.AF_UNIX, socket.SOCK_STREAM) - try: - sock.settimeout(10) - sock.connect(path) - return utils.GetSocketCredentials(sock) - finally: - sock.close() - - -class TestGetSocketCredentials(unittest.TestCase): - def setUp(self): - self.tmpdir = tempfile.mkdtemp() - self.sockpath = utils.PathJoin(self.tmpdir, "sock") - - self.listener = socket.socket(socket.AF_UNIX, socket.SOCK_STREAM) - self.listener.settimeout(10) - self.listener.bind(self.sockpath) - self.listener.listen(1) - - def tearDown(self): - self.listener.shutdown(socket.SHUT_RDWR) - self.listener.close() - shutil.rmtree(self.tmpdir) - - def test(self): - (c2pr, c2pw) = os.pipe() - - # Start child process - child = os.fork() - if child == 0: - try: - data = serializer.DumpJson(_GetSocketCredentials(self.sockpath)) - - os.write(c2pw, data) - os.close(c2pw) - - os._exit(0) - finally: - os._exit(1) - - os.close(c2pw) - - # Wait for one connection - (conn, _) = self.listener.accept() - conn.recv(1) - conn.close() - - # Wait for result - result = os.read(c2pr, 4096) - os.close(c2pr) - - # Check child's exit code - (_, status) = os.waitpid(child, 0) - self.assertFalse(os.WIFSIGNALED(status)) - self.assertEqual(os.WEXITSTATUS(status), 0) + def testInvalidInput(self): + for data in ["garbage", "0,", "0-1-2", "2-1", "1-a", "all-all"]: + self.assertRaises(errors.ParseError, utils.ParseCpuMask, data) - # Check result - (pid, uid, gid) = serializer.LoadJson(result) - self.assertEqual(pid, os.getpid()) - self.assertEqual(uid, os.getuid()) - self.assertEqual(gid, os.getgid()) +class TestGetMounts(unittest.TestCase): + """Test case for GetMounts().""" -class TestListVisibleFiles(unittest.TestCase): - """Test case for ListVisibleFiles""" + TESTDATA = ( + "rootfs / rootfs rw 0 0\n" + "none /sys sysfs rw,nosuid,nodev,noexec,relatime 0 0\n" + "none /proc proc rw,nosuid,nodev,noexec,relatime 0 0\n") def setUp(self): - self.path = tempfile.mkdtemp() - - def tearDown(self): - shutil.rmtree(self.path) - - def _test(self, files, expected): - # Sort a copy - expected = expected[:] - expected.sort() - - for name in files: - f = open(os.path.join(self.path, name), 'w') - try: - f.write("Test\n") - finally: - f.close() - - found = ListVisibleFiles(self.path) - found.sort() - - self.assertEqual(found, expected) - - def testAllVisible(self): - files = ["a", "b", "c"] - expected = files - self._test(files, expected) - - def testNoneVisible(self): - files = [".a", ".b", ".c"] - expected = [] - self._test(files, expected) - - def testSomeVisible(self): - files = ["a", "b", ".c"] - expected = ["a", "b"] - self._test(files, expected) - - def testNonAbsolutePath(self): - self.failUnlessRaises(errors.ProgrammerError, ListVisibleFiles, "abc") - - def testNonNormalizedPath(self): - self.failUnlessRaises(errors.ProgrammerError, ListVisibleFiles, - "/bin/../tmp") - - -class TestNewUUID(unittest.TestCase): - """Test case for NewUUID""" - - _re_uuid = re.compile('^[a-f0-9]{8}-[a-f0-9]{4}-[a-f0-9]{4}-' - '[a-f0-9]{4}-[a-f0-9]{12}$') - - def runTest(self): - self.failUnless(self._re_uuid.match(utils.NewUUID())) - - -class TestUniqueSequence(unittest.TestCase): - """Test case for UniqueSequence""" - - def _test(self, input, expected): - self.assertEqual(utils.UniqueSequence(input), expected) - - def runTest(self): - # Ordered input - self._test([1, 2, 3], [1, 2, 3]) - self._test([1, 1, 2, 2, 3, 3], [1, 2, 3]) - self._test([1, 2, 2, 3], [1, 2, 3]) - self._test([1, 2, 3, 3], [1, 2, 3]) - - # Unordered input - self._test([1, 2, 3, 1, 2, 3], [1, 2, 3]) - self._test([1, 1, 2, 3, 3, 1, 2], [1, 2, 3]) + self.tmpfile = tempfile.NamedTemporaryFile() + utils.WriteFile(self.tmpfile.name, data=self.TESTDATA) - # Strings - self._test(["a", "a"], ["a"]) - self._test(["a", "b"], ["a", "b"]) - self._test(["a", "b", "a"], ["a", "b"]) + def testGetMounts(self): + self.assertEqual(utils.GetMounts(filename=self.tmpfile.name), + [ + ("rootfs", "/", "rootfs", "rw"), + ("none", "/sys", "sysfs", "rw,nosuid,nodev,noexec,relatime"), + ("none", "/proc", "proc", "rw,nosuid,nodev,noexec,relatime"), + ]) class TestFirstFree(unittest.TestCase): @@ -1072,164 +110,6 @@ class TestFirstFree(unittest.TestCase): self.failUnlessRaises(AssertionError, FirstFree, [0, 3, 4, 6], base=3) -class TestTailFile(testutils.GanetiTestCase): - """Test case for the TailFile function""" - - def testEmpty(self): - fname = self._CreateTempFile() - self.failUnlessEqual(TailFile(fname), []) - self.failUnlessEqual(TailFile(fname, lines=25), []) - - def testAllLines(self): - data = ["test %d" % i for i in range(30)] - for i in range(30): - fname = self._CreateTempFile() - fd = open(fname, "w") - fd.write("\n".join(data[:i])) - if i > 0: - fd.write("\n") - fd.close() - self.failUnlessEqual(TailFile(fname, lines=i), data[:i]) - - def testPartialLines(self): - data = ["test %d" % i for i in range(30)] - fname = self._CreateTempFile() - fd = open(fname, "w") - fd.write("\n".join(data)) - fd.write("\n") - fd.close() - for i in range(1, 30): - self.failUnlessEqual(TailFile(fname, lines=i), data[-i:]) - - def testBigFile(self): - data = ["test %d" % i for i in range(30)] - fname = self._CreateTempFile() - fd = open(fname, "w") - fd.write("X" * 1048576) - fd.write("\n") - fd.write("\n".join(data)) - fd.write("\n") - fd.close() - for i in range(1, 30): - self.failUnlessEqual(TailFile(fname, lines=i), data[-i:]) - - -class _BaseFileLockTest: - """Test case for the FileLock class""" - - def testSharedNonblocking(self): - self.lock.Shared(blocking=False) - self.lock.Close() - - def testExclusiveNonblocking(self): - self.lock.Exclusive(blocking=False) - self.lock.Close() - - def testUnlockNonblocking(self): - self.lock.Unlock(blocking=False) - self.lock.Close() - - def testSharedBlocking(self): - self.lock.Shared(blocking=True) - self.lock.Close() - - def testExclusiveBlocking(self): - self.lock.Exclusive(blocking=True) - self.lock.Close() - - def testUnlockBlocking(self): - self.lock.Unlock(blocking=True) - self.lock.Close() - - def testSharedExclusiveUnlock(self): - self.lock.Shared(blocking=False) - self.lock.Exclusive(blocking=False) - self.lock.Unlock(blocking=False) - self.lock.Close() - - def testExclusiveSharedUnlock(self): - self.lock.Exclusive(blocking=False) - self.lock.Shared(blocking=False) - self.lock.Unlock(blocking=False) - self.lock.Close() - - def testSimpleTimeout(self): - # These will succeed on the first attempt, hence a short timeout - self.lock.Shared(blocking=True, timeout=10.0) - self.lock.Exclusive(blocking=False, timeout=10.0) - self.lock.Unlock(blocking=True, timeout=10.0) - self.lock.Close() - - @staticmethod - def _TryLockInner(filename, shared, blocking): - lock = utils.FileLock.Open(filename) - - if shared: - fn = lock.Shared - else: - fn = lock.Exclusive - - try: - # The timeout doesn't really matter as the parent process waits for us to - # finish anyway. - fn(blocking=blocking, timeout=0.01) - except errors.LockError, err: - return False - - return True - - def _TryLock(self, *args): - return utils.RunInSeparateProcess(self._TryLockInner, self.tmpfile.name, - *args) - - def testTimeout(self): - for blocking in [True, False]: - self.lock.Exclusive(blocking=True) - self.failIf(self._TryLock(False, blocking)) - self.failIf(self._TryLock(True, blocking)) - - self.lock.Shared(blocking=True) - self.assert_(self._TryLock(True, blocking)) - self.failIf(self._TryLock(False, blocking)) - - def testCloseShared(self): - self.lock.Close() - self.assertRaises(AssertionError, self.lock.Shared, blocking=False) - - def testCloseExclusive(self): - self.lock.Close() - self.assertRaises(AssertionError, self.lock.Exclusive, blocking=False) - - def testCloseUnlock(self): - self.lock.Close() - self.assertRaises(AssertionError, self.lock.Unlock, blocking=False) - - -class TestFileLockWithFilename(testutils.GanetiTestCase, _BaseFileLockTest): - TESTDATA = "Hello World\n" * 10 - - def setUp(self): - testutils.GanetiTestCase.setUp(self) - - self.tmpfile = tempfile.NamedTemporaryFile() - utils.WriteFile(self.tmpfile.name, data=self.TESTDATA) - self.lock = utils.FileLock.Open(self.tmpfile.name) - - # Ensure "Open" didn't truncate file - self.assertFileContent(self.tmpfile.name, self.TESTDATA) - - def tearDown(self): - self.assertFileContent(self.tmpfile.name, self.TESTDATA) - - testutils.GanetiTestCase.tearDown(self) - - -class TestFileLockWithFileObject(unittest.TestCase, _BaseFileLockTest): - def setUp(self): - self.tmpfile = tempfile.NamedTemporaryFile() - self.lock = utils.FileLock(open(self.tmpfile.name, "w"), self.tmpfile.name) - - class TestTimeFunctions(unittest.TestCase): """Test case for time functions""" @@ -1282,20 +162,19 @@ class FieldSetTestCase(unittest.TestCase): class TestForceDictType(unittest.TestCase): """Test case for ForceDictType""" - - def setUp(self): - self.key_types = { - 'a': constants.VTYPE_INT, - 'b': constants.VTYPE_BOOL, - 'c': constants.VTYPE_STRING, - 'd': constants.VTYPE_SIZE, - } + KEY_TYPES = { + "a": constants.VTYPE_INT, + "b": constants.VTYPE_BOOL, + "c": constants.VTYPE_STRING, + "d": constants.VTYPE_SIZE, + "e": constants.VTYPE_MAYBE_STRING, + } def _fdt(self, dict, allowed_values=None): if allowed_values is None: - ForceDictType(dict, self.key_types) + utils.ForceDictType(dict, self.KEY_TYPES) else: - ForceDictType(dict, self.key_types, allowed_values=allowed_values) + utils.ForceDictType(dict, self.KEY_TYPES, allowed_values=allowed_values) return dict @@ -1308,373 +187,132 @@ class TestForceDictType(unittest.TestCase): self.assertEqual(self._fdt({'b': 1, 'c': False}), {'b': True, 'c': ''}) self.assertEqual(self._fdt({'b': 'false'}), {'b': False}) self.assertEqual(self._fdt({'b': 'False'}), {'b': False}) + self.assertEqual(self._fdt({'b': False}), {'b': False}) self.assertEqual(self._fdt({'b': 'true'}), {'b': True}) self.assertEqual(self._fdt({'b': 'True'}), {'b': True}) self.assertEqual(self._fdt({'d': '4'}), {'d': 4}) self.assertEqual(self._fdt({'d': '4M'}), {'d': 4}) + self.assertEqual(self._fdt({"e": None, }), {"e": None, }) + self.assertEqual(self._fdt({"e": "Hello World", }), {"e": "Hello World", }) + self.assertEqual(self._fdt({"e": False, }), {"e": '', }) + self.assertEqual(self._fdt({"b": "hello", }, ["hello"]), {"b": "hello"}) def testErrors(self): self.assertRaises(errors.TypeEnforcementError, self._fdt, {'a': 'astring'}) + self.assertRaises(errors.TypeEnforcementError, self._fdt, {"b": "hello"}) self.assertRaises(errors.TypeEnforcementError, self._fdt, {'c': True}) self.assertRaises(errors.TypeEnforcementError, self._fdt, {'d': 'astring'}) self.assertRaises(errors.TypeEnforcementError, self._fdt, {'d': '4 L'}) + self.assertRaises(errors.TypeEnforcementError, self._fdt, {"e": object(), }) + self.assertRaises(errors.TypeEnforcementError, self._fdt, {"e": [], }) + self.assertRaises(errors.TypeEnforcementError, self._fdt, {"x": None, }) + self.assertRaises(errors.TypeEnforcementError, self._fdt, []) + self.assertRaises(errors.ProgrammerError, utils.ForceDictType, + {"b": "hello"}, {"b": "no-such-type"}) + + +class TestValidateServiceName(unittest.TestCase): + def testValid(self): + testnames = [ + 0, 1, 2, 3, 1024, 65000, 65534, 65535, + "ganeti", + "gnt-masterd", + "HELLO_WORLD_SVC", + "hello.world.1", + "0", "80", "1111", "65535", + ] - -class TestIsAbsNormPath(unittest.TestCase): - """Testing case for IsProcessAlive""" - - def _pathTestHelper(self, path, result): - if result: - self.assert_(IsNormAbsPath(path), - "Path %s should result absolute and normalized" % path) - else: - self.assert_(not IsNormAbsPath(path), - "Path %s should not result absolute and normalized" % path) - - def testBase(self): - self._pathTestHelper('/etc', True) - self._pathTestHelper('/srv', True) - self._pathTestHelper('etc', False) - self._pathTestHelper('/etc/../root', False) - self._pathTestHelper('/etc/', False) - - -class TestSafeEncode(unittest.TestCase): - """Test case for SafeEncode""" - - def testAscii(self): - for txt in [string.digits, string.letters, string.punctuation]: - self.failUnlessEqual(txt, SafeEncode(txt)) - - def testDoubleEncode(self): - for i in range(255): - txt = SafeEncode(chr(i)) - self.failUnlessEqual(txt, SafeEncode(txt)) - - def testUnicode(self): - # 1024 is high enough to catch non-direct ASCII mappings - for i in range(1024): - txt = SafeEncode(unichr(i)) - self.failUnlessEqual(txt, SafeEncode(txt)) - - -class TestFormatTime(unittest.TestCase): - """Testing case for FormatTime""" - - def testNone(self): - self.failUnlessEqual(FormatTime(None), "N/A") + for name in testnames: + self.assertEqual(utils.ValidateServiceName(name), name) def testInvalid(self): - self.failUnlessEqual(FormatTime(()), "N/A") - - def testNow(self): - # tests that we accept time.time input - FormatTime(time.time()) - # tests that we accept int input - FormatTime(int(time.time())) - - -class RunInSeparateProcess(unittest.TestCase): - def test(self): - for exp in [True, False]: - def _child(): - return exp - - self.assertEqual(exp, utils.RunInSeparateProcess(_child)) - - def testArgs(self): - for arg in [0, 1, 999, "Hello World", (1, 2, 3)]: - def _child(carg1, carg2): - return carg1 == "Foo" and carg2 == arg - - self.assert_(utils.RunInSeparateProcess(_child, "Foo", arg)) - - def testPid(self): - parent_pid = os.getpid() - - def _check(): - return os.getpid() == parent_pid - - self.failIf(utils.RunInSeparateProcess(_check)) - - def testSignal(self): - def _kill(): - os.kill(os.getpid(), signal.SIGTERM) - - self.assertRaises(errors.GenericError, - utils.RunInSeparateProcess, _kill) - - def testException(self): - def _exc(): - raise errors.GenericError("This is a test") - - self.assertRaises(errors.GenericError, - utils.RunInSeparateProcess, _exc) - - -class TestFingerprintFile(unittest.TestCase): - def setUp(self): - self.tmpfile = tempfile.NamedTemporaryFile() - - def test(self): - self.assertEqual(utils._FingerprintFile(self.tmpfile.name), - "da39a3ee5e6b4b0d3255bfef95601890afd80709") - - utils.WriteFile(self.tmpfile.name, data="Hello World\n") - self.assertEqual(utils._FingerprintFile(self.tmpfile.name), - "648a6a6ffffdaa0badb23b8baf90b6168dd16b3a") - - -class TestUnescapeAndSplit(unittest.TestCase): - """Testing case for UnescapeAndSplit""" - - def setUp(self): - # testing more that one separator for regexp safety - self._seps = [",", "+", "."] - - def testSimple(self): - a = ["a", "b", "c", "d"] - for sep in self._seps: - self.failUnlessEqual(UnescapeAndSplit(sep.join(a), sep=sep), a) - - def testEscape(self): - for sep in self._seps: - a = ["a", "b\\" + sep + "c", "d"] - b = ["a", "b" + sep + "c", "d"] - self.failUnlessEqual(UnescapeAndSplit(sep.join(a), sep=sep), b) - - def testDoubleEscape(self): - for sep in self._seps: - a = ["a", "b\\\\", "c", "d"] - b = ["a", "b\\", "c", "d"] - self.failUnlessEqual(UnescapeAndSplit(sep.join(a), sep=sep), b) - - def testThreeEscape(self): - for sep in self._seps: - a = ["a", "b\\\\\\" + sep + "c", "d"] - b = ["a", "b\\" + sep + "c", "d"] - self.failUnlessEqual(UnescapeAndSplit(sep.join(a), sep=sep), b) - - -class TestPathJoin(unittest.TestCase): - """Testing case for PathJoin""" - - def testBasicItems(self): - mlist = ["/a", "b", "c"] - self.failUnlessEqual(PathJoin(*mlist), "/".join(mlist)) - - def testNonAbsPrefix(self): - self.failUnlessRaises(ValueError, PathJoin, "a", "b") - - def testBackTrack(self): - self.failUnlessRaises(ValueError, PathJoin, "/a", "b/../c") - - def testMultiAbs(self): - self.failUnlessRaises(ValueError, PathJoin, "/a", "/b") - - -class TestHostInfo(unittest.TestCase): - """Testing case for HostInfo""" - - def testUppercase(self): - data = "AbC.example.com" - self.failUnlessEqual(HostInfo.NormalizeName(data), data.lower()) - - def testTooLongName(self): - data = "a.b." + "c" * 255 - self.failUnlessRaises(OpPrereqError, HostInfo.NormalizeName, data) - - def testTrailingDot(self): - data = "a.b.c" - self.failUnlessEqual(HostInfo.NormalizeName(data + "."), data) - - def testInvalidName(self): - data = [ - "a b", - "a/b", - ".a.b", - "a..b", - ] - for value in data: - self.failUnlessRaises(OpPrereqError, HostInfo.NormalizeName, value) - - def testValidName(self): - data = [ - "a.b", - "a-b", - "a_b", - "a.b.c", + testnames = [ + -15756, -1, 65536, 133428083, + "", "Hello World!", "!", "'", "\"", "\t", "\n", "`", + "-8546", "-1", "65536", + (129 * "A"), ] - for value in data: - HostInfo.NormalizeName(value) - - -class TestParseAsn1Generalizedtime(unittest.TestCase): - def test(self): - # UTC - self.assertEqual(utils._ParseAsn1Generalizedtime("19700101000000Z"), 0) - self.assertEqual(utils._ParseAsn1Generalizedtime("20100222174152Z"), - 1266860512) - self.assertEqual(utils._ParseAsn1Generalizedtime("20380119031407Z"), - (2**31) - 1) - # With offset - self.assertEqual(utils._ParseAsn1Generalizedtime("20100222174152+0000"), - 1266860512) - self.assertEqual(utils._ParseAsn1Generalizedtime("20100223131652+0000"), - 1266931012) - self.assertEqual(utils._ParseAsn1Generalizedtime("20100223051808-0800"), - 1266931088) - self.assertEqual(utils._ParseAsn1Generalizedtime("20100224002135+1100"), - 1266931295) - self.assertEqual(utils._ParseAsn1Generalizedtime("19700101000000-0100"), - 3600) + for name in testnames: + self.assertRaises(errors.OpPrereqError, utils.ValidateServiceName, name) - # Leap seconds are not supported by datetime.datetime - self.assertRaises(ValueError, utils._ParseAsn1Generalizedtime, - "19841231235960+0000") - self.assertRaises(ValueError, utils._ParseAsn1Generalizedtime, - "19920630235960+0000") - # Errors - self.assertRaises(ValueError, utils._ParseAsn1Generalizedtime, "") - self.assertRaises(ValueError, utils._ParseAsn1Generalizedtime, "invalid") - self.assertRaises(ValueError, utils._ParseAsn1Generalizedtime, - "20100222174152") - self.assertRaises(ValueError, utils._ParseAsn1Generalizedtime, - "Mon Feb 22 17:47:02 UTC 2010") - self.assertRaises(ValueError, utils._ParseAsn1Generalizedtime, - "2010-02-22 17:42:02") - - -class TestGetX509CertValidity(testutils.GanetiTestCase): - def setUp(self): - testutils.GanetiTestCase.setUp(self) - - pyopenssl_version = distutils.version.LooseVersion(OpenSSL.__version__) - - # Test whether we have pyOpenSSL 0.7 or above - self.pyopenssl0_7 = (pyopenssl_version >= "0.7") - - if not self.pyopenssl0_7: - warnings.warn("This test requires pyOpenSSL 0.7 or above to" - " function correctly") - - def _LoadCert(self, name): - return OpenSSL.crypto.load_certificate(OpenSSL.crypto.FILETYPE_PEM, - self._ReadTestData(name)) - - def test(self): - validity = utils.GetX509CertValidity(self._LoadCert("cert1.pem")) - if self.pyopenssl0_7: - self.assertEqual(validity, (1266919967, 1267524767)) - else: - self.assertEqual(validity, (None, None)) - - -class TestMakedirs(unittest.TestCase): +class TestReadLockedPidFile(unittest.TestCase): def setUp(self): self.tmpdir = tempfile.mkdtemp() def tearDown(self): shutil.rmtree(self.tmpdir) - def testNonExisting(self): - path = utils.PathJoin(self.tmpdir, "foo") - utils.Makedirs(path) - self.assert_(os.path.isdir(path)) + def testNonExistent(self): + path = utils.PathJoin(self.tmpdir, "nonexist") + self.assert_(utils.ReadLockedPidFile(path) is None) - def testExisting(self): - path = utils.PathJoin(self.tmpdir, "foo") - os.mkdir(path) - utils.Makedirs(path) - self.assert_(os.path.isdir(path)) + def testUnlocked(self): + path = utils.PathJoin(self.tmpdir, "pid") + utils.WriteFile(path, data="123") + self.assert_(utils.ReadLockedPidFile(path) is None) - def testRecursiveNonExisting(self): - path = utils.PathJoin(self.tmpdir, "foo/bar/baz") - utils.Makedirs(path) - self.assert_(os.path.isdir(path)) + def testLocked(self): + path = utils.PathJoin(self.tmpdir, "pid") + utils.WriteFile(path, data="123") - def testRecursiveExisting(self): - path = utils.PathJoin(self.tmpdir, "B/moo/xyz") - self.assert_(not os.path.exists(path)) - os.mkdir(utils.PathJoin(self.tmpdir, "B")) - utils.Makedirs(path) - self.assert_(os.path.isdir(path)) + fl = utils.FileLock.Open(path) + try: + fl.Exclusive(blocking=True) + self.assertEqual(utils.ReadLockedPidFile(path), 123) + finally: + fl.Close() -class TestRetry(testutils.GanetiTestCase): - def setUp(self): - testutils.GanetiTestCase.setUp(self) - self.retries = 0 + self.assert_(utils.ReadLockedPidFile(path) is None) - @staticmethod - def _RaiseRetryAgain(): - raise utils.RetryAgain() + def testError(self): + path = utils.PathJoin(self.tmpdir, "foobar", "pid") + utils.WriteFile(utils.PathJoin(self.tmpdir, "foobar"), data="") + # open(2) should return ENOTDIR + self.assertRaises(EnvironmentError, utils.ReadLockedPidFile, path) - def _WrongNestedLoop(self): - return utils.Retry(self._RaiseRetryAgain, 0.01, 0.02) - def _RetryAndSucceed(self, retries): - if self.retries < retries: - self.retries += 1 - raise utils.RetryAgain() - else: - return True +class TestFindMatch(unittest.TestCase): + def test(self): + data = { + "aaaa": "Four A", + "bb": {"Two B": True}, + re.compile(r"^x(foo|bar|bazX)([0-9]+)$"): (1, 2, 3), + } - def testRaiseTimeout(self): - self.failUnlessRaises(utils.RetryTimeout, utils.Retry, - self._RaiseRetryAgain, 0.01, 0.02) - self.failUnlessRaises(utils.RetryTimeout, utils.Retry, - self._RetryAndSucceed, 0.01, 0, args=[1]) - self.failUnlessEqual(self.retries, 1) + self.assertEqual(utils.FindMatch(data, "aaaa"), ("Four A", [])) + self.assertEqual(utils.FindMatch(data, "bb"), ({"Two B": True}, [])) - def testComplete(self): - self.failUnlessEqual(utils.Retry(lambda: True, 0, 1), True) - self.failUnlessEqual(utils.Retry(self._RetryAndSucceed, 0, 1, args=[2]), - True) - self.failUnlessEqual(self.retries, 2) + for i in ["foo", "bar", "bazX"]: + for j in range(1, 100, 7): + self.assertEqual(utils.FindMatch(data, "x%s%s" % (i, j)), + ((1, 2, 3), [i, str(j)])) - def testNestedLoop(self): - try: - self.failUnlessRaises(errors.ProgrammerError, utils.Retry, - self._WrongNestedLoop, 0, 1) - except utils.RetryTimeout: - self.fail("Didn't detect inner loop's exception") + def testNoMatch(self): + self.assert_(utils.FindMatch({}, "") is None) + self.assert_(utils.FindMatch({}, "foo") is None) + self.assert_(utils.FindMatch({}, 1234) is None) + data = { + "X": "Hello World", + re.compile("^(something)$"): "Hello World", + } -class TestLineSplitter(unittest.TestCase): - def test(self): - lines = [] - ls = utils.LineSplitter(lines.append) - ls.write("Hello World\n") - self.assertEqual(lines, []) - ls.write("Foo\n Bar\r\n ") - ls.write("Baz") - ls.write("Moo") - self.assertEqual(lines, []) - ls.flush() - self.assertEqual(lines, ["Hello World", "Foo", " Bar"]) - ls.close() - self.assertEqual(lines, ["Hello World", "Foo", " Bar", " BazMoo"]) + self.assert_(utils.FindMatch(data, "") is None) + self.assert_(utils.FindMatch(data, "Hello World") is None) - def _testExtra(self, line, all_lines, p1, p2): - self.assertEqual(p1, 999) - self.assertEqual(p2, "extra") - all_lines.append(line) - def testExtraArgsNoFlush(self): - lines = [] - ls = utils.LineSplitter(self._testExtra, lines, 999, "extra") - ls.write("\n\nHello World\n") - ls.write("Foo\n Bar\r\n ") - ls.write("") - ls.write("Baz") - ls.write("Moo\n\nx\n") - self.assertEqual(lines, []) - ls.close() - self.assertEqual(lines, ["", "", "Hello World", "Foo", " Bar", " BazMoo", - "", "x"]) +class TestTryConvert(unittest.TestCase): + def test(self): + for src, fn, result in [ + ("1", int, 1), + ("a", int, "a"), + ("", bool, False), + ("a", bool, True), + ]: + self.assertEqual(utils.TryConvert(fn, src), result) if __name__ == '__main__':