From 6fb02650ff1fc7b964ee35826efe7169b2e5541c Mon Sep 17 00:00:00 2001 From: Michael DeHaan Date: Wed, 19 Feb 2014 17:57:39 -0500 Subject: [PATCH] Remove tests for things that are now using the playbook based 'test_new' infra. Units will remain here after refactoring, WIP, more to come. --- test/TestPlayBook.py | 19 +-------- test/TestRunner.py | 93 -------------------------------------------- 2 files changed, 1 insertion(+), 111 deletions(-) diff --git a/test/TestPlayBook.py b/test/TestPlayBook.py index d558c114e5c..7458bcf95dc 100644 --- a/test/TestPlayBook.py +++ b/test/TestPlayBook.py @@ -392,23 +392,6 @@ class TestPlaybook(unittest.TestCase): assert utils.jsonify(expected, format=True) == utils.jsonify(actual,format=True) - - def test_recursive_copy(self): - pb = 'test/playbook-recursive-copy.yml' - actual = self._run(pb) - - expected = { - "localhost": { - "changed": 65, - "failures": 0, - "ok": 73, - "skipped": 0, - "unreachable": 0 - } - } - - assert utils.jsonify(expected, format=True) == utils.jsonify(actual,format=True) - def test_playbook_logging_non_ascii(self): pb = 'test/playbook-logging-non-ascii.yml' actual = self._run(pb) @@ -457,4 +440,4 @@ class TestPlaybook(unittest.TestCase): print "**EXPECTED**" print expected_lines - assert actual_lines == expected_lines \ No newline at end of file + assert actual_lines == expected_lines diff --git a/test/TestRunner.py b/test/TestRunner.py index 027a4c6c9ec..0734b9e3015 100644 --- a/test/TestRunner.py +++ b/test/TestRunner.py @@ -103,34 +103,6 @@ class TestRunner(unittest.TestCase): # result = self._run('ohai',[]) # assert "hostname" in result - def test_copy(self): - # test copy module, change trigger, etc - input_ = self._get_test_file('sample.j2') - output = self._get_stage_file('sample.out') - assert not os.path.exists(output) - result = self._run('copy', [ - "src=%s" % input_, - "dest=%s" % output, - ]) - assert os.path.exists(output) - data_in = file(input_).read() - data_out = file(output).read() - assert data_in == data_out - assert 'failed' not in result - assert result['changed'] is True - assert 'md5sum' in result - result = self._run('copy', [ - "src=%s" % input_, - "dest=%s" % output, - ]) - assert result['changed'] is False - with open(output, "a") as output_stream: - output_stream.write("output file now differs from input") - result = self._run('copy', - ["src=%s" % input_, "dest=%s" % output, "force=no"], - check_mode=True) - assert result['changed'] is False - def test_command(self): # test command module, change trigger, etc result = self._run('command', ["/bin/echo", "hi"]) @@ -225,64 +197,6 @@ class TestRunner(unittest.TestCase): self._run('file', ['path=/tmp/%s' % git_ref, 'state=absent']) self._run('file', ['path=/tmp/%s' % git_reftest, 'state=absent']) - def test_file(self): - filedemo = tempfile.mkstemp()[1] - assert self._run('file', ['dest=' + filedemo, 'state=directory'])['failed'] - assert os.path.isfile(filedemo) - - assert self._run('file', ['dest=' + filedemo, 'src=/dev/null', 'state=link'])['failed'] - assert os.path.isfile(filedemo) - - res = self._run('file', ['dest=' + filedemo, 'mode=604', 'state=file']) - assert res['changed'] - assert os.path.isfile(filedemo) and os.stat(filedemo).st_mode == 0100604 - - assert self._run('file', ['dest=' + filedemo, 'state=absent'])['changed'] - assert not os.path.exists(filedemo) - assert not self._run('file', ['dest=' + filedemo, 'state=absent'])['changed'] - - filedemo = tempfile.mkdtemp() - assert self._run('file', ['dest=' + filedemo, 'state=file'])['failed'] - assert os.path.isdir(filedemo) - - # this used to fail but will now make a 'null' symlink in the directory pointing to dev/null. - # I feel this is ok but don't want to enforce it with a test. - #result = self._run('file', ['dest=' + filedemo, 'src=/dev/null', 'state=link']) - #assert result['failed'] - #assert os.path.isdir(filedemo) - - assert self._run('file', ['dest=' + filedemo, 'mode=701', 'state=directory'])['changed'] - assert os.path.isdir(filedemo) and os.stat(filedemo).st_mode == 040701 - - assert self._run('file', ['dest=' + filedemo, 'state=absent'])['changed'] - assert not os.path.exists(filedemo) - assert not self._run('file', ['dest=' + filedemo, 'state=absent'])['changed'] - - tmp_dir = tempfile.mkdtemp() - filedemo = os.path.join(tmp_dir, 'link') - os.symlink('/dev/zero', filedemo) - assert self._run('file', ['dest=' + filedemo, 'state=file'])['failed'] - assert os.path.islink(filedemo) - - assert self._run('file', ['dest=' + filedemo, 'state=directory'])['failed'] - assert os.path.islink(filedemo) - - assert self._run('file', ['dest=' + filedemo, 'src=/dev/null', 'state=link'])['changed'] - assert os.path.islink(filedemo) and os.path.realpath(filedemo) == '/dev/null' - - assert self._run('file', ['dest=' + filedemo, 'state=absent'])['changed'] - assert not os.path.exists(filedemo) - assert not self._run('file', ['dest=' + filedemo, 'state=absent'])['changed'] - - # Make sure that we can deal safely with bad symlinks - os.symlink('/tmp/non_existent_target', filedemo) - assert self._run('file', ['dest=' + tmp_dir, 'state=directory recurse=yes mode=701'])['changed'] - assert not self._run('file', ['dest=' + tmp_dir, 'state=directory', 'recurse=yes', 'owner=' + str(os.getuid())])['changed'] - assert os.path.islink(filedemo) - assert self._run('file', ['dest=' + filedemo, 'state=absent'])['changed'] - assert not os.path.exists(filedemo) - os.rmdir(tmp_dir) - def test_large_output(self): large_path = "/usr/share/dict/words" if not os.path.exists(large_path): @@ -316,13 +230,6 @@ class TestRunner(unittest.TestCase): assert 'stdout' in result assert result['ansible_job_id'] == jid - def test_fetch(self): - input_ = self._get_test_file('sample.j2') - output = os.path.join(self.stage_dir, 'localhost', input_) - self._run('fetch', ["src=%s" % input_, "dest=%s" % self.stage_dir]) - assert os.path.exists(output) - assert open(input_).read() == open(output).read() - def test_assemble(self): input = self._get_test_file('assemble.d') output = self._get_stage_file('sample.out')