From adc055b25b9b5f3bbe9490116e89d9dc08444ee5 Mon Sep 17 00:00:00 2001 From: Siddharth Ravikumar Date: Wed, 11 Feb 2015 21:35:50 -0500 Subject: tests/file_test.py: re-wrote it. Now the test is in a Test class with proper setup and teardown methods. --- tests/file_test.py | 102 ++++++++++++++++++++++++++++------------------------- 1 file changed, 54 insertions(+), 48 deletions(-) diff --git a/tests/file_test.py b/tests/file_test.py index 4787406..829ebd3 100644 --- a/tests/file_test.py +++ b/tests/file_test.py @@ -26,71 +26,77 @@ from os import path, remove from combox.config import get_nodedirs from combox.file import (split_data, glue_data, write_file, read_file, write_shards, read_shards, - hash_file) + hash_file, rm_shards) +from tests.utils import get_config -CONFIG_DIR = path.join('tests', 'test-config') -config_file = path.join(CONFIG_DIR, 'config.yaml') -try: - config = yaml.load(file(config_file, 'r')) -except yaml.YAMLError, exc: - raise AssertionError("Error in configuration file:", exc) +class TestFile(object): + """ + Class that tests the config.py module. + """ + @classmethod + def setup_class(self): + """Set things up.""" -FILES_DIR = config['combox_dir'] -TEST_FILE = path.join(FILES_DIR,'the-red-star.jpg') + self.config = get_config() + FILES_DIR = self.config['combox_dir'] + self.TEST_FILE = path.join(FILES_DIR,'the-red-star.jpg') -def test_split(): - """Test to split, glue and create a copy of the image file from the - glued image file. - """ + def test_split(self): + """Test to split, glue and create a copy of the image file from the + glued image file. + """ + f = path.abspath(self.TEST_FILE) + f_content = read_file(f) + f_parts = split_data(f_content, 5) + f_content_glued = glue_data(f_parts) - f = path.abspath(TEST_FILE) - f_content = read_file(f) - f_parts = split_data(f_content, 5) - f_content_glued = glue_data(f_parts) + assert f_content == f_content_glued - assert f_content == f_content_glued - #f_copy = path.abspath('tests/files/the-red-star-copy.jpg') - #write_file(f_copy, f_content) + def test_shards(self): + """Split file into N shards, write them to disk, glue them together + and check if they're the same as the orginal file. -def test_shards(): - """Split file into N shards, write them to disk, glue them together -and check if they're the same as the orginal file. - """ + """ - # no. of shards = no. of nodes - SHARDS = len(config['nodes_info'].keys()) + # no. of shards = no. of nodes + SHARDS = len(self.config['nodes_info'].keys()) - f = path.abspath(TEST_FILE) - f_content = read_file(f) - f_shards = split_data(f_content, SHARDS) + f = path.abspath(self.TEST_FILE) + f_content = read_file(f) + f_shards = split_data(f_content, SHARDS) - f_basename = path.basename(f) - nodes = get_nodedirs(config) - write_shards(f_shards, nodes, f_basename) + f_basename = path.basename(f) + nodes = get_nodedirs(self.config) + write_shards(f_shards, nodes, f_basename) - # check if the shards have been created. - i = 0 - for node in nodes: - shard = "%s.shard%s" % (path.join(node, f_basename), i) - i += 1 - assert path.isfile(shard) + # check if the shards have been created. + i = 0 + for node in nodes: + shard = "%s.shard%s" % (path.join(node, f_basename), i) + i += 1 + assert path.isfile(shard) + f_shards = read_shards(nodes, f_basename) + f_content_glued = glue_data(f_shards) + assert f_content == f_content_glued - f_shards = read_shards(nodes, f_basename) - f_content_glued = glue_data(f_shards) - assert f_content == f_content_glued + def test_hashing(self): + """ + Tests the hashing function - hash_file + """ + fhash = hash_file(self.TEST_FILE) + fcontent = read_file(self.TEST_FILE) + assert fhash == sha512(fcontent).hexdigest() -def test_hashing(): - """ - Tests the hashing function - hash_file - """ - fhash = hash_file(TEST_FILE) - fcontent = read_file(TEST_FILE) - assert fhash == sha512(fcontent).hexdigest() + @classmethod + def teardown_class(self): + """Purge the mess created by this test.""" + + rm_shards(self.TEST_FILE, self.config) -- cgit v1.2.3