From 0be531db71569c10263d0ee48456b286252baabb Mon Sep 17 00:00:00 2001
From: Toshio Kuratomi <toshio@fedoraproject.org>
Date: Wed, 15 Apr 2015 15:19:40 -0700
Subject: [PATCH] Make some of the optional requirements optional for testing
 -- we'll skip the tests instead

---
 v2/test-requirements.txt      |  6 ++++--
 v2/test/plugins/test_cache.py | 22 ++++++++++++++++++++--
 2 files changed, 24 insertions(+), 4 deletions(-)

diff --git a/v2/test-requirements.txt b/v2/test-requirements.txt
index 100bdd01a00..e4822ada648 100644
--- a/v2/test-requirements.txt
+++ b/v2/test-requirements.txt
@@ -5,8 +5,10 @@ jinja2
 httplib2
 passlib
 six
-python-memcached
-redis
+
+# These are needed for various optional features
+#python-memcached
+#redis
 
 # Test requirements
 unittest2
diff --git a/v2/test/plugins/test_cache.py b/v2/test/plugins/test_cache.py
index b1273874cd3..bf94053aa33 100644
--- a/v2/test/plugins/test_cache.py
+++ b/v2/test/plugins/test_cache.py
@@ -21,9 +21,25 @@ __metaclass__ = type
 
 from ansible.compat.tests import unittest
 from ansible.plugins.cache.base import BaseCacheModule
-from ansible.plugins.cache.memcached import CacheModule as MemcachedCache
 from ansible.plugins.cache.memory import CacheModule as MemoryCache
-from ansible.plugins.cache.redis import CacheModule as RedisCache
+
+HAVE_MEMCACHED = True
+try:
+    import memcached
+except ImportError:
+    HAVE_MEMCACHED = False
+else:
+    # Use an else so that the only reason we skip this is for lack of
+    # memcached, not errors importing the plugin
+    from ansible.plugins.cache.memcached import CacheModule as MemcachedCache
+
+HAVE_REDIS = True
+try:
+    import redis
+except ImportError:
+    HAVE_REDIS = False
+else:
+    from ansible.plugins.cache.redis import CacheModule as RedisCache
 
 
 class TestAbstractClass(unittest.TestCase):
@@ -72,11 +88,13 @@ class TestAbstractClass(unittest.TestCase):
 
         self.assertIsInstance(CacheModule3(), CacheModule3)
 
+    @unittest.skipUnless(HAVE_MEMCACHED)
     def test_memcached_cachemodule(self):
         self.assertIsInstance(MemcachedCache(), MemcachedCache)
 
     def test_memory_cachemodule(self):
         self.assertIsInstance(MemoryCache(), MemoryCache)
 
+    @unittest.skipUnless(HAVE_REDIS)
     def test_redis_cachemodule(self):
         self.assertIsInstance(RedisCache(), RedisCache)