From 4f98fe10ea482fad3fffe9d655d26139dea30965 Mon Sep 17 00:00:00 2001
From: Daniel Narvaez <dwnarvaez@gmail.com>
Date: Fri, 30 Nov 2012 15:18:38 +0000
Subject: Add more tests and fix bugs

---
(limited to 'tests/devbot')

diff --git a/tests/devbot/data/os-release-ubuntu-12.10 b/tests/devbot/data/os-release-ubuntu-12.10
new file mode 100644
index 0000000..9b53281
--- /dev/null
+++ b/tests/devbot/data/os-release-ubuntu-12.10
@@ -0,0 +1,6 @@
+NAME="Ubuntu"
+VERSION="12.10, Quantal Quetzal"
+ID=ubuntu
+ID_LIKE=debian
+PRETTY_NAME="Ubuntu quantal (12.10)"
+VERSION_ID="12.10"
diff --git a/tests/devbot/test_config.py b/tests/devbot/test_config.py
index 2c3f2fa..39b3f65 100644
--- a/tests/devbot/test_config.py
+++ b/tests/devbot/test_config.py
@@ -13,27 +13,34 @@ config_dir = os.path.join(base_dir, "config")
 data_dir = os.path.join(tests_dir, "data")
 
 class TestConfig(unittest.TestCase):
-    def setUp(self):
-        config.set_config_dir(config_dir)
-
     def _set_distro(self, name, version):
-        self._orig_supported_distro = distro._supported_distros
+        if config.config_dir is None:
+            config.set_config_dir(config_dir)
+            config.load_plugins()
+
+        self._orig = distro._supported_distros
         for info_class in distro._supported_distros:
-            if "_FEDORA_RELEASE_PATH" in distro_info_class:
+            if info_class.__module__.endswith("fedora"):
                 info_class._FEDORA_RELEASE_PATH = \
-                    os.path.join(data_dir, "fedora-release-18") 
+                    os.path.join(data_dir, "fedora-release-%s" % version) 
 
-            if "_DEBIAN_VERSION_PATH" in distro_info_class:
+            if info_class.__module__.endswith("debian"):
                 info_class._DEBIAN_VERSION_PATH = \
                     os.path.join(data_dir, "debian_version-wheezy") 
 
+            if info_class.__module__.endswith("ubuntu"):
+                info_class._OS_RELEASE_PATH = \
+                    os.path.join(data_dir, "os-release-ubuntu-12.10") 
+
+        for info_class in distro._supported_distros:
             info = info_class()
             if info.name == name and info.version == version:
-                self._supported_distros = [info]
+                distro._supported_distros = [info_class]
+                distro._distro_info = None
                 break
 
     def _unset_distro(self):
-        distro._supported_distros = self._orig_supported_distro
+        distro._supported_distros = self._orig
 
     def _find_module(self, modules, name):
         for module in modules:
@@ -46,7 +53,7 @@ class TestConfig(unittest.TestCase):
         self.assertIsNotNone(self._find_module(modules, name))
 
     def _assert_no_module(self, modules, name):
-        self.assertIsNotNone(self._find_module(modules, name))
+        self.assertIsNone(self._find_module(modules, name))
 
     def test_fedora_17_modules(self):
         self._set_distro("fedora", "17")
@@ -62,10 +69,24 @@ class TestConfig(unittest.TestCase):
     def test_fedora_18_modules(self):
         self._set_distro("fedora", "18")
 
+
+        self.assertEquals("fedora", distro.get_distro_info().name)
+        self.assertEquals("18", distro.get_distro_info().version)
         modules = config.load_modules()
         self._assert_module(modules, "glib") 
-        self._assert_module(modules, "gtk+")
-        self._assert_module(modules, "gstreamer")
+        self._assert_no_module(modules, "gtk+")
+        self._assert_no_module(modules, "gstreamer")
+        self._assert_module(modules, "sugar") 
+
+        self._unset_distro()
+
+    def test_ubuntu_12_10_modules(self):
+        self._set_distro("ubuntu", "12.10")
+
+        modules = config.load_modules()
+        self._assert_module(modules, "glib") 
+        self._assert_no_module(modules, "gtk+")
+        self._assert_no_module(modules, "gstreamer")
         self._assert_module(modules, "sugar") 
 
         self._unset_distro()
--
cgit v0.9.1