From 65c017c5fe5fafe06900508f6d5a5f627dca619d Mon Sep 17 00:00:00 2001
From: Max Arnold <arnold.maxim@gmail.com>
Date: Wed, 18 Dec 2013 22:25:42 +0700
Subject: [PATCH] implement DeprecatedFilesPipelineTestCase

---
 scrapy/tests/test_pipeline_files.py | 41 +++++++++++++++++++++++++++++
 1 file changed, 41 insertions(+)

diff --git a/scrapy/tests/test_pipeline_files.py b/scrapy/tests/test_pipeline_files.py
index aba4c564f..63e1d99b7 100644
--- a/scrapy/tests/test_pipeline_files.py
+++ b/scrapy/tests/test_pipeline_files.py
@@ -1,6 +1,8 @@
 import mock
 import os
 import time
+import hashlib
+import warnings
 from tempfile import mkdtemp
 from shutil import rmtree
 
@@ -94,6 +96,44 @@ class FilesPipelineTestCase(unittest.TestCase):
             p.stop()
 
 
+class DeprecatedFilesPipeline(FilesPipeline):
+    def file_key(self, url):
+        media_guid = hashlib.sha1(url).hexdigest()
+        media_ext = os.path.splitext(url)[1]
+        return 'empty/%s%s' % (media_guid, media_ext)
+
+
+class DeprecatedFilesPipelineTestCase(unittest.TestCase):
+    def setUp(self):
+        self.tempdir = mkdtemp()
+
+    def init_pipeline(self, pipeline_class):
+        self.pipeline = pipeline_class.from_settings(Settings({'FILES_STORE': self.tempdir}))
+        self.pipeline.download_func = _mocked_download_func
+        self.pipeline.open_spider(None)
+
+    def test_default_file_key_method(self):
+        self.init_pipeline(FilesPipeline)
+        with warnings.catch_warnings(record=True) as w:
+            warnings.simplefilter('always')
+            self.assertEqual(self.pipeline.file_key("https://dev.mydeco.com/mydeco.pdf"),
+                             'full/c9b564df929f4bc635bdd19fde4f3d4847c757c5.pdf')
+            self.assertEqual(len(w), 1)
+            self.assertTrue('file_key(url) method is deprecated' in str(w[-1].message))
+
+    def test_overridden_file_key_method(self):
+        self.init_pipeline(DeprecatedFilesPipeline)
+        with warnings.catch_warnings(record=True) as w:
+            warnings.simplefilter('always')
+            self.assertEqual(self.pipeline.file_path(Request("https://dev.mydeco.com/mydeco.pdf")),
+                             'empty/c9b564df929f4bc635bdd19fde4f3d4847c757c5.pdf')
+            self.assertEqual(len(w), 1)
+            self.assertTrue('file_key(url) method is deprecated' in str(w[-1].message))
+
+    def tearDown(self):
+        rmtree(self.tempdir)
+
+
 class FilesPipelineTestCaseFields(unittest.TestCase):
 
     def test_item_fields_default(self):
@@ -144,5 +184,6 @@ def _prepare_request_object(item_url):
         item_url,
         meta={'response': Response(item_url, status=200, body='data')})
 
+
 if __name__ == "__main__":
     unittest.main()