Merge branch 'feature/local-storage-path-86' into develop

goose/configuration.py

@@ -84,9 +84,9 @@ def __init__(self):
         # Parser type
         self.parser_class = 'lxml'
 
-    @property
-    def local_storage_path(self):
-        return os.path.join(tempfile.gettempdir(), 'goose')
+        # set the local storage path
+        # make this configurable
+        self.local_storage_path = os.path.join(tempfile.gettempdir(), 'goose')
 
     def get_parser(self):
         return Parser if self.parser_class == 'lxml' else ParserSoup

tests/configuration.py

@@ -20,14 +20,21 @@
 See the License for the specific language governing permissions and
 limitations under the License.
 """
+import os
+import tempfile
 import unittest
 
-from goose.configuration import Configuration
+from goose import Goose
 
 
 class TestTempDir(unittest.TestCase):
 
-    def test_tmp_not_overwritten(self):
+    def test_tmp_defaut(self):
+        g = Goose()
+        default_local_storage_path = os.path.join(tempfile.gettempdir(), 'goose')
+        self.assertEquals(g.config.local_storage_path, default_local_storage_path)
+
+    def test_tmp_overwritten(self):
         path = '/this/directory/does/not/exist/i/assume/'
-        config = Configuration()
-        self.assertRaises(AttributeError, lambda: setattr(config, 'local_storage_path', path))
+        g = Goose({'local_storage_path': path})
+        self.assertEquals(g.config.local_storage_path, path)