diff --git a/bin/element-info b/bin/element-info index 44f2f3e84..1bdf3c010 100755 --- a/bin/element-info +++ b/bin/element-info @@ -17,7 +17,7 @@ import os import sys -from diskimage_builder.elements import main +from diskimage_builder.element_dependencies import main sys.exit(main(sys.argv)) diff --git a/diskimage_builder/elements.py b/diskimage_builder/element_dependencies.py similarity index 100% rename from diskimage_builder/elements.py rename to diskimage_builder/element_dependencies.py diff --git a/diskimage_builder/tests/__init__.py b/diskimage_builder/tests/__init__.py new file mode 100644 index 000000000..e69de29bb diff --git a/diskimage_builder/test_elementdeps.py b/diskimage_builder/tests/test_elementdeps.py similarity index 82% rename from diskimage_builder/test_elementdeps.py rename to diskimage_builder/tests/test_elementdeps.py index b1af6aba6..fc99f0ba0 100644 --- a/diskimage_builder/test_elementdeps.py +++ b/diskimage_builder/tests/test_elementdeps.py @@ -18,7 +18,7 @@ import os import fixtures import testtools -from diskimage_builder import elements +from diskimage_builder import element_dependencies data_dir = os.path.abspath( os.path.join(os.path.dirname(__file__), 'test-elements')) @@ -47,34 +47,35 @@ class TestElementDeps(testtools.TestCase): _populate_element(self.element_dir, 'circular2', ['circular1']) def test_non_transitive_deps(self): - result = elements.expand_dependencies( + result = element_dependencies.expand_dependencies( ['requires-foo'], elements_dir=self.element_dir) self.assertEqual(set(['requires-foo', 'foo']), result) def test_missing_deps(self): - self.assertRaises(SystemExit, elements.expand_dependencies, ['fake'], + self.assertRaises(SystemExit, + element_dependencies.expand_dependencies, ['fake'], self.element_dir) def test_transitive_deps(self): - result = elements.expand_dependencies( + result = element_dependencies.expand_dependencies( ['requires-requires-foo'], elements_dir=self.element_dir) self.assertEqual(set(['requires-requires-foo', 'requires-foo', 'foo']), result) def test_no_deps(self): - result = elements.expand_dependencies( + result = element_dependencies.expand_dependencies( ['foo'], elements_dir=self.element_dir) self.assertEqual(set(['foo']), result) def test_self(self): - result = elements.expand_dependencies( + result = element_dependencies.expand_dependencies( ['self'], elements_dir=self.element_dir) self.assertEqual(set(['self']), result) def test_circular(self): - result = elements.expand_dependencies( + result = element_dependencies.expand_dependencies( ['circular1'], elements_dir=self.element_dir) self.assertEqual(set(['circular1', 'circular2']), result) @@ -83,8 +84,8 @@ class TestElements(testtools.TestCase): def test_depends_on_env(self): self.useFixture( fixtures.EnvironmentVariable('ELEMENTS_PATH', '/foo/bar')) - self.assertEqual('/foo/bar', elements.get_elements_dir()) + self.assertEqual('/foo/bar', element_dependencies.get_elements_dir()) def test_env_not_set(self): self.useFixture(fixtures.EnvironmentVariable('ELEMENTS_PATH', '')) - self.assertRaises(Exception, elements.get_elements_dir, ()) + self.assertRaises(Exception, element_dependencies.get_elements_dir, ()) diff --git a/diskimage_builder/test_elements.py b/diskimage_builder/tests/test_elements.py similarity index 96% rename from diskimage_builder/test_elements.py rename to diskimage_builder/tests/test_elements.py index fa6d9d241..b24fd99dc 100644 --- a/diskimage_builder/test_elements.py +++ b/diskimage_builder/tests/test_elements.py @@ -30,7 +30,7 @@ def load_tests(loader, tests, pattern): # http://bugs.python.org/issue11218 pattern = "test*.py" this_dir = os.path.dirname(__file__) - elements_dir = os.path.join(this_dir, "..", "elements") + elements_dir = os.path.join(this_dir, "..", "..", "elements") # Make a fake elements top level package, as discovery doesn't let us # override the python path. package = StubPackage()